From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 03 Aug 2018 15:53:36 GMT
Subject: Re: [PATCH v3 1/2] sequencer: handle errors in read_author_ident()
Message-ID: <xmqqtvobzbsv.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <CAPig+cSZ3Zm=qFcvGjyj_uStn=JMAYuskMa0O_2yxkKjaRWTSg@mail.gmail.com>

Eric Sunshine <sunshine@sunshineco.com> writes:

> I think this patch can be simplified considerably by shifting one's
> perspective. If we admit that read_author_ident() is already correctly
> reporting an error by returning NULL (which is exactly what it is
> doing), then the bug is is purely on the calling side; namely, the
> caller is ignoring the error. (In fact, your commit message already
> states this.)

This approach looks quite sensible.