diff --git a/clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp b/clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp --- a/clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp +++ b/clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp @@ -55,6 +55,7 @@ m(Namespace) \ m(InlineNamespace) \ m(EnumConstant) \ + m(ScopedEnumConstant) \ m(ConstexprVariable) \ m(ConstantMember) \ m(PrivateMember) \ @@ -405,7 +406,13 @@ if (isa(D) && NamingStyles[SK_Enum]) return SK_Enum; - if (isa(D)) { + if (const auto *EnumConst = dyn_cast(D)) { + const auto *ParentEnum = dyn_cast(EnumConst->getDeclContext()); + assert(ParentEnum && "DeclContext of EnumConstant should be EnumDecl"); + + if (ParentEnum->isScoped() && NamingStyles[SK_ScopedEnumConstant]) + return SK_ScopedEnumConstant; + if (NamingStyles[SK_EnumConstant]) return SK_EnumConstant; diff --git a/clang-tools-extra/docs/ReleaseNotes.rst b/clang-tools-extra/docs/ReleaseNotes.rst --- a/clang-tools-extra/docs/ReleaseNotes.rst +++ b/clang-tools-extra/docs/ReleaseNotes.rst @@ -120,6 +120,9 @@ Added an option `GetConfigPerFile` to support including files which use different naming styles. + Added support for specifying the style of scoped enum constants. If + unspecified, will fall back to the style for regular ``enum`` constants. + - Removed `google-runtime-references` check because the rule it checks does not exist in the Google Style Guide anymore. diff --git a/clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst b/clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst --- a/clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst +++ b/clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst @@ -76,6 +76,7 @@ - :option:`ProtectedMethodCase`, :option:`ProtectedMethodPrefix`, :option:`ProtectedMethodSuffix` - :option:`PublicMemberCase`, :option:`PublicMemberPrefix`, :option:`PublicMemberSuffix` - :option:`PublicMethodCase`, :option:`PublicMethodPrefix`, :option:`PublicMethodSuffix` + - :option:`ScopedEnumConstantCase`, :option:`ScopedEnumConstantPrefix`, :option:`ScopedEnumConstantSuffix` - :option:`StaticConstantCase`, :option:`StaticConstantPrefix`, :option:`StaticConstantSuffix` - :option:`StaticVariableCase`, :option:`StaticVariablePrefix`, :option:`StaticVariableSuffix` - :option:`StructCase`, :option:`StructPrefix`, :option:`StructSuffix` @@ -1595,6 +1596,41 @@ int pre_member_method_post(); } +.. option:: ScopedEnumConstantCase + + When defined, the check will ensure enum class constant names conform to the + selected casing. + +.. option:: ScopedEnumConstantPrefix + + When defined, the check will ensure enum class constant names will add the + prefixed with the given value (regardless of casing). + +.. option:: ScopedEnumConstantSuffix + + When defined, the check will ensure enum class constant names will add the + suffix with the given value (regardless of casing). + +For example using values of: + + - ScopedEnumConstantCase of ``lower_case`` + - ScopedEnumConstantPrefix of ``pre_`` + - ScopedEnumConstantSuffix of ``_post`` + +Identifies and/or transforms enumeration constant names as follows: + +Before: + +.. code-block:: c++ + + enum class FOO { One, Two, Three }; + +After: + +.. code-block:: c++ + + enum class FOO { pre_One_post, pre_Two_post, pre_Three_post }; + .. option:: StaticConstantCase When defined, the check will ensure static constant names conform to the diff --git a/clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming.cpp b/clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming.cpp --- a/clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming.cpp +++ b/clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming.cpp @@ -18,6 +18,7 @@ // RUN: {key: readability-identifier-naming.ConstexprVariableCase, value: lower_case}, \ // RUN: {key: readability-identifier-naming.EnumCase, value: CamelCase}, \ // RUN: {key: readability-identifier-naming.EnumPrefix, value: 'E'}, \ +// RUN: {key: readability-identifier-naming.ScopedEnumConstantCase, value: CamelCase}, \ // RUN: {key: readability-identifier-naming.EnumConstantCase, value: UPPER_CASE}, \ // RUN: {key: readability-identifier-naming.FunctionCase, value: camelBack}, \ // RUN: {key: readability-identifier-naming.GlobalConstantCase, value: UPPER_CASE}, \ @@ -160,6 +161,18 @@ // CHECK-FIXES: {{^}} THIS_CONST_VALUE = 1,{{$}} }; +enum class EMyEnumeration { + myConstant = 1, +// CHECK-MESSAGES: :[[@LINE-1]]:5: warning: invalid case style for scoped enum constant 'myConstant' +// CHECK-FIXES: {{^}} MyConstant = 1,{{$}} + your_CONST = 1, +// CHECK-MESSAGES: :[[@LINE-1]]:5: warning: invalid case style for scoped enum constant 'your_CONST' +// CHECK-FIXES: {{^}} YourConst = 1,{{$}} + THIS_ConstValue = 1, +// CHECK-MESSAGES: :[[@LINE-1]]:5: warning: invalid case style for scoped enum constant 'THIS_ConstValue' +// CHECK-FIXES: {{^}} ThisConstValue = 1,{{$}} +}; + constexpr int ConstExpr_variable = MyConstant; // CHECK-MESSAGES: :[[@LINE-1]]:15: warning: invalid case style for constexpr variable 'ConstExpr_variable' // CHECK-FIXES: {{^}}constexpr int const_expr_variable = MY_CONSTANT;{{$}}