[PATCH] D26125: [clang-tidy] Fixed else-after-return warning in cascade if statement

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 30 07:39:44 PDT 2016


djasper added inline comments.


================
Comment at: clang-tidy/readability/ElseAfterReturnCheck.cpp:28
       stmt(forEach(
-          ifStmt(hasThen(stmt(
+          ifStmt(unless(hasParent(ifStmt())),
+                 hasThen(stmt(
----------------
I think this now effectively does:

  stmt(forEach(ifStmt(unless(hasParent(ifSttmt())), ...)

I think that's equivalent to:

  stmt(unless(ifStmt()), forEach(ifStmt(...)))


https://reviews.llvm.org/D26125





More information about the cfe-commits mailing list