MyDeveloperDay added inline comments.
================ Comment at: clang/lib/Format/Format.cpp:1484 LLVMStyle.PenaltyBreakFirstLessLess = 120; + LLVMStyle.PenaltyBreakReturn = 100; LLVMStyle.PenaltyBreakString = 1000; ---------------- is 100 the current penalty? ================ Comment at: clang/unittests/Format/FormatTest.cpp:22187 + " HereAVeryLongLineThatWillBeFormattedOnMultipleLineAnd" + "ShouldNotBeConsiderAsInline;\n" " });", ---------------- the fails the beyonce rule doesn't it, unless you are assuming the test is a bug? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152975/new/ https://reviews.llvm.org/D152975 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits