commit 4d0b12ef94ee38f1b0f55b7deb681f202f79f62a
author Andrew Jeffery <andrew@aj.id.au> Wed Apr 13 17:53:23 2022 +0930
committer Ed Tanous <ed@tanous.net> Fri May 06 21:29:37 2022 +0000
tree 016cf5878354de84207a5aebb98ab3777c333ffa
parent 8bc12449f2206ecf4a14112656b1e28413da0934

PerformScan: Avoid `continue` when updating unused indexes

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I0b86a2d79710f0c549a24d5dda05fa272c7a32c5

src/PerformScan.cpp

diff --git a/src/PerformScan.cpp b/src/PerformScan.cpp
index c4acf5b..7450fa1 100644
--- a/src/PerformScan.cpp
+++ b/src/PerformScan.cpp
@@ -374,11 +374,10 @@
                     auto usedIt =
                         std::find(indexes.begin(), indexes.end(), index);
 
-                    if (usedIt == indexes.end())
+                    if (usedIt != indexes.end())
                     {
-                        continue; // less items now
+                        indexes.erase(usedIt);
                     }
-                    indexes.erase(usedIt);
                 }
 
                 std::optional<std::string> replaceStr;