On Thu, Apr 04, 2013 at 10:43:14PM -0700, Junio C Hamano wrote: > Jeff King <peff@xxxxxxxx> writes: > > > I notice that you are stuck factoring out not just the setup, but also > > the cleanup, and I wondered if things could be made even simpler by just > > encapsulating the checking logic in a callback; then the setup and > > cleanup flow more naturally, as they are in a single function wrapper. > > > > Like this, which ends up saving 20 lines rather than adding 7: > > Oh, this is one of those many times I am reminded why I love having > you in the reviewer/contributor pool ;-) I didn't actually test that patch beyond compilation (but it's _obviously_ correct, right?), and I'm about to go to bed. Do you want to take care of adapting your commit message to it? -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