[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] test-exclude.c: remove unmatched #endif
|
From: |
Paul Eggert |
|
Subject: |
Re: [PATCH] test-exclude.c: remove unmatched #endif |
|
Date: |
Sun, 20 Feb 2011 15:17:07 -0800 |
|
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.13) Gecko/20101208 Thunderbird/3.1.7 |
On 02/20/2011 03:12 PM, Jim Meyering wrote:
> I do see the aesthetic value in indenting the entire cpp directive,
> but from a consistency standpoint, it is counterproductive.
I'd rather that we were consistent the other way, and
indented cpp directives like we indent other code.
Admittedly that sort of indenting is rare in gnulib,
but I have some qualms about removing it purely for
consistency's sake. It's not just aesthetics; it's
also readability.
- [PATCH] test-exclude.c: remove unmatched #endif, Jim Meyering, 2011/02/20
- Re: [PATCH] test-exclude.c: remove unmatched #endif, Bruno Haible, 2011/02/20
- Re: [PATCH] test-exclude.c: remove unmatched #endif, Jim Meyering, 2011/02/20
- Re: preprocessor directive indentation, cppi, Bruno Haible, 2011/02/20
- Re: preprocessor directive indentation, cppi, Bruce Korb, 2011/02/20
- Re: preprocessor directive indentation, cppi, Jim Meyering, 2011/02/21
- Re: preprocessor directive indentation, Bruno Haible, 2011/02/20