Fix the wrong cursor move when forward flag is false:

The forward flag is used to place the cursor behind the replaced text if it's true.
But it's not correct to move the cursor if it's false. The cursor is in front of the
replacement already after the replaceSelectionWithString() was done.

diff --git a/src/lyxfind.cpp b/src/lyxfind.cpp
index 73f6469754fe43b3ca2e903e6c893fac52491622..bd4e04e860e9825fdd4b0961402b792bc979c69e 100644 (file)
--- a/src/lyxfind.cpp
+++ b/src/lyxfind.cpp
@@ -275,9 +275,6 @@ pair<bool, int> replaceOne(BufferView * bv, docstring searchstr,
        if (forward) {
                cur.pos() += replacestr.length();
                LASSERT(cur.pos() <= cur.lastpos(), /* */);
-       } else {
-               cur.pos() -= replacestr.length();
-               LASSERT(cur.pos() >= 0, /* */);
        }
        if (findnext)
                findOne(bv, searchstr, case_sens, whole, forward, false);