From: Stefan Beller <sbeller@google.com>
Date: Mon, 09 Jan 2017 19:30:22 GMT
Subject: Re: [PATCH 4/5] unpack-trees: factor file removal out of check_updates
Message-ID: <CAGZ79kbZz5s5TdudGcOADDxYShTAm5sTCHRo81SSCc13MoF7WA@mail.gmail.com>
In-Reply-To: <20170107013602.act7aouuzgzpwb54@sigill.intra.peff.net>

On Fri, Jan 6, 2017 at 5:36 PM, Jeff King <peff@peff.net> wrote:

>> +static int remove_workingtree_files(struct unpack_trees_options *o,
>> +     unsigned cnt = 0;

>
> "cnt" is unsigned here, as it is in the caller. Should the return value
> match?

Yes, obviously. :/

Thanks for catching,
Stefan