Re: [PATCH] Code tidying for subversion/include/config.h

On 23/03/13 15:58, Daniel Shahaf wrote:
> Gabriela Gibson wrote on Sat, Mar 23, 2013 at 15:27:31 +0000:
>> [[[
>> Align variables for easier reading.
>>
>> *subversion/include/config.h(): Align variables.
>> ]]]
>>
>
> -0. The indentation is intention and reflects the nesting (e.g.,
> --config-option=servers:global:http-proxy-username=danielsh). It's
> actually easier to read the way it is now.
>
> A recent log message by cmpilato actually called out that fact
> explicitly, didn't you notice that?
>

Oh -- I hadn't noticed that at all -- I've never seen this trick before
and I think that the log message that explains it should be in the code
file, because some folks might not realize what is happening here and
miss out, I certainly did :)

Gabriela
Received on 2013-03-24 15:53:03 CET