From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 06 Apr 2022 16:41:34 GMT
Subject: Re: [PATCH v2 2/4] bisect.c: remove unnecessary include
Message-ID: <xmqqfsmqyxj5.fsf@gitster.g>
In-Reply-To: <220406.86a6cyfy4l.gmgdl@evledraar.gmail.com>

Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:

> On Tue, Apr 05 2022, Garrit Franke wrote:
>
>> Remove include "hash-lookup.h".
>
> Like your 3/4 it would be nice to have a "orphaned since xyz", or was it
> never used etc?
>
> In these cases unlike most such C changes the compiler isn't of much use
> for validation (we might be including this implicitly), so an
> explanation saying why is helpful.

Thanks for pointing it out.  I found the lack of explanation in some
but not all of the changes disturbing.