Fulfill promise to Andre: TextClass_ptr --> TextClassPtr.

[lyx.git] / src / DocIterator.cpp

diff --git a/src/DocIterator.cpp b/src/DocIterator.cpp
index ea954acaf2c04d5df5a498c4ba35228d65f8b41e..07e0d6f5e0d012a7e24dfeb18400db2c3cae2789 100644 (file)
--- a/src/DocIterator.cpp
+++ b/src/DocIterator.cpp
@@ -58,7 +58,7 @@ DocIterator doc_iterator_end(Inset & inset)
 }
 
 
-Inset * DocIterator::nextInset()
+Inset * DocIterator::nextInset() const
 {
        BOOST_ASSERT(!empty());
        if (pos() == lastpos())
@@ -73,29 +73,12 @@ Inset * DocIterator::nextInset()
 }
 
 
-Inset * DocIterator::prevInset()
+Inset * DocIterator::prevInset() const
 {
        BOOST_ASSERT(!empty());
        if (pos() == 0)
                return 0;
-       if (inMathed())
-               if (cell().empty())
-                       // FIXME: this should not happen but it does.
-                       // See bug 3189
-                       // http://bugzilla.lyx.org/show_bug.cgi?id=3189
-                       return 0;
-               else
-                       return prevAtom().nucleus();
-       return paragraph().isInset(pos() - 1) ? paragraph().getInset(pos() - 1) : 0;
-}
-
-
-Inset const * DocIterator::prevInset() const
-{
-       BOOST_ASSERT(!empty());
-       if (pos() == 0)
-               return 0;
-       if (inMathed())
+       if (inMathed()) {
                if (cell().empty())
                        // FIXME: this should not happen but it does.
                        // See bug 3189
@@ -103,6 +86,7 @@ Inset const * DocIterator::prevInset() const
                        return 0;
                else
                        return prevAtom().nucleus();
+       }
        return paragraph().isInset(pos() - 1) ? paragraph().getInset(pos() - 1) : 0;
 }
 
@@ -119,15 +103,7 @@ Inset * DocIterator::realInset() const
 }
 
 
-MathAtom const & DocIterator::prevAtom() const
-{
-       BOOST_ASSERT(!empty());
-       BOOST_ASSERT(pos() > 0);
-       return cell()[pos() - 1];
-}
-
-
-MathAtom & DocIterator::prevAtom()
+MathAtom & DocIterator::prevAtom() const
 {
        BOOST_ASSERT(!empty());
        BOOST_ASSERT(pos() > 0);
@@ -135,16 +111,7 @@ MathAtom & DocIterator::prevAtom()
 }
 
 
-MathAtom const & DocIterator::nextAtom() const
-{
-       BOOST_ASSERT(!empty());
-       //lyxerr << "lastpos: " << lastpos() << " next atom:\n" << *this << endl;
-       BOOST_ASSERT(pos() < lastpos());
-       return cell()[pos()];
-}
-
-
-MathAtom & DocIterator::nextAtom()
+MathAtom & DocIterator::nextAtom() const
 {
        BOOST_ASSERT(!empty());
        //lyxerr << "lastpos: " << lastpos() << " next atom:\n" << *this << endl;
@@ -153,20 +120,14 @@ MathAtom & DocIterator::nextAtom()
 }
 
 
-Text * DocIterator::text()
-{
-       BOOST_ASSERT(!empty());
-       return top().text();
-}
-
-Text const * DocIterator::text() const
+Text * DocIterator::text() const
 {
        BOOST_ASSERT(!empty());
        return top().text();
 }
 
 
-Paragraph & DocIterator::paragraph()
+Paragraph & DocIterator::paragraph() const
 {
        if (!inTexted())
                lyxerr << *this << endl;
@@ -175,26 +136,27 @@ Paragraph & DocIterator::paragraph()
 }
 
 
-Paragraph const & DocIterator::paragraph() const
+Paragraph & DocIterator::innerParagraph() const
 {
-       BOOST_ASSERT(inTexted());
-       return top().paragraph();
+       BOOST_ASSERT(!empty());
+       return innerTextSlice().paragraph();
 }
 
 
-Paragraph const & DocIterator::innerParagraph() const
+CursorSlice const & DocIterator::innerTextSlice() const
 {
        BOOST_ASSERT(!empty());
        // go up until first non-0 text is hit
        // (innermost text is 0 in mathed)
        for (int i = depth() - 1; i >= 0; --i)
                if (slices_[i].text())
-                       return slices_[i].paragraph();
+                       return slices_[i];
 
        // This case is in principe not possible. We _must_
-       // be inside a Paragraph.
+       // be inside a Text.
        BOOST_ASSERT(false);
-       return paragraph();
+       static CursorSlice dummy;
+       return dummy;
 }
 
 
@@ -249,32 +211,14 @@ DocIterator::col_type DocIterator::col() const
 }
 
 
-MathData const & DocIterator::cell() const
-{
-//     BOOST_ASSERT(inMathed());
-       return top().cell();
-}
-
-
-MathData & DocIterator::cell()
+MathData & DocIterator::cell() const
 {
 //     BOOST_ASSERT(inMathed());
        return top().cell();
 }
 
 
-Text * DocIterator::innerText()
-{
-       BOOST_ASSERT(!empty());
-       // Go up until first non-0 text is hit
-       // (innermost text is 0 in mathed)
-       for (int i = depth() - 1; i >= 0; --i)
-               if (slices_[i].text())
-                       return slices_[i].text();
-       return 0;
-}
-
-Text const * DocIterator::innerText() const
+Text * DocIterator::innerText() const
 {
        BOOST_ASSERT(!empty());
        // go up until first non-0 text is hit
@@ -370,19 +314,6 @@ void DocIterator::forwardPar()
 }
 
 
-void DocIterator::forwardIdx()
-{
-       CursorSlice & tip = top();
-
-       //prevent endless loops
-       BOOST_ASSERT(tip.idx() < lastidx());
-
-       ++tip.idx();
-       tip.pit() = 0;
-       tip.pos() = 0;
-}
-
-
 void DocIterator::forwardChar()
 {
        forwardPos();