lebedev.ri added inline comments.
================
Comment at: clang-tidy/readability/NumericalCostantsToMaxIntCheck.h:25
+///
+class NumericalCostantsToMaxIntCheck : public ClangTidyCheck {
+public:
----------------
I feel like the name is overly vague.
This *only* handles the cases of `-1` and `~0`.
It does not handle cases like `255` -> `std::numeric_limits<uint_8>::max()`.
(It might be nice to do that, but it is more complex i suspect.)
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52892
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
