csmulhern marked an inline comment as done. csmulhern added inline comments.
================ Comment at: clang/unittests/Format/FormatTest.cpp:22301 + auto Style = getLLVMStyle(); + Style.BreakBeforeClosingParen = true; + ---------------- HazardyKnusperkeks wrote: > Could you also add tests for `false`, even though they are spread over the > other test cases. That way the option can completely be tested with this case. Done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109557/new/ https://reviews.llvm.org/D109557 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
