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

[PHP 8.4] PDOサブクラス・新メソッド #205

Merged
merged 5 commits into from
Dec 13, 2024

Conversation

KentarouTakeda
Copy link
Collaborator

@KentarouTakeda KentarouTakeda commented Dec 7, 2024

@KentarouTakeda KentarouTakeda force-pushed the php84-pdo-subclasses branch 3 times, most recently from 06526c8 to 8bcf794 Compare December 7, 2024 12:13
@KentarouTakeda KentarouTakeda marked this pull request as ready for review December 7, 2024 12:15
@KentarouTakeda KentarouTakeda added the review wanted Review wanted. Please feel free to comment; anyone is welcome! label Dec 7, 2024
@KentarouTakeda
Copy link
Collaborator Author

@SakiTakamachi
可能であればFirebirdだけ見て頂けると非常に助かります!

@KentarouTakeda KentarouTakeda changed the title [PHP 8.4]PDOサブクラス・新メソッド [PHP 8.4] PDOサブクラス・新メソッド Dec 7, 2024
</simpara>
</listitem>
<listitem>
<simpara>
Two-dashes, C-style comments, and Hash-comments.
2 つのハイフン、C スタイルのコメント、<literal>#</literal> によるコメント
Copy link
Contributor

Choose a reason for hiding this comment

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

この文脈では

  • -- comment
  • /* comment */
  • # comment

の3種類の形式のコメントのことなので、「〜コメント」は3つそれぞれに付けるか、最後のひとつにまとめた方がいいかもしれません。

「Cスタイル」は日本語では「C言語形式」のようにも置き換えられるかもしれません。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ありがとうございます!確かに読みやすくなりますね。

465683b で修正しました。同じコミットで、ほぼ同じ文章で「新機能」も訳していたのでそちらへもバックポートしています。

@@ -15,8 +15,8 @@
<void/>
</methodsynopsis>
<simpara>
Returns the Firebird API version as defined by the C constant
<constant>FB_API_VER</constant> in <filename class="headerfile">ibase.h</filename>.
<filename class="headerfile">ibase.h</filename> 内で C 定数 <constant>FB_API_VER</constant> として定義されている
Copy link
Contributor

Choose a reason for hiding this comment

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

こちらもibase.hの定数のことなので、「ibase.h 内でC言語での定数FB_API_VERとして定義されている…」のような文章の方がわかりやすいかもしれません。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ありがとうございます! 「C 言語の定数」としました: e9f41b5

他に無いか、プルリクエストを C という文字列で検索しました。
既に翻訳済だった C API を除きすべて C 言語 となりました。

@KentarouTakeda KentarouTakeda merged commit 2c97ea1 into php:master Dec 13, 2024
1 check passed
@KentarouTakeda KentarouTakeda deleted the php84-pdo-subclasses branch December 13, 2024 03:21
@KentarouTakeda
Copy link
Collaborator Author

@Girgias cc: @mumumu
Thank you for the CI check and partial sync.

It looked like the cause of the failure was that this pull request had not been merged, so I adjusted it, including your sync, and made it pass.

@Girgias
Copy link
Member

Girgias commented Dec 13, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review wanted Review wanted. Please feel free to comment; anyone is welcome!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants