From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Sun, 21 Jan 2018 08:25:48 GMT
Subject: Re: [PATCH v2 09/12] fetch: don't redundantly NULL something calloc() gave us
Message-ID: <874lnfipo3.fsf@evledraar.gmail.com>
In-Reply-To: <20180121000304.32323-10-avarab@gmail.com>


On Sun, Jan 21 2018, Ævar Arnfjörð Bjarmason jotted:

> 2007-09-10), where wasn't any reason to do this back then either, it's

s/where wasn't/where there wasn't/

> The initial motivation for this change was to make a subsequent change
> which'll also modify the refs variable smaller, since it won't have to
> copy this redundant "NULL the last + 1 item" pattern.

Which, FWIW is not happening later in this series anymore, so this is
just a "cleanup while I'm at it" patch at this point. Will note that in
a v3 commit message pending more comments.