[PATCH] D90282: [clang-tidy] Support IgnoredRegexp configuration to selectively suppress identifier naming checks

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 25 04:35:01 PST 2020


aaron.ballman added a comment.

In D90282#2415189 <https://reviews.llvm.org/D90282#2415189>, @smhc wrote:

> There's a build failure from this merge, looks like a typo:
>
> diff --git a/clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst b/clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst
> index b744d21b6ff8..68c0abfcbdf6 100644
>
> - a/clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst
>
> +++ b/clang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst
> @@ -680,7 +680,7 @@ After:
>
>   When defined, the check will ensure enumeration names will add the
>   prefixed with the given value (regardless of casing).
>
> -.. option:: EnumConstantIgnoredRegexp
> +.. option:: EnumIgnoredRegexp
>
>   Identifier naming checks won't be enforced for enumeration names
>   matching this regular expression.

I've corrected this in ed242da0ffa28493d8a5ee6b80ecbe2441ca48a7 <https://reviews.llvm.org/rGed242da0ffa28493d8a5ee6b80ecbe2441ca48a7>, thanks!


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