[clang] Add configuration option PenaltyBreakBeforeMemberAccess (PR #118409)
Owen Pan via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 27 00:06:59 PST 2025
================
@@ -22365,6 +22365,19 @@ TEST_F(FormatTest, BreakPenaltyAfterForLoopLParen) {
Style);
}
+TEST_F(FormatTest, BreakPenaltyBeforeMemberAccess) {
+ FormatStyle Style = getLLVMStyle();
+ Style.ColumnLimit = 8;
+ Style.PenaltyExcessCharacter = 15;
+ verifyFormat("foo->bar\n"
+ " .b(a);",
+ Style);
+ Style.PenaltyBreakBeforeMemberAccess = 200;
+ verifyFormat("foo->bar.b(\n"
+ " a);",
----------------
owenca wrote:
```suggestion
auto Style = getLLVMStyle();
EXPECT_EQ(Style.PenaltyBreakBeforeMemberAccess, 150u);
Style.ColumnLimit = 60;
Style.PenaltyBreakBeforeMemberAccess = 110;
verifyFormat("aaaaaaaa.aaaaaaaa.bbbbbbbb()\n"
" .ccccccccccccccccccccc(dddddddd);\n"
"aaaaaaaa.aaaaaaaa\n"
" .bbbbbbbb(cccccccccccccccccccccccccccccccc);",
```
adapted from the examples at line 4306 and line 4316 in `TokenAnnotator.cpp`.
https://github.com/llvm/llvm-project/pull/118409
More information about the cfe-commits
mailing list