Jeff King <peff@xxxxxxxx> writes: > On Sat, Nov 17, 2007 at 01:05:35PM -0500, Daniel Barkalow wrote: > >> > + if (ref->nonfastforward) { >> > + strcat(quickref, "..."); >> > + type = '+'; >> > + msg = " (forced update)"; >> > + } >> > + else { >> >> Coding style, IIRC. > > Sorry, I don't see the style nit you're mentioning here. I think Daniel is referring to "Put 'else' on the same line as the brace that closes the corresponding 'if' opened", iow: if (...) { ... } else { ... } - 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