]> git.lyx.org Git - lyx.git/blobdiff - src/cursor.C
the pariterator stuff
[lyx.git] / src / cursor.C
index abcb92017dcac21bad3545859f329d49f04f47e6..35ccc5ad3cd89f259e9697d7a48f25be8db5c375 100644 (file)
 #include "BufferView.h"
 #include "buffer.h"
 #include "cursor.h"
+#include "CutAndPaste.h"
 #include "debug.h"
 #include "dispatchresult.h"
 #include "encoding.h"
 #include "funcrequest.h"
-#include "iterators.h"
 #include "language.h"
 #include "lfuns.h"
 #include "lyxfont.h"
 #include "lyxtext.h"
 #include "paragraph.h"
 #include "paragraph_funcs.h"
+#include "pariterator.h"
 
 #include "insets/updatableinset.h"
 #include "insets/insettabular.h"
 #include "insets/insettext.h"
 
 #include "mathed/math_data.h"
-#include "mathed/math_hullinset.h"
 #include "mathed/math_support.h"
+#include "mathed/math_inset.h"
 
 #include "support/limited_stack.h"
 #include "support/std_sstream.h"
@@ -45,6 +46,8 @@
 
 #include <boost/assert.hpp>
 
+using lyx::par_type;
+
 using std::string;
 using std::vector;
 using std::endl;
@@ -60,18 +63,37 @@ using std::swap;
 limited_stack<string> theCutBuffer;
 
 
+namespace {
+
+void region(CursorSlice const & i1, CursorSlice const & i2,
+       LCursor::row_type & r1, LCursor::row_type & r2,
+       LCursor::col_type & c1, LCursor::col_type & c2)
+{
+       InsetBase & p = i1.inset();
+       c1 = p.col(i1.idx());
+       c2 = p.col(i2.idx());
+       if (c1 > c2)
+               swap(c1, c2);
+       r1 = p.row(i1.idx());
+       r2 = p.row(i2.idx());
+       if (r1 > r2)
+               swap(r1, r2);
+}
+
+}
+
+
 LCursor::LCursor(BufferView & bv)
        : DocumentIterator(), bv_(&bv), anchor_(),
          cached_y_(0), x_target_(-1), selection_(false), mark_(false)
 {}
 
 
-void LCursor::reset()
+void LCursor::reset(InsetBase & inset)
 {
        clear();
-       push_back(CursorSlice());
+       push_back(CursorSlice(inset));
        anchor_.clear();
-       anchor_.push_back(CursorSlice());
        cached_y_ = 0;
        clearTargetX();
        selection_ = false;
@@ -89,75 +111,61 @@ void LCursor::setCursor(DocumentIterator const & cur, bool sel)
 
 DispatchResult LCursor::dispatch(FuncRequest const & cmd0)
 {
-       //lyxerr << "\nLCursor::dispatch: cmd: " << cmd0 << endl << *this << endl;
+       lyxerr << "\nLCursor::dispatch: cmd: " << cmd0 << endl << *this << endl;
        if (empty())
-               return DispatchResult(false);
+               return DispatchResult();
 
-       BOOST_ASSERT(pos() <= lastpos());
-       BOOST_ASSERT(idx() <= lastidx());
-       BOOST_ASSERT(par() <= lastpar());
        FuncRequest cmd = cmd0;
-       nopop_ = false;
-       DocumentIterator orig = *this;
-       disp_.update(true);
-       disp_.val(NONE);
-
-       // the inset's dispatch() is supposed to reset the update and
-       // val flags if necessary 
-       inset().dispatch(*this, cmd);
-               
-       // "Mutate" the request for semi-handled requests that need
-       // additional handling in outer levels.
-       switch (disp_.val()) {
-               case NONE:
-                       // the inset handled the event fully
-                       return DispatchResult(true, true);
-               case FINISHED_LEFT:
-                       // the inset handled the event partially
-                       cmd = FuncRequest(LFUN_FINISHED_LEFT);
-                       break;
-               case FINISHED_RIGHT:
-                       cmd = FuncRequest(LFUN_FINISHED_RIGHT);
-                       break;
-               case FINISHED_UP:
-                       cmd = FuncRequest(LFUN_FINISHED_UP);
-                       break;
-               case FINISHED_DOWN:
-                       cmd = FuncRequest(LFUN_FINISHED_DOWN);
-                       break;
-               default:
-                       //lyxerr << "not handled on level " << depth()
-                       //      << " val: " << disp_.val() << endl;
+       LCursor safe = *this;
+
+       for ( ; size(); pop()) {
+               //lyxerr << "\nLCursor::dispatch: cmd: " << cmd0 << endl << *this << endl;
+               BOOST_ASSERT(pos() <= lastpos());
+               BOOST_ASSERT(idx() <= lastidx());
+               BOOST_ASSERT(par() <= lastpar());
+
+               // 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);
+               disp_.dispatched(true);
+               inset().dispatch(*this, cmd);
+               if (disp_.dispatched())
                        break;
        }
-       pop();
-       if (nopop_)
-               setCursor(orig, false);
-       //lyxerr << "   result: " << res.val() << endl;
+       // it completely to get a 'bomb early' behaviour in case this
+       // object will be used again.
+       if (!disp_.dispatched()) {
+               lyxerr << "RESTORING OLD CURSOR!" << endl;
+               operator=(safe);
+       }
        return disp_;
 }
 
 
 bool LCursor::getStatus(FuncRequest const & cmd, FuncStatus & status)
 {
-       lyxerr << "\nLCursor::getStatus: cmd: " << cmd << endl << *this << endl;
-       DocumentIterator orig = *this;
-       BOOST_ASSERT(pos() <= lastpos());
-       BOOST_ASSERT(idx() <= lastidx());
-       BOOST_ASSERT(par() <= lastpar());
-       for ( ; size() != 0; pop_back()) {
-               // the inset's getStatus() will return 'true' if it made
+       // This is, of course, a mess. Better create a new doc iterator and use
+       // this in Inset::getStatus. This might require an additional
+       // BufferView * arg, though (which should be avoided)
+       LCursor safe = *this;
+       bool res = false;
+       for ( ; size(); pop()) {
+               //lyxerr << "\nLCursor::getStatus: cmd: " << cmd << endl << *this << endl;
+               BOOST_ASSERT(pos() <= lastpos());
+               BOOST_ASSERT(idx() <= lastidx());
+               BOOST_ASSERT(par() <= lastpar());
+
+               // The inset's getStatus() will return 'true' if it made
                // a definitive decision on whether it want to handle the
                // request or not. The result of this decision is put into
                // the 'status' parameter.
-               bool const res = inset().getStatus(*this, cmd, status);
-               if (res) {
-                       setCursor(orig, false);
-                       return true;
+               if (inset().getStatus(*this, cmd, status)) {
+                       res = true;
+                       break;
                }
        }
-       bool const res = bv().text()->getStatus(*this, cmd, status);
-       setCursor(orig, false);
+       operator=(safe);
        return res;
 }
 
@@ -172,7 +180,6 @@ void LCursor::pop()
 {
        BOOST_ASSERT(size() >= 1);
        pop_back();
-       anchor_.pop_back();
 }
 
 
@@ -195,12 +202,9 @@ bool LCursor::popLeft()
 {
        BOOST_ASSERT(!empty());
        //lyxerr << "Leaving inset to the left" << endl;
-       if (depth() <= 1) {
-               if (depth() == 1)
-                       inset().notifyCursorLeaves(idx());
-               return false;
-       }
        inset().notifyCursorLeaves(idx());
+       if (depth() == 1)
+               return false;
        pop();
        return true;
 }
@@ -210,12 +214,9 @@ bool LCursor::popRight()
 {
        BOOST_ASSERT(!empty());
        //lyxerr << "Leaving inset to the right" << endl;
-       if (depth() <= 1) {
-               if (depth() == 1)
-                       inset().notifyCursorLeaves(idx());
-               return false;
-       }
        inset().notifyCursorLeaves(idx());
+       if (depth() == 1)
+               return false;
        pop();
        ++pos();
        return true;
@@ -225,12 +226,12 @@ bool LCursor::popRight()
 int LCursor::currentMode()
 {
        BOOST_ASSERT(!empty());
-       for (int i = size() - 1; i >= 1; --i) {
+       for (int i = size() - 1; i >= 0; --i) {
                int res = operator[](i).inset().currentMode();
-               if (res != MathInset::UNDECIDED_MODE)
+               if (res != InsetBase::UNDECIDED_MODE)
                        return res;
        }
-       return MathInset::TEXT_MODE;
+       return InsetBase::TEXT_MODE;
 }
 
 
@@ -245,43 +246,39 @@ void LCursor::updatePos()
 
 void LCursor::getDim(int & asc, int & des) const
 {
-       BOOST_ASSERT(!empty());
        if (inMathed()) {
                BOOST_ASSERT(inset().asMathInset());
                //inset().asMathInset()->getCursorDim(asc, des);
                asc = 10;
                des = 10;
-       } else {
+       } else if (inTexted()) {
                Row const & row = textRow();
                asc = row.baseline();
                des = row.height() - asc;
+       } else {
+               lyxerr << "should this happen?" << endl;
+               asc = 10;
+               des = 10;
        }
 }
 
 
 void LCursor::getPos(int & x, int & y) const
 {
-       BOOST_ASSERT(!empty());
        x = 0;
        y = 0;
-       if (size() == 1) {
-               x = bv().text()->cursorX(front());
-               y = bv().text()->cursorY(front());
-       } else {
+       if (!empty())
                inset().getCursorPos(back(), x, y);
-               // getCursorPos gives _screen_ coordinates. We need to add
-               // top_y to get document coordinates. This is hidden in cached_y_.
-               //y += cached_y_ - inset().yo();
-               // The rest is non-obvious. The reason we have to have these
-               // extra computation is that the getCursorPos() calls rely
-               // on the inset's own knowledge of its screen position.
-               // If we scroll up or down in a big enough increment,
-               // inset->draw() is not called: this doesn't update
-               // inset.yo_, so getCursor() returns an old value.
-               // Ugly as you like.
-       }
-       //lyxerr << "#### LCursor::getPos: " << *this 
-       // << " x: " << x << " y: " << y << endl;
+       // getCursorPos gives _screen_ coordinates. We need to add
+       // top_y to get document coordinates. This is hidden in cached_y_.
+       //y += cached_y_ - inset().yo();
+       // The rest is non-obvious. The reason we have to have these
+       // extra computation is that the getCursorPos() calls rely
+       // on the inset's own knowledge of its screen position.
+       // If we scroll up or down in a big enough increment,
+       // inset->draw() is not called: this doesn't update
+       // inset.yo_, so getCursor() returns an old value.
+       // Ugly as you like.
 }
 
 
@@ -318,12 +315,14 @@ bool LCursor::posRight()
 
 CursorSlice & LCursor::anchor()
 {
+       BOOST_ASSERT(!anchor_.empty());
        return anchor_.back();
 }
 
 
 CursorSlice const & LCursor::anchor() const
 {
+       BOOST_ASSERT(!anchor_.empty());
        return anchor_.back();
 }
 
@@ -336,29 +335,27 @@ CursorSlice const & LCursor::selBegin() const
 }
 
 
-CursorSlice & LCursor::selBegin()
+CursorSlice const & LCursor::selEnd() const
 {
        if (!selection())
                return back();
-       // can't use std::min as this returns a const ref
-       return anchor() < back() ? anchor() : back();
+       return anchor() > back() ? anchor() : back();
 }
 
 
-CursorSlice const & LCursor::selEnd() const
+DocumentIterator LCursor::selectionBegin() const
 {
        if (!selection())
-               return back();
-       return anchor() > back() ? anchor() : back();
+               return *this;
+       return anchor() < back() ? anchor_ : *this;
 }
 
 
-CursorSlice & LCursor::selEnd()
+DocumentIterator LCursor::selectionEnd() const
 {
        if (!selection())
-               return back();
-       // can't use std::min as this returns a const ref
-       return anchor() > back() ? anchor() : back();
+               return *this;
+       return anchor() > back() ? anchor_ : *this;
 }
 
 
@@ -421,26 +418,6 @@ void LCursor::info(std::ostream & os) const
 }
 
 
-namespace {
-
-void region(CursorSlice const & i1, CursorSlice const & i2,
-       LCursor::row_type & r1, LCursor::row_type & r2,
-       LCursor::col_type & c1, LCursor::col_type & c2)
-{
-       InsetBase & p = i1.inset();
-       c1 = p.col(i1.idx_);
-       c2 = p.col(i2.idx_);
-       if (c1 > c2)
-               swap(c1, c2);
-       r1 = p.row(i1.idx_);
-       r2 = p.row(i2.idx_);
-       if (r1 > r2)
-               swap(r1, r2);
-}
-
-}
-
-
 string LCursor::grabSelection()
 {
        if (!selection())
@@ -449,10 +426,10 @@ string LCursor::grabSelection()
        CursorSlice i1 = selBegin();
        CursorSlice i2 = selEnd();
 
-       if (i1.idx_ == i2.idx_) {
+       if (i1.idx() == i2.idx()) {
                if (i1.inset().asMathInset()) {
                        MathArray::const_iterator it = i1.cell().begin();
-                       return asString(MathArray(it + i1.pos_, it + i2.pos_));
+                       return asString(MathArray(it + i1.pos(), it + i2.pos()));
                } else {
                        return "unknown selection 1";
                }
@@ -487,8 +464,8 @@ void LCursor::eraseSelection()
        CursorSlice const & i2 = selEnd();
 #warning FIXME
        if (i1.inset().asMathInset()) {
-               if (i1.idx_ == i2.idx_) {
-                       i1.cell().erase(i1.pos_, i2.pos_);
+               if (i1.idx() == i2.idx()) {
+                       i1.cell().erase(i1.pos(), i2.pos());
                } else {
                        MathInset * p = i1.asMathInset();
                        row_type r1, r2;
@@ -583,9 +560,19 @@ void LCursor::selClearOrDel()
 
 std::ostream & operator<<(std::ostream & os, LCursor const & cur)
 {
-       for (size_t i = 0, n = cur.size(); i != n; ++i)
-               os << " " << cur.operator[](i) << " | " << cur.anchor_[i] << "\n";
-       os << " selection: " << cur.selection_ << endl;
+       for (size_t i = 0, n = cur.size(); i != n; ++i) {
+               os << " " << cur.operator[](i) << " | ";
+               if (i < cur.anchor_.size())
+                       os << cur.anchor_[i];
+               else
+                       os << "-------------------------------";
+               os << "\n";
+       }
+       for (size_t i = cur.size(), n = cur.anchor_.size(); i < n; ++i) {
+               os << "------------------------------- | " << cur.anchor_[i] << "\n";
+       }
+       os << " selection: " << cur.selection_
+          << " x_target: " << cur.x_target_ << endl;
        return os;
 }
 
@@ -634,7 +621,7 @@ bool LCursor::openable(MathAtom const & t) const
                return true;
 
        // we can't move into anything new during selection
-       if (depth() == anchor_.size())
+       if (depth() >= anchor_.size())
                return false;
        if (!ptr_cmp(t.nucleus(), &anchor_[depth()].inset()))
                return false;
@@ -662,14 +649,8 @@ bool positionable(DocumentIterator const & cursor,
 
 void LCursor::setScreenPos(int x, int y)
 {
-       bool res = bruteFind(x, y, formula()->xlow(), formula()->xhigh(),
-               formula()->ylow(), formula()->yhigh());
-       if (!res) {
-               // this can happen on creation of "math-display"
-               idx() = 0;
-               pos() = 0;
-       }
-       clearTargetX();
+       x_target() = x;
+       bruteFind(x, y, 0, bv().workWidth(), 0, bv().workHeight());
 }
 
 
@@ -702,23 +683,21 @@ void LCursor::plainInsert(MathAtom const & t)
 void LCursor::insert(string const & str)
 {
        lyxerr << "LCursor::insert str '" << str << "'" << endl;
-       selClearOrDel();
-#if 0
        for (string::const_iterator it = str.begin(); it != str.end(); ++it)
-               plainInsert(MathAtom(new MathCharInset(*it)));
-#else
-       MathArray ar;
-       asArray(str, ar);
-       insert(ar);
-#endif
+               insert(*it);
 }
 
 
 void LCursor::insert(char c)
 {
        //lyxerr << "LCursor::insert char '" << c << "'" << endl;
-       selClearOrDel();
-       plainInsert(MathAtom(new MathCharInset(c)));
+       BOOST_ASSERT(!empty());
+       if (inMathed()) {
+               selClearOrDel();
+               plainInsert(MathAtom(new MathCharInset(c)));
+       } else {
+               text()->insertChar(*this, c);
+       }
 }
 
 
@@ -930,23 +909,12 @@ int LCursor::targetX() const
 }
 
 
-MathHullInset * LCursor::formula() const
-{
-       for (int i = size() - 1; i >= 1; --i) {
-               MathInset * inset = operator[](i).inset().asMathInset();
-               if (inset && inset->asHullInset())
-                       return static_cast<MathHullInset *>(inset);
-       }
-       return 0;
-}
-
-
 void LCursor::adjust(pos_type from, int diff)
 {
        if (pos() > from)
                pos() += diff;
-       if (anchor().pos_ > from)
-               anchor().pos_ += diff;
+       if (anchor().pos() > from)
+               anchor().pos() += diff;
        // just to be on the safe side
        // theoretically unecessary
        normalize();
@@ -982,7 +950,7 @@ MathGridInset * LCursor::enclosingGrid(idx_type & idx) const
                        return 0;
                MathGridInset * p = m->asGridInset();
                if (p) {
-                       idx = operator[](i).idx_;
+                       idx = operator[](i).idx();
                        return p;
                }
        }
@@ -1059,6 +1027,7 @@ bool LCursor::goUpDown(bool up)
        // fragile. A distance of one pixel or a '<' vs '<=' _really
        // matters. So fiddle around with it only if you think you know
        // what you are doing!
+
   int xo = 0;
        int yo = 0;
        getPos(xo, yo);
@@ -1125,13 +1094,9 @@ bool LCursor::goUpDown(bool up)
                // no such inset found, just take something "above"
                //lyxerr << "updown: handled by strange case" << endl;
                if (!popLeft()) {
-                       return
-                               bruteFind(xo, yo,
-                                       formula()->xlow(),
-                                       formula()->xhigh(),
-                                       up ? formula()->ylow() : yo + 4,
-                                       up ? yo - 4 : formula()->yhigh()
-                               );
+                       int ylow  = up ? 0 : yo + 1;
+                       int yhigh = up ? yo - 1 : bv().workHeight();
+                       return bruteFind(xo, yo, 0, bv().workWidth(), ylow, yhigh);
                }
 
                // any improvement so far?
@@ -1145,34 +1110,46 @@ bool LCursor::goUpDown(bool up)
 
 bool LCursor::bruteFind(int x, int y, int xlow, int xhigh, int ylow, int yhigh)
 {
-       DocumentIterator best_cursor;
-       double best_dist = 1e10;
+       BOOST_ASSERT(!empty());
+       par_type beg, end;
+       CursorSlice bottom = operator[](0); 
+       LyXText * text = bottom.text();
+       BOOST_ASSERT(text);
+       getParsInRange(text->paragraphs(), ylow, yhigh, beg, end);
 
-       DocumentIterator it = insetBegin(bv().buffer()->inset());
-       DocumentIterator et = insetEnd();
-       while (1) {
+       DocumentIterator it(bv().buffer()->inset());
+       DocumentIterator et;
+       //lyxerr << "x: " << x << " y: " << y << endl;
+       //lyxerr << "xlow: " << xlow << " ylow: " << ylow << endl;
+       //lyxerr << "xhigh: " << xhigh << " yhigh: " << yhigh << endl;
+
+       it.par() = beg;
+       //et.par() = text->parOffset(end);
+
+       double best_dist = 10e10;
+       DocumentIterator best_cursor = it;
+
+       for ( ; it != et; it.forwardPos()) {
                // avoid invalid nesting when selecting
                if (!selection() || positionable(it, anchor_)) {
-                       int xo, yo;
+                       int xo = 0, yo = 0;
                        CursorSlice & cur = it.back();
                        cur.inset().getCursorPos(cur, xo, yo);
                        if (xlow <= xo && xo <= xhigh && ylow <= yo && yo <= yhigh) {
                                double d = (x - xo) * (x - xo) + (y - yo) * (y - yo);
-                               //lyxerr << "x: " << x << " y: " << y << " d: " << endl;
+                               //lyxerr << "xo: " << xo << " yo: " << yo << " d: " << d << endl;
                                // '<=' in order to take the last possible position
                                // this is important for clicking behind \sum in e.g. '\sum_i a'
                                if (d <= best_dist) {
+                                       //lyxerr << "*" << endl;
                                        best_dist   = d;
                                        best_cursor = it;
                                }
                        }
                }
-
-               if (it == et)
-                       break;
-               it.forwardPos();
        }
 
+       //lyxerr << "best_dist: " << best_dist << " cur:\n" << best_cursor << endl;
        if (best_dist < 1e10)
                setCursor(best_cursor, false);
        return best_dist < 1e10;
@@ -1186,7 +1163,7 @@ void LCursor::bruteFind2(int x, int y)
        DocumentIterator it = *this;
        it.back().pos() = 0;
        DocumentIterator et = *this;
-       et.back().pos() = et.back().asMathInset()->cell(et.back().idx_).size();
+       et.back().pos() = et.back().asMathInset()->cell(et.back().idx()).size();
        for (int i = 0; ; ++i) {
                int xo, yo;
                CursorSlice & cur = it.back();
@@ -1279,27 +1256,29 @@ string LCursor::selectionAsString(bool label) const
 
        if (inTexted()) {
                Buffer const & buffer = *bv().buffer();
+               ParagraphList & pars = text()->paragraphs();
 
                // should be const ...
-               ParagraphList::iterator startpit = text()->getPar(selBegin());
-               ParagraphList::iterator endpit = text()->getPar(selEnd());
+               par_type startpit = selBegin().par();
+               par_type endpit = selEnd().par();
                size_t const startpos = selBegin().pos();
                size_t const endpos = selEnd().pos();
 
                if (startpit == endpit)
-                       return startpit->asString(buffer, startpos, endpos, label);
+                       return pars[startpit].asString(buffer, startpos, endpos, label);
 
                // First paragraph in selection
-               string result =
-                       startpit->asString(buffer, startpos, startpit->size(), label) + "\n\n";
+               string result = pars[startpit].
+                       asString(buffer, startpos, pars[startpit].size(), label) + "\n\n";
 
                // The paragraphs in between (if any)
-               ParagraphList::iterator pit = startpit;
-               for (++pit; pit != endpit; ++pit)
-                       result += pit->asString(buffer, 0, pit->size(), label) + "\n\n";
+               for (par_type pit = startpit + 1; pit != endpit; ++pit) {
+                       Paragraph & par = pars[pit];
+                       result += par.asString(buffer, 0, par.size(), label) + "\n\n";
+               }
 
                // Last paragraph in selection
-               result += endpit->asString(buffer, 0, endpos, label);
+               result += pars[endpit].asString(buffer, 0, endpos, label);
 
                return result;
        }
@@ -1324,27 +1303,6 @@ string LCursor::currentState()
 }
 
 
-// only used by the spellchecker
-void LCursor::replaceWord(string const & replacestring)
-{
-       LyXText * t = text();
-       BOOST_ASSERT(t);
-
-       t->replaceSelectionWithString(*this, replacestring);
-       t->setSelectionRange(*this, replacestring.length());
-
-       // Go back so that replacement string is also spellchecked
-       for (string::size_type i = 0; i < replacestring.length() + 1; ++i)
-               t->cursorLeft(*this);
-}
-
-
-void LCursor::update()
-{
-       bv().update();
-}
-
-
 string LCursor::getPossibleLabel()
 {
        return inMathed() ? "eq:" : text()->getPossibleLabel(*this);
@@ -1365,9 +1323,8 @@ Encoding const * LCursor::getEncoding() const
                        break;
        CursorSlice const & sl = operator[](s);
        LyXText & text = *sl.text();
-       ParagraphList::iterator pit = text.getPar(sl.par());
-       LyXFont font = pit->getFont(
-               bv().buffer()->params(), sl.pos(), outerFont(pit, text.paragraphs()));  
+       LyXFont font = text.getPar(sl.par()).getFont(
+               bv().buffer()->params(), sl.pos(), outerFont(sl.par(), text.paragraphs()));     
        return font.language()->encoding();
 }
 
@@ -1378,9 +1335,9 @@ void LCursor::undispatched()
 }
 
 
-void LCursor::dispatched(dispatch_result_t res)
+void LCursor::dispatched()
 {
-       disp_.val(res);
+       disp_.dispatched(true);
 }
 
 
@@ -1388,9 +1345,3 @@ void LCursor::noUpdate()
 {
        disp_.update(false);
 }
-
-
-void LCursor::noPop()
-{
-       nopop_ = true;
-}