On 1/6/2014, 2:56 PM, Chris Peterson wrote:
On 1/6/14, 11:37 AM, Gregory Szorc wrote:
Both are probably good clang-analyser plugins to write.
http://clang-analyzer.llvm.org/
True. Although I don't believe we run clang-analyzer in automation...
yet.
I don't think the benefit of replacing NULL with nullptr is worth the
developer and infrastructure cost of writing clang static analysis
plugins. Perhaps we can just update the Mozilla Coding Style and then
the patch lint scripts can just grep for "NULL"?
I mostly agree, but if somebody went through the pain of adding such
analysis to our clang-plugin, I'd take that patch!
Ehsan
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform
