mirror of
https://github.com/capstone-engine/llvm-capstone.git
synced 2024-12-18 05:18:43 +00:00
8edfc2f814
Auto-generated patch based on clang-tidy readability-identifier-naming. Only some manual cleanup for `extern "C"` declarations and a GTest change was required. I'm not sure if this cleanup is actually very useful. It cleans up clang-tidy findings to the number of warnings from clang-tidy should be lower. Since it was easy to do and required only little cleanup I thought I'd upload it for discussion. One pattern that keeps recurring: Test **matchers** are also supposed to start with a lowercase letter as per LLVM convention. However GTest naming convention for matchers start with upper case. I would propose to keep stay consistent with the GTest convention there. However that would imply a lot of `//NOLINT` throughout these files. To re-product this patch run: ``` run-clang-tidy -checks="-*,readability-identifier-naming" -fix -format ./clang-tools-extra/clangd ``` To convert the macro names, I was using this script with some manual cleanup afterwards: https://gist.github.com/ChristianKuehnel/a01cc4362b07c58281554ab46235a077 Differential Revision: https://reviews.llvm.org/D115634
27 lines
1.3 KiB
YAML
27 lines
1.3 KiB
YAML
Checks: '-*,clang-diagnostic-*,llvm-*,misc-*,-misc-unused-parameters,-misc-non-private-member-variables-in-classes,-misc-no-recursion,readability-identifier-naming'
|
|
CheckOptions:
|
|
- key: readability-identifier-naming.ClassCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.EnumCase
|
|
value: CamelCase
|
|
- key: readability-identifier-naming.FunctionCase
|
|
value: camelBack
|
|
# Exclude from scanning as this is an exported symbol used for fuzzing
|
|
# throughout the code base.
|
|
- key: readability-identifier-naming.FunctionIgnoredRegexp
|
|
value: "LLVMFuzzerTestOneInput"
|
|
- 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
|
|
- key: readability-identifier-naming.IgnoreMainLikeFunctions
|
|
value: 1
|
|
- key: readability-redundant-member-init.IgnoreBaseInCopyConstructors
|
|
value: 1
|
|
- key: modernize-use-default-member-init.UseAssignment
|
|
value: 1
|