From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 05 Dec 2013 18:16:10 GMT
Subject: Re: [PATCH 3/5] gitweb: Return plain booleans in validation methods
Message-ID: <xmqq61r3i2ol.fsf@gitster.dls.corp.google.com>
In-Reply-To: <1386235422.2186.7.camel@localhost.localdomain>

Krzesimir Nowak <krzesimir@endocode.com> writes:

> On Wed, 2013-12-04 at 17:07 +0100, Jakub Narębski wrote:
>> The only change that needs to be doe is replacing
>> 
>>            return $input;
>> 
>> with
>> 
>>            return 1;
>> 
>
> I prefer to use zeros instead of undefs - one might wonder if that undef
> is somewhat special that we can't use 0.

For Perl speakers, I suspect the code gives a totally opposite
impression.  Normal "false" return from a sub, when there is no
special need, is to return an undef from it, and a "return 0" would
make the readers wonder if there is something special about the way
the returned value has to be numeric zero, no?