On Mon, 2014-08-04 at 23:47 +0200, Norbert Thiebaud wrote: > I guess.. but the point of a formatting rule is easy to read and above > all _visually consistent_. > There is apparently debate about the readability of the different > braces scheme.. but surely there can't be any argument that a mix of > both is the worse of both world.
FWIW I played around with clang-format to see what combination of rules gave the smallest diff against the code of sw/source/filter/ww8 and below is what I got as far as before abandoning that line of thought. Though I do find this helpful to strip out arbitrary formatting changes when cherry-picking commits between branches where the source commit has bundled a stack of formatting changes into a titchy logic change. C.
--- # BasedOnStyle: WebKit AccessModifierOffset: -4 ConstructorInitializerIndentWidth: 4 AlignEscapedNewlinesLeft: false AlignTrailingComments: false AllowAllParametersOfDeclarationOnNextLine: true AllowShortIfStatementsOnASingleLine: false # AllowShortFunctionsOnASingleLine: SFS_Inline AllowShortLoopsOnASingleLine: false AlwaysBreakTemplateDeclarations: false AlwaysBreakBeforeMultilineStrings: false BreakBeforeBinaryOperators: true BreakBeforeTernaryOperators: true BreakConstructorInitializersBeforeComma: true BinPackParameters: true ColumnLimit: 80 ConstructorInitializerAllOnOneLineOrOnePerLine: false DerivePointerBinding: false ExperimentalAutoDetectBinPacking: false IndentCaseLabels: true MaxEmptyLinesToKeep: 1 NamespaceIndentation: Inner ObjCSpaceBeforeProtocolList: true PenaltyBreakBeforeFirstCallParameter: 19 PenaltyBreakComment: 60 PenaltyBreakString: 1000 PenaltyBreakFirstLessLess: 120 PenaltyExcessCharacter: 1000000 PenaltyReturnTypeOnItsOwnLine: 60 PointerBindsToType: true SpacesBeforeTrailingComments: 1 Cpp11BracedListStyle: false Standard: Cpp03 IndentWidth: 4 TabWidth: 8 UseTab: Never BreakBeforeBraces: Allman IndentFunctionDeclarationAfterType: false SpacesInParentheses: false SpacesInAngles: false SpaceInEmptyParentheses: false SpacesInCStyleCastParentheses: false SpaceAfterControlStatementKeyword: true SpaceBeforeAssignmentOperators: true ContinuationIndentWidth: 4
_______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice