Re: Improve readability by using designated initializers when possible - Mailing list pgsql-hackers

From Japin Li
Subject Re: Improve readability by using designated initializers when possible
Date
Msg-id ME3P282MB3166DEAB9FAFA07C54CEF3EDB6582@ME3P282MB3166.AUSP282.PROD.OUTLOOK.COM
Whole thread Raw
In response to Re: Improve readability by using designated initializers when possible  (Jelte Fennema-Nio <postgres@jeltef.nl>)
Responses Re: Improve readability by using designated initializers when possible
List pgsql-hackers
On Wed, 28 Feb 2024 at 00:06, Jelte Fennema-Nio <postgres@jeltef.nl> wrote:
> On Tue, 27 Feb 2024 at 16:04, Japin Li <japinli@hotmail.com> wrote:
>> I see the config_group_names[] needs null-terminated because of help_config,
>> however, I didn't find the reference in help_config.c.  Is this comment
>> outdated?
>
> Yeah, you're correct. That comment has been outdated for more than 20
> years. The commit that made it unnecessary to null-terminate the array
> was 9d77708d83ee.
>
> Attached is v5 of the patchset that also includes this change (with
> you listed as author).

Thanks for updating the patch!

It looks good to me except there is an outdated comment.

diff --git a/src/common/encnames.c b/src/common/encnames.c
index bd012fe3a0..dba6bd2c9e 100644
--- a/src/common/encnames.c
+++ b/src/common/encnames.c
@@ -297,7 +297,6 @@ static const pg_encname pg_encname_tbl[] =

 /* ----------
  * These are "official" encoding names.
- * XXX must be sorted by the same order as enum pg_enc (in mb/pg_wchar.h)
  * ----------
  */
 #ifndef WIN32



pgsql-hackers by date:

Previous
From: Jeff Davis
Date:
Subject: Re: MAINTAIN privilege -- what do we need to un-revert it?
Next
From: Andrew Dunstan
Date:
Subject: Re: Logging parallel worker draught