[PATCH] D117520: [clang-format] Fix SeparateDefinitionBlocks issues

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 24 00:11:45 PST 2022


curdeius accepted this revision.
curdeius added a comment.

LGTM.



================
Comment at: clang/lib/Format/DefinitionBlockSeparator.cpp:141-145
+        if (NextLine->MightBeFunctionDecl &&
+            NextLine->mightBeFunctionDefinition())
+          if (NextLine->First->NewlinesBefore == 1 &&
+              OperateLine->First->is(TT_FunctionLikeOrFreestandingMacro))
+            return true;
----------------
Please merge these two `if`s into one.


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

https://reviews.llvm.org/D117520



More information about the cfe-commits mailing list