alexfh added inline comments.
Comment at: clang-tools-extra/clang-tidy/abseil/TimeSubtractionCheck.cpp:96
void TimeSubtractionCheck::registerMatchers(MatchFinder *Finder) {
- for (auto ScaleName :
+ for (const auto *ScaleName :
{"Hours", "Minutes", "Seconds", "Millis",
alexfh updated this revision to Diff 319979.
alexfh added a comment.
Manually cleaned up suboptimal fixes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95614/new/
https://reviews.llvm.org/D95614
Files:
clang-tools-extra/clang-tidy/abseil/TimeSu
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM, thank you for the cleanup!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95614/new/
https://reviews.llvm.org/D95614
___
