PiotrZSL wrote:

> Sure thing. I don't have a strong feeling about the category.

I'ts not only about category but also name, check suggests std::numbers, then 
better name it modernize-use-std-numbers, reason for that is simple. Before 
C++20 we may want to have a check for example <something>-prefer-math-constants 
that would change 3.14 hardcoded in code into M_PI. And in same way we could 
have check <something>-use-std-numeric-limits that would change -2147483648 
into std::numeric_limits<std::sint32_t>::min(). Simply we may have many of such 
checks.

https://github.com/llvm/llvm-project/pull/66583
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to