fix regex error message

far/changelog

@@ -1,3 +1,7 @@
+drkns 22.07.2017 13:25:18 +0000 - build 4992
+
+1. С форума: Ошибочная индикация позиции ошибки в регэкспе
+
 drkns 21.07.2017 19:29:12 +0000 - build 4991
 
 1. Уточнение 4989.

far/editor.cpp

@@ -3430,18 +3430,16 @@ bool Editor::Search(bool Next)
 
 		auto CurPtr = FindAllReferences? FirstLine() : m_it_CurLine, TmpPtr = CurPtr;
 
-		string strSlash(strSearchStr);
-		InsertRegexpQuote(strSlash);
 		std::vector<RegExpMatch> m;
 		MatchHash hm;
 		RegExp re;
 
 		if (Regexp)
 		{
 			// Q: что важнее: опция диалога или опция RegExp`а?
-			if (!re.Compile(strSlash.data(), OP_PERLSTYLE|OP_OPTIMIZE|(!Case?OP_IGNORECASE:0)))
+			if (!re.Compile(strSearchStr.data(), OP_OPTIMIZE | (Case? 0 : OP_IGNORECASE)))
 			{
-				ReCompileErrorMessage(re, strSlash);
+				ReCompileErrorMessage(re, strSearchStr);
 				return false; //BUGBUG
 			}
 			m.resize(re.GetBracketsCount() * 2);

far/help.cpp

@@ -1896,18 +1896,16 @@ void Help::Search(const os::fs::file& HelpFile,uintptr_t nCodePage)
 	GetFileString GetStr(HelpFile, nCodePage);
 	string strCurTopic, strEntryName, strReadStr;
 
-	string strSlash(strLastSearchStr);
-	InsertRegexpQuote(strSlash);
 	std::vector<RegExpMatch> m;
 	MatchHash hm;
 	RegExp re;
 
 	if (LastSearchRegexp)
 	{
 		// Q: что важнее: опция диалога или опция RegExp`а?
-		if (!re.Compile(strSlash.data(), OP_PERLSTYLE|OP_OPTIMIZE|(!LastSearchCase?OP_IGNORECASE:0)))
+		if (!re.Compile(strLastSearchStr.data(), OP_OPTIMIZE | (LastSearchCase? 0 : OP_IGNORECASE)))
 		{
-			ReCompileErrorMessage(re, strSlash);
+			ReCompileErrorMessage(re, strLastSearchStr);
 			return; //BUGBUG
 		}
 

far/strmix.cpp

@@ -138,18 +138,6 @@ string& InsertQuote(string &strStr)
 	return InsertCustomQuote(strStr,L'\"');
 }
 
-string& InsertRegexpQuote(string &strStr)
-{
-	//выражение вида /regexp/i не дополняем слешами
-	if (!strStr.empty() && strStr[0] != L'/')
-	{
-		strStr.insert(0, 1, L'/');
-		strStr += L'/';
-	}
-
-	return strStr;
-}
-
 string &QuoteSpace(string &strStr)
 {
 	if (strStr.find_first_of(Global->Opt->strQuotedSymbols) != string::npos)

far/strmix.hpp

@@ -60,7 +60,6 @@ inline string InsertQuote(string&& strStr) { InsertQuote(strStr); return strStr;
 string& Unquote(string &strStr);
 inline string Unquote(string&& strStr) { Unquote(strStr); return strStr; }
 
-string& InsertRegexpQuote(string& strStr);
 void UnquoteExternal(string &strStr);
 string& RemoveLeadingSpaces(string &strStr);
 string& RemoveTrailingSpaces(string &strStr);

far/vbuild.m4

@@ -1 +1 @@
-m4_define(BUILD,4991)m4_dnl
+m4_define(BUILD,4992)m4_dnl

far/viewer.cpp

@@ -3429,16 +3429,15 @@ void Viewer::Search(int Next,int FirstChar)
 		sd.ch_size = getCharSize();
 		sd.search_text = strSearchStr.data();
 
+		InsertQuote(strMsgStr);
+
 		if (SearchRegexp)
 		{
 			WholeWords = false;
 			searcher = (ReverseSearch ? &Viewer::search_regex_backward : &Viewer::search_regex_forward);
-			InsertRegexpQuote(strMsgStr);
-			string strSlash = strSearchStr;
-			InsertRegexpQuote(strSlash);
-			if (!sd.InitRegEx(strSlash, OP_PERLSTYLE | OP_OPTIMIZE | (Case ? 0 : OP_IGNORECASE)))
+			if (!sd.InitRegEx(strSearchStr, OP_OPTIMIZE | (Case ? 0 : OP_IGNORECASE)))
 			{
-				ReCompileErrorMessage(*sd.pRex, strSlash);
+				ReCompileErrorMessage(*sd.pRex, strSearchStr);
 				return; // wrong regular expression...
 			}
 			sd.RexMatchCount = sd.pRex->GetBracketsCount();
@@ -3447,7 +3446,6 @@ void Viewer::Search(int Next,int FirstChar)
 		else
 		{
 			searcher = (ReverseSearch ? &Viewer::search_text_backward : &Viewer::search_text_forward);
-			InsertQuote(strMsgStr);
 		}
 	}