[PATCH] D35078: [clang-tidy] Fix modernize-use-override incorrect replacement

2017-07-06 Thread Victor Gao via Phabricator via cfe-commits
vgao created this revision. Herald added subscribers: xazax.hun, JDevlieghere. For the following code: `modernize-use-override` generates a replacement with incorrect location. struct IntPair { int first, second; }; struct A { virtual void il(IntPair); }; struct B : A

[PATCH] D35078: [clang-tidy] Fix modernize-use-override incorrect replacement

2017-07-06 Thread Victor Gao via Phabricator via cfe-commits
vgao updated this revision to Diff 105554. vgao added a comment. Change variable name `Parens` to `NestedParens` https://reviews.llvm.org/D35078 Files: clang-tidy/modernize/UseOverrideCheck.cpp test/clang-tidy/modernize-use-override.cpp Index: test/clang-tidy/modernize-use-override.cpp ==

[PATCH] D35078: [clang-tidy] Fix modernize-use-override incorrect replacement

2017-07-06 Thread Victor Gao via Phabricator via cfe-commits
vgao added a comment. @alexfh Could you please land this for me? Thanks! https://reviews.llvm.org/D35078 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits