[PATCH] D136154: [clang-format] Fix the continuation indenter
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 19 08:02:55 PDT 2022
MyDeveloperDay added inline comments.
================
Comment at: clang/lib/Format/ContinuationIndenter.cpp:814
+ if (Style.BreakBeforeBinaryOperators == FormatStyle::BOS_None ||
+ Style.BreakBeforeBinaryOperators == FormatStyle::BOS_NonAssignment) {
CurrentState.LastSpace = State.Column;
----------------
ok so now you don't actually have a test for this, so I don't know if this is needed any more.
================
Comment at: clang/unittests/Format/FormatTest.cpp:6602
+ Style.ContinuationIndentWidth = 2;
+ Style.BreakBeforeBinaryOperators = FormatStyle::BOS_NonAssignment;
+ verifyFormat(
----------------
I don't think you need this line
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136154/new/
https://reviews.llvm.org/D136154
More information about the cfe-commits
mailing list