From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 14 Aug 2023 16:55:21 GMT
Subject: Re: [PATCH 2/3] setup: add discover_git_directory_reason()
Message-ID: <xmqq8radlg1y.fsf@gitster.g>
In-Reply-To: <xmqqpm3plh8n.fsf@gitster.g>

Junio C Hamano <gitster@pobox.com> writes:

>> 1. The zero value of the enum is actually GIT_DIR_NONE, so nonpositive
>>    results are errors.
>
> True. discover_git_directory() already knows that negative return
> values from setup_git_directory_gently_1() signal errors while 0 or
> positive are OK.

NOnononono.  negative are not.  0 is not returned, so if we saw one,
it would be an error.  And positives are OK.

Sorry for the confusion.