From ac3b6f7fea58a7f599bd661339e1d2ac8d0a48b0 Mon Sep 17 00:00:00 2001
From: "METANEOCORTEX\\Kotti" <rainer.kottenhoff@gmail.com>
Date: Fri, 3 Feb 2023 17:29:44 +0100
Subject: [PATCH] +fix: infinite loop using SCI_REPLACEMINIMAL in replace all
 loop in range

---
 src/Edit.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/Edit.c b/src/Edit.c
index 987b90350..4162bb5cc 100644
--- a/src/Edit.c
+++ b/src/Edit.c
@@ -7382,6 +7382,10 @@ bool EditReplace(HWND hwnd, LPEDITFINDREPLACE lpefr)
 //  EditReplaceAllInRange()
 //
 //
+
+// force NOT minimal replacement to calculate next match correctly
+#define REPLACE_ALL_MSG(M) ((M) == SCI_REPLACETARGETRE ? SCI_REPLACETARGETRE : SCI_REPLACETARGET)
+
 int EditReplaceAllInRange(HWND hwnd, LPEDITFINDREPLACE lpefr, DocPos iStartPos, DocPos iEndPos, DocPos *enlargement)
 {
     if (iStartPos > iEndPos) {
@@ -7400,13 +7404,11 @@ int EditReplaceAllInRange(HWND hwnd, LPEDITFINDREPLACE lpefr, DocPos iStartPos,
     bool const bRegexStartOfLine = bIsRegExpr && (wchFind[0] == L'^');
 
     // SCI_REPLACETARGET or SCI_REPLACETARGETRE
-    int iReplaceMsg = SciCall_GetChangeHistory() ? SCI_REPLACETARGETMINIMAL : SCI_REPLACETARGET;
+    int iReplaceMsg = SCI_REPLACETARGET;
     char* const pszReplace = _GetReplaceString(hwnd, lpefr, &iReplaceMsg);
     if (!pszReplace) {
         return -1; // recoding of clipboard canceled
     }
-    bool const bMinRepl = (SCI_REPLACETARGETMINIMAL == iReplaceMsg);
-    DocPos const pszReplaceLen = strlen(pszReplace);
 
     DocPos const _saveTargetBeg_ = SciCall_GetTargetStart();
     DocPos const _saveTargetEnd_ = SciCall_GetTargetEnd();
@@ -7426,14 +7428,14 @@ int EditReplaceAllInRange(HWND hwnd, LPEDITFINDREPLACE lpefr, DocPos iStartPos,
     while ((iPos >= 0LL) && (start <= iEndPos)) {
         DocLn const lnCnt = bRegexStartOfLine ? SciCall_GetLineCount() : 0;
         SciCall_SetTargetRange(iPos, end);
-        DocPos const replLen = Sci_ReplaceTargetEx(iReplaceMsg, -1, pszReplace);
+        DocPos const replLen = Sci_ReplaceTargetEx(REPLACE_ALL_MSG(iReplaceMsg), -1, pszReplace);
         ++iCount;
         // start next search behind replacement
         iStartPos = SciCall_GetTargetEnd();
         if (bRegexStartOfLine && (Sci_GetLineStartPosition(iStartPos) == iStartPos) && (SciCall_GetLineCount() == lnCnt)) {
             iStartPos = SciCall_PositionAfter(iStartPos);
         }
-        iEndPos += (bMinRepl ? pszReplaceLen : replLen) - (end - iPos);
+        iEndPos += replLen - (end - iPos);
         start = iStartPos;
         end   = iEndPos;
         iPos = (start <= end) ? _FindInTarget(wchFind, sFlags, &start, &end, true, FRMOD_NORM) : -1LL;