Re: [PATCH] D20909: [clang-tidy] Ignore function context in misc-unused-using-decls.

2016-06-03 Thread Haojian Wu via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL271632: [clang-tidy] Ignore function context in misc-unused-using-decls. (authored by hokein). Changed prior to commit: http://reviews.llvm.org/D20909?vs=59492&id=59494#toc Repository: rL LLVM http:

Re: [PATCH] D20909: [clang-tidy] Ignore function context in misc-unused-using-decls.

2016-06-03 Thread Haojian Wu via cfe-commits
hokein updated this revision to Diff 59492. hokein added a comment. Address code comments. http://reviews.llvm.org/D20909 Files: clang-tidy/misc/UnusedUsingDeclsCheck.cpp clang-tidy/misc/UnusedUsingDeclsCheck.h test/clang-tidy/misc-unused-using-decls.cpp Index: test/clang-tidy/misc-unuse

Re: [PATCH] D20909: [clang-tidy] Ignore function context in misc-unused-using-decls.

2016-06-02 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:109 @@ -100,1 +108,3 @@ + // giving an incorrect message, we mark all of them as used. for (auto &Context : Contex

[PATCH] D20909: [clang-tidy] Ignore function context in misc-unused-using-decls.

2016-06-02 Thread Haojian Wu via cfe-commits
hokein created this revision. hokein added a reviewer: alexfh. hokein added a subscriber: cfe-commits. Make the check's behavior more correct when handling using-decls in multiple scopes. http://reviews.llvm.org/D20909 Files: clang-tidy/misc/UnusedUsingDeclsCheck.cpp clang-tidy/misc/UnusedU