[cfe-commits] [patch] Let NULL and MSVC headers coexist
Eli Friedman
eli.friedman at gmail.com
Tue Apr 24 14:06:59 PDT 2012
On Tue, Apr 24, 2012 at 1:49 PM, Nico Weber <thakis at chromium.org> wrote:
> Hi,
>
> the attached patch fixes both issues mentioned in
> http://llvm.org/PR12146 . Like suggested by Eli in that bug, it does
> this by defining NULL to 0 instead of __null on windows. I think it
> would be nice to have a __clang_null, but this patch fixes the
> immediate issues I'm seeing without me having to argue with Eli :-)
> Maybe I'll do that in a follow-up patch.
>
> Ok?
Not exactly sure what you did to make the test show up twice in the patch...
Are you sure we want to do this for __MINGW32__? I don't know of
anyone having issues here with MinGW.
Otherwise, the patch is fine.
-Eli
More information about the cfe-commits
mailing list