[PATCH] D135476: [clang-tidy] Support concepts in `bugprone-forwarding-reference-overload`

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 7 15:26:28 PDT 2022


Eugene.Zelenko added a subscriber: MyDeveloperDay.
Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/bugprone/forwarding-reference-overload.rst:54
 constructor could hide in this case. We also suppress warnings for constructors
-like C3 and C4 that are guarded with an ``enable_if``, assuming the programmer was
+like C3-C6 that are guarded with an ``enable_if`` or a concept, assuming the programmer was
 aware of the possible hiding.
----------------
Izaron wrote:
> Eugene.Zelenko wrote:
> > Please follow 80 characters limit.
> Thanks! I ran `git clang-format HEAD~1` before commiting, but it didn't modify this file. Seems like it doesn't see `.rst` files.
If I my memory is not failing me, @MyDeveloperDay once tried to introduce documentation linter, but I don't think that it reached production.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135476



More information about the cfe-commits mailing list