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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 12 07:07:58 PST 2020


aaron.ballman added a comment.

In D90282#2391005 <https://reviews.llvm.org/D90282#2391005>, @njames93 wrote:

> Should this be a NamingStyle option instead.
> `{key: readability-identifier-naming.ParameterShortSizeThreshold, value: 2}`
> WDYT?

I think that makes a lot of sense -- I can imagine wanting to enforce different identifier lengths depending on whether we're spelling a type name vs a local variable name, etc.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90282



More information about the cfe-commits mailing list