Skip to content
Snippets Groups Projects

Initial provision of a proposal for a Clang-Tidy configuration file.

3 unresolved threads

The concrete values and checks can be discussed in the course of further development.

The program was tested solely for our own use cases, which might differ from yours.

Jupp Tscheak jupp.tscheak@daimler.com, Mercedes-Benz AG on behalf of Daimler TSS GmbH

https://github.com/Daimler/daimler-foss/blob/master/PROVIDER_INFORMATION.md

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
.clang-tidy 0 → 100644
54 - key: readability-identifier-naming.PrivateMemberSuffix
55 value: _
56 - key: readability-identifier-naming.ProtectedMemberSuffix
57 value: _
58 - key: readability-identifier-naming.StaticConstantCase
59 value: CamelCase
60 - key: readability-identifier-naming.StaticConstantPrefix
61 value: k
62 - key: readability-identifier-naming.StructCase
63 value: CamelCase
64 - key: readability-identifier-naming.TemplateParameterCase
65 value: CamelCase
66 - key: readability-identifier-naming.VariableCase
67 value: lower_case
68 - key: readability-magic-numbers.IgnoredFloatingPointValues
69 value: 1e-3;1e-6;
  • I'm not a fan of this. In opsimulation we try to use constants even for this such as EPSILON_GEOMETRY which makes the code (a) more readable and (b) easier to change. Do we have any use case right know, where we could not apply this here?

  • Author Developer

    You're right. IgnoredFloatingPointValues can be ignored :smiley_cat:

  • Please register or sign in to reply
  • René Paris
    René Paris @rparisha2 started a thread on commit 6af2f511
  • .clang-tidy 0 → 100644
    55 value: _
    56 - key: readability-identifier-naming.ProtectedMemberSuffix
    57 value: _
    58 - key: readability-identifier-naming.StaticConstantCase
    59 value: CamelCase
    60 - key: readability-identifier-naming.StaticConstantPrefix
    61 value: k
    62 - key: readability-identifier-naming.StructCase
    63 value: CamelCase
    64 - key: readability-identifier-naming.TemplateParameterCase
    65 value: CamelCase
    66 - key: readability-identifier-naming.VariableCase
    67 value: lower_case
    68 - key: readability-magic-numbers.IgnoredFloatingPointValues
    69 value: 1e-3;1e-6;
    70 - key: cppcoreguidelines-avoid-magic-numbers.IgnoredFloatingPointValues
  • René Paris
    René Paris @rparisha2 started a thread on commit 6af2f511
  • .clang-tidy 0 → 100644
    32 - key: readability-identifier-naming.ClassCase
    33 value: CamelCase
    34 - key: readability-identifier-naming.ConstexprVariableCase
    35 value: CamelCase
    36 - key: readability-identifier-naming.ConstexprVariablePrefix
    37 value: k
    38 - key: readability-identifier-naming.EnumConstantCase
    39 value: CamelCase
    40 - key: readability-identifier-naming.EnumConstantPrefix
    41 value: k
    42 - key: readability-identifier-naming.FunctionCase
    43 value: CamelCase
    44 - key: readability-identifier-naming.GlobalConstantCase
    45 value: CamelCase
    46 - key: readability-identifier-naming.GlobalConstantPrefix
    47 value: k
  • Martin Stump mentioned in issue #18

    mentioned in issue #18

  • René Paris mentioned in merge request !78 (merged)

    mentioned in merge request !78 (merged)

  • closed

  • Please register or sign in to reply
    Loading