[PATCH] D20254: [Clang] Fix some Clang-tidy modernize-use-bool-literals warnings; other minor fixes.

Hans Wennborg via cfe-commits cfe-commits at lists.llvm.org
Mon May 16 08:55:38 PDT 2016


hans added a comment.

I'd leave out the blank lines between DEF_TRAVERSE_STMTs, but the rest looks good.


================
Comment at: include/clang/AST/RecursiveASTVisitor.h:2014
@@ -2002,2 +2013,3 @@
 DEF_TRAVERSE_STMT(ObjCAutoreleasePoolStmt, {})
+
 DEF_TRAVERSE_STMT(CXXForRangeStmt, {
----------------
I'm not sure about the blank lines in these DEF_TRAVERSE_STMTs. I don't really see what the logic is.


Repository:
  rL LLVM

http://reviews.llvm.org/D20254





More information about the cfe-commits mailing list