mirror of
https://github.com/capstone-engine/llvm-capstone.git
synced 2024-12-29 11:17:28 +00:00
d226bc8397
Summary:
Some tidy checks have too many hits in the codebase, making it hard to spot
other results from clang-tidy, therefore rendering the tool less useful.
Two checks were disabled:
- misc-non-private-member-variable-in-classes in the whole LLVM monorepo,
it is very common to have those in LLVM and the style guide does not forbid
them.
- readability-identifier-naming in the clang subtree. There are thousands of
violations in 'Sema.h' alone.
Before the change, 'Sema.h' had >1000 tidy warnings, after the change the number
dropped to 3 warnings (unterminated namespace comments).
Reviewers: alexfh, hokein
Reviewed By: hokein
Subscribers: llvm-commits, cfe-commits
Tags: #clang, #llvm
Differential Revision: https://reviews.llvm.org/D57573
llvm-svn: 352862
23 lines
1.1 KiB
YAML
23 lines
1.1 KiB
YAML
Checks: '-*,clang-diagnostic-*,llvm-*,misc-*,-misc-unused-parameters,-misc-non-private-member-variables-in-classes,-readability-identifier-naming'
|
|
# Note that the readability-identifier-naming check is disabled, there are too
|
|
# many violations in the codebase and they create too much noise in clang-tidy
|
|
# results.
|
|
# Naming settings are kept for documentation purposes and allowing to run the
|
|
# check if the users would override this file, e.g. via a command-line arg.
|
|
CheckOptions:
|
|
- key: readability-identifier-naming.ClassCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.EnumCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.FunctionCase
|
|
value: camelBack
|
|
- key: readability-identifier-naming.MemberCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.ParameterCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.UnionCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.VariableCase
|
|
value: CamelCase
|
|
|