From: | "Tristan Partin" <tristan(at)neon(dot)tech> |
---|---|
To: | "Alvaro Herrera" <alvherre(at)alvh(dot)no-ip(dot)org>, "Heikki Linnakangas" <hlinnaka(at)iki(dot)fi> |
Cc: | "Pg Hackers" <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: make BuiltinTrancheNames less ugly |
Date: | 2024-03-01 16:42:19 |
Message-ID: | CZIKAT2WH7YR.31FQYQ4XBO58J@neon.tech |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri Mar 1, 2024 at 8:00 AM CST, Alvaro Herrera wrote:
> On 2024-Feb-23, Heikki Linnakangas wrote:
>
> > On 12/02/2024 19:01, Tristan Partin wrote:
> > > On Wed Jan 24, 2024 at 8:09 AM CST, Alvaro Herrera wrote:
> > > > IMO it would be less ugly to have the origin file lwlocknames.txt be
> > > > not a text file but a .h with a macro that can be defined by
> > > > interested parties so that they can extract what they want from the
> > > > file, like PG_CMDTAG or PG_KEYWORD. Using Perl for this seems dirty...
> > >
> > > I really like this idea, and would definitely be more inclined to see
> > > a solution like this.
> >
> > +1 to that idea from me too. Seems pretty straightforward.
>
> OK, here's a patch that does it. I have not touched Meson yet.
>
> I'm pretty disappointed of not being able to remove
> generate-lwlocknames.pl (it now generates the .h, no longer the .c
> file), but I can't find a way to do the equivalent #defines in CPP ...
> it'd require invoking the C preprocessor twice. Maybe an intermediate
> .h file would solve the problem, but I have no idea how would that work
> with Meson. I guess I'll do it in Make and let somebody suggest a Meson
> way.
I can help you with Meson if you get the Make implementation done.
--
Tristan Partin
Neon (https://neon.tech)
From | Date | Subject | |
---|---|---|---|
Next Message | Melanie Plageman | 2024-03-01 16:51:44 | Re: BitmapHeapScan streaming read user and prelim refactoring |
Previous Message | Tom Lane | 2024-03-01 16:12:56 | Re: double precisoin type |