[cfe-commits] r148324 - in /cfe/trunk: include/clang/Sema/Sema.h lib/Sema/SemaChecking.cpp lib/Sema/SemaExprObjC.cpp test/SemaObjC/format-strings-objc.m

[Hans Wennborg]

On Mon, Jan 30, 2012 at 4:37 PM, Nico Weber <thakis at chromium.org> wrote:
> Since %S can expect utf32 (lib) or utf16 (AppKit), the %S warning is
> currently fairly useless. %S seems to be an Apple extension, so should
> we just hardcode %S to look for a 32bit type in c strings and for a
> 16bit type in @"" strings? Should we disable the warning for %S
> altogether?

Scary.

I'm not sure that it's really an Apple extension (though I've seen
comments that say that in the code). "man 3 printf" on my Ubuntu
machine says "S  (Not in C99, but in SUSv2.)  Synonym for ls.  Don't
use."