]> git.lyx.org Git - lyx.git/blobdiff - src/DocIterator.cpp
Andre's s/getTextClass/textClass/ cleanup.
[lyx.git] / src / DocIterator.cpp
index a9769ff8a5bd9dd172733483a0523d88387432f3..a29f364aa3deab5f974dc6adcb64f25a646bca12 100644 (file)
@@ -14,7 +14,6 @@
 
 #include "DocIterator.h"
 
-#include "debug.h"
 #include "InsetList.h"
 #include "Paragraph.h"
 #include "Text.h"
 
 #include "insets/InsetTabular.h"
 
+#include "support/debug.h"
+
 #include <boost/assert.hpp>
-#include <boost/current_function.hpp>
 
-using std::endl;
+#include <ostream>
 
+using namespace std;
 
 namespace lyx {
 
@@ -59,18 +60,25 @@ DocIterator doc_iterator_end(Inset & inset)
 }
 
 
+LyXErr & operator<<(LyXErr & os, DocIterator const & it)
+{
+       os.stream() << it;
+       return os;
+}
+
+
 Inset * DocIterator::nextInset() const
 {
        BOOST_ASSERT(!empty());
        if (pos() == lastpos())
                return 0;
        if (pos() > lastpos()) {
-               lyxerr << "Should not happen, but it does. " << endl;
+               LYXERR0("Should not happen, but it does. ");
                return 0;
        }
        if (inMathed())
                return nextAtom().nucleus();
-       return paragraph().isInset(pos()) ? paragraph().getInset(pos()) : 0;
+       return paragraph().getInset(pos());
 }
 
 
@@ -88,7 +96,7 @@ Inset * DocIterator::prevInset() const
                else
                        return prevAtom().nucleus();
        }
-       return paragraph().isInset(pos() - 1) ? paragraph().getInset(pos() - 1) : 0;
+       return paragraph().getInset(pos() - 1);
 }
 
 
@@ -131,7 +139,7 @@ Text * DocIterator::text() const
 Paragraph & DocIterator::paragraph() const
 {
        if (!inTexted())
-               lyxerr << *this << endl;
+               LYXERR0(*this);
        BOOST_ASSERT(inTexted());
        return top().paragraph();
 }
@@ -240,24 +248,16 @@ Inset * DocIterator::innerInsetOfType(int code) const
 }
 
 
-void DocIterator::forwardPos(bool ignorecollapsed)
+// This duplicates code above, but is in the critical path.
+// So please think twice before adding stuff
+void DocIterator::forwardPos()
 {
-       //this dog bites his tail
+       // this dog bites his tail
        if (empty()) {
                push_back(CursorSlice(*inset_));
                return;
        }
 
-       Inset * const nextinset = nextInset();
-       // jump over collapsables if they are collapsed
-       // FIXME: the check for asInsetMath() shouldn't be necessary
-       // but math insets do not return a sensible editable() state yet.
-       if (ignorecollapsed && nextinset && (!nextinset->asInsetMath()
-           && nextinset->editable() != Inset::HIGHLY_EDITABLE)) {
-               ++top().pos();
-               return;
-       }
-
        CursorSlice & tip = top();
        //lyxerr << "XXX\n" << *this << endl;
 
@@ -269,12 +269,10 @@ void DocIterator::forwardPos(bool ignorecollapsed)
 
        if (tip.pos() != lastp) {
                // this is impossible for pos() == size()
-               if (inMathed()) {
+               if (inMathed())
                        n = (tip.cell().begin() + tip.pos())->nucleus();
-               } else {
-                       if (paragraph().isInset(tip.pos()))
-                               n = paragraph().getInset(tip.pos());
-               }
+               else
+                       n = paragraph().getInset(tip.pos());
        }
 
        if (n && n->isActive()) {
@@ -295,6 +293,20 @@ void DocIterator::forwardPos(bool ignorecollapsed)
 }
 
 
+void DocIterator::forwardPosIgnoreCollapsed()
+{
+       Inset * const nextinset = nextInset();
+       // FIXME: the check for asInsetMath() shouldn't be necessary
+       // but math insets do not return a sensible editable() state yet.
+       if (nextinset && !nextinset->asInsetMath()
+           && nextinset->editable() != Inset::HIGHLY_EDITABLE) {
+               ++top().pos();
+               return;
+       }
+       forwardPos();
+}
+
+
 void DocIterator::forwardPar()
 {
        forwardPos();
@@ -371,12 +383,10 @@ void DocIterator::backwardPos()
        // move into an inset to the left if possible
        Inset * n = 0;
 
-       if (inMathed()) {
+       if (inMathed())
                n = (top().cell().begin() + top().pos())->nucleus();
-       } else {
-               if (paragraph().isInset(top().pos()))
-                       n = paragraph().getInset(top().pos());
-       }
+       else
+               n = paragraph().getInset(top().pos());
 
        if (n && n->isActive()) {
                push_back(CursorSlice(*n));
@@ -418,6 +428,9 @@ void DocIterator::updateInsets(Inset * inset)
 
 bool DocIterator::fixIfBroken()
 {
+       if (empty())
+               return false;
+
        // Go through the slice stack from the bottom. 
        // Check that all coordinates (idx, pit, pos) are correct and
        // that the inset is the one which is claimed to be there
@@ -429,29 +442,29 @@ bool DocIterator::fixIfBroken()
                if (&cs.inset() != inset) {
                        // the whole slice is wrong, chop off this as well
                        --i;
-                       LYXERR(Debug::DEBUG) << "fixIfBroken(): inset changed" << endl;
+                       LYXERR(Debug::DEBUG, "fixIfBroken(): inset changed");
                        break;
                } else if (cs.idx() > cs.lastidx()) {
                        cs.idx() = cs.lastidx();
                        cs.pit() = cs.lastpit();
                        cs.pos() = cs.lastpos();
-                       LYXERR(Debug::DEBUG) << "fixIfBroken(): idx fixed" << endl;
+                       LYXERR(Debug::DEBUG, "fixIfBroken(): idx fixed");
                        break;
                } else if (cs.pit() > cs.lastpit()) {
                        cs.pit() = cs.lastpit();
                        cs.pos() = cs.lastpos();
-                       LYXERR(Debug::DEBUG) << "fixIfBroken(): pit fixed" << endl;
+                       LYXERR(Debug::DEBUG, "fixIfBroken(): pit fixed");
                        break;
                } else if (cs.pos() > cs.lastpos()) {
                        cs.pos() = cs.lastpos();
-                       LYXERR(Debug::DEBUG) << "fixIfBroken(): pos fixed" << endl;
+                       LYXERR(Debug::DEBUG, "fixIfBroken(): pos fixed");
                        break;
                } else if (i != n - 1 && cs.pos() != cs.lastpos()) {
                        // get inset which is supposed to be in the next slice
                        if (cs.inset().inMathed())
                                inset = (cs.cell().begin() + cs.pos())->nucleus();
-                       else if (cs.paragraph().isInset(cs.pos()))
-                               inset = cs.paragraph().getInset(cs.pos());
+                       else if (Inset * csInset = cs.paragraph().getInset(cs.pos()))
+                               inset = csInset;
                        else {
                                // there are slices left, so there must be another inset
                                break;
@@ -462,7 +475,7 @@ bool DocIterator::fixIfBroken()
        // Did we make it through the whole slice stack? Otherwise there
        // was a problem at slice i, and we have to chop off above
        if (i < n) {
-               LYXERR(Debug::DEBUG) << "fixIfBroken(): cursor chopped at " << i << endl;
+               LYXERR(Debug::DEBUG, "fixIfBroken(): cursor chopped at " << i);
                resize(i + 1);
                return true;
        } else
@@ -470,7 +483,7 @@ bool DocIterator::fixIfBroken()
 }
 
 
-DocIterator::idx_type DocIterator::find(MathData const & cell) const
+int DocIterator::find(MathData const & cell) const
 {
        for (size_t l = 0; l != slices_.size(); ++l) {
                if (slices_[l].asInsetMath() && &slices_[l].cell() == &cell)
@@ -480,7 +493,7 @@ DocIterator::idx_type DocIterator::find(MathData const & cell) const
 }
 
 
-DocIterator::idx_type DocIterator::find(InsetMath const * inset) const 
+int DocIterator::find(InsetMath const * inset) const 
 {
        for (size_t l = 0; l != slices_.size(); ++l) {
                if (slices_[l].asInsetMath() == inset)
@@ -490,20 +503,20 @@ DocIterator::idx_type DocIterator::find(InsetMath const * inset) const
 }
 
 
-void DocIterator::cutOff(DocIterator::idx_type above, std::vector<CursorSlice> & cut)
+void DocIterator::cutOff(int above, vector<CursorSlice> & cut)
 {
-       cut = std::vector<CursorSlice>(slices_.begin() + above + 1, slices_.end());
+       cut = vector<CursorSlice>(slices_.begin() + above + 1, slices_.end());
        slices_.resize(above + 1);
 }
 
 
-void DocIterator::cutOff(DocIterator::idx_type above) 
+void DocIterator::cutOff(int above)
 {
        slices_.resize(above + 1);
 }
 
 
-void DocIterator::append(std::vector<CursorSlice> const & x) 
+void DocIterator::append(vector<CursorSlice> const & x) 
 {
        slices_.insert(slices_.end(), x.begin(), x.end());
 }
@@ -517,7 +530,7 @@ void DocIterator::append(DocIterator::idx_type idx, pos_type pos)
 }
 
 
-std::ostream & operator<<(std::ostream & os, DocIterator const & dit)
+ostream & operator<<(ostream & os, DocIterator const & dit)
 {
        for (size_t i = 0, n = dit.depth(); i != n; ++i)
                os << " " << dit[i] << "\n";
@@ -525,29 +538,6 @@ std::ostream & operator<<(std::ostream & os, DocIterator const & dit)
 }
 
 
-bool operator<(DocIterator const & p, DocIterator const & q)
-{
-       size_t depth = std::min(p.depth(), q.depth());
-       for (size_t i = 0 ; i < depth ; ++i) {
-               if (p[i] != q[i])
-                       return p[i] < q[i];
-       }
-       return p.depth() < q.depth();
-}
-
-
-bool operator>(DocIterator const & p, DocIterator const & q)
-{
-       return q < p;
-}
-
-
-bool operator<=(DocIterator const & p, DocIterator const & q)
-{
-       return !(q < p);
-}
-
-
 ///////////////////////////////////////////////////////
 
 StableDocIterator::StableDocIterator(DocIterator const & dit)
@@ -566,11 +556,11 @@ DocIterator StableDocIterator::asDocIterator(Inset * inset) const
        for (size_t i = 0, n = data_.size(); i != n; ++i) {
                if (inset == 0) {
                        // FIXME
-                       lyxerr << BOOST_CURRENT_FUNCTION
-                              << " Should not happen, but does e.g. after C-n C-l C-z S-C-z\n"
-                                  << " or when a Buffer has been concurently edited by two views"
+                       LYXERR0(" Should not happen, but does e.g. after "
+                               "C-n C-l C-z S-C-z\n"
+                               << " or when a Buffer has been concurrently edited by two views"
                                << '\n' << "dit: " << dit << '\n'
-                               << " lastpos: " << dit.lastpos() << endl;
+                               << " lastpos: " << dit.lastpos());
                        dit.fixIfBroken();
                        break;
                }
@@ -586,7 +576,7 @@ DocIterator StableDocIterator::asDocIterator(Inset * inset) const
 }
 
 
-std::ostream & operator<<(std::ostream & os, StableDocIterator const & dit)
+ostream & operator<<(ostream & os, StableDocIterator const & dit)
 {
        for (size_t i = 0, n = dit.data_.size(); i != n; ++i)
                os << " " << dit.data_[i] << "\n";