[PATCH] D144216: [clang-tidy] Extract string header from redundant-string-cstr checker

Mike Crowe via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 17 09:45:39 PST 2023


mikecrowe added a comment.

In D144216#4135431 <https://reviews.llvm.org/D144216#4135431>, @carlosgalvezp wrote:

> Fair enough! I think we should merge this patch as is first, so that people can benefit from it already, and work through the technical debt afterwards.

Great. Thank you. I'm still interested in answers to the questions in my comment since I have the necessary changes to migrate the tests I mentioned to using `<string>` ready and I'd be happy to have a go at migrating more.

Thanks.

Mike.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144216/new/

https://reviews.llvm.org/D144216



More information about the cfe-commits mailing list