[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