[PATCH] D90282: [clang-tidy] Add IgnoreShortNames config to identifier naming checks

Shane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 22 15:03:31 PST 2020


smhc added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp:127-128
 getFileStyleFromOptions(const ClangTidyCheck::OptionsView &Options) {
   SmallVector<llvm::Optional<IdentifierNamingCheck::NamingStyle>, 0> Styles(
       SK_Count);
   SmallString<64> StyleString;
----------------
njames93 wrote:
> Making this change removes the need to NamingStyle to be copy constructable/assignable.
Thanks - was looking for a solution to this.


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

https://reviews.llvm.org/D90282



More information about the cfe-commits mailing list