Ramkumar Ramachandra wrote: > Actually, I intended to put it in the commit message. Seems to be > idiomatic: grep the log for "\[jc:" etc. More important than the idiom is the intent. Presumably that intent was something like "I wrote the commit message, so if it makes you scratch your head, blame me, not Jonathan; and I made some other (minor) improvements, so consider blaming me even if it's the functional part that makes you scratch your head." Sorry I haven't had a chance to look over the patch yet. Is it supposed to introduce a behavior change, does it prepare for some future change, or is it just a cleanup? -- 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