On Fri, Nov 09, 2007 at 01:49:42AM +0100, René Scharfe wrote: > + strbuf_expand(sb, format, placeholders, format_commit_item, (void *)commit); This void cast is pointless, since all pointers types convert implicitly to void pointers anyway. At best, it does nothing, and at worst, it hides an actual type error if the function signature or the type of 'commit' change. (In the patch I just sent out, I had to change this line anyway, and removed the cast). -Peff - 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
