From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Nathan Bossart <nathandbossart(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Why does pgindent's README say to download typedefs.list from the buildfarm? |
Date: | 2024-05-16 10:03:30 |
Message-ID: | 195c6c45-abce-4331-be6a-e87724e1d060@eisentraut.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 16.05.24 01:32, Tom Lane wrote:
> As for the remainder, they aren't showing up because no variable
> or field is declared using them, which means no debug symbol
> table entry is made for them. This means we could just drop those
> typedefs and be little the worse off notationally. I experimented
> with a patch for that, as attached. (In the case of NotificationHash,
> I thought it better to arrange for there to be a suitable variable;
> but it could certainly be done the other way too.) Is this too anal?
I agree we should get rid of these.
Over the last release cycle, I've been leaning a bit more toward not
typdef'ing enums and structs that are only in local use, in part because
of the implied need to keep the typedefs list up to date.
In these cases, I think for
NotificationHash
ResourceOwnerData
WalSyncMethod
we can just get rid of the typedef.
ReadBuffersFlags shouldn't be an enum at all, because its values are
used as flag bits.
WaitEventExtension, I'm not sure, it's like, an extensible enum? I
guess let's remove the typedef there, too.
Attached is a variant patch.
Attachment | Content-Type | Size |
---|---|---|
remove-useless-typedefs-pe.patch | text/plain | 3.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2024-05-16 10:07:32 | Re: An improved README experience for PostgreSQL |
Previous Message | Quan Zongliang | 2024-05-16 10:01:21 | Re: Fix log_line_prefix to display the transaction id (%x) for statements not in a transaction block |