[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: parsing of boolean config options

From: Branko Čibej <brane_at_xbc.nu>
Date: Sun, 15 Jun 2008 23:21:04 +0200

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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.