lebedev.ri added a comment. Passing-by remark:
> I wrote a Clang warning [not pictured] to diagnose any use of `T(x)` which > was not equivalent to `static_cast<T>(x)`. I'm not sure whether or not this will pass the bar for a clang diagnostic, but that does sound like a good clang-tidy readability check. I also have a bit of pet peeve with `T(x)` (i.e. including those that should be `static_cast<>`), so if you do go with clang-tidy direction, it may be good to generalize it to handle all `T(x)`, but diagnose only those cast types that are specified in the config. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76572/new/ https://reviews.llvm.org/D76572 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
