X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Flyxfind.C;h=932cbec30e0a31df48b9cd37ab980a076802070f;hb=8765ab59cdddad67284007813ef25934ea0042ce;hp=8ca56cbc4ace2c6ea1f21abbd5e191a6264b3e53;hpb=a6444784dca48da8b1f7723be12069b2e1dbb4c5;p=lyx.git

diff --git a/src/lyxfind.C b/src/lyxfind.C
index 8ca56cbc4a..932cbec30e 100644
--- a/src/lyxfind.C
+++ b/src/lyxfind.C
@@ -18,6 +18,7 @@
 #include "buffer.h"
 #include "cursor.h"
 #include "CutAndPaste.h"
+#include "buffer_funcs.h"
 #include "BufferView.h"
 #include "debug.h"
 #include "funcrequest.h"
@@ -28,51 +29,46 @@
 #include "undo.h"
 
 #include "frontends/Alert.h"
-#include "frontends/LyXView.h"
 
 #include "support/convert.h"
+#include "support/docstream.h"
 
-#include <sstream>
-
-using lyx::support::lowercase;
-using lyx::support::uppercase;
-using lyx::support::split;
+namespace lyx {
 
-using lyx::pit_type;
-using lyx::pos_type;
+using support::lowercase;
+using support::uppercase;
+using support::split;
 
 using std::advance;
-using std::ostringstream;
-using std::string;
 
 
 namespace {
 
-bool parse_bool(string & howto)
+bool parse_bool(docstring & howto)
 {
 	if (howto.empty())
 		return false;
-	string var;
+	docstring var;
 	howto = split(howto, var, ' ');
 	return (var == "1");
 }
 
 
-class MatchString : public std::binary_function<Paragraph, lyx::pos_type, bool>
+class MatchString : public std::binary_function<Paragraph, pos_type, bool>
 {
 public:
-	MatchString(string const & str, bool cs, bool mw)
+	MatchString(docstring const & str, bool cs, bool mw)
 		: str(str), cs(cs), mw(mw)
 	{}
 
 	// returns true if the specified string is at the specified position
-	bool operator()(Paragraph const & par, lyx::pos_type pos) const
+	bool operator()(Paragraph const & par, pos_type pos) const
 	{
-		string::size_type const size = str.length();
-		lyx::pos_type i = 0;
-		lyx::pos_type const parsize = par.size();
+		docstring::size_type const size = str.length();
+		pos_type i = 0;
+		pos_type const parsize = par.size();
 		for (i = 0; pos + i < parsize; ++i) {
-			if (string::size_type(i) >= size)
+			if (docstring::size_type(i) >= size)
 				break;
 			if (cs && str[i] != par.getChar(pos + i))
 				break;
@@ -80,14 +76,14 @@ public:
 				break;
 		}
 
-		if (size != string::size_type(i))
+		if (size != docstring::size_type(i))
 			return false;
 
 		// if necessary, check whether string matches word
 		if (mw) {
 			if (pos > 0 && par.isLetter(pos - 1))
 				return false;
-			if (pos + lyx::pos_type(size) < parsize
+			if (pos + pos_type(size) < parsize
 			    && par.isLetter(pos + size))
 				return false;
 		}
@@ -97,7 +93,7 @@ public:
 
 private:
 	// search string
-	string str;
+	docstring str;
 	// case sensitive
 	bool cs;
 	// match whole words only
@@ -128,24 +124,24 @@ bool findBackwards(DocIterator & cur, MatchString const & match)
 bool findChange(DocIterator & cur)
 {
 	for (; cur; cur.forwardPos())
-		if (cur.inTexted() && cur.paragraph().lookupChange(cur.pos()).type
-		    != Change::UNCHANGED)
+		if (cur.inTexted() && !cur.paragraph().isUnchanged(cur.pos()))
 			return true;
 	return false;
 }
 
 
-bool searchAllowed(BufferView * bv, string const & str)
+bool searchAllowed(BufferView * bv, docstring const & str)
 {
 	if (str.empty()) {
-		Alert::error(_("Search error"), _("Search string is empty"));
+		frontend::Alert::error(_("Search error"),
+		                            _("Search string is empty"));
 		return false;
 	}
-	return bv->available();
+	return bv->buffer();
 }
 
 
-bool find(BufferView * bv, string const & searchstr, bool cs, bool mw, bool fw)
+bool find(BufferView * bv, docstring const & searchstr, bool cs, bool mw, bool fw)
 {
 	if (!searchAllowed(bv, searchstr))
 		return false;
@@ -164,7 +160,7 @@ bool find(BufferView * bv, string const & searchstr, bool cs, bool mw, bool fw)
 
 
 int replaceAll(BufferView * bv,
-	       string const & searchstr, string const & replacestr,
+	       docstring const & searchstr, docstring const & replacestr,
 	       bool cs, bool mw)
 {
 	Buffer & buf = *bv->buffer();
@@ -182,17 +178,20 @@ int replaceAll(BufferView * bv,
 
 	DocIterator cur = doc_iterator_begin(buf.inset());
 	while (findForward(cur, match)) {
-		lyx::pos_type pos = cur.pos();
+		pos_type pos = cur.pos();
 		LyXFont const font
 			= cur.paragraph().getFontSettings(buf.params(), pos);
-		int striked = ssize - cur.paragraph().erase(pos, pos + ssize);
-		cur.paragraph().insert(pos, replacestr, font);
+		int striked = ssize - cur.paragraph().eraseChars(pos, pos + ssize,
+							    buf.params().trackChanges);
+		cur.paragraph().insert(pos, replacestr, font,
+		                       Change(buf.params().trackChanges ?
+		                              Change::INSERTED : Change::UNCHANGED));
 		for (int i = 0; i < rsize + striked; ++i)
 			cur.forwardChar();
 		++num;
 	}
 
-	bv->text()->init(bv);
+	updateLabels(buf);
 	bv->putSelectionAt(doc_iterator_begin(buf.inset()), 0, false);
 	if (num)
 		buf.markDirty();
@@ -200,13 +199,13 @@ int replaceAll(BufferView * bv,
 }
 
 
-bool stringSelected(BufferView * bv, string const & searchstr,
+bool stringSelected(BufferView * bv, docstring const & searchstr,
 		    bool cs, bool mw, bool fw)
 {
 	// if nothing selected or selection does not equal search
 	// string search and select next occurance and return
-	string const & str1 = searchstr;
-	string const str2 = lyx::to_utf8(bv->cursor().selectionAsString(false));
+	docstring const & str1 = searchstr;
+	docstring const str2 = bv->cursor().selectionAsString(false);
 	if ((cs && str1 != str2) || lowercase(str1) != lowercase(str2)) {
 		find(bv, searchstr, cs, mw, fw);
 		return false;
@@ -216,8 +215,8 @@ bool stringSelected(BufferView * bv, string const & searchstr,
 }
 
 
-int replace(BufferView * bv, string const & searchstr,
-	    string const & replacestr, bool cs, bool mw, bool fw)
+int replace(BufferView * bv, docstring const & searchstr,
+	    docstring const & replacestr, bool cs, bool mw, bool fw)
 {
 	if (!searchAllowed(bv, searchstr) || bv->buffer()->isReadonly())
 		return 0;
@@ -226,7 +225,7 @@ int replace(BufferView * bv, string const & searchstr,
 		return 0;
 
 	LCursor & cur = bv->cursor();
-	lyx::cap::replaceSelectionWithString(cur, replacestr, fw);
+	cap::replaceSelectionWithString(cur, replacestr, fw);
 	bv->buffer()->markDirty();
 	find(bv, searchstr, cs, mw, fw);
 	bv->update();
@@ -237,13 +236,10 @@ int replace(BufferView * bv, string const & searchstr,
 } // namespace anon
 
 
-namespace lyx {
-namespace find {
-
-string const find2string(string const & search,
+docstring const find2string(docstring const & search,
 			 bool casesensitive, bool matchword, bool forward)
 {
-	ostringstream ss;
+	odocstringstream ss;
 	ss << search << '\n'
 	   << int(casesensitive) << ' '
 	   << int(matchword) << ' '
@@ -252,11 +248,11 @@ string const find2string(string const & search,
 }
 
 
-string const replace2string(string const & search, string const & replace,
+docstring const replace2string(docstring const & search, docstring const & replace,
 			    bool casesensitive, bool matchword,
 			    bool all, bool forward)
 {
-	ostringstream ss;
+	odocstringstream ss;
 	ss << search << '\n'
 	   << replace << '\n'
 	   << int(casesensitive) << ' '
@@ -277,14 +273,14 @@ void find(BufferView * bv, FuncRequest const & ev)
 	// data is of the form
 	// "<search>
 	//  <casesensitive> <matchword> <forward>"
-	string search;
-	string howto = split(lyx::to_utf8(ev.argument()), search, '\n');
+	docstring search;
+	docstring howto = split(ev.argument(), search, '\n');
 
 	bool casesensitive = parse_bool(howto);
 	bool matchword     = parse_bool(howto);
 	bool forward       = parse_bool(howto);
 
-	bool const found = ::find(bv, search,
+	bool const found = find(bv, search,
 				  casesensitive, matchword, forward);
 
 	if (!found)
@@ -302,10 +298,10 @@ void replace(BufferView * bv, FuncRequest const & ev)
 	// "<search>
 	//  <replace>
 	//  <casesensitive> <matchword> <all> <forward>"
-	string search;
-	string replace;
-	string howto = split(lyx::to_utf8(ev.argument()), search, '\n');
-	howto = split(howto, replace, '\n');
+	docstring search;
+	docstring rplc;
+	docstring howto = split(ev.argument(), search, '\n');
+	howto = split(howto, rplc, '\n');
 
 	bool casesensitive = parse_bool(howto);
 	bool matchword     = parse_bool(howto);
@@ -315,8 +311,8 @@ void replace(BufferView * bv, FuncRequest const & ev)
 	Buffer * buf = bv->buffer();
 
 	int const replace_count = all
-		? ::replaceAll(bv, search, replace, casesensitive, matchword)
-		: ::replace(bv, search, replace, casesensitive, matchword, forward);
+		? replaceAll(bv, search, rplc, casesensitive, matchword)
+		: replace(bv, search, rplc, casesensitive, matchword, forward);
 
 	if (replace_count == 0) {
 		// emit message signal.
@@ -337,7 +333,7 @@ void replace(BufferView * bv, FuncRequest const & ev)
 
 bool findNextChange(BufferView * bv)
 {
-	if (!bv->available())
+	if (!bv->buffer())
 		return false;
 
 	DocIterator cur = bv->cursor();
@@ -351,21 +347,26 @@ bool findNextChange(BufferView * bv)
 	Change orig_change = cur.paragraph().lookupChange(cur.pos());
 
 	DocIterator et = doc_iterator_end(cur.inset());
+	DocIterator ok = cur;   // see below
 	for (; cur != et; cur.forwardPosNoDescend()) {
+		ok = cur;
 		Change change = cur.paragraph().lookupChange(cur.pos());
 		if (change != orig_change) {
 			break;
 		}
 	}
+
+	// avoid crash (assertion violation) if the imaginary end-of-par
+	// character of the last paragraph of the document is marked as changed 
+	if (cur == et) {
+		cur = ok;
+	}
+
 	// Now put cursor to end of selection:
 	bv->cursor().setCursor(cur);
 	bv->cursor().setSelection();
-	// if we used a lfun like in find/replace, dispatch would do
-	// that for us
-	bv->update();
 
 	return true;
 }
 
-} // find namespace
 } // lyx namespace