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

[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