[PATCH] D43847: [clang-tidy] Add check: replace string::find(...) == 0 with absl::StartsWith

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 2 12:01:02 PST 2018


Eugene.Zelenko added a comment.

In https://reviews.llvm.org/D43847#1025833, @niko wrote:

> Thanks everyone for your comments! I renamed the namespace and filenames to 'abseil'.
>
> @Eugene.Zelenko, definitely interested in extending this to a C++20 modernize check and adding `absl::EndsWith()` support,  would it be OK though to do this in a later patch?


I think will be better to do this now, since you'll need to create base class and specializations for Abseil and C++17.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D43847





More information about the cfe-commits mailing list