njames93 marked an inline comment as done.
njames93 added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/modernize/UseRangesCheck.cpp:77
+ MatchCallTo((ID + "Begin").str(), namedDecl().bind(Range),
+ hasAnyName("::std::begin", "::std::cbegin"),
+ hasAnyName("begin", "cbegin"))),
----------------
JonasToth wrote:
> i would at `rbegin()` and the likes, too.
How should `rbegin` be handled
`std::<func>(std::rbegin(X), std::rbegin(X))` ->
`std::ranges::<func>(std::ranges::reverse_view(X))` I almost think that looks
less pleasing, but thats just my opinion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81923/new/
https://reviews.llvm.org/D81923
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
