[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Douglas Chen via Phabricator via cfe-commits 
dougpuob added a comment.

In D107325#2923518 , @thakis wrote:

> The bot is happy again, thanks.

Nice, thank you for your help :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Nico Weber via Phabricator via cfe-commits 
thakis added a comment.

The bot is happy again, thanks.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Nico Weber via Phabricator via cfe-commits 
This revision was automatically updated to reflect the committed changes.
Closed by commit rG559d142331c2: [clang-tidy] Fix command line is too long 
issue which breaks test on Windows (authored by dougpuob, committed by thakis).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

Files:
  clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp
  
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation1/.clang-tidy
  
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation2/.clang-tidy
  
clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation-cfgfile.cpp
  
clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp

Index: clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp
===
--- clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp
+++ clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp
@@ -1,62 +1,5 @@
-// RUN: %check_clang_tidy %s readability-identifier-naming %t -- \
-// RUN:   -config='{ CheckOptions: [ \
-// RUN: { key: readability-identifier-naming.AbstractClassCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassConstantCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassMemberCase , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantMemberCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantParameterCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantPointerParameterCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstexprVariableCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.EnumConstantCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalConstantPointerCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalPointerCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalVariableCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalConstantCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalConstantPointerCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalPointerCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalVariableCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.MemberCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ParameterCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PointerParameterCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PrivateMemberCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ProtectedMemberCase , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PublicMemberCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ScopedEnumConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.StaticConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.StaticVariableCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.VariableCase,

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Douglas Chen via Phabricator via cfe-commits 
dougpuob added a comment.

Hi @thakis:
I can't access the repo, could you please help me to land?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Douglas Chen via Phabricator via cfe-commits 
dougpuob updated this revision to Diff 363717.
dougpuob marked 3 inline comments as done.
dougpuob added a comment.

- Improved code review suggestions.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

Files:
  clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp
  
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation1/.clang-tidy
  
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation2/.clang-tidy
  
clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation-cfgfile.cpp
  
clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp

Index: clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp
===
--- clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp
+++ clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp
@@ -1,62 +1,5 @@
-// RUN: %check_clang_tidy %s readability-identifier-naming %t -- \
-// RUN:   -config='{ CheckOptions: [ \
-// RUN: { key: readability-identifier-naming.AbstractClassCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassConstantCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassMemberCase , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantMemberCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantParameterCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantPointerParameterCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstexprVariableCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.EnumConstantCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalConstantPointerCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalPointerCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalVariableCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalConstantCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalConstantPointerCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalPointerCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalVariableCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.MemberCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ParameterCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PointerParameterCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PrivateMemberCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ProtectedMemberCase , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PublicMemberCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ScopedEnumConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.StaticConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.StaticVariableCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.VariableCase, value: CamelCase }, \
-// RUN: { key:

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Douglas Chen via Phabricator via cfe-commits 
dougpuob marked 3 inline comments as done.
dougpuob added inline comments.



Comment at: 
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation1/.clang-tidy:1
+Checks: readability-identifier-naming
+CheckOptions:

whisperity wrote:
> I do not think this file should have the //execute// bit set...
I will change it to 664.



Comment at: 
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation1/.clang-tidy:3
+CheckOptions:
+  - { key: readability-identifier-naming.AbstractClassCase 
  , value: CamelCase }
+  - { key: readability-identifier-naming.ClassCase 
  , value: CamelCase }

whisperity wrote:
> thakis wrote:
> > can we drop all the spaces here? :)
> And maybe even the object brackets.
Sure.



Comment at: 
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation1/.clang-tidy:3
+CheckOptions:
+  - { key: readability-identifier-naming.AbstractClassCase 
  , value: CamelCase }
+  - { key: readability-identifier-naming.ClassCase 
  , value: CamelCase }

dougpuob wrote:
> whisperity wrote:
> > thakis wrote:
> > > can we drop all the spaces here? :)
> > And maybe even the object brackets.
> Sure.
OK


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Whisperity via Phabricator via cfe-commits 
whisperity added a comment.

In general, would it be possible to specify the configuration file as a custom 
path, and not use the hidden filename `.clang-tidy` in an otherwise empty 
directory? I think there's a `--config` flag on Tidy's interface for giving any 
file from any path for this. So it could be `config1.yaml` or something like 
that instead.




Comment at: 
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation1/.clang-tidy:1
+Checks: readability-identifier-naming
+CheckOptions:

I do not think this file should have the //execute// bit set...



Comment at: 
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation1/.clang-tidy:3
+CheckOptions:
+  - { key: readability-identifier-naming.AbstractClassCase 
  , value: CamelCase }
+  - { key: readability-identifier-naming.ClassCase 
  , value: CamelCase }

thakis wrote:
> can we drop all the spaces here? :)
And maybe even the object brackets.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Nico Weber via Phabricator via cfe-commits 
thakis accepted this revision.
thakis added a comment.
This revision is now accepted and ready to land.

Thanks!




Comment at: 
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation1/.clang-tidy:3
+CheckOptions:
+  - { key: readability-identifier-naming.AbstractClassCase 
  , value: CamelCase }
+  - { key: readability-identifier-naming.ClassCase 
  , value: CamelCase }

can we drop all the spaces here? :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Douglas Chen via Phabricator via cfe-commits 
dougpuob added a comment.

In D107325#2921844 , @thakis wrote:

> Thanks!
>
> Since the config file contents are fixed as far as I can tell, maybe we could 
> instead just add a file with the right contents to 
> `clang-tools-extra/test/clang-tidy/checkers/Inputs` and refer to that? (Use 
> `%S/Inputs/myfile.txt`) (Alternatively you could use the `split-file` 
> utility, but putting the file in Inputs is probably easier if you're new, and 
> it's not any worse :) )

Hi @thakis:
It's a really good idea, thank you for the suggestion. I just upload the latest 
diff.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Douglas Chen via Phabricator via cfe-commits 
dougpuob updated this revision to Diff 363698.
dougpuob added a comment.
Herald added a subscriber: aheejin.

- Moved config content of regression test to 
`clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation*/.clang-tidy`,
 then refer to those files in tests.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

Files:
  clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp
  
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation1/.clang-tidy
  
clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/hungarian-notation2/.clang-tidy
  
clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation-cfgfile.cpp
  
clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp

Index: clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp
===
--- clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp
+++ clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation.cpp
@@ -1,62 +1,5 @@
-// RUN: %check_clang_tidy %s readability-identifier-naming %t -- \
-// RUN:   -config='{ CheckOptions: [ \
-// RUN: { key: readability-identifier-naming.AbstractClassCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassConstantCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassMemberCase , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantMemberCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantParameterCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantPointerParameterCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstexprVariableCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.EnumConstantCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalConstantPointerCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalPointerCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalVariableCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalConstantCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalConstantPointerCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalPointerCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalVariableCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.MemberCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ParameterCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PointerParameterCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PrivateMemberCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ProtectedMemberCase , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PublicMemberCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ScopedEnumConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.StaticConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.StaticVariableCase  , value: CamelCase }, \
-// RUN: { key:

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Nico Weber via Phabricator via cfe-commits 
thakis added a comment.

Thanks!

Since the config file contents are fixed as far as I can tell, maybe we could 
instead just add a file with the right contents to 
`clang-tools-extra/test/clang-tidy/checkers/Inputs` and refer to that? (Use 
`%S/Inputs/myfile.txt`) (Alternatively you could use the `split-file` utility, 
but putting the file in Inputs is probably easier if you're new, and it's not 
any worse :) )


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107325/new/

https://reviews.llvm.org/D107325

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D107325: [clang-tidy] Fix command line is too long issue which breaks test on Windows

2021-08-03 Thread Douglas Chen via Phabricator via cfe-commits 
dougpuob created this revision.
Herald added a subscriber: xazax.hun.
dougpuob requested review of this revision.
Herald added a project: clang-tools-extra.
Herald added a subscriber: cfe-commits.

Command line is too long with check_clang_tidy.py program on Windows, because 
the configuration is too long for regression test. Fix this issue by creating a 
temporary response file then pass to target program. (error log: 
http://45.33.8.238/win/43180/step_8.txt)


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D107325

Files:
  clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp
  
clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation-cfgfile.cpp

Index: clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation-cfgfile.cpp
===
--- clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation-cfgfile.cpp
+++ clang-tools-extra/test/clang-tidy/checkers/readability-identifier-naming-hungarian-notation-cfgfile.cpp
@@ -1,141 +1,142 @@
-// RUN: %check_clang_tidy %s readability-identifier-naming %t -- \
-// RUN:   -config='{ CheckOptions: [ \
-// RUN: { key: readability-identifier-naming.AbstractClassCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassConstantCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ClassMemberCase , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantMemberCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantParameterCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstantPointerParameterCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ConstexprVariableCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.EnumConstantCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalConstantPointerCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalPointerCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.GlobalVariableCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalConstantCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalConstantPointerCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalPointerCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.LocalVariableCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.MemberCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ParameterCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PointerParameterCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PrivateMemberCase   , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ProtectedMemberCase , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.PublicMemberCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.ScopedEnumConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.StaticConstantCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.StaticVariableCase  , value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.VariableCase, value: CamelCase }, \
-// RUN: { key: readability-identifier-naming.AbstractClassHungarianPrefix,