Re: str_casefold: fix typo in error message

From: Steven Niu <niushiji(at)gmail(dot)com>
To: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
Cc: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: str_casefold: fix typo in error message
Date: 2026-01-13 05:33:47
Message-ID: MN2PR15MB302106EAB88C45ACEDFD5E3BA78EA@MN2PR15MB3021.namprd15.prod.outlook.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

From: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
Sent: Tuesday, January 13, 2026 10:32
To: Steven Niu <niushiji(at)gmail(dot)com>
Cc: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: str_casefold: fix typo in error message

> On Jan 13, 2026, at 10:16, Steven Niu <niushiji(at)gmail(dot)com> wrote:
>
> From: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
> Sent: Tuesday, January 13, 2026 10:09
> To: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
> Subject: str_casefold: fix typo in error message
>
> Hi Hacker,Found this issue while reviewing the other patch. That looks like a copy-paste error from the initial commit.Best regards,--Chao Li (Evan)HighGo Software Co., Ltd.https://www.highgo.com/
>
>
> Hi, Evan,
>
> I have a comment. Since it is collid that's being checked and will be used by function pg_newlocale_from_collation(), so I think the change should be:
>
> - "lower()"),
> + "pg_newlocale_from_collation(()"),
>
> thought?

If you look at other functions in the file, there is a pattern for what it should be. In this patch, I just want to fix the error without breaking the pattern.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/

Yeah, I took a look at another function str_initcap() and you are correct.

Thanks,
Steven

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message shveta malik 2026-01-13 05:34:06 Re: [Patch] add new parameter to pg_replication_origin_session_setup
Previous Message John Naylor 2026-01-13 05:07:13 Re: Proposal for enabling auto-vectorization for checksum calculations