socket_accept: wrong statements about socket_set_blocking, false and socket_last_error
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 1/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 89/100
- issue の種類
- ドキュメント
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- php
調査の方向性
socket-acceptのマニュアルページと、issueで言及されているsocketsの例のページから始めてください。ブロッキング呼び出しを更新し、falseは接続が保留されていないことを意味すると説明し、両方の箇所で引数なしのsocket_last_error()を使用してください。両方のページにこれらの修正が反映されたら完了です。
索引モデルが issue の本文から書いたものです。
説明
Affected page
https://www.php.net/manual/en/function.socket-accept.php
Issue description
socket_set_blocking()doesn't work on aSocket(it is an alias ofstream_set_blocking()); onlysocket_set_nonblock()does.falseis returned only when no connection is pending (EAGAIN), without a warning.socket_last_error()gives the error only without argument: after a failed accept,socket_last_error($listener)returns 0. The example in the sockets examples page does exactly that.
Steps to reproduce
$l = socket_create(AF_INET, SOCK_STREAM, 0);
socket_set_blocking($l, false);
// TypeError: Argument #1 ($stream) must be of type resource, Socket given
Suggested fix
Mention socket_set_nonblock() instead, say false means no pending connection, and call socket_last_error() without argument (in the page and in the example).
- 主要言語
- XML
- スター
- 600
- フォーク
- 903
- 平均マージ
- 1日 5時間
- マージ済み PR(30日)
- 81
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
php/doc-en のほかの issue
-
session_write_close: the return value is not success or failure対応中かも @qodelicious が今日担当しました。 オープンbug good first issue
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
SID is deprecated対応中かも @jordikroon が 8 日前に担当しました。 オープンbug
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
enchant_dict_*: document the ValueError and fix the return values対応中かも @qodelicious が 2 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
cran-task-views/Survival#76 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
type:docs
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
sandtable-ai/sandtable#300 ·
メンテナーはふだん 1 日以内に返信
-
content good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100