On 12/18/24 10:42 AM, Peter Eisentraut wrote:
I can fix that with the attached patch.

The symbol YY_TYPEDEF_YY_SCANNER_T isn't documented, but we already use it elsewhere in the code.

Note that in replication/syncrep.h and replication/walsender_private.h we have to have an #ifndef wrapper because there are files that end up including both headers.  Maybe we should put that #ifndef wrapper everywhere for consistency?

Any thoughts?

Seems like a sane fix to me and as for the ifndef I have no strong opinion either way but I would personally probably have added it everywhere for consistency.

Andreas



Reply via email to