Re: parsing of boolean config options

Karl Fogel wrote:
> Branko Čibej <brane_at_xbc.nu> writes:
>
>> Karl Fogel wrote:
>>
>>> [[[
>>> * subversion/libsvn_subr/config_file.c
>>> (svn_config_ensure): Just recommend "yes", don't recommend "true" as
>>> an alternative.
>>> ]]]
>>>
>> The patch looks fine. "Yes" is stylistically more pleasing to the eye
>> than "true" here ... though it's your vermilion to my crimson.
>>
>> But you really should read up on how we write log messages. :)
>>
>
> ? I'm staring at it, but don't see the problem... What am I missing?
>

Changes to a couple more files in that patch? :)

> By the way, both my patch and log message favored "yes" over "true", so
> it's my crimson to your crimson.
>

Oh, I forgot to mention that I prefer "true" sorry sorry!

> Thus, I am confused by nearly everything in your response :-). Help?
>

That seems to be SOP ... /me ducks

-- Brane

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-06-15 23:21:52 CEST