]> git.lyx.org Git - lyx.git/blobdiff - src/cursor.C
fix table crash;
[lyx.git] / src / cursor.C
index 52d971d211d9b399c73337a6287f634f68ac9d6a..6e91d9d9c27fb770b5080201289244e5f302c290 100644 (file)
@@ -20,7 +20,6 @@
 #include "dispatchresult.h"
 #include "encoding.h"
 #include "funcrequest.h"
-#include "iterators.h"
 #include "language.h"
 #include "lfuns.h"
 #include "lyxfont.h"
@@ -30,6 +29,7 @@
 #include "lyxtext.h"
 #include "paragraph.h"
 #include "paragraph_funcs.h"
+#include "pariterator.h"
 
 #include "insets/updatableinset.h"
 #include "insets/insettabular.h"
@@ -84,8 +84,9 @@ void region(CursorSlice const & i1, CursorSlice const & i2,
 
 
 LCursor::LCursor(BufferView & bv)
-       : DocumentIterator(), bv_(&bv), anchor_(),
-         cached_y_(0), x_target_(-1), selection_(false), mark_(false)
+       : DocIterator(), bv_(&bv),
+         anchor_(), cached_y_(0), x_target_(-1),
+         selection_(false), mark_(false)
 {}
 
 
@@ -93,7 +94,7 @@ void LCursor::reset(InsetBase & inset)
 {
        clear();
        push_back(CursorSlice(inset));
-       anchor_.clear();
+       anchor_ = DocIterator(inset);
        cached_y_ = 0;
        clearTargetX();
        selection_ = false;
@@ -101,10 +102,10 @@ void LCursor::reset(InsetBase & inset)
 }
 
 
-void LCursor::setCursor(DocumentIterator const & cur, bool sel)
+void LCursor::setCursor(DocIterator const & cur, bool sel)
 {
        // this (intentionally) does not touch the anchor
-       DocumentIterator::operator=(cur);
+       DocIterator::operator=(cur);
        selection() = sel;
 }
 
@@ -124,7 +125,7 @@ DispatchResult LCursor::dispatch(FuncRequest const & cmd0)
                BOOST_ASSERT(idx() <= lastidx());
                BOOST_ASSERT(par() <= lastpar());
 
-               // The common case is 'LFUN handled, need update', so make the 
+               // The common case is 'LFUN handled, need update', so make the
                // LFUN handler's life easier by assuming this as default value.
                // The handler can reset the update and val flags if necessary.
                disp_.update(true);
@@ -202,7 +203,7 @@ bool LCursor::popLeft()
 {
        BOOST_ASSERT(!empty());
        //lyxerr << "Leaving inset to the left" << endl;
-       inset().notifyCursorLeaves(idx());
+       inset().notifyCursorLeaves(*this);
        if (depth() == 1)
                return false;
        pop();
@@ -214,7 +215,7 @@ bool LCursor::popRight()
 {
        BOOST_ASSERT(!empty());
        //lyxerr << "Leaving inset to the right" << endl;
-       inset().notifyCursorLeaves(idx());
+       inset().notifyCursorLeaves(*this);
        if (depth() == 1)
                return false;
        pop();
@@ -315,15 +316,27 @@ bool LCursor::posRight()
 
 CursorSlice & LCursor::anchor()
 {
+       if (anchor_.size() < size()) {
+               lyxerr << "anchor_.size() < cursor_.size() "
+                       "should not happen when accessing the anchor" << endl;
+               BOOST_ASSERT(false);
+       }
        BOOST_ASSERT(!anchor_.empty());
-       return anchor_.back();
+       // this size is cursor_.size()
+       return anchor_[size() - 1];
 }
 
 
 CursorSlice const & LCursor::anchor() const
 {
+       if (anchor_.size() < size()) {
+               lyxerr << "anchor_.size() < cursor_.size() "
+                       "should not happen when accessing the anchor" << endl;
+               BOOST_ASSERT(false);
+       }
+       // this size is cursor_.size()
        BOOST_ASSERT(!anchor_.empty());
-       return anchor_.back();
+       return anchor_[size() - 1];
 }
 
 
@@ -343,7 +356,7 @@ CursorSlice const & LCursor::selEnd() const
 }
 
 
-DocumentIterator LCursor::selectionBegin() const
+DocIterator LCursor::selectionBegin() const
 {
        if (!selection())
                return *this;
@@ -351,7 +364,7 @@ DocumentIterator LCursor::selectionBegin() const
 }
 
 
-DocumentIterator LCursor::selectionEnd() const
+DocIterator LCursor::selectionEnd() const
 {
        if (!selection())
                return *this;
@@ -363,15 +376,15 @@ void LCursor::setSelection()
 {
        selection() = true;
        // a selection with no contents is not a selection
+#warning doesnt look ok
        if (par() == anchor().par() && pos() == anchor().pos())
                selection() = false;
 }
 
 
-void LCursor::setSelection(DocumentIterator const & where, size_t n)
+void LCursor::setSelection(DocIterator const & where, size_t n)
 {
-       selection() = true;
-       setCursor(where, false);
+       setCursor(where, true);
        anchor_ = where;
        pos() += n;
 }
@@ -462,7 +475,9 @@ void LCursor::eraseSelection()
        //lyxerr << "LCursor::eraseSelection" << endl;
        CursorSlice const & i1 = selBegin();
        CursorSlice const & i2 = selEnd();
+#ifdef WITH_WARNINGS
 #warning FIXME
+#endif
        if (i1.inset().asMathInset()) {
                if (i1.idx() == i2.idx()) {
                        i1.cell().erase(i1.pos(), i2.pos());
@@ -494,13 +509,6 @@ string LCursor::grabAndEraseSelection()
 }
 
 
-void LCursor::selClear()
-{
-       resetAnchor();
-       clearSelection();
-}
-
-
 void LCursor::selCopy()
 {
        if (selection()) {
@@ -591,7 +599,6 @@ std::ostream & operator<<(std::ostream & os, LCursor const & cur)
 #include "mathed/math_factory.h"
 #include "mathed/math_gridinset.h"
 #include "mathed/math_macroarg.h"
-#include "mathed/math_macrotemplate.h"
 #include "mathed/math_mathmlstream.h"
 #include "mathed/math_scriptinset.h"
 #include "mathed/math_support.h"
@@ -630,8 +637,8 @@ bool LCursor::openable(MathAtom const & t) const
 }
 
 
-bool positionable(DocumentIterator const & cursor,
-       DocumentIterator const & anchor)
+bool positionable(DocIterator const & cursor,
+       DocIterator const & anchor)
 {
        // avoid deeper nested insets when selecting
        if (cursor.size() > anchor.size())
@@ -663,7 +670,7 @@ void LCursor::plainErase()
 
 void LCursor::markInsert()
 {
-       cell().insert(pos(), MathAtom(new MathCharInset(0)));
+       insert(char(0));
 }
 
 
@@ -682,7 +689,7 @@ void LCursor::plainInsert(MathAtom const & t)
 
 void LCursor::insert(string const & str)
 {
-       lyxerr << "LCursor::insert str '" << str << "'" << endl;
+       //lyxerr << "LCursor::insert str '" << str << "'" << endl;
        for (string::const_iterator it = str.begin(); it != str.end(); ++it)
                insert(*it);
 }
@@ -694,7 +701,7 @@ void LCursor::insert(char c)
        BOOST_ASSERT(!empty());
        if (inMathed()) {
                selClearOrDel();
-               plainInsert(MathAtom(new MathCharInset(c)));
+               insert(new MathCharInset(c));
        } else {
                text()->insertChar(*this, c);
        }
@@ -707,6 +714,7 @@ void LCursor::insert(MathAtom const & t)
        macroModeClose();
        selClearOrDel();
        plainInsert(t);
+       lyxerr << "LCursor::insert MathAtom: cur:\n" << *this << endl;
 }
 
 
@@ -783,6 +791,7 @@ bool LCursor::backspace()
        if (pos() != 0 && prevAtom()->nargs() > 0) {
                // let's require two backspaces for 'big stuff' and
                // highlight on the first
+               resetAnchor();
                selection() = true;
                --pos();
        } else {
@@ -821,7 +830,9 @@ bool LCursor::erase()
                return true;
        }
 
+       // 'clever' UI hack: only erase large items if previously slected
        if (pos() != lastpos() && inset().nargs() > 0) {
+               resetAnchor();
                selection() = true;
                ++pos();
        } else {
@@ -835,7 +846,7 @@ bool LCursor::erase()
 bool LCursor::up()
 {
        macroModeClose();
-       DocumentIterator save = *this;
+       DocIterator save = *this;
        if (goUpDown(true))
                return true;
        setCursor(save, false);
@@ -847,7 +858,7 @@ bool LCursor::up()
 bool LCursor::down()
 {
        macroModeClose();
-       DocumentIterator save = *this;
+       DocIterator save = *this;
        if (goUpDown(false))
                return true;
        setCursor(save, false);
@@ -936,31 +947,11 @@ MathUnknownInset * LCursor::activeMacro()
 }
 
 
-bool LCursor::inMacroArgMode() const
-{
-       return pos() > 0 && prevAtom()->getChar() == '#';
-}
-
-
-MathGridInset * LCursor::enclosingGrid(idx_type & idx) const
-{
-       for (MathInset::difference_type i = depth() - 1; i >= 0; --i) {
-               MathInset * m = operator[](i).inset().asMathInset();
-               if (!m)
-                       return 0;
-               MathGridInset * p = m->asGridInset();
-               if (p) {
-                       idx = operator[](i).idx();
-                       return p;
-               }
-       }
-       return 0;
-}
-
-
 void LCursor::pullArg()
 {
+#ifdef WITH_WARNINGS
 #warning Look here
+#endif
        MathArray ar = cell();
        if (popLeft() && inMathed()) {
                plainErase();
@@ -974,10 +965,12 @@ void LCursor::pullArg()
 
 void LCursor::touch()
 {
+#ifdef WITH_WARNINGS
 #warning look here
+#endif
 #if 0
-       DocumentIterator::const_iterator it = begin();
-       DocumentIterator::const_iterator et = end();
+       DocIterator::const_iterator it = begin();
+       DocIterator::const_iterator et = end();
        for ( ; it != et; ++it)
                it->cell().touch();
 #endif
@@ -1005,22 +998,6 @@ void LCursor::normalize()
 }
 
 
-char LCursor::valign()
-{
-       idx_type idx;
-       MathGridInset * p = enclosingGrid(idx);
-       return p ? p->valign() : '\0';
-}
-
-
-char LCursor::halign()
-{
-       idx_type idx;
-       MathGridInset * p = enclosingGrid(idx);
-       return p ? p->halign(idx % p->ncols()) : '\0';
-}
-
-
 bool LCursor::goUpDown(bool up)
 {
        // Be warned: The 'logic' implemented in this function is highly
@@ -1112,13 +1089,13 @@ bool LCursor::bruteFind(int x, int y, int xlow, int xhigh, int ylow, int yhigh)
 {
        BOOST_ASSERT(!empty());
        par_type beg, end;
-       CursorSlice bottom = operator[](0); 
+       CursorSlice bottom = operator[](0);
        LyXText * text = bottom.text();
        BOOST_ASSERT(text);
        getParsInRange(text->paragraphs(), ylow, yhigh, beg, end);
 
-       DocumentIterator it(bv().buffer()->inset());
-       DocumentIterator et;
+       DocIterator it = doc_iterator_begin(bv().buffer()->inset());
+       DocIterator et = doc_iterator_end(bv().buffer()->inset());
        //lyxerr << "x: " << x << " y: " << y << endl;
        //lyxerr << "xlow: " << xlow << " ylow: " << ylow << endl;
        //lyxerr << "xhigh: " << xhigh << " yhigh: " << yhigh << endl;
@@ -1127,7 +1104,7 @@ bool LCursor::bruteFind(int x, int y, int xlow, int xhigh, int ylow, int yhigh)
        //et.par() = text->parOffset(end);
 
        double best_dist = 10e10;
-       DocumentIterator best_cursor = it;
+       DocIterator best_cursor = it;
 
        for ( ; it != et; it.forwardPos()) {
                // avoid invalid nesting when selecting
@@ -1160,9 +1137,9 @@ void LCursor::bruteFind2(int x, int y)
 {
        double best_dist = 1e10;
 
-       DocumentIterator it = *this;
+       DocIterator it = *this;
        it.back().pos() = 0;
-       DocumentIterator et = *this;
+       DocIterator et = *this;
        et.back().pos() = et.back().asMathInset()->cell(et.back().idx()).size();
        for (int i = 0; ; ++i) {
                int xo, yo;
@@ -1283,7 +1260,9 @@ string LCursor::selectionAsString(bool label) const
                return result;
        }
 
+#ifdef WITH_WARNINGS
 #warning and mathed?
+#endif
        return string();
 }
 
@@ -1324,7 +1303,7 @@ Encoding const * LCursor::getEncoding() const
        CursorSlice const & sl = operator[](s);
        LyXText & text = *sl.text();
        LyXFont font = text.getPar(sl.par()).getFont(
-               bv().buffer()->params(), sl.pos(), outerFont(sl.par(), text.paragraphs()));     
+               bv().buffer()->params(), sl.pos(), outerFont(sl.par(), text.paragraphs()));
        return font.language()->encoding();
 }