Remove pass-by-reference & from example code
メンテナーはふだん 1 日以内に返信
関連するプルリクエストがすでにマージされています。
- #5952 @warren5236 による — マージ済み
評価
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 初心者へのやさしさ
- 88/100
- issue の種類
- ドキュメント
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- php, xml
調査の方向性
対象は英語版マニュアルのオブジェクト比較のページにある Example #1 Example of object comparison です。そのため php/doc-en のソースで compareObjects を検索し、マークアップを見つけてください。パラメータを &$o1, &$o2 から $o1, $o2 に変更し、それ以外の例はそのままにしてください。シグネチャに & がなくなり、PHP で実行したときに例の出力が変わらなければ完了です。
索引モデルが issue の本文から書いたものです。
説明
Affected page
https://www.php.net/manual/en/language.oop5.object-comparison.php
Current issue
Example #1 Example of object comparison defines function compareObjects(&$o1, &$o2) taking $o1 and $o2 by reference.
There is no need for a reference. The pass-by-reference makes the example code confusing.
Suggested improvement
Change the function to function compareObjects($o1, $o2). The output remains exactly identical as expected.
See the change with https://3v4l.org/3HGT8#v8.4.26
Additional context (optional)
No response
- 主要言語
- 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 日以内に返信
-
socket_accept: wrong statements about socket_set_blocking, false and socket_last_error対応中かも @qodelicious が今日担当しました。 オープンbug
難易度 1/5 1〜3時間 初心者へのやさしさ 89/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 日以内に返信
似ている issue
-
documentation good first issue hacktoberfest
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
level/task
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
wazuh/wazuh-dashboard-plugins#9452 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 1/5 1〜3時間 初心者へのやさしさ 77/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 82/100
cran-task-views/Agriculture#146 ·