From: Jean-Marc Lasgouttes <lasgouttes@lyx.org>
Date: Wed, 24 Apr 2013 10:04:49 +0000 (+0200)
Subject: Fix bug #3199: change-next may leave empty lines behind
X-Git-Tag: 2.1.0beta1~352
X-Git-Url: https://git.lyx.org/gitweb/?a=commitdiff_plain;h=77bf805155609ad869238d5ab69eb325ee83b389;p=lyx.git

Fix bug #3199: change-next may leave empty lines behind

Using Cursor::setCursor or even BufferView::setCursor is often a bad
idea since it does not run DEPM. In this case (and other cases in
f&replace code) it is better to use BufferView::mouseSetCursor (which
should maybe be renamed...).
---

diff --git a/src/lyxfind.cpp b/src/lyxfind.cpp
index a01fbd6309..3338c2f9a6 100644
--- a/src/lyxfind.cpp
+++ b/src/lyxfind.cpp
@@ -417,20 +417,15 @@ bool findPreviousChange(BufferView * bv)
 
 bool findChange(BufferView * bv, bool next)
 {
-	if (bv->cursor().selection()) {
-		// set the cursor at the beginning or at the end of the selection
-		// before searching. Otherwise, the current change will be found.
-		if (next != (bv->cursor().top() > bv->cursor().normalAnchor()))
-			bv->cursor().setCursorToAnchor();
-	}
-
-	DocIterator cur = bv->cursor();
+	Cursor cur(*bv);
+	cur.setCursor(next ? bv->cursor().selectionEnd()
+		      : bv->cursor().selectionBegin());
 
 	// Are we within a change ? Then first search forward (backward),
 	// clear the selection and search the other way around (see the end
 	// of this function). This will avoid changes to be selected half.
 	bool search_both_sides = false;
-	DocIterator tmpcur = cur;
+	Cursor tmpcur = cur;
 	// Leave math first
 	while (tmpcur.inMathed())
 		tmpcur.pop_back();
@@ -449,8 +444,7 @@ bool findChange(BufferView * bv, bool next)
 	if (!findChange(cur, next))
 		return false;
 
-	bv->cursor().setCursor(cur);
-	bv->cursor().resetAnchor();
+	bv->mouseSetCursor(cur, false);
 
 	CursorSlice & tip = cur.top();
 
@@ -478,12 +472,11 @@ bool findChange(BufferView * bv, bool next)
 		}
 	}
 
-	// Now put cursor to end of selection:
-	bv->cursor().setCursor(cur);
-	bv->cursor().setSelection();
-
-	if (search_both_sides) {
-		bv->cursor().setSelection(false);
+	if (!search_both_sides) {
+		// Now set the selection.
+		bv->mouseSetCursor(cur, true);
+	} else {
+		bv->mouseSetCursor(cur, false);
 		findChange(bv, !next);
 	}