| From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
|---|---|
| To: | Quan Zongliang <quanzongliang(at)yeah(dot)net> |
| Cc: | Dongpo Liu <poe(dot)liu(at)pm(dot)me>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: [PATCH] Remove unused include from analyze.c |
| Date: | 2026-07-29 11:46:58 |
| Message-ID: | 98202165-6579-4CA3-88C2-EA77AF66CBAE@yesql.se |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On 23 Jul 2026, at 04:23, Quan Zongliang <quanzongliang(at)yeah(dot)net> wrote:
> On 7/5/26 9:27 PM, Dongpo Liu wrote:
>> The attached patch removes it. The file compiles cleanly without it,
>> and I verified that no other symbol from procarray.h is referenced there.
> Agree.
>
> Furthermore, I also discovered that there are other files that contain this unnecessary references. Please take a look at the new patch in the attachment.
This version doesn't seem to work.
../contrib/amcheck/verify_heapam.c:773:5: error: call to undeclared function 'TransactionIdIsInProgress'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
TransactionIdIsInProgress(curr_xmin))
^
../contrib/amcheck/verify_heapam.c:2168:12: error: call to undeclared function 'TransactionIdIsInProgress'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
else if (TransactionIdIsInProgress(xid))
^
2 errors generated.
--
Daniel Gustafsson
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ants Aasma | 2026-07-29 11:51:53 | Re: pg_rewind does not rewind diverging timelines |
| Previous Message | Andrey Borodin | 2026-07-29 11:45:14 | Re: nbtree backwards scan test coverage |