Skip to content
  • Dr. David Alan Gilbert's avatar
    7387863d
    virtiofsd: Format imported files to qemu style · 7387863d
    Dr. David Alan Gilbert authored
    
    
    Mostly using a set like:
    
    indent -nut -i 4 -nlp -br -cs -ce --no-space-after-function-call-names file
    clang-format -style=file -i -- file
    clang-tidy -fix-errors -checks=readability-braces-around-statements file
    clang-format -style=file -i -- file
    
    With manual cleanups.
    
    The .clang-format used is below.
    
    Signed-off-by: default avatarDr. David Alan Gilbert <dgilbert@redhat.com>
    Reviewed-by: default avatarDaniel P. Berrangé <berrange@redhat.com>
    Reviewed by: Aleksandar Markovic <amarkovic@wavecomp.com>
    
    Language:        Cpp
    AlignAfterOpenBracket: Align
    AlignConsecutiveAssignments: false # although we like it, it creates churn
    AlignConsecutiveDeclarations: false
    AlignEscapedNewlinesLeft: true
    AlignOperands:   true
    AlignTrailingComments: false # churn
    AllowAllParametersOfDeclarationOnNextLine: true
    AllowShortBlocksOnASingleLine: false
    AllowShortCaseLabelsOnASingleLine: false
    AllowShortFunctionsOnASingleLine: None
    AllowShortIfStatementsOnASingleLine: false
    AllowShortLoopsOnASingleLine: false
    AlwaysBreakAfterReturnType: None # AlwaysBreakAfterDefinitionReturnType is taken into account
    AlwaysBreakBeforeMultilineStrings: false
    BinPackArguments: true
    BinPackParameters: true
    BraceWrapping:
      AfterControlStatement: false
      AfterEnum:       false
      AfterFunction:   true
      AfterStruct:     false
      AfterUnion:      false
      BeforeElse:      false
      IndentBraces:    false
    BreakBeforeBinaryOperators: None
    BreakBeforeBraces: Custom
    BreakBeforeTernaryOperators: false
    BreakStringLiterals: true
    ColumnLimit:     80
    ContinuationIndentWidth: 4
    Cpp11BracedListStyle: false
    DerivePointerAlignment: false
    DisableFormat:   false
    ForEachMacros:   [
      'CPU_FOREACH',
      'CPU_FOREACH_REVERSE',
      'CPU_FOREACH_SAFE',
      'IOMMU_NOTIFIER_FOREACH',
      'QLIST_FOREACH',
      'QLIST_FOREACH_ENTRY',
      'QLIST_FOREACH_RCU',
      'QLIST_FOREACH_SAFE',
      'QLIST_FOREACH_SAFE_RCU',
      'QSIMPLEQ_FOREACH',
      'QSIMPLEQ_FOREACH_SAFE',
      'QSLIST_FOREACH',
      'QSLIST_FOREACH_SAFE',
      'QTAILQ_FOREACH',
      'QTAILQ_FOREACH_REVERSE',
      'QTAILQ_FOREACH_SAFE',
      'QTAILQ_RAW_FOREACH',
      'RAMBLOCK_FOREACH'
    ]
    IncludeCategories:
      - Regex:           '^"qemu/osdep.h'
        Priority:        -3
      - Regex:           '^"(block|chardev|crypto|disas|exec|fpu|hw|io|libdecnumber|migration|monitor|net|qapi|qemu|qom|standard-headers|sysemu|ui)/'
        Priority:        -2
      - Regex:           '^"(elf.h|qemu-common.h|glib-compat.h|qemu-io.h|trace-tcg.h)'
        Priority:        -1
      - Regex:           '.*'
        Priority:        1
    IncludeIsMainRegex: '$'
    IndentCaseLabels: false
    IndentWidth:     4
    IndentWrappedFunctionNames: false
    KeepEmptyLinesAtTheStartOfBlocks: false
    MacroBlockBegin: '.*_BEGIN$' # only PREC_BEGIN ?
    MacroBlockEnd:   '.*_END$'
    MaxEmptyLinesToKeep: 2
    PointerAlignment: Right
    ReflowComments:  true
    SortIncludes:    true
    SpaceAfterCStyleCast: false
    SpaceBeforeAssignmentOperators: true
    SpaceBeforeParens: ControlStatements
    SpaceInEmptyParentheses: false
    SpacesBeforeTrailingComments: 1
    SpacesInContainerLiterals: true
    SpacesInParentheses: false
    SpacesInSquareBrackets: false
    Standard:        Auto
    UseTab:          Never
    ...
    
    Signed-off-by: default avatarDr. David Alan Gilbert <dgilbert@redhat.com>
    7387863d
    virtiofsd: Format imported files to qemu style
    Dr. David Alan Gilbert authored
    
    
    Mostly using a set like:
    
    indent -nut -i 4 -nlp -br -cs -ce --no-space-after-function-call-names file
    clang-format -style=file -i -- file
    clang-tidy -fix-errors -checks=readability-braces-around-statements file
    clang-format -style=file -i -- file
    
    With manual cleanups.
    
    The .clang-format used is below.
    
    Signed-off-by: default avatarDr. David Alan Gilbert <dgilbert@redhat.com>
    Reviewed-by: default avatarDaniel P. Berrangé <berrange@redhat.com>
    Reviewed by: Aleksandar Markovic <amarkovic@wavecomp.com>
    
    Language:        Cpp
    AlignAfterOpenBracket: Align
    AlignConsecutiveAssignments: false # although we like it, it creates churn
    AlignConsecutiveDeclarations: false
    AlignEscapedNewlinesLeft: true
    AlignOperands:   true
    AlignTrailingComments: false # churn
    AllowAllParametersOfDeclarationOnNextLine: true
    AllowShortBlocksOnASingleLine: false
    AllowShortCaseLabelsOnASingleLine: false
    AllowShortFunctionsOnASingleLine: None
    AllowShortIfStatementsOnASingleLine: false
    AllowShortLoopsOnASingleLine: false
    AlwaysBreakAfterReturnType: None # AlwaysBreakAfterDefinitionReturnType is taken into account
    AlwaysBreakBeforeMultilineStrings: false
    BinPackArguments: true
    BinPackParameters: true
    BraceWrapping:
      AfterControlStatement: false
      AfterEnum:       false
      AfterFunction:   true
      AfterStruct:     false
      AfterUnion:      false
      BeforeElse:      false
      IndentBraces:    false
    BreakBeforeBinaryOperators: None
    BreakBeforeBraces: Custom
    BreakBeforeTernaryOperators: false
    BreakStringLiterals: true
    ColumnLimit:     80
    ContinuationIndentWidth: 4
    Cpp11BracedListStyle: false
    DerivePointerAlignment: false
    DisableFormat:   false
    ForEachMacros:   [
      'CPU_FOREACH',
      'CPU_FOREACH_REVERSE',
      'CPU_FOREACH_SAFE',
      'IOMMU_NOTIFIER_FOREACH',
      'QLIST_FOREACH',
      'QLIST_FOREACH_ENTRY',
      'QLIST_FOREACH_RCU',
      'QLIST_FOREACH_SAFE',
      'QLIST_FOREACH_SAFE_RCU',
      'QSIMPLEQ_FOREACH',
      'QSIMPLEQ_FOREACH_SAFE',
      'QSLIST_FOREACH',
      'QSLIST_FOREACH_SAFE',
      'QTAILQ_FOREACH',
      'QTAILQ_FOREACH_REVERSE',
      'QTAILQ_FOREACH_SAFE',
      'QTAILQ_RAW_FOREACH',
      'RAMBLOCK_FOREACH'
    ]
    IncludeCategories:
      - Regex:           '^"qemu/osdep.h'
        Priority:        -3
      - Regex:           '^"(block|chardev|crypto|disas|exec|fpu|hw|io|libdecnumber|migration|monitor|net|qapi|qemu|qom|standard-headers|sysemu|ui)/'
        Priority:        -2
      - Regex:           '^"(elf.h|qemu-common.h|glib-compat.h|qemu-io.h|trace-tcg.h)'
        Priority:        -1
      - Regex:           '.*'
        Priority:        1
    IncludeIsMainRegex: '$'
    IndentCaseLabels: false
    IndentWidth:     4
    IndentWrappedFunctionNames: false
    KeepEmptyLinesAtTheStartOfBlocks: false
    MacroBlockBegin: '.*_BEGIN$' # only PREC_BEGIN ?
    MacroBlockEnd:   '.*_END$'
    MaxEmptyLinesToKeep: 2
    PointerAlignment: Right
    ReflowComments:  true
    SortIncludes:    true
    SpaceAfterCStyleCast: false
    SpaceBeforeAssignmentOperators: true
    SpaceBeforeParens: ControlStatements
    SpaceInEmptyParentheses: false
    SpacesBeforeTrailingComments: 1
    SpacesInContainerLiterals: true
    SpacesInParentheses: false
    SpacesInSquareBrackets: false
    Standard:        Auto
    UseTab:          Never
    ...
    
    Signed-off-by: default avatarDr. David Alan Gilbert <dgilbert@redhat.com>
Loading