> > I propose to remove '.cppi-disable'.
>
> Sounds good to me. I don't use cppi.
Done:
2024-06-15 Bruno Haible
Drop outdated cppi configuration.
* lib/.cppi-disable: Remove file.
On 2024-06-14 15:45, Bruno Haible wrote:
I propose to remove '.cppi-disable'.
Sounds good to me. I don't use cppi.
Paul Eggert did:
> * lib/.cppi-disable: Remove getpagesize.h.
This file had not been touched for more than 16 years.
When I run
$ cd lib
$ cppi -c *.h *.c
it emits 23812 diagnostics. So, clearly, 'cppi' does not reflect the
indentation habits that we are using for many years.
FYI,
>From af0285db3e8dee3f40ba573e845362335ac0cff2 Mon Sep 17 00:00:00 2001
From: Jim Meyering
Date: Wed, 2 Nov 2011 09:15:34 +0100
Subject: [PATCH] putenv: indent #definition of "environ" to placate cppi
* lib/putenv.c (environ): Make indentation reflect cpp nesting.
---
Change
Bruno Haible wrote:
> Hi Jim,
>
>> Would you mind if I adjust whichever of the 25 in tests/ that are yours ?
>
> Yes, I do mind changing 160 lines of code in just one file because of this.
>
> My point is that 'cppi' wants to enforce a coding style that is
On 02/20/11 16:58, Bruno Haible wrote:
>> it lets you distinguish conditional and unconditional
>> #include directives and #define directives based solely
>> on presence or absence of spaces after the "#".
>
> This heuristic does not always work:
> - sleep.c:# define WIN32_LEAN_AND_MEAN
> is
Hi Jim,
> Would you mind if I adjust whichever of the 25 in tests/ that are yours ?
Yes, I do mind changing 160 lines of code in just one file because of this.
My point is that 'cppi' wants to enforce a coding style that is pretty far
away from the current, reasonable coding st