It will resolve the large number of these warnings from
https://github.com/pgcentralfoundation/pgrx/blob/6dfb9d1/cargo-pgrx/src/command/init.rs#L411-L412:
<command-line>: note: this is the location of the previous definition
./../../src/include/pg_config_manual.h:274: warning:
"MEMORY_CONTEXT_CHECKING" redefined
| #define MEMORY_CONTEXT_CHECKING
|
<command-line>: note: this is the location of the previous definition
In file included from ../../../src/include/c.h:55,
from ../../../src/include/postgres.h:46,
from xactdesc.c:15:
../../../src/include/pg_config_manual.h:265: warning:
"CLOBBER_FREED_MEMORY" redefined
| #define CLOBBER_FREED_MEMORY
and yes will be sending them a patch also. But there's no harm in not
redefining symbols, so not sure why this is a controversial patch.
On Fri, Aug 23, 2024 at 4:11 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Samuel Marks <samuelmarks@gmail.com> writes:
> > Subject: [PATCH] [src/include/pg_config_manual.h] Guard `CLOBBER_FREED_MEMORY`
> > & `MEMORY_CONTEXT_CHECKING`
>
> Why is this a good idea?
>
> regards, tom lane