Skip to content

Commit 1565d52

Browse files
committed
[Lint] Remove clang-tidy suppression for _-prefix
Part of OpenAssetIO#1091. Previous versions of clang-tidy would incorrectly flag the `_1`, `_2`, etc placeholder macros from the Trompeloeil library as violating our naming convention. This no longer seems to be the case for clang-tidy v19. Signed-off-by: David Feltell <david.feltell@foundry.com>
1 parent 0cbc6b8 commit 1565d52

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

.clang-tidy

-5
Original file line numberDiff line numberDiff line change
@@ -64,11 +64,6 @@ CheckOptions:
6464
- { key: readability-identifier-naming.TemplateParameterCase, value: CamelCase }
6565
- { key: readability-identifier-naming.FunctionCase, value: camelBack }
6666
- { key: readability-identifier-naming.VariableCase, value: camelBack }
67-
# Allow _1, _2, etc as variable (placeholder) names - used for
68-
# Trompeloeil mock setup.
69-
# TODO(DF): This should be placed in `tests/.clang-tidy` but that
70-
# doesn't work. Reported upstream: https://github.com/llvm/llvm-project/issues/54781
71-
- { key: readability-identifier-naming.VariableIgnoredRegexp, value: "^_[0-9]+$"}
7267
- { key: readability-identifier-naming.ClassMemberCase, value: camelBack }
7368
- { key: readability-identifier-naming.PrivateMemberSuffix, value: _ }
7469
- { key: readability-identifier-naming.ProtectedMemberSuffix, value: _ }

0 commit comments

Comments
 (0)