Re: [PATCH] D18274: [clang-tidy] Add boost module

2016-04-29 Thread Piotr Padlewski via cfe-commits
Prazek abandoned this revision. Prazek added a comment. Merged with boost-use-to-string http://reviews.llvm.org/D18274 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D18274: [clang-tidy] Add boost module

2016-04-27 Thread Piotr Padlewski via cfe-commits
Prazek updated this revision to Diff 55287. http://reviews.llvm.org/D18274 Files: clang-tidy/CMakeLists.txt clang-tidy/boost/BoostTidyModule.cpp clang-tidy/boost/CMakeLists.txt clang-tidy/plugin/CMakeLists.txt clang-tidy/tool/CMakeLists.txt clang-tidy/tool/ClangTidyMain.cpp docs/Rel

Re: [PATCH] D18274: [clang-tidy] Add boost module

2016-03-29 Thread Piotr Padlewski via cfe-commits
Prazek added a comment. I was planning to do so. Can you look at http://reviews.llvm.org/D18136 ? http://reviews.llvm.org/D18274 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D18274: [clang-tidy] Add boost module

2016-03-24 Thread Alexander Kornienko via cfe-commits
alexfh requested changes to this revision. alexfh added a comment. This revision now requires changes to proceed. I would add the module together with the first check in it, since an empty module doesn't make sense. http://reviews.llvm.org/D18274 _

Re: [PATCH] D18274: [clang-tidy] Add boost module

2016-03-20 Thread Piotr Padlewski via cfe-commits
Prazek updated this revision to Diff 51087. http://reviews.llvm.org/D18274 Files: clang-tidy/CMakeLists.txt clang-tidy/boost/BoostTidyModule.cpp clang-tidy/boost/CMakeLists.txt clang-tidy/plugin/CMakeLists.txt clang-tidy/tool/CMakeLists.txt clang-tidy/tool/ClangTidyMain.cpp docs/cla

Re: [PATCH] D18274: [clang-tidy] Add boost module

2016-03-19 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko added a subscriber: Eugene.Zelenko. Eugene.Zelenko added a comment. I think you also need to add new module to clang-tidy/plugin/CMakeLists.txt. http://reviews.llvm.org/D18274 ___ cfe-commits mailing list cfe-commits@lists.llvm.org ht

Re: [PATCH] D18274: [clang-tidy] Add boost module

2016-03-19 Thread Piotr Padlewski via cfe-commits
Prazek updated this revision to Diff 51086. http://reviews.llvm.org/D18274 Files: clang-tidy/CMakeLists.txt clang-tidy/boost/BoostTidyModule.cpp clang-tidy/boost/CMakeLists.txt clang-tidy/plugin/CMakeLists.txt clang-tidy/tool/CMakeLists.txt clang-tidy/tool/ClangTidyMain.cpp docs/cla