[PATCH] D30412: [clang-tidy] Fix a false positive on modernize-use-nullptr check.

2017-02-28 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL296479: [clang-tidy] Fix a false positive on modernize-use-nullptr check. (authored by hokein). Changed prior to commit: https://reviews.llvm.org/D30412?vs=89894&id=90032#toc Repository: rL LLVM htt

[PATCH] D30412: [clang-tidy] Fix a false positive on modernize-use-nullptr check.

2017-02-28 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG. Thanks for tracking this down! https://reviews.llvm.org/D30412 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/

[PATCH] D30412: [clang-tidy] Fix a false positive on modernize-use-nullptr check.

2017-02-27 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. Herald added a subscriber: JDevlieghere. The false positive happens on two neighbour CXXDefaultArgExpr AST nodes. like below: CXXFunctionalCastExpr 0x85c9670 'struct ZZ' functional cast to struct ZZ `-CXXConstructExpr 0x85c9518 'struct ZZ' 'void (uint64, const