On Mon, Dec 08, 2014 at 03:22:06AM -0500, Jeff King wrote: > /* Print bidirectional transfer loop debug message. */ > __attribute__((format (printf, 1, 2))) > static void transfer_debug(const char *fmt, ...) > { > va_list args; > char msgbuf[PBUFFERSIZE]; > - static int debug_enabled = -1; > > - if (debug_enabled < 0) > - debug_enabled = getenv("GIT_TRANSLOOP_DEBUG") ? 1 : 0; > - if (!debug_enabled) > + if (!transport_debug_enabled) > return; I think in my cover letter I made clear that this was not really meant for inclusion as-is, but just in case: this obviously is missing "()" at the end of the function call (interestingly gcc -Wall complains, but clang does not). -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