From 6fa6ebe8dbc9f1910ea6d91dd6eea20f2c934f65 Mon Sep 17 00:00:00 2001
From: Christian Mollekopf <chrigi_1@fastmail.fm>
Date: Fri, 6 May 2016 20:39:58 +0200
Subject: Properly count replayed results.

We used to count one too many.
---
 common/queryrunner.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'common/queryrunner.cpp')

diff --git a/common/queryrunner.cpp b/common/queryrunner.cpp
index 38fc779..b572d11 100644
--- a/common/queryrunner.cpp
+++ b/common/queryrunner.cpp
@@ -184,11 +184,11 @@ qint64 QueryWorker<DomainType>::replaySet(ResultSet &resultSet, Sink::ResultProv
 {
     Trace() << "Skipping over " << offset << " results";
     resultSet.skip(offset);
-    int counter;
-    for (counter = 0; !batchSize || (counter < batchSize); counter++) {
+    int counter = 0;
+    while (!batchSize || (counter < batchSize)) {
         const bool ret =
             resultSet.next([this, &resultProvider, &counter, &properties, batchSize](const Sink::ApplicationDomain::ApplicationDomainType::Ptr &value, Sink::Operation operation) -> bool {
-                // FIXME allow maildir resource to set the mimeMessage property
+                counter++;
                 auto valueCopy = Sink::ApplicationDomain::ApplicationDomainType::getInMemoryRepresentation<DomainType>(*value, properties).template staticCast<DomainType>();
                 if (mResultTransformation) {
                     mResultTransformation(*valueCopy);
-- 
cgit v1.2.3