Thanks everyone. I should have tested a later version of clang before I sent the patch. It might still be worthwhile to munge the flags for old clangs so that git doesn't error out on it, but if 1.0 doesn't make it into some major OS release and gcc remains the default it's not much of an issue. On Tue, Aug 3, 2010 at 14:23, Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> wrote: > Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> writes: > >> (other than that, it's cool to see someone testing another >> compiler ;-) ) > > BTW, the only warnings remaining with -Wall with my clang are: > > imap-send.c:548:27: warning: data argument not used by format string [-Wformat-extra-args] > cmd->tag, cmd->cmd, cmd->cb.dlen); > ^ I didn't look into that one. It'd be usefu to check out if the format string is really incomplete there, or if clang is just failing it its analysis. That's some hairy code, in any case. > imap-send.c:1089:41: warning: conversion specifies type 'unsigned short' but the argument has type 'int' [-Wformat] > snprintf(portstr, sizeof(portstr), "%hu", srvc->port); > ~~^ ~~~~~~~~~~ > 2 diagnostics generated. Looks like that needs a cast. -- 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