On Fri, Jan 6, 2017 at 5:36 PM, Jeff King <peff@xxxxxxxx> 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