owenpan added inline comments.
================ Comment at: clang/unittests/Format/FormatTest.cpp:3213 Style.BraceWrapping.AfterFunction = true; - Style.BraceWrapping.AfterControlStatement = FormatStyle::BWACS_MultiLine; Style.AllowShortFunctionsOnASingleLine = FormatStyle::SFS_All; ---------------- curdeius wrote: > This seems unrelated, is it so really? It's redundant but unrelated. We can delete it in a separate patch when replacing every `EXPECT_EQ` with a `verifyFormat`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125959/new/ https://reviews.llvm.org/D125959 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
