[PATCH] D26301: [clang-tidy] Fix a regression issue introduced by r285239.

2016-11-07 Thread Haojian Wu via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL286156: [clang-tidy] Fix a regression issue introduced by r285239. (authored by hokein). Changed prior to commit: https://reviews.llvm.org/D26301?vs=76920&id=77096#toc Repository: rL LLVM https://re

[PATCH] D26301: [clang-tidy] Fix a regression issue introduced by r285239.

2016-11-07 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. Thank you for tracking and fixing this! One nit inline. Comment at: clang-tidy/modernize/UseNullptrCheck.cpp:194 +// Catch the castExpr inside cxxDefaultArgExpr. +if

[PATCH] D26301: [clang-tidy] Fix a regression issue introduced by r285239.

2016-11-04 Thread Haojian Wu via cfe-commits
hokein created this revision. hokein added a reviewer: alexfh. hokein added a subscriber: cfe-commits. r285239 changes the behavior of AST CXXDefaultArgExpr node. Update `modernize-use-nullptr` to handle CXXDefaultArgExpr correctly. https://reviews.llvm.org/D26301 Files: clang-tidy/modernize