Skip to content
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

Document PHP 8.4 ext-hash changes を取り込み #240

Merged
merged 2 commits into from
Jan 13, 2025

Conversation

siwa32
Copy link
Contributor

@siwa32 siwa32 commented Jan 11, 2025

refs #150

php/doc-en#4137 を取り込みました。

この文の翻訳に苦労しました。

Passing configurations options of the wrong type in
<parameter>options</parameter> will now emit an
<constant>E_DEPRECATED</constant> error because they can be interpreted
incorrectly.

以下の挙動の違いを表したくて試行錯誤しましたが、かえってわかりにくい日本語になってしまったかもしれません。

hash_init('murmur3f', 0, "", ["seed" => "10"]);// E_DEPRECATED
hash_init('murmur3f', 0, "", ["seed" => 10]);// OK
hash_init('murmur3f', 0, "", ["sed" => 10]);// オプション項目名は間違っていてもエラーや警告はなく処理される

incorrect と wrong の違いも難しく、ただ違う言い回しにしただけになってしまいました。
両者は微妙にニュアンスが違うらしいですが、ネイティブではないので違いをつかめずでした。

せっかく訳したので参考訳としてプルリクします。マージせずでもまったく問題ありません!

siwa32 and others added 2 commits January 11, 2025 16:16
E_DEPRECATEDを発生させるコード例
hash_init('murmur3f', 0, "", ["seed" => "10"]);// E_DEPRECATED
hash_init('murmur3f', 0, "", ["seed" => 10]);// OK
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.

ありがとうございます!

頂いた訳でも通じるように思われたのですが、お悩みのようだったので私なりに少し変えてみました。

  • 日本語としての言い回しを優先し訳を崩す
  • 他の個所とあわせ E_DEPRECATED エラーE_DEPRECATED に修正

いかがでしょうか?余計おかしくなってたらすみません!

@KentarouTakeda KentarouTakeda merged commit 1e7b7da into php:master Jan 13, 2025
1 check passed
@siwa32
Copy link
Contributor Author

siwa32 commented Jan 13, 2025

すごくいいです!
ありがとうございます!

@siwa32 siwa32 deleted the doc-en/4137_hash-84 branch January 13, 2025 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants