jiridanek commented on pull request #169: URL: https://github.com/apache/qpid-proton/pull/169#issuecomment-633959533
I messed with it for a while, and I found out I am not the correct person to propose a code style for the project. So, let me propose to adopt whatever dispatch ends up choosing, in https://github.com/apache/qpid-dispatch/pull/744/files#diff-88d5907f11b8b6ca0b213de568bf4582 Currently the proposal there is ``` BasedOnStyle: Google IndentWidth: 4 ColumnLimit: 120 AllowShortFunctionsOnASingleLine: false SpaceAfterCStyleCast: true DerivePointerAlignment: false PointerAlignment: Left AlignConsecutiveAssignments: true AlignConsecutiveDeclarations: true KeepEmptyLinesAtTheStartOfBlocks: false ``` On the other hand, my favorite settings include something like ``` BasedOnStyle: LLVM ExperimentalAutoDetectBinPacking: true ColumnLimit: 100 AccessModifierOffset: -4 AllowShortIfStatementsOnASingleLine: Always AlignConsecutiveDeclarations: true FixNamespaceComments: true NamespaceIndentation: None DerivePointerAlignment: false PointerAlignment: Right SpaceAfterCStyleCast: true ``` ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
