[cfe-commits] r156092 - /cfe/trunk/lib/Sema/SemaChecking.cpp

Douglas Gregor dgregor at apple.com
Fri May 4 08:37:14 PDT 2012


On May 4, 2012, at 6:11 AM, Jean-Daniel Dupas <devlists at shadowlab.org> wrote:

> When I updated -Wformat to works with NSString methods, I had to include a similar workaround for NSLocalizedString.
> At that time, I wasn't aware of the existence of isInSystemMacro().
> 
> I wrote a patch to use this method instead of disabling the warning in all macros in my existing workaround, 
> and wrote a test that check for both NSLocalizedString and NSAssert cases.
> 
> If you're OK with this patch, I will commit it.

Patch looks good!

	- Doug



More information about the cfe-commits mailing list