[PATCH] D137762: [clang-format] avoid breaking )( with BlockIndent

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 10 02:45:14 PST 2022


MyDeveloperDay added a comment.

Definitely needs tests



================
Comment at: clang/lib/Format/ContinuationIndenter.cpp:720
       (!Previous.Previous || !Previous.Previous->isOneOf(
-                                 tok::kw_for, tok::kw_while, tok::kw_switch)) &&
+                  tok::r_paren,  tok::kw_for, tok::kw_while, tok::kw_switch)) &&
       // Don't do this for simple (no expressions) one-argument function calls
----------------
this would likely have been added for a reason, did this not break any tests?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137762/new/

https://reviews.llvm.org/D137762



More information about the cfe-commits mailing list