From 5ae37f2f842784c6eaea87bfee4b7cf302a9671a Mon Sep 17 00:00:00 2001
From: Jarek Kobus <jaroslaw.kobus@qt.io>
Date: Thu, 25 Jan 2024 14:26:13 +0100
Subject: [PATCH] ImageScaling: Reuse LoopList

Instead of LoopRepeat + container capture.

Change-Id: Ic37c4ff53c4ce2a217a258aa4af1272ac3d8ab29
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
Reviewed-by: hjk <hjk@qt.io>
---
 .../manual/tasking/imagescaling/imagescaling.cpp | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/tests/manual/tasking/imagescaling/imagescaling.cpp b/tests/manual/tasking/imagescaling/imagescaling.cpp
index 8d74301cf64..186042435ec 100644
--- a/tests/manual/tasking/imagescaling/imagescaling.cpp
+++ b/tests/manual/tasking/imagescaling/imagescaling.cpp
@@ -56,7 +56,7 @@ void Images::process()
     const auto urls = downloadDialog->getUrls();
     initLayout(urls.size());
 
-    const LoopRepeat repeater(urls.size());
+    const LoopList iterator(urls);
     const Storage<QByteArray> storage;
 
     const auto onRootSetup = [this] {
@@ -68,13 +68,13 @@ void Images::process()
         cancelButton->setEnabled(false);
     };
 
-    const auto onDownloadSetup = [this, urls, repeater](NetworkQuery &query) {
+    const auto onDownloadSetup = [this, iterator](NetworkQuery &query) {
         query.setNetworkAccessManager(&qnam);
-        query.setRequest(QNetworkRequest(urls.at(repeater.iteration())));
+        query.setRequest(QNetworkRequest(*iterator));
     };
-    const auto onDownloadDone = [this, storage, repeater](const NetworkQuery &query,
+    const auto onDownloadDone = [this, storage, iterator](const NetworkQuery &query,
                                                                   DoneWith result) {
-        const int it = repeater.iteration();
+        const int it = iterator.iteration();
         if (result == DoneWith::Success)
             *storage = query.reply()->readAll();
         else
@@ -84,9 +84,9 @@ void Images::process()
     const auto onScalingSetup = [storage](ConcurrentCall<QImage> &data) {
         data.setConcurrentCallData(&scale, *storage);
     };
-    const auto onScalingDone = [this, repeater](const ConcurrentCall<QImage> &data,
+    const auto onScalingDone = [this, iterator](const ConcurrentCall<QImage> &data,
                                                 DoneWith result) {
-        const int it = repeater.iteration();
+        const int it = iterator.iteration();
         if (result == DoneWith::Success)
             labels[it]->setPixmap(QPixmap::fromImage(data.result()));
         else
@@ -96,7 +96,7 @@ void Images::process()
     const QList<GroupItem> tasks {
         finishAllAndSuccess,
         parallel,
-        repeater,
+        iterator,
         onGroupSetup(onRootSetup),
         Group {
             storage,