Jeff King <peff@xxxxxxxx> writes:
> On Thu, Apr 14, 2016 at 11:07:08AM -0700, Junio C Hamano wrote:
>
>> The pattern taken by all the validations in this function is:
>>
>> if (notice a violation exists) {
>> err = report(... VIOLATION_KIND ...);
>> if (err)
>> return err;
>> }
>>
>> where report() returns zero if specified kind of violation is set to
>> be ignored, and otherwise shows an error message and returns non-zero.
>>
>> The last validation in the function immediately after the function
s/after/before/
>> returns 0 to declare "all good" can cheat and directly return the
>> return value from report(), and the current code does so, i.e.
>>
>> if (notice a violation exists)
>> return report(... VIOLATION_KIND ...);
>> return 0;
>>
>> But that is a selfish code that declares it is the ultimate and
>> final form of the function, never to be enhanced later. To allow
>> and invite future enhancements, make the last test follow the same
>> pattern.
>>
>> Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx>
>> ---
>> fsck.c | 8 +++++---
>> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> Patch looks good, and nicely explained.
>
> -Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html