[clang] [clang-format] Add BinPackBinaryOperations configuration (PR #95013)

Owen Pan via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 10 00:39:47 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.
+static bool startsNextOperand(const FormatToken &Current) {
+  const FormatToken &Previous = *Current.Previous;
+  return Previous.is(TT_BinaryOperator) && !Current.isTrailingComment() &&
+         (Previous.getPrecedence() > prec::Conditional) &&
+         (Previous.getPrecedence() < prec::PointerToMember);
----------------
owenca wrote:

Please remove the redundant parens.

https://github.com/llvm/llvm-project/pull/95013


More information about the cfe-commits mailing list