Michael Haggerty <mhagger@xxxxxxxxxxxx> writes: > On 04/27/2012 01:25 AM, Junio C Hamano wrote: >> Please write this like this: >> >> if (...) { >> ; /* silently ignore */ >> } >> >> to make the "emptyness" stand out (I amended the previous round when I >> queued them to 'pu', but I forgot to point it out in my review message). > > OK. A similar construct is in patch 2 of the same series. I've fixed > them in my repo and will use the fixed versions if there are any > future re-rolls. OK. Has there been any re-roll I missed, or is what I have in 'pu' with fixups ready for 'next'? -- 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