forked from mhx/dwarfs
-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy path.clang-tidy
More file actions
74 lines (74 loc) · 2.86 KB
/
.clang-tidy
File metadata and controls
74 lines (74 loc) · 2.86 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
Checks:
- bugprone-*
- -bugprone-easily-swappable-parameters
- -bugprone-narrowing-conversions
- -bugprone-implicit-widening-of-multiplication-result
- -bugprone-unchecked-optional-access
# TODO: definitely revisit this one
- -bugprone-exception-escape
# TODO: good idea, but needed for things like std::hash
- -bugprone-std-namespace-modification
- cert-*
- clang-diagnostic-*
- clang-analyzer-*
- clang-analyzer-core.*
- clang-analyzer-cplusplus.*
- clang-analyzer-nullability.*
- clang-analyzer-unix.*
- cppcoreguidelines-*
- -cppcoreguidelines-avoid-const-or-ref-data-members
- -cppcoreguidelines-avoid-do-while
- -cppcoreguidelines-avoid-magic-numbers
- -cppcoreguidelines-macro-usage
- -cppcoreguidelines-narrowing-conversions
- -cppcoreguidelines-pro-bounds-constant-array-index
- -cppcoreguidelines-special-member-functions
# fully agree, but there are some cases where it's currently not
# done on purpose; TODO: revisit
- -cppcoreguidelines-use-enum-class
# TODO: good idea, but too many places that would need touching
- -cppcoreguidelines-pro-bounds-avoid-unchecked-container-access
- modernize-*
# a good idea, but only for more than one mutex
- -modernize-use-scoped-lock
- -modernize-use-trailing-return-type
- -modernize-macro-to-enum
# already covered by cppcoreguidelines-avoid-c-arrays
- -modernize-avoid-c-arrays
- performance-*
- -performance-enum-size
- portability-*
- -portability-avoid-pragma-once
- -portability-template-virtual-member-function
- readability-*
- -readability-magic-numbers
- -readability-identifier-length
- -readability-implicit-bool-conversion
- -readability-named-parameter
- -readability-redundant-member-init
- -readability-math-missing-parentheses
- -readability-isolate-declaration
- -readability-qualified-auto
- -readability-use-concise-preprocessor-directives
# TODO: revisit once we drop support for gcc-12, which still chokes on some cases of this
- -readability-redundant-typename
# TODO
- -cppcoreguidelines-init-variables
- -cppcoreguidelines-pro-bounds-pointer-arithmetic
- -cppcoreguidelines-pro-type-member-init
- -cppcoreguidelines-pro-type-reinterpret-cast
- -modernize-pass-by-value
- -modernize-use-nodiscard
- -modernize-use-designated-initializers
- -readability-avoid-nested-conditional-operator
- -readability-convert-member-functions-to-static
- -readability-function-cognitive-complexity
- -readability-function-size
WarningsAsErrors: 'bugprone-use-after-move'
UseColor: true
ExtraArgsBefore: ['-Wno-unknown-warning-option']
CheckOptions:
- key: cppcoreguidelines-avoid-do-while.IgnoreMacros
value: true
HeaderFilterRegex: '.*/(include/dwarfs/.*|ricepp/[^/]*\.h|ricepp/include/ricepp/.*|frozen/thrift/lib/cpp/.*|frozen/thrift/lib/cpp2/frozen/[^/]*\.h|frozen/thrift/lib/cpp2/frozen/schema/[^/]*\.h)'
SystemHeaders: false