Re: [RFC/PATCH] git-compat-util.h: Don't define NORETURN under __clang__

["Ævar Arnfjörð Bjarmason" <avarab@gmail.com>]

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@grenoble-inp.fr> wrote:
> Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> 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.