Postgres Pro
Downloads
Home   >   mailing lists

Re: Avoid a possible null pointer (src/backend/utils/adt/pg_locale.c) - Mailing list pgsql-hackers

From Jeff Davis
Subject Re: Avoid a possible null pointer (src/backend/utils/adt/pg_locale.c)
Date
Msg-id b99ce0835797a6659de1a54c9f973e7b0e9a0e92.camel@j-davis.com
Whole thread Raw
In response to Re: Avoid a possible null pointer (src/backend/utils/adt/pg_locale.c)  (Michael Paquier <michael@paquier.xyz>)
Responses Re: Avoid a possible null pointer (src/backend/utils/adt/pg_locale.c)
List pgsql-hackers
Tree view 
On Wed, 2023-09-13 at 11:48 +0900, Michael Paquier wrote:
> Hmm.  I see your point, one could be confused that the function name
> is the provider with this wording.  How about that instead:
>  ERROR:  unsupported collprovider for %s: %c
>
> I've hidden that in a macro that uses __func__ as Jeff has suggested.
>
> What do you think?

Looks good to me, thank you.

Regards,
    Jeff Davis

pgsql-hackers by date:

Previous
From: Aleksander Alekseev
Date:
Subject: Re: pg_resetwal tests, logging, and docs update
Next
From: Nathan Bossart
Date:
Subject: Re: Add 'worker_type' to pg_stat_subscription