From 38f3a4cacbd4ed1c4b37c7fb7c0b6c1c7bc1661c Mon Sep 17 00:00:00 2001 From: jeremy Date: Wed, 30 Nov 2022 16:28:01 +0100 Subject: [PATCH] Use range for loop instead of idx based --- src/query/v2/multiframe.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/query/v2/multiframe.cpp b/src/query/v2/multiframe.cpp index 5bae5c108..e6e8685fc 100644 --- a/src/query/v2/multiframe.cpp +++ b/src/query/v2/multiframe.cpp @@ -101,12 +101,11 @@ ValidFramesConsumer::Iterator ValidFramesConsumer::end() { InvalidFramesPopulator::InvalidFramesPopulator(MultiFrame &multiframe) : multiframe_(multiframe) {} InvalidFramesPopulator::Iterator InvalidFramesPopulator::begin() { - for (auto idx = 0UL; idx < multiframe_.frames_.size(); ++idx) { - if (!multiframe_.frames_[idx].IsValid()) { - return Iterator{&multiframe_.frames_[idx]}; + for (auto &frame : multiframe_.frames_) { + if (!frame.IsValid()) { + return Iterator{&frame}; } } - return end(); }