-
Notifications
You must be signed in to change notification settings - Fork 72
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] Add manual for pg_result_memory_syze()
#145
Conversation
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.
関数と英語版ドキュメントの作者です。
訳して頂き恐縮です、ありがとうございます!
マージされたPRに改善点があり、メンテナの方がxmlのネストをより良くして下さいました:
こちらも一緒にすると良いかもしれません。
(リスト要素を段落で括る必要はない、ということだと思います)
<para> | ||
<variablelist> | ||
<varlistentry> | ||
<term><parameter>result</parameter></term> | ||
<listitem> | ||
&pgsql.parameter.result; | ||
</listitem> | ||
</varlistentry> | ||
</variablelist> | ||
</para> |
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.
<para> | |
<variablelist> | |
<varlistentry> | |
<term><parameter>result</parameter></term> | |
<listitem> | |
&pgsql.parameter.result; | |
</listitem> | |
</varlistentry> | |
</variablelist> | |
</para> | |
<variablelist> | |
<varlistentry> | |
<term><parameter>result</parameter></term> | |
<listitem> | |
&pgsql.parameter.result; | |
</listitem> | |
</varlistentry> | |
</variablelist> |
<para> | ||
<simplelist> | ||
<member><function>pg_free_result</function></member> | ||
</simplelist> | ||
</para> |
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.
<para> | |
<simplelist> | |
<member><function>pg_free_result</function></member> | |
</simplelist> | |
</para> | |
<simplelist> | |
<member><function>pg_free_result</function></member> | |
</simplelist> |
@KentarouTakeda ありがとうございます。修正しました🙏 |
e63ed56
to
3685405
Compare
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.
@Ippey
ありがとうございました!
スタイルを他と統一させて頂いたのと、ヘッダのクレジットにお名前を載せて良いか解らなかったのでひとまず私の名前にしたのですが、ご自身の方が良ければ仰って下さい。
php/doc-en#3972 の変更を取り込みました。
reference/pgsql/version.xml はja側に存在しなかったので、不要だと思いコピーしていません。必要であれば、コピーして作成します。