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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
