Merge pull request #539 from MKadaner/mzk-search-string

Fixed two bugs in the string search algorithm.

Author: alabuzhev

far/changelog

@@ -1,3 +1,8 @@
+--------------------------------------------------------------------------------
+MZK 06.08.2022 18:26:39 -0700 - build 6010
+
+1. Fixed two bugs in the string search algorithm.
+
 --------------------------------------------------------------------------------
 drkns 06.08.2022 15:34:28 +0100 - build 6009
 

far/findfile.cpp

@@ -1222,9 +1222,9 @@ bool background_searcher::LookForString(string_view const FileName)
 
 				string_view Where{ buffer, bufferCount };
 
-				const auto Next = [&](size_t const Offset, size_t const Size)
+				const auto Next = [&](size_t const Offset)
 				{
-					Where.remove_prefix(Offset + Size);
+					Where.remove_prefix(Offset + 1);
 				};
 
 				while (!Where.empty())
@@ -1262,7 +1262,7 @@ bool background_searcher::LookForString(string_view const FileName)
 							firstWordDiv = true;
 					}
 
-					// Проверяем разделитель в конце, только если найден разделитель вначале
+					// Проверяем разделитель в конце, только если найден разделитель в начале
 					if (firstWordDiv)
 					{
 						// Если блок выбран не до конца
@@ -1278,7 +1278,7 @@ bool background_searcher::LookForString(string_view const FileName)
 							i.WordFound = true;
 					}
 
-					Next(FoundOffset, FoundSize);
+					Next(FoundOffset);
 				}
 
 				// Выходим, если мы вышли за пределы количества байт разрешённых для поиска

far/strmix.cpp

@@ -920,11 +920,11 @@ bool SearchAndReplaceString(
 		Haystack.substr(0, Position + 1) :
 		Haystack.substr(Position);
 
-	const auto Next = [&](size_t const Offset, size_t const Size)
+	const auto Next = [&](size_t const Offset)
 	{
 		Where = Reverse?
-			Where.substr(0, Offset - Size + 1) :
-			Where.substr(Offset + Size);
+			Where.substr(0, Offset > 0? Offset - 1 : 0) :
+			Where.substr(Offset + 1);
 	};
 
 	while (!Where.empty())
@@ -939,7 +939,7 @@ bool SearchAndReplaceString(
 
 		if (WholeWords && !CanContainWholeWord(Haystack, AbsoluteOffset, FoundSize, WordDiv))
 		{
-			Next(FoundOffset, FoundSize);
+			Next(FoundOffset);
 			continue;
 		}
 

far/vbuild.m4

@@ -1 +1 @@
-6009
+6010