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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
