[PATCH] D33103: [clang-tidy] TwineLocalCheck: add param # checking

2017-05-12 Thread Benjamin Kramer via Phabricator via cfe-commits
bkramer accepted this revision. bkramer added a comment. This revision is now accepted and ready to land. looks good, thanks! https://reviews.llvm.org/D33103 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailm

[PATCH] D33103: [clang-tidy] TwineLocalCheck: add param # checking

2017-05-11 Thread Chih-Hung Hsieh via Phabricator via cfe-commits
chh added a comment. LGTM. Leave approval to bkramer. https://reviews.llvm.org/D33103 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D33103: [clang-tidy] TwineLocalCheck: add param # checking

2017-05-11 Thread Yan Wang via Phabricator via cfe-commits
yawanng updated this revision to Diff 98705. yawanng added a comment. Add more tests. https://reviews.llvm.org/D33103 Files: clang-tidy/llvm/TwineLocalCheck.cpp test/clang-tidy/llvm-twine-local.cpp Index: test/clang-tidy/llvm-twine-local.cpp ===

[PATCH] D33103: [clang-tidy] TwineLocalCheck: add param # checking

2017-05-11 Thread Yan Wang via Phabricator via cfe-commits
yawanng updated this revision to Diff 98684. https://reviews.llvm.org/D33103 Files: clang-tidy/llvm/TwineLocalCheck.cpp test/clang-tidy/llvm-twine-local.cpp Index: test/clang-tidy/llvm-twine-local.cpp === --- test/clang-tidy/ll