nicovank wrote: Ping. @carlosgalvezp could you please weigh in? modernize / performance? Anyone else?
I have no real preference either way. Now that modernize is fully integrated in Clang-Tidy, I don't think historical reasons should lead to modernize overbloat. If this check was only replacing `strncmp` with `starts_with` it would definitely belong in modernize, as-is it sits in-between. https://github.com/llvm/llvm-project/pull/72385 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
