]> git.lyx.org Git - features.git/blobdiff - src/Cursor.cpp
start work on less-likely-to-misuse iterators.
[features.git] / src / Cursor.cpp
index 791c2602bb97bda44446b8414c49727df1dab07c..6868651e212bfed80d54da1defde1fad39b06e67 100644 (file)
 #include <config.h>
 
 #include "Bidi.h"
-#include "BufferView.h"
 #include "Buffer.h"
-#include "Cursor.h"
+#include "BufferView.h"
 #include "CoordCache.h"
+#include "Cursor.h"
 #include "CutAndPaste.h"
-#include "debug.h"
 #include "DispatchResult.h"
 #include "Encoding.h"
+#include "Font.h"
 #include "FuncRequest.h"
 #include "Language.h"
 #include "lfuns.h"
-#include "Font.h"
 #include "LyXFunc.h" // only for setMessage()
 #include "LyXRC.h"
-#include "Row.h"
-#include "Text.h"
-#include "Paragraph.h"
 #include "paragraph_funcs.h"
+#include "Paragraph.h"
 #include "ParIterator.h"
+#include "Row.h"
+#include "Text.h"
 #include "TextMetrics.h"
+#include "TocBackend.h"
+
+#include "support/debug.h"
+#include "support/docstream.h"
 
 #include "insets/InsetTabular.h"
 #include "insets/InsetText.h"
 
 #include <boost/assert.hpp>
 #include <boost/bind.hpp>
-#include <boost/current_function.hpp>
 
 #include <sstream>
 #include <limits>
 #include <map>
 
-using std::string;
-using std::vector;
-using std::endl;
-using std::min;
-using std::for_each;
-
+using namespace std;
 
 namespace lyx {
 
 namespace {
 
-       bool
-       positionable(DocIterator const & cursor, DocIterator const & anchor)
-       {
-               // avoid deeper nested insets when selecting
-               if (cursor.depth() > anchor.depth())
+bool positionable(DocIterator const & cursor, DocIterator const & anchor)
+{
+       // avoid deeper nested insets when selecting
+       if (cursor.depth() > anchor.depth())
+               return false;
+
+       // anchor might be deeper, should have same path then
+       for (size_t i = 0; i < cursor.depth(); ++i)
+               if (&cursor[i].inset() != &anchor[i].inset())
                        return false;
 
-               // anchor might be deeper, should have same path then
-               for (size_t i = 0; i < cursor.depth(); ++i)
-                       if (&cursor[i].inset() != &anchor[i].inset())
-                               return false;
+       // position should be ok.
+       return true;
+}
+
 
-               // position should be ok.
-               return true;
-       }
+// Find position closest to (x, y) in cell given by iter.
+// Used only in mathed
+DocIterator bruteFind2(Cursor const & c, int x, int y)
+{
+       double best_dist = numeric_limits<double>::max();
 
+       DocIterator result;
 
-       // Find position closest to (x, y) in cell given by iter.
-       // Used only in mathed
-       DocIterator bruteFind2(Cursor const & c, int x, int y)
-       {
-               double best_dist = std::numeric_limits<double>::max();
-
-               DocIterator result;
-
-               DocIterator it = c;
-               it.top().pos() = 0;
-               DocIterator et = c;
-               et.top().pos() = et.top().asInsetMath()->cell(et.top().idx()).size();
-               for (size_t i = 0;; ++i) {
-                       int xo;
-                       int yo;
-                       Inset const * inset = &it.inset();
-                       std::map<Inset const *, Geometry> const & data =
-                               c.bv().coordCache().getInsets().getData();
-                       std::map<Inset const *, Geometry>::const_iterator I = data.find(inset);
-
-                       // FIXME: in the case where the inset is not in the cache, this
-                       // means that no part of it is visible on screen. In this case
-                       // we don't do elaborate search and we just return the forwarded
-                       // DocIterator at its beginning.
-                       if (I == data.end()) {
-                               it.top().pos() = 0;
-                               return it;
-                       }
+       DocIterator it = c;
+       it.top().pos() = 0;
+       DocIterator et = c;
+       et.top().pos() = et.top().asInsetMath()->cell(et.top().idx()).size();
+       for (size_t i = 0;; ++i) {
+               int xo;
+               int yo;
+               Inset const * inset = &it.inset();
+               map<Inset const *, Geometry> const & data =
+                       c.bv().coordCache().getInsets().getData();
+               map<Inset const *, Geometry>::const_iterator I = data.find(inset);
 
-                       Point o = I->second.pos;
-                       inset->cursorPos(c.bv(), it.top(), c.boundary(), xo, yo);
-                       // Convert to absolute
-                       xo += o.x_;
-                       yo += o.y_;
-                       double d = (x - xo) * (x - xo) + (y - yo) * (y - yo);
-                       // '<=' in order to take the last possible position
-                       // this is important for clicking behind \sum in e.g. '\sum_i a'
-                       LYXERR(Debug::DEBUG) << "i: " << i << " d: " << d
-                               << " best: " << best_dist << endl;
-                       if (d <= best_dist) {
-                               best_dist = d;
-                               result = it;
-                       }
-                       if (it == et)
-                               break;
-                       it.forwardPos();
+               // FIXME: in the case where the inset is not in the cache, this
+               // means that no part of it is visible on screen. In this case
+               // we don't do elaborate search and we just return the forwarded
+               // DocIterator at its beginning.
+               if (I == data.end()) {
+                       it.top().pos() = 0;
+                       return it;
                }
-               return result;
-       }
 
-
-       /*
-       /// moves position closest to (x, y) in given box
-       bool bruteFind(Cursor & cursor,
-               int x, int y, int xlow, int xhigh, int ylow, int yhigh)
-       {
-               BOOST_ASSERT(!cursor.empty());
-               Inset & inset = cursor[0].inset();
-               BufferView & bv = cursor.bv();
-
-               CoordCache::InnerParPosCache const & cache =
-                       bv.coordCache().getParPos().find(cursor.bottom().text())->second;
-               // Get an iterator on the first paragraph in the cache
-               DocIterator it(inset);
-               it.push_back(CursorSlice(inset));
-               it.pit() = cache.begin()->first;
-               // Get an iterator after the last paragraph in the cache
-               DocIterator et(inset);
-               et.push_back(CursorSlice(inset));
-               et.pit() = boost::prior(cache.end())->first;
-               if (et.pit() >= et.lastpit())
-                       et = doc_iterator_end(inset);
-               else
-                       ++et.pit();
-
-               double best_dist = std::numeric_limits<double>::max();;
-               DocIterator best_cursor = et;
-
-               for ( ; it != et; it.forwardPos(true)) {
-                       // avoid invalid nesting when selecting
-                       if (!cursor.selection() || positionable(it, cursor.anchor_)) {
-                               Point p = bv.getPos(it, false);
-                               int xo = p.x_;
-                               int yo = p.y_;
-                               if (xlow <= xo && xo <= xhigh && ylow <= yo && yo <= yhigh) {
-                                       double const dx = xo - x;
-                                       double const dy = yo - y;
-                                       double const d = dx * dx + dy * dy;
-                                       // '<=' 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;
-                                       }
+               Point o = I->second.pos;
+               inset->cursorPos(c.bv(), it.top(), c.boundary(), xo, yo);
+               // Convert to absolute
+               xo += o.x_;
+               yo += o.y_;
+               double d = (x - xo) * (x - xo) + (y - yo) * (y - yo);
+               // '<=' in order to take the last possible position
+               // this is important for clicking behind \sum in e.g. '\sum_i a'
+               LYXERR(Debug::DEBUG, "i: " << i << " d: " << d
+                       << " best: " << best_dist);
+               if (d <= best_dist) {
+                       best_dist = d;
+                       result = it;
+               }
+               if (it == et)
+                       break;
+               it.forwardPos();
+       }
+       return result;
+}
+
+
+/*
+/// moves position closest to (x, y) in given box
+bool bruteFind(Cursor & cursor,
+       int x, int y, int xlow, int xhigh, int ylow, int yhigh)
+{
+       BOOST_ASSERT(!cursor.empty());
+       Inset & inset = cursor[0].inset();
+       BufferView & bv = cursor.bv();
+
+       CoordCache::InnerParPosCache const & cache =
+               bv.coordCache().getParPos().find(cursor.bottom().text())->second;
+       // Get an iterator on the first paragraph in the cache
+       DocIterator it(inset);
+       it.push_back(CursorSlice(inset));
+       it.pit() = cache.begin()->first;
+       // Get an iterator after the last paragraph in the cache
+       DocIterator et(inset);
+       et.push_back(CursorSlice(inset));
+       et.pit() = boost::prior(cache.end())->first;
+       if (et.pit() >= et.lastpit())
+               et = doc_iterator_end(inset);
+       else
+               ++et.pit();
+
+       double best_dist = numeric_limits<double>::max();;
+       DocIterator best_cursor = et;
+
+       for ( ; it != et; it.forwardPos(true)) {
+               // avoid invalid nesting when selecting
+               if (!cursor.selection() || positionable(it, cursor.anchor_)) {
+                       Point p = bv.getPos(it, false);
+                       int xo = p.x_;
+                       int yo = p.y_;
+                       if (xlow <= xo && xo <= xhigh && ylow <= yo && yo <= yhigh) {
+                               double const dx = xo - x;
+                               double const dy = yo - y;
+                               double const d = dx * dx + dy * dy;
+                               // '<=' 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 (best_cursor != et) {
-                       cursor.setCursor(best_cursor);
-                       return true;
-               }
-
-               return false;
+       if (best_cursor != et) {
+               cursor.setCursor(best_cursor);
+               return true;
        }
-       */
 
+       return false;
+}
+*/
 
-       /// moves position closest to (x, y) in given box
-       bool bruteFind3(Cursor & cur, int x, int y, bool up)
-       {
-               BufferView & bv = cur.bv();
-               int ylow  = up ? 0 : y + 1;
-               int yhigh = up ? y - 1 : bv.workHeight();
-               int xlow = 0;
-               int xhigh = bv.workWidth();
+
+/// moves position closest to (x, y) in given box
+bool bruteFind3(Cursor & cur, int x, int y, bool up)
+{
+       BufferView & bv = cur.bv();
+       int ylow  = up ? 0 : y + 1;
+       int yhigh = up ? y - 1 : bv.workHeight();
+       int xlow = 0;
+       int xhigh = bv.workWidth();
 
 // FIXME: bit more work needed to get 'from' and 'to' right.
-               pit_type from = cur.bottom().pit();
-               //pit_type to = cur.bottom().pit();
-               //lyxerr << "Pit start: " << from << endl;
-
-               //lyxerr << "bruteFind3: x: " << x << " y: " << y
-               //      << " xlow: " << xlow << " xhigh: " << xhigh
-               //      << " ylow: " << ylow << " yhigh: " << yhigh
-               //      << endl;
-               Inset & inset = bv.buffer().inset();
-               DocIterator it = doc_iterator_begin(inset);
-               it.pit() = from;
-               DocIterator et = doc_iterator_end(inset);
-
-               double best_dist = std::numeric_limits<double>::max();
-               DocIterator best_cursor = et;
-
-               for ( ; it != et; it.forwardPos()) {
-                       // avoid invalid nesting when selecting
-                       if (bv.cursorStatus(it) == CUR_INSIDE
-                           && (!cur.selection() || positionable(it, cur.anchor_))) {
-                               Point p = bv.getPos(it, false);
-                               int xo = p.x_;
-                               int yo = p.y_;
-                               if (xlow <= xo && xo <= xhigh && ylow <= yo && yo <= yhigh) {
-                                       double const dx = xo - x;
-                                       double const dy = yo - y;
-                                       double const d = dx * dx + dy * dy;
-                                       //lyxerr << "itx: " << xo << " ity: " << yo << " d: " << d
-                                       //      << " dx: " << dx << " dy: " << dy
-                                       //      << " idx: " << it.idx() << " pos: " << it.pos()
-                                       //      << " it:\n" << it
-                                       //      << 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;
-                                       }
+       pit_type from = cur.bottom().pit();
+       //pit_type to = cur.bottom().pit();
+       //lyxerr << "Pit start: " << from << endl;
+
+       //lyxerr << "bruteFind3: x: " << x << " y: " << y
+       //      << " xlow: " << xlow << " xhigh: " << xhigh
+       //      << " ylow: " << ylow << " yhigh: " << yhigh
+       //      << endl;
+       Inset & inset = bv.buffer().inset();
+       DocIterator it = doc_iterator_begin(inset);
+       it.pit() = from;
+       DocIterator et = doc_iterator_end(inset);
+
+       double best_dist = numeric_limits<double>::max();
+       DocIterator best_cursor = et;
+
+       for ( ; it != et; it.forwardPos()) {
+               // avoid invalid nesting when selecting
+               if (bv.cursorStatus(it) == CUR_INSIDE
+                               && (!cur.selection() || positionable(it, cur.anchor_))) {
+                       Point p = bv.getPos(it, false);
+                       int xo = p.x_;
+                       int yo = p.y_;
+                       if (xlow <= xo && xo <= xhigh && ylow <= yo && yo <= yhigh) {
+                               double const dx = xo - x;
+                               double const dy = yo - y;
+                               double const d = dx * dx + dy * dy;
+                               //lyxerr << "itx: " << xo << " ity: " << yo << " d: " << d
+                               //      << " dx: " << dx << " dy: " << dy
+                               //      << " idx: " << it.idx() << " pos: " << it.pos()
+                               //      << " it:\n" << it
+                               //      << 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;
                                }
                        }
                }
-
-               //lyxerr << "best_dist: " << best_dist << " cur:\n" << best_cursor << endl;
-               if (best_cursor == et)
-                       return false;
-               cur.setCursor(best_cursor);
-               return true;
        }
 
-       docstring parbreak(Paragraph const & par)
-       {
-               odocstringstream ods;
+       //lyxerr << "best_dist: " << best_dist << " cur:\n" << best_cursor << endl;
+       if (best_cursor == et)
+               return false;
+       cur.setCursor(best_cursor);
+       return true;
+}
+
+docstring parbreak(Paragraph const & par)
+{
+       odocstringstream ods;
+       ods << '\n';
+       // only add blank line if we're not in an ERT or Listings inset
+       if (par.ownerCode() != ERT_CODE
+                       && par.ownerCode() != LISTINGS_CODE)
                ods << '\n';
-               // only add blank line if we're not in an ERT or Listings inset
-               if (par.ownerCode() != ERT_CODE
-                   && par.ownerCode() != LISTINGS_CODE)
-                       ods << '\n';
-               return ods.str();
-       }
+       return ods.str();
+}
 
 } // namespace anon
 
@@ -276,7 +272,8 @@ void Cursor::reset(Inset & inset)
 {
        clear();
        push_back(CursorSlice(inset));
-       anchor_ = DocIterator(inset);
+       anchor_ = doc_iterator_begin(inset);
+       anchor_.clear();
        clearTargetX();
        selection_ = false;
        mark_ = false;
@@ -292,9 +289,7 @@ void Cursor::setCursor(DocIterator const & cur)
 
 void Cursor::dispatch(FuncRequest const & cmd0)
 {
-       LYXERR(Debug::DEBUG) << BOOST_CURRENT_FUNCTION
-                            << " cmd: " << cmd0 << '\n'
-                            << *this << endl;
+       LYXERR(Debug::DEBUG, "cmd: " << cmd0 << '\n' << *this);
        if (empty())
                return;
 
@@ -303,11 +298,10 @@ void Cursor::dispatch(FuncRequest const & cmd0)
        Cursor safe = *this;
        
        // store some values to be used inside of the handlers
-       getPos(beforeDispX_, beforeDispY_);
        beforeDispatchCursor_ = *this;
        for (; depth(); pop()) {
-               LYXERR(Debug::DEBUG) << "Cursor::dispatch: cmd: "
-                       << cmd0 << endl << *this << endl;
+               LYXERR(Debug::DEBUG, "Cursor::dispatch: cmd: "
+                       << cmd0 << endl << *this);
                BOOST_ASSERT(pos() <= lastpos());
                BOOST_ASSERT(idx() <= lastidx());
                BOOST_ASSERT(pit() <= lastpit());
@@ -325,7 +319,7 @@ void Cursor::dispatch(FuncRequest const & cmd0)
        // it completely to get a 'bomb early' behaviour in case this
        // object will be used again.
        if (!disp_.dispatched()) {
-               LYXERR(Debug::DEBUG) << "RESTORING OLD CURSOR!" << endl;
+               LYXERR(Debug::DEBUG, "RESTORING OLD CURSOR!");
                operator=(safe);
                disp_.update(Update::None);
                disp_.dispatched(false);
@@ -427,7 +421,8 @@ void Cursor::getPos(int & x, int & y) const
 
 Row const & Cursor::textRow() const
 {
-       ParagraphMetrics const & pm = bv().parMetrics(text(), pit());
+       CursorSlice const & cs = innerTextSlice();
+       ParagraphMetrics const & pm = bv().parMetrics(cs.text(), cs.pit());
        BOOST_ASSERT(!pm.rows().empty());
        return pm.getRow(pos(), boundary());
 }
@@ -585,6 +580,8 @@ void Cursor::info(odocstream & os) const
 bool Cursor::selHandle(bool sel)
 {
        //lyxerr << "Cursor::selHandle" << endl;
+       if (mark())
+               sel = true;
        if (sel == selection())
                return false;
 
@@ -597,7 +594,7 @@ bool Cursor::selHandle(bool sel)
 }
 
 
-std::ostream & operator<<(std::ostream & os, Cursor const & cur)
+ostream & operator<<(ostream & os, Cursor const & cur)
 {
        os << "\n cursor:                                | anchor:\n";
        for (size_t i = 0, n = cur.depth(); i != n; ++i) {
@@ -616,11 +613,20 @@ std::ostream & operator<<(std::ostream & os, Cursor const & cur)
        return os;
 }
 
+
+LyXErr & operator<<(LyXErr & os, Cursor const & cur)
+{
+       os.stream() << cur;
+       return os;
+}
+
+
 } // namespace lyx
 
 
 ///////////////////////////////////////////////////////////////////
 //
+// FIXME: Look here
 // The part below is the non-integrated rest of the original math
 // cursor. This should be either generalized for texted or moved
 // back to mathed (in most cases to InsetMathNest).
@@ -641,7 +647,7 @@ namespace lyx {
 //#define FILEDEBUG 1
 
 
-bool Cursor::isInside(Inset const * p)
+bool Cursor::isInside(Inset const * p) const
 {
        for (size_t i = 0; i != depth(); ++i)
                if (&operator[](i).inset() == p)
@@ -948,7 +954,7 @@ bool Cursor::macroModeClose()
        MathMacro * atomAsMacro = atom.nucleus()->asMacro();
        if (atomAsMacro) {
                // make non-greedy, i.e. don't eat parameters from the right
-               atomAsMacro->setDisplayMode(MathMacro::DISPLAY_NONGREEDY_INIT);
+               atomAsMacro->setDisplayMode(MathMacro::DISPLAY_INTERACTIVE_INIT);
        }
        plainInsert(atom);
        return true;
@@ -1072,8 +1078,8 @@ bool Cursor::upDownInMath(bool up)
        int xo = 0;
        int yo = 0;
        getPos(xo, yo);
-       xo = beforeDispX_;
-
+       xo = theLyXFunc().cursorBeforeDispatchX();
+       
        // check if we had something else in mind, if not, this is the future
        // target
        if (x_target_ == -1)
@@ -1122,8 +1128,9 @@ bool Cursor::upDownInMath(bool up)
                                int x;
                                int y;
                                getPos(x, y);
-                               if ((!up && y <= beforeDispY_) ||
-                                               (up && y >= beforeDispY_))
+                               int oy = theLyXFunc().cursorBeforeDispatchY();
+                               if ((!up && y <= oy) ||
+                                               (up && y >= oy))
                                        operator=(old);
                                else
                                        return true;
@@ -1142,8 +1149,9 @@ bool Cursor::upDownInMath(bool up)
                                int x;
                                int y;
                                getPos(x, y);
-                               if ((!up && y <= beforeDispY_) ||
-                                               (up && y >= beforeDispY_))
+                               int oy = theLyXFunc().cursorBeforeDispatchY();
+                               if ((!up && y <= oy) ||
+                                               (up && y >= oy))
                                        operator=(old);
                                else
                                        return true;
@@ -1165,8 +1173,9 @@ bool Cursor::upDownInMath(bool up)
                //lyxerr << "updown: popBackward succeeded" << endl;
                int xnew;
                int ynew;
+               int yold = theLyXFunc().cursorBeforeDispatchY();
                getPos(xnew, ynew);
-               if (up ? ynew < beforeDispY_ : ynew > beforeDispY_)
+               if (up ? ynew < yold : ynew > yold)
                        return true;
        }
        
@@ -1184,8 +1193,8 @@ bool Cursor::upDownInText(bool up, bool & updateNeeded)
        int xo = 0;
        int yo = 0;
        getPos(xo, yo);
-       xo = beforeDispX_;
-       
+       xo = theLyXFunc().cursorBeforeDispatchX();
+
        // update the targetX - this is here before the "return false"
        // to set a new target which can be used by InsetTexts above
        // if we cannot move up/down inside this inset anymore
@@ -1267,18 +1276,18 @@ bool Cursor::upDownInText(bool up, bool & updateNeeded)
                Cursor old = *this;
                if (up) {
                        if (row > 0) {
-                               top().pos() = std::min(tm.x2pos(pit(), row - 1, xo), top().lastpos());
+                               top().pos() = min(tm.x2pos(pit(), row - 1, xo), top().lastpos());
                        } else if (pit() > 0) {
                                --pit();
                                ParagraphMetrics const & pmcur = bv_->parMetrics(text(), pit());
-                               top().pos() = std::min(tm.x2pos(pit(), pmcur.rows().size() - 1, xo), top().lastpos());
+                               top().pos() = min(tm.x2pos(pit(), pmcur.rows().size() - 1, xo), top().lastpos());
                        }
                } else {
                        if (row + 1 < int(pm.rows().size())) {
-                               top().pos() = std::min(tm.x2pos(pit(), row + 1, xo), top().lastpos());
+                               top().pos() = min(tm.x2pos(pit(), row + 1, xo), top().lastpos());
                        } else if (pit() + 1 < int(text()->paragraphs().size())) {
                                ++pit();
-                               top().pos() = std::min(tm.x2pos(pit(), 0, xo), top().lastpos());
+                               top().pos() = min(tm.x2pos(pit(), 0, xo), top().lastpos());
                        }
                }
 
@@ -1292,7 +1301,7 @@ bool Cursor::upDownInText(bool up, bool & updateNeeded)
 
 void Cursor::handleFont(string const & font)
 {
-       LYXERR(Debug::DEBUG) << BOOST_CURRENT_FUNCTION << ": " << font << endl;
+       LYXERR(Debug::DEBUG, font);
        docstring safe;
        if (selection()) {
                macroModeClose();
@@ -1405,13 +1414,7 @@ Encoding const * Cursor::getEncoding() const
 {
        if (empty())
                return 0;
-       int s = 0;
-       // go up until first non-0 text is hit
-       // (innermost text is 0 in mathed)
-       for (s = depth() - 1; s >= 0; --s)
-               if (operator[](s).text())
-                       break;
-       CursorSlice const & sl = operator[](s);
+       CursorSlice const & sl = innerTextSlice();
        Text const & text = *sl.text();
        Font font = text.getPar(sl.pit()).getFont(
                bv().buffer().params(), sl.pos(), outerFont(sl.pit(), text.paragraphs()));
@@ -1450,13 +1453,8 @@ Font Cursor::getFont() const
        // if a character is entered.
        
        // HACK. far from being perfect...
-       // go up until first non-0 text is hit
-       // (innermost text is 0 in mathed)
-       int s = 0;
-       for (s = depth() - 1; s >= 0; --s)
-               if (operator[](s).text())
-                       break;
-       CursorSlice const & sl = operator[](s);
+
+       CursorSlice const & sl = innerTextSlice();
        Text const & text = *sl.text();
        Paragraph const & par = text.getPar(sl.pit());
        
@@ -1632,4 +1630,14 @@ void Cursor::recordUndoSelection()
 }
 
 
+void Cursor::checkBufferStructure()
+{
+       if (paragraph().layout()->toclevel == Layout::NOT_IN_TOC)
+               return;
+       Buffer const * master = buffer().masterBuffer();
+       master->tocBackend().updateItem(ParConstIterator(*this));
+       master->structureChanged();
+}
+
+
 } // namespace lyx