lebedev.ri added a comment.

In https://reviews.llvm.org/D36836#845417, @JonasToth wrote:

> Did I run it over clang or athletic like that? I would be interested in an 
> overall output.


No, not yet.
But if it is to be enabled for LLVM repos by default, then the `Threshold` will 
most definitively need adjusting in a `.clang-tidy` :)

> I added my thoughts.

Should be fixed.

> This check seems very valuable.

Thank you :)
I hope others agree...


Repository:
  rL LLVM

https://reviews.llvm.org/D36836



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

Reply via email to