Jeff King <peff@xxxxxxxx> writes: > Because this function is static and used only by the > http-walker, when NO_CURL is defined, gcc emits a "defined > but not used" warning. > > Signed-off-by: Jeff King <peff@xxxxxxxx> > --- > On master. I like to compile with -Werror to make sure I don't miss > warnings as the compile scrolls by. > > This fix feels a little wrong, since the function isn't specific to http > support, but hopefully the comment should be obvious if we ever add > another similar commit walker that needs it. Yeah, while I share your -Werror desire it sure feels a bit dirty. As it does not look like we will be taking any new walkers, I think your patch is reasonable, though. - 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
