On 1/6/2014, 1:20 PM, Gregory Szorc wrote:
On 1/6/14, 7:12 AM, Ehsan Akhgari wrote:
With Birunthan's restless efforts in bug 784739, we have finally removed
the usage of NULL in our C++ code. Please stop using NULL in new C++
code,
and use nullptr instead!
Can we update the Clang plugin to emit an error (and turn the tree red)
if a NULL is detected in non-whitelisted file paths? It won't be
perfect, but it's better than nothing.
NULL is a preprocessor macro which means that by the time that the
semantic analysis phase is run, the compiler doesn't even see it. This
means that we need a new analysis phase on the lexer, which I think is
doable in a clang plugin (but Joshua can correct me if I'm wrong.)
More interesting to me would be for somebody to fix bug 778984.
Cheers,
Ehsan
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform
