Clean-up FileFilterList API.

[lyx.git] / src / dociterator.C

diff --git a/src/dociterator.C b/src/dociterator.C
index 2df577b5f4adfffe1cc6caf21a2fcc57c700a139..8f97436531265301fec1027abc86bcc05c1df413 100644 (file)
--- a/src/dociterator.C
+++ b/src/dociterator.C
@@ -57,6 +57,10 @@ InsetBase * DocIterator::nextInset()
        BOOST_ASSERT(!empty());
        if (pos() == lastpos())
                return 0;
+       if (pos() > lastpos()) {
+               lyxerr << "Should not happen, but it does. " << endl;
+               return 0;
+       }
        if (inMathed())
                return nextAtom().nucleus();
        return paragraph().isInset(pos()) ? paragraph().getInset(pos()) : 0;
@@ -152,7 +156,7 @@ Row const & DocIterator::textRow() const
 }
 
 
-DocIterator::par_type DocIterator::lastpar() const
+DocIterator::pit_type DocIterator::lastpit() const
 {
        return inMathed() ? 0 : text()->paragraphs().size() - 1;
 }
@@ -304,9 +308,9 @@ void DocIterator::forwardPos()
        //lyxerr << "... no next pos" << endl;
 
        // otherwise move on one paragraph if possible
-       if (top.par() < lastpar()) {
+       if (top.pit() < lastpit()) {
                //lyxerr << "... next par" << endl;
-               ++top.par();
+               ++top.pit();
                top.pos() = 0;
                return;
        }
@@ -316,7 +320,7 @@ void DocIterator::forwardPos()
        if (top.idx() < lastidx()) {
                //lyxerr << "... next idx" << endl;
                ++top.idx();
-               top.par() = 0;
+               top.pit() = 0;
                top.pos() = 0;
                return;
        }
@@ -368,7 +372,7 @@ void DocIterator::backwardPos()
        if (empty()) {
                push_back(CursorSlice(*inset_));
                back().idx() = lastidx();
-               back().par() = lastpar();
+               back().pit() = lastpit();
                back().pos() = lastpos();
                return;
        }
@@ -377,13 +381,13 @@ void DocIterator::backwardPos()
 
        if (top.pos() != 0) {
                --top.pos();
-       } else if (top.par() != 0) {
-               --top.par();
+       } else if (top.pit() != 0) {
+               --top.pit();
                top.pos() = lastpos();
                return;
        } else if (top.idx() != 0) {
                --top.idx();
-               top.par() = lastpar();
+               top.pit() = lastpit();
                top.pos() = lastpos();
                return;
        } else {
@@ -404,7 +408,7 @@ void DocIterator::backwardPos()
        if (n && n->isActive()) {
                push_back(CursorSlice(*n));
                back().idx() = lastidx();
-               back().par() = lastpar();
+               back().pit() = lastpit();
                back().pos() = lastpos();
        }
 }
@@ -440,13 +444,20 @@ StableDocIterator::StableDocIterator(const DocIterator & dit)
 }
 
 
-DocIterator
-StableDocIterator::asDocIterator(InsetBase * inset) const
+DocIterator StableDocIterator::asDocIterator(InsetBase * inset) const
 {
        // this function re-creates the cache of inset pointers
        //lyxerr << "converting:\n" << *this << endl;
        DocIterator dit = DocIterator(*inset);
        for (size_t i = 0, n = data_.size(); i != n; ++i) {
+               if (inset == 0) {
+                       // FIXME
+                       lyxerr << "Should not happen, but does e.g. after C-n C-l C-z S-C-z"
+                               << endl << "dit: " << dit << endl
+                               << " lastpos: " << dit.lastpos() << endl;
+                       //break;
+                       BOOST_ASSERT(false);
+               }
                dit.push_back(data_[i]);
                dit.back().inset_ = inset;
                if (i + 1 != n)