[cfe-commits] r150128 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaDecl.cpp test/SemaCXX/function-extern-c.cpp

Rafael Espíndola rafael.espindola at gmail.com
Thu Feb 16 18:11:34 PST 2012


> I don't think it's been finding false positives, actually.  I think
> the real problem is that there's no way for the programmer to specify
> their intentions.

One false positive was in a (strange, ABI dependent) code in firefox.
The javascript engine used to be C and still has a C interface. The
actual implementation is c++. The basic "javascript value" type is
declared as a C or C++ type with the same layout.

The code should probably be changed to use a C type on the interfaces
and have a implicitly constructed c++ type that wraps it, but that
would be a massive change to the codebase, so I had to disable this
warning for us too.

>
> ~Aaron
>

Cheers,
Rafael




More information about the cfe-commits mailing list