> On Jan 26, 2017, at 9:14 AM, Andrew Trick <[email protected]> wrote: > > >> On Jan 26, 2017, at 9:11 AM, Ben Langmuir <[email protected] >> <mailto:[email protected]>> wrote: >>> >>> ** Option 1: Add a simple configuration option to swift/.clang-format: >>> >>> 1a. BreakBeforeBinaryOperators: All >>> >>> 1b. BreakBeforeBinaryOperators: NonAssignment >> >>> >>> I have absolutely no preference between 1a and 1b. It's purely style. >>> >>> 1a: >>> SomeLongTypeName someLongVariableName = >>> someLongExpression(); >>> >>> 1b: >>> SomeLongTypeName someLongVariableName >>> = someLongExpression(); >> >> 1b sounds good to me. > > I contradicted myself above. If you like the style shown in (1b), the > configuration option is actually BreakBeforeBinaryOperators: All.
Glad you mentioned it, because I prefer “NonAssignment”, but didn’t check your example code against the above description :-) > -Andy
_______________________________________________ swift-dev mailing list [email protected] https://lists.swift.org/mailman/listinfo/swift-dev
