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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to