[PATCH] D76496: [clang-tidy] add support to 3-arg find() in StringFindStartswith
Niko Weh via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 1 17:16:13 PDT 2020
niko added a comment.
In D76496#1976713 <https://reviews.llvm.org/D76496#1976713>, @njames93 wrote:
> In D76496#1949851 <https://reviews.llvm.org/D76496#1949851>, @niko wrote:
>
> > Correct me if I'm wrong, but that seems to be in violation of IWYU? Maybe I'm misreading this, or is the idea that higher-lever tooling (e.g. IWYU fixer) is supposed to address that?
>
>
> It is but it sort of gets a pass as the absl/strings/match.h has to include it.
> Similar to how if you `#include <vector>`, you wouldn't then `#include <initializer_list>` so you could construct a `std::vector` using a `std::initializer_list`.
Makes sense. Thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76496/new/
https://reviews.llvm.org/D76496
More information about the cfe-commits
mailing list