-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
language-snippets.entの翻訳 #161
Conversation
フォークしたphp/doc-jaのリポジトリのmasterにコミットした名残で、いらないコミットが残っていますが気にしないでください。 |
language-snippets.ent
Outdated
呼び出す関数またはメソッド。有効な <link linkend="gearman.constants">Gearmanの値</link> を返す必要があります。 | ||
</para> | ||
<para> | ||
return ステートメントが存在しない場合、デフォルトで <constant>GEARMAN_SUCCESS</constant> が返されます。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
既存の訳ではstatement
は「文」と訳されていそうです。
return ステートメントが存在しない場合、デフォルトで <constant>GEARMAN_SUCCESS</constant> が返されます。 | |
return 文が存在しない場合、デフォルトで <constant>GEARMAN_SUCCESS</constant> が返されます。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
修正しました
language-snippets.ent
Outdated
|
||
<!ENTITY gearman.note.callback '<note xmlns="http://docbook.org/ns/docbook"> | ||
<para> | ||
この callback は、このメソッドを呼び出した後に追加されたタスク(例: <methodname>GearmanClient::addTask</methodname> を呼び出して追加されたタスク)に対してのみ実行されます。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
上と同じく「コールバック」とした方が全体として一貫性がありそうです。
この callback は、このメソッドを呼び出した後に追加されたタスク(例: <methodname>GearmanClient::addTask</methodname> を呼び出して追加されたタスク)に対してのみ実行されます。 | |
このコールバックは、このメソッドを呼び出した後に追加されたタスク(例: <methodname>GearmanClient::addTask</methodname> を呼び出して追加されたタスク)に対してのみ実行されます。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
修正しました
こちらについて削除するだけの内容になっているのですが、こちらでも削除しておいた方がよろしいでしょうか? |
このファイル内の |
</itemizedlist> | ||
'> | ||
|
||
<!ENTITY strings.stripped.unicode ' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@youkidearitai
mbstring
関数関連のものを追加しました。
お手数をおかけしますが、この部分のレビュー(校正)をお願いしたいです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jdkfx ありがとうございます!この部分問題ないと思います。
はい、削除お願いします。どこからも参照されてないもう不要な変数、だと思います。
今回は、このPR中で取り扱った原文のコミットの最新に合わせる、で良いと思います。 過去のコミット履歴を見た所、厳密なルールが存在するわけでもなさそうな気がしました。 |
@KentarouTakeda |
@KentarouTakeda |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ほとんどLGTMですが、文章の工夫で少し解りやすくできそうな気がしました。
ご検討お願いします!
Co-authored-by: KentarouTakeda <takeda@youmind.jp>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
広範に渡る大変な作業だったと思います。お疲れ様でした。
mb_trim()
の充実した説明も含め、非常に有益なコンテンツになったと感じます。大変ありがとうございました!
https://github.com/php/doc-en/blame/master/language-snippets.ent
取り込んだもののうち、関連する物を列挙
warn.deprecated.feature-8-4-0
to language-snippets.ent doc-en#3991example.outputs.84
to language-snippets.ent doc-en#3893