Re: BUG #19450: Where is checksum_block.inc.c after master install? - Mailing list pgsql-bugs

From John Naylor
Subject Re: BUG #19450: Where is checksum_block.inc.c after master install?
Date
Msg-id CANWCAZb968EFqj1Q67yPNn2xVaWg6ASShz+kbJ5G-pP0s3BEEA@mail.gmail.com
Whole thread
In response to Re: BUG #19450: Where is checksum_block.inc.c after master install?  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-bugs
On Mon, Apr 6, 2026 at 10:07 AM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> John Naylor <johncnaylorls@gmail.com> writes:
> > On Mon, Apr 6, 2026 at 9:50 AM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> >> One could argue that the real bug is having put a .c file into
> >> the include/ tree in the first place.  Why was it done like that?
> >> Couldn't it be a .h file?
>
> > That was the way it was first coded. I thought of this way to avoid
> > adding an exception to headerscheck. I can reverse that decision
> > easily, but I may not get to it today.
>
> Ah, the good ol' law of conservation of cruft.  But on the whole
> I think naming it .h not .c is less crufty.  Agreed that there's
> no great urgency about changing it.

Okay, done that way as of commit 8c3e22a8f.

--
John Naylor
Amazon Web Services



pgsql-bugs by date:

Previous
From: Chris Hofstaedtler
Date:
Subject: Re: BUG #19416: Backend SIGSEGV in ExecShutdownHashJoin/ExecHashTableDetach/dsa_free
Next
From: Bruce Momjian
Date:
Subject: Re: BUG #19451: In postgreSQL v13 For 1000 tables how to update from rrn column to GUID. Each table more than 2crore