[PATCH] D145642: [clang-format] Annotate lambdas with requires clauses.

Emilia Dreamer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 9 06:37:04 PST 2023


rymiel marked an inline comment as done.
rymiel added inline comments.


================
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:3393
+        break;
+      } else {
+        return;
----------------
HazardyKnusperkeks wrote:
> don't need `else` after `break`.
> In fact I would negate the condition and return, and keep the remainder out of any `if` or `else`.
Thank you! I clearly wasn't thinking clearly there


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145642



More information about the cfe-commits mailing list