[cfe-commits] [Patch] -Wformat: warn about using length modifiers with %n

[Ted Kremenek]

No worries.  Thanks for doing this.

On Aug 2, 2012, at 10:35 AM, Hans Wennborg <hans at chromium.org> wrote:

> On Thu, Aug 2, 2012 at 6:31 PM, Ted Kremenek <kremenek at apple.com> wrote:
>> C99 says you can use %n with the length modifiers 'hh', 'h', 'l', 'j', 'z', and 't'.
> 
> My apologies, apparently I can't read. I'll update the patch.
> 
> Thanks,
> Hans
> 
>> 
>> On Aug 1, 2012, at 6:43 AM, Hans Wennborg <hans at chromium.org> wrote:
>> 
>>> Hi all,
>>> 
>>> As far as I know, using any length modifier with %n is undefined
>>> behavior. For some reason, Clang doesn't think so, but that's probably
>>> unintentional? The attached patch makes Clang warn about it.
>>> 
>>> Please take a look.
>>> 
>>> Thanks,
>>> Hans
>>> <wformat_n_length_modifiers.patch>_______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>