[PATCH] D27700: [clang-tidy] refactor ExprSequence out of misc-use-after-move check

Piotr Padlewski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 13 03:04:36 PST 2016


Prazek added inline comments.


================
Comment at: clang-tidy/misc/UseAfterMoveCheck.cpp:18
 using namespace clang::ast_matchers;
+using namespace clang::tidy::utils;
+
----------------
I don't think it is required


================
Comment at: clang-tidy/utils/ExprSequence.cpp:180-182
+}
+}
+}
----------------
same here


================
Comment at: clang-tidy/utils/ExprSequence.h:121-123
+}
+}
+}
----------------
Run clang-tidy llvm checks on this patch. These braces requires comments like 
  // namespace clang


https://reviews.llvm.org/D27700





More information about the cfe-commits mailing list