Reconsider usages of `gethostbyname_r()`
オープン
まだ誰も着手していません。
Category: Engine
Refactoring
- 主要言語
- C
- スター
- 40.4k
- フォーク
- 8.2k
- 平均マージ
- 2日 13時間
- マージ済み PR（30日）
- 96
説明
Description
According to some sources, this function is obsolete and should be replaced with:
Applications should use getaddrinfo(3), getnameinfo(3), and gai_strerror(3) instead.
Moreover, getting rid of them would allow us to get remove some conditionally defined globals:
https://github.com/php/php-src/blob/69d9c12df64e829befd843175bfc9617aabb7450/ext/standard/file.h#L106-L110
Initially found while working on #15511
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
リポジトリ内で gethostbyname_r() の使用箇所を検索し、106-110 行付近の ext/standard/file.h と #15511 のコンテキストを確認してください。各使用箇所を getaddrinfo()、getnameinfo()、gai_strerror() と比較してください。完了とは、廃止された呼び出しと、不要になった条件付きグローバル変数が一貫して対処されていることを意味します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- c
- 領域
- networking
- issue の種類
- リファクタリング
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100