On Tue, Feb 12, 2013 at 10:55:05AM -0800, Junio C Hamano wrote: > Michal Nazarewicz <mpn@xxxxxxxxxx> writes: > > > From: Michal Nazarewicz <mina86@xxxxxxxxxx> > > > > The body of the loop in command_close_bidi_pipe function is identical to > > what _cmd_close function does so instead of duplicating, refactor change > > _cmd_close so that it accepts list of file handlers to be closed, which > > s/file handlers/file handles/, I think. And s/refactor change/refactor/. Other than that, I think the series looks OK. I have one style micro-nit on patch 4 which I'll reply in-line. But it is either "fix while applying" or "ignore", I don't think it will be worth a re-roll. -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