Unify dns_*() function definitions for all platforms
未关闭
还没有人认领这个 Issue。
Category: DNS
Extension: standard
Feature
Status: Verified
- 主要语言
- C
- 星标
- 40.4k
- 派生
- 8.2k
- 平均合并
- 2 天 13 小时
- 30 天内合并 PR
- 96
描述
Description
This is a todo in the code
I'm putting it here so it gets on the radar for a todo.
If this is a wontfix then remove the todo from the code.
Changes to this code segment caused an issue #7748 with the output of gethostbyaddr() showing as a binary string in Windows.
/* TODO: Reimplement the gethostby* functions using the new winxp+ API, in dns_win32.c, then
we can have a dns.c, dns_unix.c and dns_win32.c instead of a messy dns.c full of #ifdef
*/
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
调研方向
先从 ext/standard/dns.c 中约第 138 行的 TODO 开始，然后比较所引用的 dns.c、dns_unix.c 和 dns_win32.c 的组织方式。查看 issue #7748，了解 Windows 上 gethostbyaddr() 的回归问题。完成的标准是：在不引入该回归的情况下统一各平台上的 dns_*() 定义，或者如果项目决定将其标记为 wontfix，则删除 TODO。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- c
- 领域
- networking
- Issue 类型
- 重构
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100