[cfe-commits] [Patch] -Wformat fix-its: preserve original conversion specifiers (PR11975)
Ted Kremenek
kremenek at apple.com
Tue Feb 14 13:49:30 PST 2012
Looks great to me.
On Feb 14, 2012, at 7:43 AM, Hans Wennborg <hans at chromium.org> wrote:
> Hi all,
>
> The attached patch makes PrintfSpecifier::fixType() and
> ScanfSpecifier::fixType() only fix the conversion specification enough
> that Clang wouldn't warn about it, as opposed to always changing it to
> use the "canonical" conversion specifier. (PR11975)
>
> This preserves the user's choice of conversion specifier in cases like:
>
> printf("%a", (long double)1); where we previously suggested "%Lf", we
> now suggest "%La"
>
> and
>
> printf("%x", (long)1); where we previously suggested "%ld", we now
> suggest "%lx".
>
>
> Please take a look.
>
> Thanks,
> Hans
> <preserve-conversion-specifiers.diff>
More information about the cfe-commits
mailing list