| From: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
|---|---|
| To: | Sugamoto Shinya <shinya34892(at)gmail(dot)com> |
| Cc: | Nathan Bossart <nathandbossart(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [PATCH] Add hints for invalid binary encoding names in encode/decode functions |
| Date: | 2025-11-19 13:01:03 |
| Message-ID: | CAHGQGwFMy5T15RfujkL8UdG8us_qtO8pJgOjsK6cO2kF0tizTQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Nov 19, 2025 at 8:08 PM Sugamoto Shinya <shinya34892(at)gmail(dot)com> wrote:
>
> Thanks everyone for reviewing my proposal. Please let me know if you have something to discuss here.
+ errhint("Valid encodings are \"%s\",
\"%s\", \"%s\", and \"%s\".",
+ "base64",
"base64url", "escape", "hex")));
You changed the HINT message to use format arguments,
but I still think it's better to embed the values directly (e.g.,
Valid encodings are "base64", "base64url", "escape", and "hex".)
for consistency with similar HINT messages. If this message is
used in many places a parameterized style would help translators,
but that benefit doesn't apply here.
Attached is an updated version of the patch. I switched
the HINT message to the embedded form and updated the commit message.
Barring any objections, I'm thinking to commit this.
Regards,
--
Fujii Masao
| Attachment | Content-Type | Size |
|---|---|---|
| v4-0001-Add-HINT-listing-valid-encodings-to-encode-and-de.patch | application/octet-stream | 3.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Daniel Gustafsson | 2025-11-19 13:04:44 | Re: [PATCH] Add hints for invalid binary encoding names in encode/decode functions |
| Previous Message | BharatDB | 2025-11-19 12:29:09 | Re: BUG #19095: Test if function exit() is used fail when linked static |