[cfe-commits] [PATCH] Teach printf warnings about small integer types.

[Ted Kremenek]

Hi Justin,

I went ahead applied this patch (with a few modifications) here:

  http://llvm.org/viewvc/llvm-project?view=rev&revision=116996

I mainly incorporated my minor nits I commented on in my last email.  I also needed to modify the test case slightly to use 'typedef' instead of '#define'.  Even though they might appear to be correct, please make sure that the test cases actually pass before submitting patches.

Cheers,
Ted

On Oct 2, 2010, at 12:49 AM, Justin Bogner wrote:

> Previously, the printf warnings would say your arguments type was
> `int` when it was really a `char` or a `short`. This fixes that and
> allows the hints to suggest `h` modifiers for small ints.
> 
> Typedef'd `char` types are treated as ints, since the most common case
> of uint8_t shouldn't normally be formatted as a character.
> ---
> lib/Analysis/PrintfFormatString.cpp |   16 +++++++++++++++-
> lib/Sema/SemaChecking.cpp           |    6 ++++--
> test/Sema/format-strings.c          |   11 ++++++++++-
> 3 files changed, 29 insertions(+), 4 deletions(-)
> 
> <informative-printf.patch>_______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits