Horton

Thread

Commits

  1. Update src/tools/pginclude/README to match recent changes to cpluspluscheck

  2. Combine headerscheck and cpluspluscheck scripts

  1. Combine headerscheck and cpluspluscheck scripts

    Peter Eisentraut <peter@eisentraut.org> — 2024-02-07T16:01:40Z

    headerscheck started in 55ea1091884 (2019) essentially as an adjusted 
copy of cpluspluscheck.  Since then two scripts have not drifted far 
apart.  But there are occasionally mistakes keeping the two exclude 
lists updated together.  I figure we can just combine the two scripts 
into one, so it's easier to keep updated.

The attached patch adds an option --cplusplus to headerscheck, with 
which it does the same thing as cpluspluscheck, and cpluspluscheck is 
removed.  The top-level make targets stay the same.

  2. Re: Combine headerscheck and cpluspluscheck scripts

    Thomas Munro <thomas.munro@gmail.com> — 2024-03-07T00:37:36Z

    +1

  3. Re: Combine headerscheck and cpluspluscheck scripts

    Michael Paquier <michael@paquier.xyz> — 2024-03-07T07:30:11Z

    On Thu, Mar 07, 2024 at 01:37:36PM +1300, Thomas Munro wrote:
> +1

Looking at the patch, nice cleanup.
--
Michael

  4. Re: Combine headerscheck and cpluspluscheck scripts

    Peter Eisentraut <peter@eisentraut.org> — 2024-03-10T09:03:00Z

    On 07.03.24 08:30, Michael Paquier wrote:
> On Thu, Mar 07, 2024 at 01:37:36PM +1300, Thomas Munro wrote:
>> +1
> 
> Looking at the patch, nice cleanup.

Committed, thanks.

  5. Re: Combine headerscheck and cpluspluscheck scripts

    Anton Voloshin <a.voloshin@postgrespro.ru> — 2024-04-16T15:17:14Z

    Hello, hackers,

On 10/03/2024 12:03, Peter Eisentraut wrote:
> Committed, thanks.

This commit (7b8e2ae2f) have turned cpluspluscheck script into a 
--cplusplus option for headerscheck.  I propose to update the 
src/tools/pginclude/README correspondingly, please see the attached patch.

-- 
Anton Voloshin
Postgres Professional, The Russian Postgres Company
https://postgrespro.ru

  6. Re: Combine headerscheck and cpluspluscheck scripts

    Peter Eisentraut <peter@eisentraut.org> — 2024-04-18T09:37:58Z

    On 16.04.24 17:17, Anton Voloshin wrote:
> On 10/03/2024 12:03, Peter Eisentraut wrote:
>> Committed, thanks.
> 
> This commit (7b8e2ae2f) have turned cpluspluscheck script into a 
> --cplusplus option for headerscheck.  I propose to update the 
> src/tools/pginclude/README correspondingly, please see the attached patch.

Fixed, thanks!