Re: lwlocknames.h beautification attempt

On Sat, Mar 1, 2025 at 10:26 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Gurjeet Singh <gurjeet(at)singh(dot)im> writes:
> > I propose the following change to the generation script,
> > generate-lwlocknames.pl
> > ...
> > which produces the lock names in this format
>
> > #define ShmemIndexLock (&MainLWLockArray[1].lock)
> > #define OidGenLock (&MainLWLockArray[2].lock)
> > #define XidGenLock (&MainLWLockArray[3].lock)
> > #define ProcArrayLock (&MainLWLockArray[4].lock)
> > #define SInvalReadLock (&MainLWLockArray[5].lock)
>
> This looks reasonably in line with project style ...

Should I create a commitfest entry for this patch, or is it
uncontroversial enough and small enough to not warrant that?

> > Yet another format, which I prefer, can be achieved by right-aligning the
> > lock names.
>
> > #define ShmemIndexLock (&MainLWLockArray[1].lock)
> > #define OidGenLock (&MainLWLockArray[2].lock)
> > #define XidGenLock (&MainLWLockArray[3].lock)
> > #define ProcArrayLock (&MainLWLockArray[4].lock)
> > #define SInvalReadLock (&MainLWLockArray[5].lock)
>
> ... but that doesn't. I challenge you to provide even one example
> of that layout in our source tree, or to explain why it's better.

I haven't seen this style in any other project, let alone in Postgres. I
just mentioned it since I personally liked it slightly better after
trying a couple of different styles, because of the aligned suffix in
the lock names.

Best regards,
Gurjeet
http://Gurje.et