Re: ECPG cleanup and fix for clang compile-time problem

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: John Naylor <johncnaylorls(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Andres Freund <andres(at)anarazel(dot)de>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Subject: Re: ECPG cleanup and fix for clang compile-time problem
Date: 2024-10-04 20:46:44
Message-ID: 707324.1728074804@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I wrote:
> The cfbot points out that I should probably have marked progname
> as "static" in 0008. I'm not going to repost the patchset just for
> that, though.

Rebase needed after f22e84df1, so here's an update that rebases
up to HEAD and adds the missing "static". No other changes.

(Anybody want to review this? I'm getting tired of rebasing it,
and we're missing out on the clang build time savings.)

regards, tom lane

Attachment Content-Type Size
v5-0001-Clean-up-documentation-of-parse.pl-and-add-more-i.patch text/plain 12.4 KB
v5-0002-Major-cleanup-simplification-and-documentation-of.patch text/x-diff 24.0 KB
v5-0003-Re-implement-ecpg-preprocessor-s-string-managemen.patch text/x-diff 109.5 KB
v5-0004-Move-some-functions-into-a-new-file-ecpg-preproc-.patch text/x-diff 10.1 KB
v5-0005-Improve-ecpg-preprocessor-s-memory-management.patch text/x-diff 88.9 KB
v5-0006-Fix-some-memory-leakage-of-data-type-related-stru.patch text/x-diff 8.6 KB
v5-0007-Make-all-string-valued-tokens-returned-by-pgc.l-b.patch text/x-diff 7.1 KB
v5-0008-Clean-up-some-other-assorted-ecpg-memory-leaks.patch text/x-diff 16.1 KB
v5-0009-Remove-ecpg-s-check_rules.pl.patch text/x-diff 6.9 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jelte Fennema-Nio 2024-10-04 21:05:51 Re: Extension security improvement: Add support for extensions with an owned schema
Previous Message Noah Misch 2024-10-04 18:57:48 Re: IPC::Run accepts bug reports