-
Notifications
You must be signed in to change notification settings - Fork 148
/
.clang-tidy
42 lines (41 loc) · 1.97 KB
/
.clang-tidy
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
---
Checks: '-*,bugprone-*,
-bugprone-branch-clone,
-bugprone-exception-escape,
-bugprone-lambda-function-name,
-bugprone-misplaced-widening-cast,
-bugprone-not-null-terminated-result,
performance-*,
-performance-noexcept-move-constructor,
misc-*,
-misc-non-private-member-variables-in-classes,
-misc-unused-parameters
'
WarningsAsErrors: 'bugprone-*,
-bugprone-branch-clone,
-bugprone-exception-escape,
-bugprone-lambda-function-name,
-bugprone-misplaced-widening-cast,
-bugprone-not-null-terminated-result,
performance-*,
-performance-noexcept-move-constructor,
misc-*,
-misc-non-private-member-variables-in-classes,
-misc-unused-parameters
'
# -bugprone-branch-clone - does not work well with switch cases
# -bugprone-exception-escape - main functions throwing exceptions are OK: we prefer to analyze core dumps
# -bugprone-lambda-function-name - Assert macro leads to many false positives
# -bugprone-misplaced-widening-cast - does not seem to be a big issue
# -bugprone-not-null-terminated-result - many false positives when std::string is used for storing raw bytes
# -misc-non-private-member-variables-in-classes - seems irrelevant
# -misc-unused-parameters - too many errors, fix later
# Include all headers in the project
# Except for protobuf/gRPC generated header files
# Note: '(?<!' starts a negative look behind expression
HeaderFilterRegex: 'concord-bft.*(?<!\.pb\.h)$'
AnalyzeTemporaryDtors: false
FormatStyle: file
CheckOptions:
- { key: performance-unnecessary-value-param.AllowedTypes, value: 'shared_ptr;SharedPtr;[Ll]og;BlockId' }
...