On Sat, May 02, 2015 at 07:15:08PM -0700, Junio C Hamano wrote: > Jeff King <peff@xxxxxxxx> writes: > > > On Fri, May 01, 2015 at 03:35:37PM -0700, Stefan Beller wrote: > > > >> Subject: Re: [PATCH] checkout-index.c: Unconditionally free memory > > > > Looks like the patch has expanded beyond checkout-index.c. Maybe: > > > > unconditionally free result of prefix_path > > > > would be more descriptive? I usually like the "area:" prefix, but I > > think here the common thread is not an area, but that they are return > > values from prefix_path. > > Sure, the prefix could even be "prefix_path(): $message", I would > think. I almost suggested that, but it not a change to prefix_path at all, but rather to its callers. That may be getting nit-picky, though. :) -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html