tmroeder marked 2 inline comments as done. tmroeder added a comment. In D59963#1447985 <https://reviews.llvm.org/D59963#1447985>, @alexfh wrote:
> Looks reasonable in general, but we usually add modules with at least one > check. Let's do the same here. OK, will do. ================ Comment at: clang-tools-extra/clang-tidy/linux/LinuxTidyModule.cpp:13 + +using namespace clang::ast_matchers; + ---------------- lebedev.ri wrote: > You don't need this here. Thanks. I've removed it now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59963/new/ https://reviews.llvm.org/D59963 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
