Re: [PATCH] D19819: [clang-tidy] Cleanup namespace in utils folder.

2016-05-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. Thank you! http://reviews.llvm.org/D19819 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listi

Re: [PATCH] D19819: [clang-tidy] Cleanup namespace in utils folder.

2016-05-02 Thread Etienne Bergeron via cfe-commits
etienneb updated this revision to Diff 55876. etienneb added a comment. fix unittests http://reviews.llvm.org/D19819 Files: clang-tidy/cppcoreguidelines/ProBoundsConstantArrayIndexCheck.cpp clang-tidy/cppcoreguidelines/ProBoundsConstantArrayIndexCheck.h clang-tidy/cppcoreguidelines/ProTyp

Re: [PATCH] D19819: [clang-tidy] Cleanup namespace in utils folder.

2016-05-02 Thread Etienne Bergeron via cfe-commits
etienneb updated this revision to Diff 55858. etienneb added a comment. add missing case http://reviews.llvm.org/D19819 Files: clang-tidy/cppcoreguidelines/ProBoundsConstantArrayIndexCheck.cpp clang-tidy/cppcoreguidelines/ProBoundsConstantArrayIndexCheck.h clang-tidy/cppcoreguidelines/Pro

[PATCH] D19819: [clang-tidy] Cleanup namespace in utils folder.

2016-05-02 Thread Etienne Bergeron via cfe-commits
etienneb created this revision. etienneb added a reviewer: alexfh. etienneb added a subscriber: cfe-commits. This is a step forward cleaning up the namespaces in clang-tidy/utils. There is no behavior change. http://reviews.llvm.org/D19819 Files: clang-tidy/cppcoreguidelines/ProBoundsConstantA