]> git.lyx.org Git - features.git/commitdiff
backport fix for #7661 to branch - check buffer of cursors before using compare opera...
authorStephan Witt <switt@lyx.org>
Mon, 4 Jul 2011 18:39:25 +0000 (18:39 +0000)
committerStephan Witt <switt@lyx.org>
Mon, 4 Jul 2011 18:39:25 +0000 (18:39 +0000)
git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/branches/BRANCH_2_0_X@39245 a592a061-630c-0410-9148-cb99ea01b6c8

src/frontends/qt4/GuiSpellchecker.cpp
status.20x

index f6da10a9ebd28828a65bc86ff108f50f50b96fe4..5e7305e75a9897ffe7409753e6d20b9d33e4290a 100644 (file)
@@ -79,6 +79,8 @@ struct SpellcheckerWidget::Private
                return false;
        }
        void canCheck() { incheck_ = false; }
+       /// check for wrap around of current position
+       bool isWrapAround(DocIterator cursor) const;
        ///
        Ui::SpellcheckerUi ui;
        ///
@@ -200,6 +202,11 @@ bool SpellcheckerWidget::Private::continueFromBeginning()
        return true;
 }
 
+bool SpellcheckerWidget::Private::isWrapAround(DocIterator cursor) const
+{
+       return wrap_around_ && start_.buffer() == cursor.buffer() && start_ < cursor;
+}
+
 
 void SpellcheckerWidget::Private::forward()
 {
@@ -216,7 +223,7 @@ void SpellcheckerWidget::Private::forward()
                //FIXME we must be at the end of a cell
                dispatch(FuncRequest(LFUN_CHAR_FORWARD));
        }
-       if (wrap_around_ && start_ < bv->cursor()) {
+       if (isWrapAround(bv->cursor())) {
                dv_->hide();
        }
 }
@@ -397,7 +404,7 @@ void SpellcheckerWidget::Private::check()
                return;
        }
 
-       if (wrap_around_ && start_ < from) {
+       if (isWrapAround(from)) {
                dv_->hide();
                return;
        }
index 0d2911de4ada6bb840578421303fe4997637923a..ce3e1d94825878f035a3f1cb3c3d463b7d8da683 100644 (file)
@@ -198,6 +198,9 @@ What's new
 
 - Don't assert when accepting/rejecting non-existent changes (bug 7654).
 
+- Fix assertion when using "Ignore All" in spell checker dialog
+  after changing buffer view (bug 7661).
+
 
 * ADVANCED FIND AND REPLACE