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 @@ -173,6 +173,7 @@ m(unsigned-short-int) \ m(unsigned-short) \ m(unsigned-int) \ + m(unsigned-char) \ m(unsigned) \ m(long-long-int) \ m(long-double) \ @@ -180,6 +181,7 @@ m(long-int) \ m(long) \ m(ptrdiff_t) \ + m(void) \ static StringRef const HungarainNotationPrimitiveTypes[] = { #define STRINGIZE(v) #v, @@ -751,13 +753,15 @@ {"unsigned short int", "usi" }, {"unsigned short", "us" }, {"unsigned int", "ui" }, + {"unsigned char", "uc" }, {"unsigned", "u" }, {"long long int", "lli" }, {"long double", "ld" }, {"long long", "ll" }, {"long int", "li" }, {"long", "l" }, - {"ptrdiff_t", "p" }}; + {"ptrdiff_t", "p" }, + {"void", "" }}; // clang-format on for (const auto &PT : PrimitiveTypes) HNOption.PrimitiveType.try_emplace(PT.first, PT.second); diff --git a/clang-tools-extra/test/clang-tidy/checkers/readability/Inputs/identifier-naming/hungarian-notation2/.clang-tidy b/clang-tools-extra/test/clang-tidy/checkers/readability/Inputs/identifier-naming/hungarian-notation2/.clang-tidy --- a/clang-tools-extra/test/clang-tidy/checkers/readability/Inputs/identifier-naming/hungarian-notation2/.clang-tidy +++ b/clang-tools-extra/test/clang-tidy/checkers/readability/Inputs/identifier-naming/hungarian-notation2/.clang-tidy @@ -200,6 +200,8 @@ value: us - key: readability-identifier-naming.HungarianNotation.PrimitiveType.unsigned-int value: ui + - key: readability-identifier-naming.HungarianNotation.PrimitiveType.unsigned-char + value: uc - key: readability-identifier-naming.HungarianNotation.PrimitiveType.unsigned value: u - key: readability-identifier-naming.HungarianNotation.PrimitiveType.long-long-int @@ -214,6 +216,8 @@ value: l - key: readability-identifier-naming.HungarianNotation.PrimitiveType.ptrdiff_t value: p + - key: readability-identifier-naming.HungarianNotation.PrimitiveType.void + value: v - key: readability-identifier-naming.HungarianNotation.UserDefinedType.BOOL value: b - key: readability-identifier-naming.HungarianNotation.UserDefinedType.BOOLEAN diff --git a/clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation-cfgfile.cpp b/clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation-cfgfile.cpp --- a/clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation-cfgfile.cpp +++ b/clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation-cfgfile.cpp @@ -366,15 +366,15 @@ void *BufferPtr1; // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: invalid case style for global pointer 'BufferPtr1' [readability-identifier-naming] -// CHECK-FIXES: {{^}}void *pBufferPtr1; +// CHECK-FIXES: {{^}}void *pvBufferPtr1; void **BufferPtr2; // CHECK-MESSAGES: :[[@LINE-1]]:8: warning: invalid case style for global pointer 'BufferPtr2' [readability-identifier-naming] -// CHECK-FIXES: {{^}}void **ppBufferPtr2; +// CHECK-FIXES: {{^}}void **ppvBufferPtr2; void **pBufferPtr3; // CHECK-MESSAGES: :[[@LINE-1]]:8: warning: invalid case style for global pointer 'pBufferPtr3' [readability-identifier-naming] -// CHECK-FIXES: {{^}}void **ppBufferPtr3; +// CHECK-FIXES: {{^}}void **ppvBufferPtr3; int *pBufferPtr4; // CHECK-MESSAGES: :[[@LINE-1]]:6: warning: invalid case style for global pointer 'pBufferPtr4' [readability-identifier-naming] @@ -387,7 +387,7 @@ void *ValueVoidPtr = NULL; // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: invalid case style for global pointer 'ValueVoidPtr' [readability-identifier-naming] -// CHECK-FIXES: {{^}}void *pValueVoidPtr = NULL; +// CHECK-FIXES: {{^}}void *pvValueVoidPtr = NULL; ptrdiff_t PtrDiff = NULL; // CHECK-MESSAGES: :[[@LINE-1]]:11: warning: invalid case style for global variable 'PtrDiff' [readability-identifier-naming] @@ -403,7 +403,7 @@ void MyFunc2(void* Val){} // CHECK-MESSAGES: :[[@LINE-1]]:20: warning: invalid case style for pointer parameter 'Val' [readability-identifier-naming] -// CHECK-FIXES: {{^}}void MyFunc2(void* pVal){} +// CHECK-FIXES: {{^}}void MyFunc2(void* pvVal){} //===----------------------------------------------------------------------===// @@ -567,6 +567,10 @@ // CHECK-MESSAGES: :[[@LINE-1]]:14: warning: invalid case style for global variable 'ValueUnsignedInt' [readability-identifier-naming] // CHECK-FIXES: {{^}}unsigned int uiValueUnsignedInt = 0; +unsigned char ValueUnsignedChar = 0; +// CHECK-MESSAGES: :[[@LINE-1]]:15: warning: invalid case style for global variable 'ValueUnsignedChar' [readability-identifier-naming] +// CHECK-FIXES: {{^}}unsigned char ucValueUnsignedChar = 0; + long int ValueLongInt = 0; // CHECK-MESSAGES: :[[@LINE-1]]:10: warning: invalid case style for global variable 'ValueLongInt' [readability-identifier-naming] // CHECK-FIXES: {{^}}long int liValueLongInt = 0; diff --git a/clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation.cpp b/clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation.cpp --- a/clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation.cpp +++ b/clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation.cpp @@ -567,6 +567,10 @@ // CHECK-MESSAGES: :[[@LINE-1]]:14: warning: invalid case style for global variable 'ValueUnsignedInt' [readability-identifier-naming] // CHECK-FIXES: {{^}}unsigned int uiValueUnsignedInt = 0; +unsigned char ValueUnsignedChar = 0; +// CHECK-MESSAGES: :[[@LINE-1]]:15: warning: invalid case style for global variable 'ValueUnsignedChar' [readability-identifier-naming] +// CHECK-FIXES: {{^}}unsigned char ucValueUnsignedChar = 0; + long int ValueLongInt = 0; // CHECK-MESSAGES: :[[@LINE-1]]:10: warning: invalid case style for global variable 'ValueLongInt' [readability-identifier-naming] // CHECK-FIXES: {{^}}long int liValueLongInt = 0;