added values to Find and Replace Column

daddel80 · Jan 20, 2024 · dfa97f4 · dfa97f4
1 parent 674023c
commit dfa97f4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/MultiReplacePanel.cpp b/src/MultiReplacePanel.cpp
@@ -847,10 +847,10 @@ void MultiReplace::resetCountColumns() {
     InvalidateRect(_replaceListView, NULL, TRUE);
 }
 
-void MultiReplace::updateCountColumns(int itemIndex, int findCount, int replaceCount)
+void MultiReplace::updateCountColumns(size_t itemIndex, int findCount, int replaceCount)
 {
     // Check if the itemIndex is valid
-    if (itemIndex < 0 || itemIndex >= replaceListData.size()) {
+    if (itemIndex >= replaceListData.size()) {
         return;
     }
 
@@ -1543,7 +1543,7 @@ void MultiReplace::handleReplaceAllButton() {
             return;
         }
         ::SendMessage(_hScintilla, SCI_BEGINUNDOACTION, 0, 0);
-        for (int i = 0; i < replaceListData.size(); ++i)
+        for (size_t i = 0; i < replaceListData.size(); ++i)
         {
             if (replaceListData[i].isSelected)
             {
@@ -2694,7 +2694,7 @@ void MultiReplace::handleMarkMatchesButton() {
             return;
         }
 
-        for (int i = 0; i < replaceListData.size(); ++i) {
+        for (size_t i = 0; i < replaceListData.size(); ++i) {
             if (replaceListData[i].isSelected) {
                 std::string findTextUtf8 = convertAndExtend(replaceListData[i].findText, replaceListData[i].extended);
                 int searchFlags = (replaceListData[i].wholeWord * SCFIND_WHOLEWORD)

diff --git a/src/MultiReplacePanel.h b/src/MultiReplacePanel.h
@@ -358,7 +358,7 @@ class MultiReplace : public StaticDialog
     void selectRows(const std::vector<ReplaceItemData>& rowsToSelect);
     void handleCopyToListButton();
     void resetCountColumns();
-    void updateCountColumns(int itemIndex, int findCount, int replaceCount = -1);
+    void updateCountColumns(size_t itemIndex, int findCount, int replaceCount = -1);
 
     //Replace
     void handleReplaceAllButton();