Unify dns_*() function definitions for all platforms
オープン
まだ誰も着手していません。
Category: DNS
Extension: standard
Feature
Status: Verified
- 主要言語
- C
- スター
- 40.4k
- フォーク
- 8.2k
- 平均マージ
- 2日 13時間
- マージ済み PR（30日）
- 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 にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず ext/standard/dns.c の138行目付近にある TODO から始め、次に参照されている dns.c、dns_unix.c、dns_win32.c の構成を比較します。gethostbyaddr() に関する Windows のリグレッションについて issue #7748 を確認してください。完了条件は、そのリグレッションを発生させずにプラットフォーム間で dns_*() の定義を統一するか、プロジェクトが wontfix と判断した場合に TODO を削除することです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- c
- 領域
- networking
- issue の種類
- リファクタリング
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100