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

[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