Hi, On Sun, 29 Jun 2008, Junio C Hamano wrote: > Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: > > > Anyway, I'm fine with having them only in my personal fork. > > It will cost you some "trust point" next time you try to sneak something > in as a part of a largely unrelated topic. Please don't. Fine. Fine! I thought it made the code more readable, and that it also might have helped other sites where you have to strbuf_init() first, and often need curly brackets only because of that. Making for ultra-ugly code. Please note that I also added a simple test case for the vaddf() function, so your "trying to sneak through" is only half as bad as you made it sound. Also, please note that the situation with SNPRINTF_RETURNS_BOGUS works, but is a thorn in this developers eye. Anyway, I will not try to post this patch again, Dscho -- 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