[clang] [clang-format] Add BinPackBinaryOperations configuration (PR #95013)
Ameer J via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 10 09:00:17 PDT 2024
================
@@ -146,6 +146,14 @@ static bool startsNextParameter(const FormatToken &Current,
Style.BreakInheritanceList != FormatStyle::BILS_BeforeComma));
}
+// Returns \c true if \c Current starts a new operand in a binary operation.
----------------
ameerj wrote:
The wording was meant to match the comment for `startsNextParameter`, but I changed the wording to clarify a bit.
In a case like this: `op1 * (op2 + op3)`
The function would return true for `(` since that's what starts the next operand for the `*` operation.
It would be a bit inaccurate to say `(` "`is the right operand of a binary operator`"
https://github.com/llvm/llvm-project/pull/95013
More information about the cfe-commits
mailing list