From: Junio C Hamano <gitster@pobox.com>
Date: Sat, 09 Feb 2008 06:04:42 GMT
Subject: Re: [Janitors] value could be NULL in config parser
Message-ID: <7v63wyfyx1.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <200802090641.20171.chriscool@tuxfamily.org>

Christian Couder <chriscool@tuxfamily.org> writes:

> Le samedi 9 février 2008, Govind Salinas a écrit :
>
>> Question.  Wouldn't it reduce the amount of validation we have to do
>> if whoever is calling back checked null and assigned an empty string?
>> If so, we can probably replace all these patches with one patch.
>
> This was discussed in this thread:
>
> http://thread.gmane.org/gmane.comp.version-control.git/72107/
>
> In short we can't do that now, but it may be possible latter, if we first 
> deprecate using an empty string as boolean value "false" (while a NULL is 
> boolean value "true").

It may be the case that many in-tree config parser functions are
sloppy right now.  It however is never an excuse to break user's
existing repositories.