Skip to content

Conversation

@jdkfx
Copy link
Contributor Author

jdkfx commented Nov 9, 2024

フォークしたphp/doc-jaのリポジトリのmasterにコミットした名残で、いらないコミットが残っていますが気にしないでください。

呼び出す関数またはメソッド。有効な <link linkend="gearman.constants">Gearmanの値</link> を返す必要があります。
</para>
<para>
return ステートメントが存在しない場合、デフォルトで <constant>GEARMAN_SUCCESS</constant> が返されます。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

既存の訳ではstatementは「文」と訳されていそうです。

Suggested change
return ステートメントが存在しない場合、デフォルトで <constant>GEARMAN_SUCCESS</constant> が返されます。
return 文が存在しない場合、デフォルトで <constant>GEARMAN_SUCCESS</constant> が返されます。
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正しました


<!ENTITY gearman.note.callback '<note xmlns="http://docbook.org/ns/docbook">
<para>
この callback は、このメソッドを呼び出した後に追加されたタスク(例: <methodname>GearmanClient::addTask</methodname> を呼び出して追加されたタスク)に対してのみ実行されます。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

上と同じく「コールバック」とした方が全体として一貫性がありそうです。

Suggested change
この callback は、このメソッドを呼び出した後に追加されたタスク(例: <methodname>GearmanClient::addTask</methodname> を呼び出して追加されたタスク)に対してのみ実行されます。
このコールバックは、このメソッドを呼び出した後に追加されたタスク(例: <methodname>GearmanClient::addTask</methodname> を呼び出して追加されたタスク)に対してのみ実行されます。
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正しました

@jdkfx jdkfx marked this pull request as ready for review November 14, 2024 15:22
@jdkfx
Copy link
Contributor Author

jdkfx commented Nov 14, 2024

こちらについて削除するだけの内容になっているのですが、こちらでも削除しておいた方がよろしいでしょうか?
php/doc-en#3967

@jdkfx
Copy link
Contributor Author

jdkfx commented Nov 14, 2024

このファイル内の EN-Revision: の扱いが分からなかったため、どなたか教えていただけますと幸いです。

</itemizedlist>
'>

<!ENTITY strings.stripped.unicode '
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@youkidearitai
mbstring関数関連のものを追加しました。
お手数をおかけしますが、この部分のレビュー(校正)をお願いしたいです。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jdkfx ありがとうございます!この部分問題ないと思います。

@jdkfx jdkfx changed the title WIP: language-snippets.entの翻訳 language-snippets.entの翻訳 Nov 14, 2024
@KentarouTakeda
Copy link
Collaborator

こちらについて削除するだけの内容になっているのですが、こちらでも削除しておいた方がよろしいでしょうか?
php/doc-en#3967

はい、削除お願いします。どこからも参照されてないもう不要な変数、だと思います。
(ビルドが通れば大丈夫です)

このファイル内の EN-Revision: の扱いが分からなかったため、どなたか教えていただけますと幸いです。

今回は、このPR中で取り扱った原文のコミットの最新に合わせる、で良いと思います。

過去のコミット履歴を見た所、厳密なルールが存在するわけでもなさそうな気がしました。
かなり昔の変更の未反映、なども残っているようですが、一旦、最近の変更にフォーカスしましょう。

@jdkfx
Copy link
Contributor Author

jdkfx commented Nov 15, 2024

@KentarouTakeda
ありがとうございます。
時間が空いた時にやっておきます。

@jdkfx
Copy link
Contributor Author

jdkfx commented Nov 16, 2024

@KentarouTakeda
修正を行い、手元でビルドが通ることを確認しました。
レビューをお願いいたします!

Copy link
Collaborator

@KentarouTakeda KentarouTakeda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ほとんどLGTMですが、文章の工夫で少し解りやすくできそうな気がしました。
ご検討お願いします!

jdkfx and others added 2 commits November 16, 2024 04:21
Co-authored-by: KentarouTakeda <takeda@youmind.jp>
Co-authored-by: KentarouTakeda <takeda@youmind.jp>
Copy link
Collaborator

@KentarouTakeda KentarouTakeda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

広範に渡る大変な作業だったと思います。お疲れ様でした。

mb_trim() の充実した説明も含め、非常に有益なコンテンツになったと感じます。大変ありがとうございました!

@KentarouTakeda KentarouTakeda merged commit 5f0376f into php:master Nov 16, 2024
1 check passed
@jdkfx jdkfx deleted the #150_language-snippets branch November 23, 2024 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants