[clang-tools-extra] [clang-tidy] add check to suggest replacement of nested std::min or std::max with initializer lists (PR #85572)

Julian Schmidt via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 1 12:40:00 PDT 2024


https://github.com/5chmidti requested changes to this pull request.

Functionality-wise, this looks good to me, I only have some comments regarding cleanup.

Please also add tests for min and max calls that are true-negatives, i.e.: `std::max(1, 2);`, `std::max({1, 2, 3});` and `std::max({1, 2, 3}, less_than);` are probably enough

https://github.com/llvm/llvm-project/pull/85572


More information about the cfe-commits mailing list