mboehme accepted this revision.
mboehme added a comment.
This revision is now accepted and ready to land.
LG apart from minor comments by others and me
================
Comment at: clang-tidy/misc/UseAfterMoveCheck.cpp:18
using namespace clang::ast_matchers;
+using namespace clang::tidy::utils;
+
----------------
Prazek wrote:
> Prazek wrote:
> > I don't think it is required
> ok I guess I am wrong
I would suggest instead adding an explicit "utils::" qualifier -- it's only
needed in two places anyway. I don't feel strongly about this though.
================
Comment at: clang-tidy/utils/ExprSequence.cpp:52
+
+bool isDescendantOrEqual(const Stmt *Descendant, const Stmt *Ancestor,
+ ASTContext *Context) {
----------------
Prazek wrote:
> staronj wrote:
> > Shouldn't isDescendantOrEqual be static or in inline namespace?
> Goot catch. I guess putting it with getParentStmts into anonymous namespace
> is the best solution.
> btw inline namespace is not the same as anonymous namespace :)
Thanks for the catch -- this was already a bug in the original code...
================
Comment at: clang-tidy/utils/ExprSequence.cpp:179
+
+
+}
----------------
Double newline
================
Comment at: clang-tidy/utils/ExprSequence.h:120
+
+
+}
----------------
Double newline
https://reviews.llvm.org/D27700
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
