On Sat, 26 Jul 2008, Miklos Vajna wrote: > + if (!prefix) > + prefix = "git-"; > + prefix_len = strlen(prefix); The whitespace gave me a start: the diff markup moved the prefix_len line to the next tab stop, so at first glance it seems there are missing braces here. But it is an illusion. (I mention this so others might avoid wasting time worrying about it.) I like the patch so far. Thanks for the pleasant reading. -- 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