Fixes for clang-tidy-9
[alexxy/gromacs.git] / src / .clang-tidy
1 # List of rationales for check suppressions (where known).
2 # This have to precede the list because inline comments are not
3 # supported by clang-tidy.
4 #
5 #         -cppcoreguidelines-non-private-member-variables-in-classes,
6 #         -misc-non-private-member-variables-in-classes,
7 # We intend a gradual transition to conform to this guideline, but it
8 # is not practical to implement yet.
9 #
10 #         -readability-isolate-declaration,
11 # Declarations like "int a, b;" are readable. Some forms are not, and
12 # those might reasonably be suggested against during code review.
13 #
14 #         -cppcoreguidelines-avoid-c-arrays,
15 # C arrays are still necessary in many places with legacy code
16 #
17 #         -cppcoreguidelines-avoid-magic-numbers,
18 #         -readability-magic-numbers,
19 # We have many legitimate use cases for magic numbers
20 #
21 #         -cppcoreguidelines-macro-usage,
22 # We do use too many macros, and we should fix many of them, but there
23 # is no reasonable way to suppress the check e.g. in src/config.h and
24 # configuring the build is a major legitimate use of macros.
25 #
26 #         -cppcoreguidelines-narrowing-conversions,
27 #         -bugprone-narrowing-conversions
28 # We have many cases where int is converted to float and we don't care
29 # enough about such potential loss of precision to use explicit casts
30 # in large numbers of places.
31 #
32 #         -google-readability-avoid-underscore-in-googletest-name
33 # We need to use underscores for readability for our legacy types
34 # and command-line parameter names
35 #
36 Checks:  clang-diagnostic-*,-clang-analyzer-*,-clang-analyzer-security.insecureAPI.strcpy,
37          bugprone-*,misc-*,readability-*,performance-*,mpi-*,
38          -readability-inconsistent-declaration-parameter-name,
39          -readability-function-size,-readability-else-after-return,
40          modernize-use-nullptr,modernize-use-emplace,
41          modernize-make-unique,modernize-make-shared,
42          modernize-avoid-bind,
43          modernize-use-override,
44          modernize-redundant-void-arg,modernize-use-bool-literals,
45          cppcoreguidelines-*,-cppcoreguidelines-pro-*,-cppcoreguidelines-owning-memory,
46          -cppcoreguidelines-no-malloc,-cppcoreguidelines-special-member-functions,
47          -cppcoreguidelines-avoid-goto,
48          google-*,-google-build-using-namespace,-google-explicit-constructor,
49          -google-readability-function-size,-google-readability-todo,-google-runtime-int,
50          -cppcoreguidelines-non-private-member-variables-in-classes,
51          -misc-non-private-member-variables-in-classes,
52          -readability-isolate-declaration,
53          -cppcoreguidelines-avoid-c-arrays,
54          -cppcoreguidelines-avoid-magic-numbers,
55          -readability-magic-numbers,
56          -cppcoreguidelines-macro-usage,
57          -cppcoreguidelines-narrowing-conversions,
58          -bugprone-narrowing-conversions,
59          -google-readability-avoid-underscore-in-googletest-name
60 HeaderFilterRegex: .*
61 CheckOptions:
62   - key:           cppcoreguidelines-special-member-functions.AllowSoleDefaultDtor
63     value:         1
64   - key:           modernize-make-unique.IncludeStyle
65     value:         google
66   - key:           modernize-make-shared.IncludeStyle
67     value:         google
68   - key:           readability-implicit-bool-conversion.AllowIntegerConditions
69     value:         1
70   - key:           readability-implicit-bool-conversion.AllowPointerConditions
71     value:         1
72   - key:           bugprone-dangling-handle.HandleClasses
73     value:         std::basic_string_view; nonstd::sv_lite::basic_string_view
74 # Permit passing shard pointers by value for sink parameters
75   - key:           performance-unnecessary-copy-initialization.AllowedTypes
76     value:         shared_ptr
77   - key:           performance-unnecessary-value-param.AllowedTypes
78     value:         shared_ptr