]> git.lyx.org Git - lyx.git/blobdiff - src/text2.C
Partial fix bug 2092: branches not propagated to child documents
[lyx.git] / src / text2.C
index 8e0afeab877607650c35a7ad6163c8b1d4a1780e..059cd30e1be81c40756bf56bdace6a8707cdf191 100644 (file)
 #include "bufferparams.h"
 #include "BufferView.h"
 #include "Bullet.h"
-#include "counters.h"
+#include "coordcache.h"
 #include "cursor.h"
 #include "CutAndPaste.h"
 #include "debug.h"
 #include "dispatchresult.h"
 #include "errorlist.h"
-#include "Floating.h"
-#include "FloatList.h"
 #include "funcrequest.h"
 #include "gettext.h"
 #include "language.h"
 #include "paragraph.h"
 #include "paragraph_funcs.h"
 #include "ParagraphParameters.h"
-#include "PosIterator.h"
+#include "pariterator.h"
 #include "undo.h"
 #include "vspace.h"
 
 #include "frontends/font_metrics.h"
 #include "frontends/LyXView.h"
 
-#include "insets/insetbibitem.h"
 #include "insets/insetenv.h"
-#include "insets/insetfloat.h"
-#include "insets/insetwrap.h"
 
-#include "support/lstrings.h"
 #include "support/textutils.h"
-#include "support/tostr.h"
-#include "support/std_sstream.h"
 
-#include <boost/tuple/tuple.hpp>
+#include <boost/current_function.hpp>
 
+#include <sstream>
+
+using lyx::pit_type;
 using lyx::pos_type;
-using lyx::paroffset_type;
-using lyx::support::bformat;
 
 using std::endl;
 using std::ostringstream;
 using std::string;
+using std::min;
 
 
-LyXText::LyXText(BufferView * bv, bool in_inset)
-       : height(0), width(0), textwidth_(bv ? bv->workWidth() : 100),
-               background_color_(LColor::background),
-         bv_owner(bv), in_inset_(in_inset), xo_(0), yo_(0)
+LyXText::LyXText(BufferView * bv)
+       : maxwidth_(bv ? bv->workWidth() : 100),
+         background_color_(LColor::background),
+         bv_owner(bv),
+         autoBreakRows_(false)
 {}
 
 
 void LyXText::init(BufferView * bv)
 {
+       BOOST_ASSERT(bv);
        bv_owner = bv;
+       maxwidth_ = bv->workWidth();
+       dim_.wid = maxwidth_;
+       dim_.asc = 10;
+       dim_.des = 10;
+
+       pit_type const end = paragraphs().size();
+       for (pit_type pit = 0; pit != end; ++pit)
+               pars_[pit].rows().clear();
 
-       ParagraphList::iterator const beg = paragraphs().begin();
-       ParagraphList::iterator const end = paragraphs().end();
-       for (ParagraphList::iterator pit = beg; pit != end; ++pit)
-               pit->rows.clear();
+       current_font = getFont(pars_[0], 0);
+       updateCounters(*bv->buffer());
+}
 
-       width = 0;
-       height = 0;
 
-       current_font = getFont(beg, 0);
+bool LyXText::isMainText() const
+{
+       return &bv()->buffer()->text() == this;
+}
 
-       redoParagraphs(beg, end);
-       bv->cursor().resetAnchor();
 
-       updateCounters();
+//takes screen x,y coordinates
+InsetBase * LyXText::checkInsetHit(int x, int y) const
+{
+       pit_type pit = getPitNearY(y);
+       BOOST_ASSERT(pit != -1);
+
+       Paragraph const & par = pars_[pit];
+
+       lyxerr[Debug::DEBUG]
+               << BOOST_CURRENT_FUNCTION
+               << ": x: " << x
+               << " y: " << y
+               << "  pit: " << pit
+               << endl;
+       InsetList::const_iterator iit = par.insetlist.begin();
+       InsetList::const_iterator iend = par.insetlist.end();
+       for (; iit != iend; ++iit) {
+               InsetBase * inset = iit->inset;
+#if 1
+               lyxerr[Debug::DEBUG]
+                       << BOOST_CURRENT_FUNCTION
+                       << ": examining inset " << inset << endl;
+
+               if (theCoords.getInsets().has(inset))
+                       lyxerr[Debug::DEBUG]
+                               << BOOST_CURRENT_FUNCTION
+                               << ": xo: " << inset->xo() << "..."
+                               << inset->xo() + inset->width()
+                               << " yo: " << inset->yo() - inset->ascent()
+                               << "..."
+                               << inset->yo() + inset->descent()
+                               << endl;
+               else
+                       lyxerr[Debug::DEBUG]
+                               << BOOST_CURRENT_FUNCTION
+                               << ": inset has no cached position" << endl;
+#endif
+               if (inset->covers(x, y)) {
+                       lyxerr[Debug::DEBUG]
+                               << BOOST_CURRENT_FUNCTION
+                               << ": Hit inset: " << inset << endl;
+                       return inset;
+               }
+       }
+       lyxerr[Debug::DEBUG]
+               << BOOST_CURRENT_FUNCTION
+               << ": No inset hit. " << endl;
+       return 0;
 }
 
 
+
 // Gets the fully instantiated font at a given position in a paragraph
 // Basically the same routine as Paragraph::getFont() in paragraph.C.
 // The difference is that this one is used for displaying, and thus we
 // are allowed to make cosmetic improvements. For instance make footnotes
 // smaller. (Asger)
-LyXFont LyXText::getFont(ParagraphList::iterator pit, pos_type pos) const
+LyXFont LyXText::getFont(Paragraph const & par, pos_type const pos) const
 {
        BOOST_ASSERT(pos >= 0);
 
-       LyXLayout_ptr const & layout = pit->layout();
+       LyXLayout_ptr const & layout = par.layout();
+#ifdef WITH_WARNINGS
 #warning broken?
+#endif
        BufferParams const & params = bv()->buffer()->params();
-       pos_type const body_pos = pit->beginOfBody();
+       pos_type const body_pos = par.beginOfBody();
 
        // We specialize the 95% common case:
-       if (!pit->getDepth()) {
-               LyXFont f = pit->getFontSettings(params, pos);
-               if (in_inset_)
-                       f.realize(font_);
+       if (!par.getDepth()) {
+               LyXFont f = par.getFontSettings(params, pos);
+               if (!isMainText())
+                       applyOuterFont(f);
                if (layout->labeltype == LABEL_MANUAL && pos < body_pos)
                        return f.realize(layout->reslabelfont);
                else
@@ -132,25 +184,39 @@ LyXFont LyXText::getFont(ParagraphList::iterator pit, pos_type pos) const
        else
                layoutfont = layout->font;
 
-       LyXFont font = pit->getFontSettings(params, pos);
+       LyXFont font = par.getFontSettings(params, pos);
        font.realize(layoutfont);
 
-       if (in_inset_)
-               font.realize(font_);
+       if (!isMainText())
+               applyOuterFont(font);
 
        // Realize with the fonts of lesser depth.
-       //font.realize(outerFont(pit, paragraphs()));
        font.realize(defaultfont_);
 
        return font;
 }
 
+// There are currently two font mechanisms in LyX:
+// 1. The font attributes in a lyxtext, and
+// 2. The inset-specific font properties, defined in an inset's
+// metrics() and draw() methods and handed down the inset chain through
+// the pi/mi parameters, and stored locally in a lyxtext in font_.
+// This is where the two are integrated in the final fully realized
+// font.
+void LyXText::applyOuterFont(LyXFont & font) const {
+       LyXFont lf(font_);
+       lf.reduce(defaultfont_);
+       lf.realize(font);
+       lf.setLanguage(font.language());
+       font = lf;
+}
+
 
-LyXFont LyXText::getLayoutFont(ParagraphList::iterator pit) const
+LyXFont LyXText::getLayoutFont(pit_type const pit) const
 {
-       LyXLayout_ptr const & layout = pit->layout();
+       LyXLayout_ptr const & layout = pars_[pit].layout();
 
-       if (!pit->getDepth())
+       if (!pars_[pit].getDepth())
                return layout->resfont;
 
        LyXFont font = layout->font;
@@ -162,45 +228,43 @@ LyXFont LyXText::getLayoutFont(ParagraphList::iterator pit) const
 }
 
 
-LyXFont LyXText::getLabelFont(ParagraphList::iterator pit) const
+LyXFont LyXText::getLabelFont(Paragraph const & par) const
 {
-       LyXLayout_ptr const & layout = pit->layout();
+       LyXLayout_ptr const & layout = par.layout();
 
-       if (!pit->getDepth())
+       if (!par.getDepth())
                return layout->reslabelfont;
 
        LyXFont font = layout->labelfont;
        // Realize with the fonts of lesser depth.
-       font.realize(outerFont(pit, paragraphs()));
        font.realize(defaultfont_);
 
        return font;
 }
 
 
-void LyXText::setCharFont(
-       ParagraphList::iterator pit, pos_type pos, LyXFont const & fnt)
+void LyXText::setCharFont(pit_type pit, pos_type pos, LyXFont const & fnt)
 {
        LyXFont font = fnt;
-       LyXLayout_ptr const & layout = pit->layout();
+       LyXLayout_ptr const & layout = pars_[pit].layout();
 
        // Get concrete layout font to reduce against
        LyXFont layoutfont;
 
-       if (pos < pit->beginOfBody())
+       if (pos < pars_[pit].beginOfBody())
                layoutfont = layout->labelfont;
        else
                layoutfont = layout->font;
 
        // Realize against environment font information
-       if (pit->getDepth()) {
-               ParagraphList::iterator tp = pit;
+       if (pars_[pit].getDepth()) {
+               pit_type tp = pit;
                while (!layoutfont.resolved() &&
-                      tp != paragraphs().end() &&
-                      tp->getDepth()) {
+                      tp != pit_type(paragraphs().size()) &&
+                      pars_[tp].getDepth()) {
                        tp = outerHook(tp, paragraphs());
-                       if (tp != paragraphs().end())
-                               layoutfont.realize(tp->layout()->font);
+                       if (tp != pit_type(paragraphs().size()))
+                               layoutfont.realize(pars_[tp].layout()->font);
                }
        }
 
@@ -209,39 +273,7 @@ void LyXText::setCharFont(
        // Now, reduce font against full layout font
        font.reduce(layoutfont);
 
-       pit->setFont(pos, font);
-}
-
-
-InsetBase * LyXText::getInset() const
-{
-       ParagraphList::iterator pit = cursorPar();
-       pos_type const pos = cursor().pos();
-
-       if (pos < pit->size() && pit->isInset(pos)) {
-               return pit->getInset(pos);
-       }
-       return 0;
-}
-
-
-bool LyXText::toggleInset()
-{
-       InsetBase * inset = getInset();
-       // is there an editable inset at cursor position?
-       if (!isEditableInset(inset))
-               return false;
-       //bv()->owner()->message(inset->editMessage());
-
-       // do we want to keep this?? (JMarc)
-       if (!isHighlyEditableInset(inset))
-               recUndo(cursor().par());
-
-       if (inset->isOpen())
-               inset->close();
-       else
-               inset->open();
-       return true;
+       pars_[pit].setFont(pos, font);
 }
 
 
@@ -268,42 +300,38 @@ void LyXText::makeFontEntriesLayoutSpecific(BufferParams const & params,
 
 
 // return past-the-last paragraph influenced by a layout change on pit
-ParagraphList::iterator LyXText::undoSpan(ParagraphList::iterator pit)
+pit_type LyXText::undoSpan(pit_type pit)
 {
-       ParagraphList::iterator end = paragraphs().end();
-       ParagraphList::iterator nextpit = boost::next(pit);
+       pit_type end = paragraphs().size();
+       pit_type nextpit = pit + 1;
        if (nextpit == end)
                return nextpit;
        //because of parindents
-       if (!pit->getDepth())
+       if (!pars_[pit].getDepth())
                return boost::next(nextpit);
        //because of depth constrains
        for (; nextpit != end; ++pit, ++nextpit) {
-               if (!pit->getDepth())
+               if (!pars_[pit].getDepth())
                        break;
        }
        return nextpit;
 }
 
 
-ParagraphList::iterator
-LyXText::setLayout(ParagraphList::iterator start,
-                  ParagraphList::iterator end,
-                  string const & layout)
+pit_type LyXText::setLayout(pit_type start, pit_type end, string const & layout)
 {
        BOOST_ASSERT(start != end);
-       ParagraphList::iterator undopit = undoSpan(boost::prior(end));
-       recUndo(parOffset(start), parOffset(undopit) - 1);
+       pit_type undopit = undoSpan(end - 1);
+       recUndo(start, undopit - 1);
 
        BufferParams const & bufparams = bv()->buffer()->params();
-       LyXLayout_ptr const & lyxlayout =
-               bufparams.getLyXTextClass()[layout];
+       LyXLayout_ptr const & lyxlayout = bufparams.getLyXTextClass()[layout];
 
-       for (ParagraphList::iterator pit = start; pit != end; ++pit) {
-               pit->applyLayout(lyxlayout);
-               makeFontEntriesLayoutSpecific(bufparams, *pit);
+       for (pit_type pit = start; pit != end; ++pit) {
+               pars_[pit].applyLayout(lyxlayout);
+               makeFontEntriesLayoutSpecific(bufparams, pars_[pit]);
                if (lyxlayout->margintype == MARGIN_MANUAL)
-                       pit->setLabelWidthString(lyxlayout->labelstring());
+                       pars_[pit].setLabelWidthString(lyxlayout->labelstring());
        }
 
        return undopit;
@@ -311,64 +339,45 @@ LyXText::setLayout(ParagraphList::iterator start,
 
 
 // set layout over selection and make a total rebreak of those paragraphs
-void LyXText::setLayout(string const & layout)
+void LyXText::setLayout(LCursor & cur, string const & layout)
 {
+       BOOST_ASSERT(this == cur.text());
        // special handling of new environment insets
-       BufferParams const & params = bv()->buffer()->params();
+       BufferView & bv = cur.bv();
+       BufferParams const & params = bv.buffer()->params();
        LyXLayout_ptr const & lyxlayout = params.getLyXTextClass()[layout];
        if (lyxlayout->is_environment) {
                // move everything in a new environment inset
-               lyxerr << "setting layout " << layout << endl;
-               bv()->owner()->dispatch(FuncRequest(LFUN_HOME));
-               bv()->owner()->dispatch(FuncRequest(LFUN_ENDSEL));
-               bv()->owner()->dispatch(FuncRequest(LFUN_CUT));
+               lyxerr[Debug::DEBUG] << "setting layout " << layout << endl;
+               bv.owner()->dispatch(FuncRequest(LFUN_HOME));
+               bv.owner()->dispatch(FuncRequest(LFUN_ENDSEL));
+               bv.owner()->dispatch(FuncRequest(LFUN_CUT));
                InsetBase * inset = new InsetEnvironment(params, layout);
-               if (bv()->insertInset(inset)) {
-                       //inset->edit(bv());
-                       //bv()->owner()->dispatch(FuncRequest(LFUN_PASTE));
-               } else
-                       delete inset;
+               insertInset(cur, inset);
+               //inset->edit(cur, true);
+               //bv.owner()->dispatch(FuncRequest(LFUN_PASTE));
                return;
        }
 
-       ParagraphList::iterator start =
-               getPar(bv()->cursor().selBegin().par());
-       ParagraphList::iterator end =
-               boost::next(getPar(bv()->cursor().selEnd().par()));
-       ParagraphList::iterator endpit = setLayout(start, end, layout);
-
-       redoParagraphs(start, endpit);
-       updateCounters();
+       pit_type start = cur.selBegin().pit();
+       pit_type end = cur.selEnd().pit() + 1;
+       setLayout(start, end, layout);
+       updateCounters(cur.buffer());
 }
 
 
 namespace {
 
 
-void getSelectionSpan(LyXText & text,
-       ParagraphList::iterator & beg,
-       ParagraphList::iterator & end)
-{
-       if (!text.bv()->cursor().selection()) {
-               beg = text.cursorPar();
-               end = boost::next(beg);
-       } else {
-               beg = text.getPar(text.bv()->cursor().selBegin());
-               end = boost::next(text.getPar(text.bv()->cursor().selEnd()));
-       }
-}
-
-
-bool changeDepthAllowed(bv_funcs::DEPTH_CHANGE type,
-                       Paragraph const & par,
-                       int max_depth)
+bool changeDepthAllowed(LyXText::DEPTH_CHANGE type,
+                       Paragraph const & par, int max_depth)
 {
        if (par.layout()->labeltype == LABEL_BIBLIO)
                return false;
        int const depth = par.params().depth();
-       if (type == bv_funcs::INC_DEPTH && depth < max_depth)
+       if (type == LyXText::INC_DEPTH && depth < max_depth)
                return true;
-       if (type == bv_funcs::DEC_DEPTH && depth > 0)
+       if (type == LyXText::DEC_DEPTH && depth > 0)
                return true;
        return false;
 }
@@ -377,66 +386,64 @@ bool changeDepthAllowed(bv_funcs::DEPTH_CHANGE type,
 }
 
 
-bool LyXText::changeDepthAllowed(bv_funcs::DEPTH_CHANGE type)
+bool LyXText::changeDepthAllowed(LCursor & cur, DEPTH_CHANGE type) const
 {
-       ParagraphList::iterator beg, end; 
-       getSelectionSpan(*this, beg, end);
-       int max_depth = 0;
-       if (beg != paragraphs().begin())
-               max_depth = boost::prior(beg)->getMaxDepthAfter();
-
-       for (ParagraphList::iterator pit = beg; pit != end; ++pit) {
-               if (::changeDepthAllowed(type, *pit, max_depth))
+       BOOST_ASSERT(this == cur.text());
+       pit_type const beg = cur.selBegin().pit();
+       pit_type const end = cur.selEnd().pit() + 1;
+       int max_depth = (beg != 0 ? pars_[beg - 1].getMaxDepthAfter() : 0);
+
+       for (pit_type pit = beg; pit != end; ++pit) {
+               if (::changeDepthAllowed(type, pars_[pit], max_depth))
                        return true;
-               max_depth = pit->getMaxDepthAfter();
+               max_depth = pars_[pit].getMaxDepthAfter();
        }
        return false;
 }
 
 
-void LyXText::changeDepth(bv_funcs::DEPTH_CHANGE type)
+void LyXText::changeDepth(LCursor & cur, DEPTH_CHANGE type)
 {
-       ParagraphList::iterator beg, end;
-       getSelectionSpan(*this, beg, end);
-       
-       recUndo(parOffset(beg), parOffset(end) - 1);
-
-       int max_depth = 0;
-       if (beg != paragraphs().begin())
-               max_depth = boost::prior(beg)->getMaxDepthAfter();
-
-       for (ParagraphList::iterator pit = beg; pit != end; ++pit) {
-               if (::changeDepthAllowed(type, *pit, max_depth)) {
-                       int const depth = pit->params().depth();
-                       if (type == bv_funcs::INC_DEPTH)
-                               pit->params().depth(depth + 1);
+       BOOST_ASSERT(this == cur.text());
+       pit_type const beg = cur.selBegin().pit();
+       pit_type const end = cur.selEnd().pit() + 1;
+       recordUndoSelection(cur);
+       int max_depth = (beg != 0 ? pars_[beg - 1].getMaxDepthAfter() : 0);
+
+       for (pit_type pit = beg; pit != end; ++pit) {
+               Paragraph & par = pars_[pit];
+               if (::changeDepthAllowed(type, par, max_depth)) {
+                       int const depth = par.params().depth();
+                       if (type == INC_DEPTH)
+                               par.params().depth(depth + 1);
                        else
-                               pit->params().depth(depth - 1);
+                               par.params().depth(depth - 1);
                }
-               max_depth = pit->getMaxDepthAfter();
+               max_depth = par.getMaxDepthAfter();
        }
        // this handles the counter labels, and also fixes up
        // depth values for follow-on (child) paragraphs
-       updateCounters();
+       updateCounters(cur.buffer());
 }
 
 
-// set font over selection and make a total rebreak of those paragraphs
-void LyXText::setFont(LyXFont const & font, bool toggleall)
+// set font over selection
+void LyXText::setFont(LCursor & cur, LyXFont const & font, bool toggleall)
 {
-       LCursor & cur = bv()->cursor();
+       BOOST_ASSERT(this == cur.text());
        // if there is no selection just set the current_font
        if (!cur.selection()) {
                // Determine basis font
                LyXFont layoutfont;
-               if (cursor().pos() < cursorPar()->beginOfBody())
-                       layoutfont = getLabelFont(cursorPar());
+               pit_type pit = cur.pit();
+               if (cur.pos() < pars_[pit].beginOfBody())
+                       layoutfont = getLabelFont(pars_[pit]);
                else
-                       layoutfont = getLayoutFont(cursorPar());
+                       layoutfont = getLayoutFont(pit);
 
                // Update current font
                real_current_font.update(font,
-                                        bv()->buffer()->params().language,
+                                        cur.buffer().params().language,
                                         toggleall);
 
                // Reduce to implicit settings
@@ -448,149 +455,152 @@ void LyXText::setFont(LyXFont const & font, bool toggleall)
                return;
        }
 
-       // ok we have a selection.
-       recUndo(cur.selBegin().par(), cur.selEnd().par());
-       freezeUndo();
+       // Ok, we have a selection.
+       recordUndoSelection(cur);
 
-       ParagraphList::iterator beg = getPar(cur.selBegin().par());
-       ParagraphList::iterator end = getPar(cur.selEnd().par());
-       
-       PosIterator pos(&paragraphs(), beg, cur.selBegin().pos());
-       PosIterator posend(&paragraphs(), end, cur.selEnd().pos());
+       DocIterator dit = cur.selectionBegin();
+       DocIterator ditend = cur.selectionEnd();
 
-       BufferParams const & params = bv()->buffer()->params();
+       BufferParams const & params = cur.buffer().params();
 
-       for (; pos != posend; ++pos) {
-               LyXFont f = getFont(pos.pit(), pos.pos());
-               f.update(font, params.language, toggleall);
-               setCharFont(pos.pit(), pos.pos(), f);
+       // Don't use forwardChar here as ditend might have
+       // pos() == lastpos() and forwardChar would miss it.
+       // Can't use forwardPos either as this descends into
+       // nested insets.
+       for (; dit != ditend; dit.forwardPosNoDescend()) {
+               if (dit.pos() != dit.lastpos()) {
+                       LyXFont f = getFont(dit.paragraph(), dit.pos());
+                       f.update(font, params.language, toggleall);
+                       setCharFont(dit.pit(), dit.pos(), f);
+               }
        }
-       
-       unFreezeUndo();
-
-       redoParagraphs(beg, ++end);
 }
 
 
 // the cursor set functions have a special mechanism. When they
 // realize you left an empty paragraph, they will delete it.
 
-void LyXText::cursorHome()
+void LyXText::cursorHome(LCursor & cur)
 {
-       ParagraphList::iterator cpit = cursorPar();
-       setCursor(cpit, cpit->getRow(cursor().pos())->pos());
+       BOOST_ASSERT(this == cur.text());
+       Row const & row = cur.paragraph().getRow(cur.pos(),cur.boundary());
+
+       setCursor(cur, cur.pit(), row.pos());
 }
 
 
-void LyXText::cursorEnd()
+void LyXText::cursorEnd(LCursor & cur)
 {
-       ParagraphList::iterator cpit = cursorPar();
-       pos_type end = cpit->getRow(cursor().pos())->endpos();
+       BOOST_ASSERT(this == cur.text());
        // if not on the last row of the par, put the cursor before
-       // the final space
-       setCursor(cpit, end == cpit->size() ? end : end - 1);
+       // the final space exept if I have a spanning inset or one string
+       // is so long that we force a break.
+       pos_type end = cur.textRow().endpos();
+       if (end == 0)
+               // empty text, end-1 is no valid position
+               return;
+       bool boundary = false;
+       if (end != cur.lastpos()) {
+               if (!cur.paragraph().isLineSeparator(end-1)
+                   && !cur.paragraph().isNewline(end-1))
+                       boundary = true;
+               else
+                       --end;
+       }
+       setCursor(cur, cur.pit(), end, true, boundary);
 }
 
 
-void LyXText::cursorTop()
+void LyXText::cursorTop(LCursor & cur)
 {
-       setCursor(paragraphs().begin(), 0);
+       BOOST_ASSERT(this == cur.text());
+       setCursor(cur, 0, 0);
 }
 
 
-void LyXText::cursorBottom()
+void LyXText::cursorBottom(LCursor & cur)
 {
-       ParagraphList::iterator lastpit =
-               boost::prior(paragraphs().end());
-       setCursor(lastpit, lastpit->size());
+       BOOST_ASSERT(this == cur.text());
+       setCursor(cur, cur.lastpit(), boost::prior(paragraphs().end())->size());
 }
 
 
-void LyXText::toggleFree(LyXFont const & font, bool toggleall)
+void LyXText::toggleFree(LCursor & cur, LyXFont const & font, bool toggleall)
 {
+       BOOST_ASSERT(this == cur.text());
        // If the mask is completely neutral, tell user
        if (font == LyXFont(LyXFont::ALL_IGNORE)) {
                // Could only happen with user style
-               bv()->owner()->message(_("No font change defined. "
-                       "Use Character under the Layout menu to define font change."));
+               cur.message(_("No font change defined. "
+                             "Use Character under the Layout menu to define font change."));
                return;
        }
 
        // Try implicit word selection
        // If there is a change in the language the implicit word selection
        // is disabled.
-       CursorSlice resetCursor = cursor();
+       CursorSlice resetCursor = cur.top();
        bool implicitSelection =
                font.language() == ignore_language
                && font.number() == LyXFont::IGNORE
-               && selectWordWhenUnderCursor(lyx::WHOLE_WORD_STRICT);
+               && selectWordWhenUnderCursor(cur, lyx::WHOLE_WORD_STRICT);
 
        // Set font
-       setFont(font, toggleall);
+       setFont(cur, font, toggleall);
 
        // Implicit selections are cleared afterwards
-       //and cursor is set to the original position.
+       // and cursor is set to the original position.
        if (implicitSelection) {
-               bv()->cursor().clearSelection();
-               cursor() = resetCursor;
-               bv()->cursor().resetAnchor();
+               cur.clearSelection();
+               cur.top() = resetCursor;
+               cur.resetAnchor();
        }
 }
 
 
-string LyXText::getStringToIndex()
+string LyXText::getStringToIndex(LCursor const & cur)
 {
-       LCursor & cur = bv()->cursor();
-       // Try implicit word selection
-       // If there is a change in the language the implicit word selection
-       // is disabled.
-       CursorSlice const reset_cursor = cursor();
-       bool const implicitSelection =
-               selectWordWhenUnderCursor(lyx::PREVIOUS_WORD);
+       BOOST_ASSERT(this == cur.text());
 
        string idxstring;
-       if (!cur.selection())
-               bv()->owner()->message(_("Nothing to index!"));
-       else if (cur.selBegin().par() != cur.selEnd().par())
-               bv()->owner()->message(_("Cannot index more than one paragraph!"));
-       else
-               idxstring = selectionAsString(*bv()->buffer(), false);
-
-       // Reset cursors to their original position.
-       cursor() = reset_cursor;
-       cur.resetAnchor();
-
-       // Clear the implicit selection.
-       if (implicitSelection)
-               cur.clearSelection();
+       if (cur.selection()) {
+               idxstring = cur.selectionAsString(false);
+       } else {
+               // Try implicit word selection. If there is a change
+               // in the language the implicit word selection is
+               // disabled.
+               LCursor tmpcur = cur;
+               selectWord(tmpcur, lyx::PREVIOUS_WORD);
+
+               if (!tmpcur.selection())
+                       cur.message(_("Nothing to index!"));
+               else if (tmpcur.selBegin().pit() != tmpcur.selEnd().pit())
+                       cur.message(_("Cannot index more than one paragraph!"));
+               else
+                       idxstring = tmpcur.selectionAsString(false);
+       }
 
        return idxstring;
 }
 
 
-// the DTP switches for paragraphs(). LyX will store them in the first
-// physical paragraph. When a paragraph is broken, the top settings rest,
-// the bottom settings are given to the new one. So I can make sure,
-// they do not duplicate themself and you cannot play dirty tricks with
-// them!
-
-void LyXText::setParagraph(Spacing const & spacing, LyXAlignment align,
-       string const & labelwidthstring, bool noindent)
+void LyXText::setParagraph(LCursor & cur,
+                          Spacing const & spacing, LyXAlignment align,
+                          string const & labelwidthstring, bool noindent)
 {
-       LCursor & cur = bv()->cursor();
+       BOOST_ASSERT(cur.text());
        // make sure that the depth behind the selection are restored, too
-       ParagraphList::iterator undopit = undoSpan(getPar(cur.selEnd()));
-       recUndo(cur.selBegin().par(), parOffset(undopit) - 1);
-
-       ParagraphList::reverse_iterator pit(getPar(cur.selEnd().par()));
-       ParagraphList::reverse_iterator beg(getPar(cur.selBegin().par()));
+       pit_type undopit = undoSpan(cur.selEnd().pit());
+       recUndo(cur.selBegin().pit(), undopit - 1);
 
-       for (--pit; pit != beg; ++pit) {
-               ParagraphParameters & params = pit->params();
+       for (pit_type pit = cur.selBegin().pit(), end = cur.selEnd().pit();
+            pit <= end; ++pit) {
+               Paragraph & par = pars_[pit];
+               ParagraphParameters & params = par.params();
                params.spacing(spacing);
 
                // does the layout allow the new alignment?
-               LyXLayout_ptr const & layout = pit->layout();
+               LyXLayout_ptr const & layout = par.layout();
 
                if (align == LYX_ALIGN_LAYOUT)
                        align = layout->align;
@@ -600,515 +610,37 @@ void LyXText::setParagraph(Spacing const & spacing, LyXAlignment align,
                        else
                                params.align(align);
                }
-               pit->setLabelWidthString(labelwidthstring);
+               par.setLabelWidthString(labelwidthstring);
                params.noindent(noindent);
        }
-
-       redoParagraphs(getPar(cur.selBegin()), undopit);
-}
-
-
-string expandLabel(LyXTextClass const & textclass,
-       LyXLayout_ptr const & layout, bool appendix)
-{
-       string fmt = appendix ?
-               layout->labelstring_appendix() : layout->labelstring();
-
-       // handle 'inherited level parts' in 'fmt',
-       // i.e. the stuff between '@' in   '@Section@.\arabic{subsection}'
-       size_t const i = fmt.find('@', 0);
-       if (i != string::npos) {
-               size_t const j = fmt.find('@', i + 1);
-               if (j != string::npos) {
-                       string parent(fmt, i + 1, j - i - 1);
-                       string label = expandLabel(textclass, textclass[parent], appendix);
-                       fmt = string(fmt, 0, i) + label + string(fmt, j + 1, string::npos);
-               }
-       }
-
-       return textclass.counters().counterLabel(fmt);
-}
-
-
-namespace {
-
-void incrementItemDepth(ParagraphList::iterator pit,
-                       ParagraphList::iterator first_pit)
-{
-       int const cur_labeltype = pit->layout()->labeltype;
-
-       if (cur_labeltype != LABEL_ENUMERATE && cur_labeltype != LABEL_ITEMIZE)
-               return;
-
-       int const cur_depth = pit->getDepth();
-
-       ParagraphList::iterator prev_pit = boost::prior(pit);
-       while (true) {
-               int const prev_depth = prev_pit->getDepth();
-               int const prev_labeltype = prev_pit->layout()->labeltype;
-               if (prev_depth == 0 && cur_depth > 0) {
-                       if (prev_labeltype == cur_labeltype) {
-                               pit->itemdepth = prev_pit->itemdepth + 1;
-                       }
-                       break;
-               } else if (prev_depth < cur_depth) {
-                       if (prev_labeltype == cur_labeltype) {
-                               pit->itemdepth = prev_pit->itemdepth + 1;
-                               break;
-                       }
-               } else if (prev_depth == cur_depth) {
-                       if (prev_labeltype == cur_labeltype) {
-                               pit->itemdepth = prev_pit->itemdepth;
-                               break;
-                       }
-               }
-               if (prev_pit == first_pit)
-                       break;
-
-               --prev_pit;
-       }
-}
-
-
-void resetEnumCounterIfNeeded(ParagraphList::iterator pit,
-                             ParagraphList::iterator firstpit,
-                             Counters & counters)
-{
-       if (pit == firstpit)
-               return;
-
-       int const cur_depth = pit->getDepth();
-       ParagraphList::iterator prev_pit = boost::prior(pit);
-       while (true) {
-               int const prev_depth = prev_pit->getDepth();
-               int const prev_labeltype = prev_pit->layout()->labeltype;
-               if (prev_depth <= cur_depth) {
-                       if (prev_labeltype != LABEL_ENUMERATE) {
-                               switch (pit->itemdepth) {
-                               case 0:
-                                       counters.reset("enumi");
-                               case 1:
-                                       counters.reset("enumii");
-                               case 2:
-                                       counters.reset("enumiii");
-                               case 3:
-                                       counters.reset("enumiv");
-                               }
-                       }
-                       break;
-               }
-
-               if (prev_pit == firstpit)
-                       break;
-
-               --prev_pit;
-       }
-}
-
-} // anon namespace
-
-
-// set the counter of a paragraph. This includes the labels
-void LyXText::setCounter(Buffer const & buf, ParagraphList::iterator pit)
-{
-       BufferParams const & bufparams = buf.params();
-       LyXTextClass const & textclass = bufparams.getLyXTextClass();
-       LyXLayout_ptr const & layout = pit->layout();
-       ParagraphList::iterator first_pit = paragraphs().begin();
-       Counters & counters = textclass.counters();
-
-       // Always reset
-       pit->itemdepth = 0;
-
-       if (pit == first_pit) {
-               pit->params().appendix(pit->params().startOfAppendix());
-       } else {
-               pit->params().appendix(boost::prior(pit)->params().appendix());
-               if (!pit->params().appendix() &&
-                   pit->params().startOfAppendix()) {
-                       pit->params().appendix(true);
-                       textclass.counters().reset();
-               }
-
-               // Maybe we have to increment the item depth.
-               incrementItemDepth(pit, first_pit);
-       }
-
-       // erase what was there before
-       pit->params().labelString(string());
-
-       if (layout->margintype == MARGIN_MANUAL) {
-               if (pit->params().labelWidthString().empty())
-                       pit->setLabelWidthString(layout->labelstring());
-       } else {
-               pit->setLabelWidthString(string());
-       }
-
-       // is it a layout that has an automatic label?
-       if (layout->labeltype == LABEL_COUNTER) {
-               BufferParams const & bufparams = buf.params();
-               LyXTextClass const & textclass = bufparams.getLyXTextClass();
-               counters.step(layout->counter);
-               string label = expandLabel(textclass, layout, pit->params().appendix());
-               pit->params().labelString(label);
-       } else if (layout->labeltype == LABEL_ITEMIZE) {
-               // At some point of time we should do something more
-               // clever here, like:
-               //   pit->params().labelString(
-               //    bufparams.user_defined_bullet(pit->itemdepth).getText());
-               // for now, use a simple hardcoded label
-               string itemlabel;
-               switch (pit->itemdepth) {
-               case 0:
-                       itemlabel = "*";
-                       break;
-               case 1:
-                       itemlabel = "-";
-                       break;
-               case 2:
-                       itemlabel = "@";
-                       break;
-               case 3:
-                       itemlabel = "·";
-                       break;
-               }
-
-               pit->params().labelString(itemlabel);
-       } else if (layout->labeltype == LABEL_ENUMERATE) {
-               // Maybe we have to reset the enumeration counter.
-               resetEnumCounterIfNeeded(pit, first_pit, counters);
-
-               // FIXME
-               // Yes I know this is a really, really! bad solution
-               // (Lgb)
-               string enumcounter = "enum";
-
-               switch (pit->itemdepth) {
-               case 2:
-                       enumcounter += 'i';
-               case 1:
-                       enumcounter += 'i';
-               case 0:
-                       enumcounter += 'i';
-                       break;
-               case 3:
-                       enumcounter += "iv";
-                       break;
-               default:
-                       // not a valid enumdepth...
-                       break;
-               }
-
-               counters.step(enumcounter);
-
-               pit->params().labelString(counters.enumLabel(enumcounter));
-       } else if (layout->labeltype == LABEL_BIBLIO) {// ale970302
-               counters.step("bibitem");
-               int number = counters.value("bibitem");
-               if (pit->bibitem()) {
-                       pit->bibitem()->setCounter(number);
-                       pit->params().labelString(layout->labelstring());
-               }
-               // In biblio should't be following counters but...
-       } else {
-               string s = buf.B_(layout->labelstring());
-
-               // the caption hack:
-               if (layout->labeltype == LABEL_SENSITIVE) {
-                       ParagraphList::iterator end = paragraphs().end();
-                       ParagraphList::iterator tmppit = pit;
-                       InsetBase * in = 0;
-                       bool isOK = false;
-                       while (tmppit != end && tmppit->inInset()
-                              // the single '=' is intended below
-                              && (in = tmppit->inInset()->owner()))
-                       {
-                               if (in->lyxCode() == InsetBase::FLOAT_CODE ||
-                                   in->lyxCode() == InsetBase::WRAP_CODE) {
-                                       isOK = true;
-                                       break;
-                               } else {
-                                       Paragraph const * owner = &ownerPar(buf, in);
-                                       tmppit = first_pit;
-                                       for ( ; tmppit != end; ++tmppit)
-                                               if (&*tmppit == owner)
-                                                       break;
-                               }
-                       }
-
-                       if (isOK) {
-                               string type;
-
-                               if (in->lyxCode() == InsetBase::FLOAT_CODE)
-                                       type = static_cast<InsetFloat*>(in)->params().type;
-                               else if (in->lyxCode() == InsetBase::WRAP_CODE)
-                                       type = static_cast<InsetWrap*>(in)->params().type;
-                               else
-                                       BOOST_ASSERT(false);
-
-                               Floating const & fl = textclass.floats().getType(type);
-
-                               counters.step(fl.type());
-
-                               // Doesn't work... yet.
-                               s = bformat(_("%1$s #:"), buf.B_(fl.name()));
-                       } else {
-                               // par->SetLayout(0);
-                               // s = layout->labelstring;
-                               s = _("Senseless: ");
-                       }
-               }
-               pit->params().labelString(s);
-
-       }
-}
-
-
-// Updates all counters.
-void LyXText::updateCounters()
-{
-       // start over
-       bv()->buffer()->params().getLyXTextClass().counters().reset();
-
-       bool update_pos = false;
-       
-       ParagraphList::iterator beg = paragraphs().begin();
-       ParagraphList::iterator end = paragraphs().end();
-       for (ParagraphList::iterator pit = beg; pit != end; ++pit) {
-               string const oldLabel = pit->params().labelString();
-               size_t maxdepth = 0;
-               if (pit != beg)
-                       maxdepth = boost::prior(pit)->getMaxDepthAfter();
-
-               if (pit->params().depth() > maxdepth)
-                       pit->params().depth(maxdepth);
-
-               // setCounter can potentially change the labelString.
-               setCounter(*bv()->buffer(), pit);
-               string const & newLabel = pit->params().labelString();
-               if (oldLabel != newLabel) {
-                       redoParagraphInternal(pit);
-                       update_pos = true;
-               }
-               
-       }
-       if (update_pos)
-               updateParPositions();
-}
-
-
-void LyXText::insertInset(InsetBase * inset)
-{
-       if (!cursorPar()->insetAllowed(inset->lyxCode()))
-               return;
-
-       recUndo(cursor().par());
-       freezeUndo();
-       cursorPar()->insertInset(cursor().pos(), inset);
-       // Just to rebreak and refresh correctly.
-       // The character will not be inserted a second time
-       insertChar(Paragraph::META_INSET);
-       // If we enter a highly editable inset the cursor should be before
-       // the inset. After an undo LyX tries to call inset->edit(...)
-       // and fails if the cursor is behind the inset and getInset
-       // does not return the inset!
-       if (isHighlyEditableInset(inset))
-               cursorLeft(true);
-
-       unFreezeUndo();
-}
-
-
-void LyXText::cutSelection(bool doclear, bool realcut)
-{
-       LCursor & cur = bv()->cursor();
-       // Stuff what we got on the clipboard. Even if there is no selection.
-
-       // There is a problem with having the stuffing here in that the
-       // larger the selection the slower LyX will get. This can be
-       // solved by running the line below only when the selection has
-       // finished. The solution used currently just works, to make it
-       // faster we need to be more clever and probably also have more
-       // calls to stuffClipboard. (Lgb)
-       bv()->stuffClipboard(selectionAsString(*bv()->buffer(), true));
-
-       // This doesn't make sense, if there is no selection
-       if (!cur.selection())
-               return;
-
-       // OK, we have a selection. This is always between cur.selBegin()
-       // and cur.selEnd()
-
-       // make sure that the depth behind the selection are restored, too
-       ParagraphList::iterator begpit = getPar(cur.selBegin().par());
-       ParagraphList::iterator endpit = getPar(cur.selEnd().par());
-       ParagraphList::iterator undopit = undoSpan(endpit);
-       recUndo(cur.selBegin().par(), parOffset(undopit) - 1);
-
-       int endpos = cur.selEnd().pos();
-
-       BufferParams const & bufparams = bv()->buffer()->params();
-       boost::tie(endpit, endpos) = realcut ?
-               CutAndPaste::cutSelection(bufparams,
-                                         paragraphs(),
-                                         begpit , endpit,
-                                         cur.selBegin().pos(), endpos,
-                                         bufparams.textclass,
-                                         doclear)
-               : CutAndPaste::eraseSelection(bufparams,
-                                             paragraphs(),
-                                             begpit, endpit,
-                                             cur.selBegin().pos(), endpos,
-                                             doclear);
-       // sometimes necessary
-       if (doclear)
-               begpit->stripLeadingSpaces();
-
-       redoParagraphs(begpit, undopit);
-       // cutSelection can invalidate the cursor so we need to set
-       // it anew. (Lgb)
-       // we prefer the end for when tracking changes
-       cursor().pos(endpos);
-       cursor().par(parOffset(endpit));
-
-       // need a valid cursor. (Lgb)
-       cur.clearSelection();
-       updateCounters();
-}
-
-
-void LyXText::copySelection()
-{
-       LCursor & cur = bv()->cursor();
-       // stuff the selection onto the X clipboard, from an explicit copy request
-       bv()->stuffClipboard(selectionAsString(*bv()->buffer(), true));
-
-       // this doesnt make sense, if there is no selection
-       if (!cur.selection())
-               return;
-
-       // ok we have a selection. This is always between cur.selBegin()
-       // and sel_end cursor
-
-       // copy behind a space if there is one
-       while (getPar(cur.selBegin())->size() > cur.selBegin().pos()
-              && getPar(cur.selBegin())->isLineSeparator(cur.selBegin().pos())
-              && (cur.selBegin().par() != cur.selEnd().par()
-                  || cur.selBegin().pos() < cur.selEnd().pos()))
-               cur.selBegin().pos(cur.selBegin().pos() + 1);
-
-       CutAndPaste::copySelection(getPar(cur.selBegin().par()),
-                                  getPar(cur.selEnd().par()),
-                                  cur.selBegin().pos(), 
-                                  cur.selEnd().pos(),
-                                  bv()->buffer()->params().textclass);
-}
-
-
-void LyXText::pasteSelection(size_t sel_index)
-{
-       LCursor & cur = bv()->cursor();
-       // this does not make sense, if there is nothing to paste
-       if (!CutAndPaste::checkPastePossible())
-               return;
-
-       recUndo(cursor().par());
-
-       ParagraphList::iterator endpit;
-       PitPosPair ppp;
-
-       ErrorList el;
-
-       boost::tie(ppp, endpit) =
-               CutAndPaste::pasteSelection(*bv()->buffer(),
-                                           paragraphs(),
-                                           cursorPar(), cursor().pos(),
-                                           bv()->buffer()->params().textclass,
-                                           sel_index, el);
-       bufferErrors(*bv()->buffer(), el);
-       bv()->showErrorList(_("Paste"));
-
-       redoParagraphs(cursorPar(), endpit);
-
-       cur.clearSelection();
-       cur.resetAnchor();
-       setCursor(ppp.first, ppp.second);
-       cur.setSelection();
-       updateCounters();
 }
 
 
-void LyXText::setSelectionRange(lyx::pos_type length)
+// this really should just insert the inset and not move the cursor.
+void LyXText::insertInset(LCursor & cur, InsetBase * inset)
 {
-       if (!length)
-               return;
-
-       LCursor & cur = bv()->cursor();
-       cur.resetAnchor();
-       while (length--)
-               cursorRight(true);
-       cur.setSelection();
-}
-
-
-// simple replacing. The font of the first selected character is used
-void LyXText::replaceSelectionWithString(string const & str)
-{
-       LCursor & cur = bv()->cursor();
-       recUndo(cur.par());
-       freezeUndo();
-
-       // Get font setting before we cut
-       pos_type pos = cur.selEnd().pos();
-       LyXFont const font = getPar(cur.selBegin())
-               ->getFontSettings(bv()->buffer()->params(),
-                                 cur.selBegin().pos());
-
-       // Insert the new string
-       string::const_iterator cit = str.begin();
-       string::const_iterator end = str.end();
-       for (; cit != end; ++cit) {
-               getPar(cur.selEnd())->insertChar(pos, (*cit), font);
-               ++pos;
-       }
-
-       // Cut the selection
-       cutSelection(true, false);
-
-       unFreezeUndo();
+       BOOST_ASSERT(this == cur.text());
+       BOOST_ASSERT(inset);
+       cur.paragraph().insertInset(cur.pos(), inset);
 }
 
 
 // needed to insert the selection
-void LyXText::insertStringAsLines(string const & str)
+void LyXText::insertStringAsLines(LCursor & cur, string const & str)
 {
-       LCursor & cur = bv()->cursor();
-       ParagraphList::iterator pit = cursorPar();
-       pos_type pos = cursor().pos();
-       ParagraphList::iterator endpit = boost::next(cursorPar());
-
-       recUndo(cursor().par());
-
-       // only to be sure, should not be neccessary
-       cur.clearSelection();
-       bv()->buffer()->insertStringAsLines(pit, pos, current_font, str);
-
-       redoParagraphs(cursorPar(), endpit);
-       cur.resetAnchor();
-       setCursor(pit, pos);
-       cur.setSelection();
+       cur.buffer().insertStringAsLines(pars_, cur.pit(), cur.pos(),
+                                        current_font, str, autoBreakRows_);
 }
 
 
 // turn double CR to single CR, others are converted into one
 // blank. Then insertStringAsLines is called
-void LyXText::insertStringAsParagraphs(string const & str)
+void LyXText::insertStringAsParagraphs(LCursor & cur, string const & str)
 {
-       string linestr(str);
+       string linestr = str;
        bool newline_inserted = false;
-       string::size_type const siz = linestr.length();
 
-       for (string::size_type i = 0; i < siz; ++i) {
+       for (string::size_type i = 0, siz = linestr.size(); i < siz; ++i) {
                if (linestr[i] == '\n') {
                        if (newline_inserted) {
                                // we know that \r will be ignored by
@@ -1124,117 +656,85 @@ void LyXText::insertStringAsParagraphs(string const & str)
                        newline_inserted = false;
                }
        }
-       insertStringAsLines(linestr);
-}
-
-
-void LyXText::setCursor(ParagraphList::iterator pit, pos_type pos)
-{
-       setCursor(parOffset(pit), pos);
+       insertStringAsLines(cur, linestr);
 }
 
 
-bool LyXText::setCursor(paroffset_type par, pos_type pos, bool setfont,
-       bool boundary)
+bool LyXText::setCursor(LCursor & cur, pit_type par, pos_type pos,
+                       bool setfont, bool boundary)
 {
-       CursorSlice old_cursor = cursor();
-       setCursorIntern(par, pos, setfont, boundary);
-       return deleteEmptyParagraphMechanism(old_cursor);
+       LCursor old = cur;
+       setCursorIntern(cur, par, pos, setfont, boundary);
+       return deleteEmptyParagraphMechanism(cur, old);
 }
 
 
-void LyXText::setCursor(CursorSlice & cur, paroffset_type par,
-       pos_type pos, bool boundary)
+void LyXText::setCursor(CursorSlice & cur, pit_type par, pos_type pos)
 {
        BOOST_ASSERT(par != int(paragraphs().size()));
-
-       cur.par(par);
-       cur.pos(pos);
-       cur.boundary(boundary);
-
-       // no rows, no fun...
-       if (paragraphs().begin()->rows.empty())
-               return;
+       cur.pit() = par;
+       cur.pos() = pos;
 
        // now some strict checking
-       Paragraph & para = *getPar(par);
-       Row const & row = *para.getRow(pos);
-       pos_type const end = row.endpos();
+       Paragraph & para = getPar(par);
 
        // None of these should happen, but we're scaredy-cats
        if (pos < 0) {
                lyxerr << "dont like -1" << endl;
-               pos = 0;
-               cur.pos(0);
                BOOST_ASSERT(false);
-       } else if (pos > para.size()) {
+       }
+
+       if (pos > para.size()) {
                lyxerr << "dont like 1, pos: " << pos
                       << " size: " << para.size()
-                      << " row.pos():" << row.pos()
-                      << " paroffset: " << par << endl;
-               pos = 0;
-               cur.pos(0);
-               BOOST_ASSERT(false);
-       } else if (pos > end) {
-               lyxerr << "dont like 2 please report" << endl;
-               // This shouldn't happen.
-               pos = end;
-               cur.pos(pos);
-               BOOST_ASSERT(false);
-       } else if (pos < row.pos()) {
-               lyxerr << "dont like 3 please report pos:" << pos
-                      << " size: " << para.size()
-                      << " row.pos():" << row.pos()
-                      << " paroffset: " << par << endl;
-               pos = row.pos();
-               cur.pos(pos);
+                      << " par: " << par << endl;
                BOOST_ASSERT(false);
        }
 }
 
 
-void LyXText::setCursorIntern(paroffset_type par,
-                             pos_type pos, bool setfont, bool boundary)
+void LyXText::setCursorIntern(LCursor & cur,
+                             pit_type par, pos_type pos, bool setfont, bool boundary)
 {
-       setCursor(cursor(), par, pos, boundary);
-       bv()->cursor().x_target() = cursorX(cursor());
+       cur.boundary(boundary);
+       setCursor(cur.top(), par, pos);
+       cur.setTargetX();
        if (setfont)
-               setCurrentFont();
+               setCurrentFont(cur);
 }
 
 
-void LyXText::setCurrentFont()
+void LyXText::setCurrentFont(LCursor & cur)
 {
-       LCursor & cur = bv()->cursor();
+       BOOST_ASSERT(this == cur.text());
        pos_type pos = cur.pos();
-       ParagraphList::iterator pit = cursorPar();
+       Paragraph & par = cur.paragraph();
 
-       if (cursor().boundary() && pos > 0)
+       if (cur.boundary() && pos > 0)
                --pos;
 
        if (pos > 0) {
-               if (pos == pit->size())
+               if (pos == cur.lastpos())
                        --pos;
                else // potentional bug... BUG (Lgb)
-                       if (pit->isSeparator(pos)) {
-                               if (pos > pit->getRow(pos)->pos() &&
+                       if (par.isSeparator(pos)) {
+                               if (pos > cur.textRow().pos() &&
                                    bidi.level(pos) % 2 ==
                                    bidi.level(pos - 1) % 2)
                                        --pos;
-                               else if (pos + 1 < pit->size())
+                               else if (pos + 1 < cur.lastpos())
                                        ++pos;
                        }
        }
 
-       BufferParams const & bufparams = bv()->buffer()->params();
-       current_font = pit->getFontSettings(bufparams, pos);
-       real_current_font = getFont(pit, pos);
-
-       if (cursor().pos() == pit->size() &&
-           bidi.isBoundary(*bv()->buffer(), *pit, cursor().pos()) &&
-           !cursor().boundary()) {
-               Language const * lang =
-                       pit->getParLanguage(bufparams);
+       BufferParams const & bufparams = cur.buffer().params();
+       current_font = par.getFontSettings(bufparams, pos);
+       real_current_font = getFont(par, pos);
+
+       if (cur.pos() == cur.lastpos()
+           && bidi.isBoundary(cur.buffer(), par, cur.pos())
+           && !cur.boundary()) {
+               Language const * lang = par.getParLanguage(bufparams);
                current_font.setLanguage(lang);
                current_font.setNumber(LyXFont::OFF);
                real_current_font.setLanguage(lang);
@@ -1243,34 +743,36 @@ void LyXText::setCurrentFont()
 }
 
 
+// x is an absolute screen coord
 // returns the column near the specified x-coordinate of the row
 // x is set to the real beginning of this column
-pos_type LyXText::getColumnNearX(ParagraphList::iterator pit,
-       Row const & row, int & x, bool & boundary) const
+pos_type LyXText::getColumnNearX(pit_type const pit,
+                                Row const & row, int & x, bool & boundary) const
 {
-       x -= xo_;
-       double tmpx             = row.x();
-       double fill_separator   = row.fill_separator();
-       double fill_hfill       = row.fill_hfill();
-       double fill_label_hfill = row.fill_label_hfill();
+       int const xo = theCoords.get(this, pit).x_;
+       x -= xo;
+       RowMetrics const r = computeRowMetrics(pit, row);
+       Paragraph const & par = pars_[pit];
 
        pos_type vc = row.pos();
        pos_type end = row.endpos();
        pos_type c = 0;
-       LyXLayout_ptr const & layout = pit->layout();
+       LyXLayout_ptr const & layout = par.layout();
 
        bool left_side = false;
 
-       pos_type body_pos = pit->beginOfBody();
+       pos_type body_pos = par.beginOfBody();
+
+       double tmpx = r.x;
        double last_tmpx = tmpx;
 
        if (body_pos > 0 &&
-           (body_pos > end || !pit->isLineSeparator(body_pos - 1)))
+           (body_pos > end || !par.isLineSeparator(body_pos - 1)))
                body_pos = 0;
 
        // check for empty row
        if (vc == end) {
-               x = int(tmpx) + xo_;
+               x = int(tmpx) + xo;
                return 0;
        }
 
@@ -1278,24 +780,24 @@ pos_type LyXText::getColumnNearX(ParagraphList::iterator pit,
                c = bidi.vis2log(vc);
                last_tmpx = tmpx;
                if (body_pos > 0 && c == body_pos - 1) {
-                       tmpx += fill_label_hfill +
-                               font_metrics::width(layout->labelsep, getLabelFont(pit));
-                       if (pit->isLineSeparator(body_pos - 1))
-                               tmpx -= singleWidth(pit, body_pos - 1);
+                       tmpx += r.label_hfill +
+                               font_metrics::width(layout->labelsep, getLabelFont(par));
+                       if (par.isLineSeparator(body_pos - 1))
+                               tmpx -= singleWidth(par, body_pos - 1);
                }
 
-               if (hfillExpansion(*pit, row, c)) {
-                       tmpx += singleWidth(pit, c);
+               if (hfillExpansion(par, row, c)) {
+                       tmpx += singleWidth(par, c);
                        if (c >= body_pos)
-                               tmpx += fill_hfill;
+                               tmpx += r.hfill;
                        else
-                               tmpx += fill_label_hfill;
-               } else if (pit->isSeparator(c)) {
-                       tmpx += singleWidth(pit, c);
+                               tmpx += r.label_hfill;
+               } else if (par.isSeparator(c)) {
+                       tmpx += singleWidth(par, c);
                        if (c >= body_pos)
-                               tmpx += fill_separator;
+                               tmpx += r.separator;
                } else {
-                       tmpx += singleWidth(pit, c);
+                       tmpx += singleWidth(par, c);
                }
                ++vc;
        }
@@ -1310,16 +812,14 @@ pos_type LyXText::getColumnNearX(ParagraphList::iterator pit,
        boundary = false;
        // This (rtl_support test) is not needed, but gives
        // some speedup if rtl_support == false
-       bool const lastrow = lyxrc.rtl_support && row.endpos() == pit->size();
+       bool const lastrow = lyxrc.rtl_support && row.endpos() == par.size();
 
        // If lastrow is false, we don't need to compute
        // the value of rtl.
-       bool const rtl = lastrow
-               ? pit->isRightToLeftPar(bv()->buffer()->params())
-               : false;
+       bool const rtl = lastrow ? isRTL(par) : false;
        if (lastrow &&
-                ((rtl  &&  left_side && vc == row.pos() && x < tmpx - 5) ||
-                 (!rtl && !left_side && vc == end  && x > tmpx + 5)))
+           ((rtl  &&  left_side && vc == row.pos() && x < tmpx - 5) ||
+            (!rtl && !left_side && vc == end  && x > tmpx + 5)))
                c = end;
        else if (vc == row.pos()) {
                c = bidi.vis2log(vc);
@@ -1330,250 +830,333 @@ pos_type LyXText::getColumnNearX(ParagraphList::iterator pit,
                bool const rtl = (bidi.level(c) % 2 == 1);
                if (left_side == rtl) {
                        ++c;
-                       boundary = bidi.isBoundary(*bv()->buffer(), *pit, c);
+                       boundary = bidi.isBoundary(*bv()->buffer(), par, c);
                }
        }
 
-       if (row.pos() < end && c >= end && pit->isNewline(end - 1)) {
+// I believe this code is not needed anymore (Jug 20050717)
+#if 0
+       // The following code is necessary because the cursor position past
+       // the last char in a row is logically equivalent to that before
+       // the first char in the next row. That's why insets causing row
+       // divisions -- Newline and display-style insets -- must be treated
+       // specially, so cursor up/down doesn't get stuck in an air gap -- MV
+       // Newline inset, air gap below:
+       if (row.pos() < end && c >= end && par.isNewline(end - 1)) {
                if (bidi.level(end -1) % 2 == 0)
-                       tmpx -= singleWidth(pit, end - 1);
+                       tmpx -= singleWidth(par, end - 1);
                else
-                       tmpx += singleWidth(pit, end - 1);
+                       tmpx += singleWidth(par, end - 1);
+               c = end - 1;
+       }
+
+       // Air gap above display inset:
+       if (row.pos() < end && c >= end && end < par.size()
+           && par.isInset(end) && par.getInset(end)->display()) {
                c = end - 1;
        }
+       // Air gap below display inset:
+       if (row.pos() < end && c >= end && par.isInset(end - 1)
+           && par.getInset(end - 1)->display()) {
+               c = end - 1;
+       }
+#endif
+
+       x = int(tmpx) + xo;
+       pos_type const col = c - row.pos();
+
+       if (!c || end == par.size())
+               return col;
+
+       if (c==end && !par.isLineSeparator(c-1) && !par.isNewline(c-1)) {
+               boundary = true;
+               return col;
+       }
 
-       x = int(tmpx) + xo_;
-       return c - row.pos();
+       return min(col, end - 1 - row.pos());
 }
 
 
-void LyXText::setCursorFromCoordinates(int x, int y)
+// y is screen coordinate
+pit_type LyXText::getPitNearY(int y) const
 {
-       CursorSlice old_cursor = cursor();
-       setCursorFromCoordinates(cursor(), x, y);
-       setCurrentFont();
-       deleteEmptyParagraphMechanism(old_cursor);
+       BOOST_ASSERT(!paragraphs().empty());
+       BOOST_ASSERT(theCoords.getParPos().find(this) != theCoords.getParPos().end());
+       CoordCache::InnerParPosCache const & cc = theCoords.getParPos().find(this)->second;
+       lyxerr[Debug::DEBUG]
+               << BOOST_CURRENT_FUNCTION
+               << ": y: " << y << " cache size: " << cc.size()
+               << endl;
+
+       // look for highest numbered paragraph with y coordinate less than given y
+       pit_type pit = 0;
+       int yy = -1;
+       CoordCache::InnerParPosCache::const_iterator it = cc.begin();
+       CoordCache::InnerParPosCache::const_iterator et = cc.end();
+       for (; it != et; ++it) {
+               lyxerr[Debug::DEBUG]
+                       << BOOST_CURRENT_FUNCTION
+                       << "  examining: pit: " << it->first
+                       << " y: " << it->second.y_
+                       << endl;
+
+               if (it->first >= pit && int(it->second.y_) - int(pars_[it->first].ascent()) <= y) {
+                       pit = it->first;
+                       yy = it->second.y_;
+               }
+       }
+
+       lyxerr[Debug::DEBUG]
+               << BOOST_CURRENT_FUNCTION
+               << ": found best y: " << yy << " for pit: " << pit
+               << endl;
+
+       return pit;
 }
 
 
-// x,y are coordinates relative to this LyXText
-void LyXText::setCursorFromCoordinates(CursorSlice & cur, int x, int y)
+Row const & LyXText::getRowNearY(int y, pit_type pit) const
 {
-       ParagraphList::iterator pit;
-       Row const & row = *getRowNearY(y, pit);
-       bool bound = false;
-       pos_type const pos = row.pos() + getColumnNearX(pit, row, x, bound);
-       cur.par() = parOffset(pit);
-       cur.pos() = pos;
-       cur.boundary() = bound;
+       Paragraph const & par = pars_[pit];
+       int yy = theCoords.get(this, pit).y_ - par.ascent();
+       BOOST_ASSERT(!par.rows().empty());
+       RowList::const_iterator rit = par.rows().begin();
+       RowList::const_iterator const rlast = boost::prior(par.rows().end());
+       for (; rit != rlast; yy += rit->height(), ++rit)
+               if (yy + rit->height() > y)
+                       break;
+       return *rit;
 }
 
 
 // x,y are absolute screen coordinates
-void LyXText::edit(LCursor & cur, int x, int y)
+// sets cursor recursively descending into nested editable insets
+InsetBase * LyXText::editXY(LCursor & cur, int x, int y)
 {
-       int xx = x; // is modified by getColumnNearX
-       ParagraphList::iterator pit;
-       Row const & row = *getRowNearY(y, pit);
+       pit_type pit = getPitNearY(y);
+       BOOST_ASSERT(pit != -1);
+       Row const & row = getRowNearY(y, pit);
        bool bound = false;
+
+       int xx = x; // is modified by getColumnNearX
        pos_type const pos = row.pos() + getColumnNearX(pit, row, xx, bound);
-       cur.par() = parOffset(pit);
+       cur.pit() = pit;
        cur.pos() = pos;
-       cur.boundary() = bound;
+       cur.boundary(bound);
+       cur.x_target() = x;
 
        // try to descend into nested insets
        InsetBase * inset = checkInsetHit(x, y);
-       if (inset) {
-               // This should be just before or just behind the cursor position
-               // set above.
-               BOOST_ASSERT((pos != 0 && inset == pit->getInset(pos - 1))
-                            || inset == pit->getInset(pos));
-               // Make sure the cursor points to the position before this inset.
-               if (inset == pit->getInset(pos - 1))
-                       --cur.pos();
-               inset->edit(cur, x, y);
+       //lyxerr << "inset " << inset << " hit at x: " << x << " y: " << y << endl;
+       if (!inset) {
+               // Either we deconst editXY or better we move current_font
+               // and real_current_font to LCursor
+               setCurrentFont(cur);
+               return 0;
        }
+
+       // This should be just before or just behind the
+       // cursor position set above.
+       BOOST_ASSERT((pos != 0 && inset == pars_[pit].getInset(pos - 1))
+                    || inset == pars_[pit].getInset(pos));
+       // Make sure the cursor points to the position before
+       // this inset.
+       if (inset == pars_[pit].getInset(pos - 1))
+               --cur.pos();
+       inset = inset->editXY(cur, x, y);
+       if (cur.top().text() == this)
+               setCurrentFont(cur);
+       return inset;
 }
 
 
-bool LyXText::checkAndActivateInset(bool front)
+bool LyXText::checkAndActivateInset(LCursor & cur, bool front)
 {
-       if (cursor().pos() == cursorPar()->size())
+       if (cur.selection())
                return false;
-       InsetBase * inset = cursorPar()->getInset(cursor().pos());
+       if (cur.pos() == cur.lastpos())
+               return false;
+       InsetBase * inset = cur.nextInset();
        if (!isHighlyEditableInset(inset))
                return false;
-       inset->edit(bv()->cursor(), front);
+       inset->edit(cur, front);
        return true;
 }
 
 
-DispatchResult LyXText::moveRight()
+bool LyXText::cursorLeft(LCursor & cur)
 {
-       if (cursorPar()->isRightToLeftPar(bv()->buffer()->params()))
-               return moveLeftIntern(false, true, false);
-       else
-               return moveRightIntern(true, true, false);
-}
-
+       if (!cur.boundary() && cur.pos() > 0 &&
+           cur.textRow().pos() == cur.pos() &&
+           !cur.paragraph().isLineSeparator(cur.pos()-1) &&
+           !cur.paragraph().isNewline(cur.pos()-1)) {
+               return setCursor(cur, cur.pit(), cur.pos(), true, true);
+       }
+       if (cur.pos() != 0) {
+               bool boundary = cur.boundary();
+               bool updateNeeded = setCursor(cur, cur.pit(), cur.pos() - 1, true, false);
+               if (!checkAndActivateInset(cur, false)) {
+                       if (false && !boundary &&
+                           bidi.isBoundary(cur.buffer(), cur.paragraph(), cur.pos() + 1))
+                               updateNeeded |=
+                                       setCursor(cur, cur.pit(), cur.pos() + 1, true, true);
+               }
+               return updateNeeded;
+       }
 
-DispatchResult LyXText::moveLeft()
-{
-       if (cursorPar()->isRightToLeftPar(bv()->buffer()->params()))
-               return moveRightIntern(true, true, false);
-       else
-               return moveLeftIntern(false, true, false);
+       if (cur.pit() != 0) {
+               // Steps into the paragraph above
+               return setCursor(cur, cur.pit() - 1, getPar(cur.pit() - 1).size());
+       }
+       return false;
 }
 
 
-DispatchResult LyXText::moveRightIntern(bool front, bool activate_inset, bool selecting)
+bool LyXText::cursorRight(LCursor & cur)
 {
-       ParagraphList::iterator c_par = cursorPar();
-       if (boost::next(c_par) == paragraphs().end()
-               && cursor().pos() >= c_par->size())
-               return DispatchResult(false, FINISHED_RIGHT);
-       if (activate_inset && checkAndActivateInset(front))
-               return DispatchResult(true, true);
-       cursorRight(true);
-       if (!selecting)
-               bv()->cursor().clearSelection();
-       return DispatchResult(true);
-}
-
+       if (cur.pos() != cur.lastpos()) {
+               if (cur.boundary())
+                       return setCursor(cur, cur.pit(), cur.pos(),
+                                        true, false);
+
+               bool updateNeeded = false;
+               if (!checkAndActivateInset(cur, true)) {
+                       if (cur.textRow().endpos() == cur.pos() + 1 &&
+                           cur.textRow().endpos() != cur.lastpos() &&
+                           !cur.paragraph().isLineSeparator(cur.pos()) &&
+                           !cur.paragraph().isNewline(cur.pos())) {
+                               cur.boundary(true);
+                       }
+                       updateNeeded |= setCursor(cur, cur.pit(), cur.pos() + 1, true, cur.boundary());
+                       if (false && bidi.isBoundary(cur.buffer(), cur.paragraph(),
+                                                    cur.pos()))
+                               updateNeeded |= setCursor(cur, cur.pit(), cur.pos(), true, true);
+               }
+               return updateNeeded;
+       }
 
-DispatchResult LyXText::moveLeftIntern(bool front,
-                         bool activate_inset, bool selecting)
-{
-       if (cursor().par() == 0 && cursor().pos() <= 0)
-               return DispatchResult(false, FINISHED);
-       cursorLeft(true);
-       if (!selecting)
-               bv()->cursor().clearSelection();
-       if (activate_inset && checkAndActivateInset(front))
-               return DispatchResult(true, true);
-       return DispatchResult(true);
+       if (cur.pit() != cur.lastpit())
+               return setCursor(cur, cur.pit() + 1, 0);
+       return false;
 }
 
 
-DispatchResult LyXText::moveUp()
+bool LyXText::cursorUp(LCursor & cur)
 {
-       if (cursorPar() == firstPar() && cursorRow() == firstRow())
-               return DispatchResult(false, FINISHED_UP);
-       cursorUp(false);
-       bv()->cursor().clearSelection();
-       return DispatchResult(true);
-}
+       Paragraph const & par = cur.paragraph();
+       int row;
+       int const x = cur.targetX();
 
+       if (cur.pos() && cur.boundary())
+               row = par.pos2row(cur.pos()-1);
+       else
+               row = par.pos2row(cur.pos());
 
-DispatchResult LyXText::moveDown()
-{
-       LCursor & cur = bv()->cursor();
-       if (cursorPar() == lastPar() && cursorRow() == lastRow())
-               return DispatchResult(false, FINISHED_DOWN);
-       cursorDown(false);
-       cur.clearSelection();
-       return DispatchResult(true);
-}
+       if (!cur.selection()) {
+               int const y = bv_funcs::getPos(cur, cur.boundary()).y_;
+               LCursor old = cur;
+               editXY(cur, x, y - par.rows()[row].ascent() - 1);
+               cur.clearSelection();
 
+               // This happens when you move out of an inset.
+               // And to give the DEPM the possibility of doing
+               // something we must provide it with two different
+               // cursors. (Lgb)
+               LCursor dummy = cur;
+               if (dummy == old)
+                       ++dummy.pos();
 
-bool LyXText::cursorLeft(bool internal)
-{
-       LCursor & cur = bv()->cursor();
-       if (cur.pos() > 0) {
-               bool boundary = cur.boundary();
-               setCursor(cur.par(), cur.pos() - 1, true, false);
-               if (!internal && !boundary &&
-                   bidi.isBoundary(*bv()->buffer(), cur.paragraph(), cur.pos() + 1))
-                       setCursor(cur.par(), cur.pos() + 1, true, true);
-               return true;
+               return deleteEmptyParagraphMechanism(dummy, old);
        }
 
-       if (cur.par() != 0) {
-               // steps into the paragraph above
-               setCursor(cur.par() - 1, boost::prior(cursorPar())->size());
-               return true;
+       bool updateNeeded = false;
+
+       if (row > 0) {
+               updateNeeded |= setCursor(cur, cur.pit(),
+                                         x2pos(cur.pit(), row - 1, x));
+       } else if (cur.pit() > 0) {
+               --cur.pit();
+               //cannot use 'par' now
+               updateNeeded |= setCursor(cur, cur.pit(),
+                                         x2pos(cur.pit(), cur.paragraph().rows().size() - 1, x));
        }
 
-       return false;
+       cur.x_target() = x;
+
+       return updateNeeded;
 }
 
 
-bool LyXText::cursorRight(bool internal)
+bool LyXText::cursorDown(LCursor & cur)
 {
-       LCursor & cur = bv()->cursor();
-       if (!internal && cur.boundary()) {
-               setCursor(cur.par(), cur.pos(), true, false);
-               return true;
-       }
+       Paragraph const & par = cur.paragraph();
+       int row;
+       int const x = cur.targetX();
 
-       if (cur.pos() != cur.lastpos()) {
-               setCursor(cur.par(), cur.pos() + 1, true, false);
-               if (!internal && bidi.isBoundary(*bv()->buffer(), cur.paragraph(),
-                                                cur.pos()))
-                       setCursor(cur.par(), cur.pos(), true, true);
-               return true;
-       }
+       if (cur.pos() && cur.boundary())
+               row = par.pos2row(cur.pos()-1);
+       else
+               row = par.pos2row(cur.pos());
 
-       if (cur.par() + 1 != int(paragraphs().size())) {
-               setCursor(cur.par() + 1, 0);
-               return true;
-       }
+       if (!cur.selection()) {
+               int const y = bv_funcs::getPos(cur, cur.boundary()).y_;
+               LCursor old = cur;
+               editXY(cur, x, y + par.rows()[row].descent() + 1);
+               cur.clearSelection();
 
-       return false;
-}
+               // This happens when you move out of an inset.
+               // And to give the DEPM the possibility of doing
+               // something we must provide it with two different
+               // cursors. (Lgb)
+               LCursor dummy = cur;
+               if (dummy == old)
+                       ++dummy.pos();
 
+               bool const changed = deleteEmptyParagraphMechanism(dummy, old);
 
-void LyXText::cursorUp(bool selecting)
-{
-       LCursor & cur = bv()->cursor();
-       Row const & row = *cursorRow();
-       int x = cur.x_target();
-       int y = cursorY(cur.current()) - row.baseline() - 1;
-       setCursorFromCoordinates(x, y);
-
-       if (!selecting) {
-               InsetBase * inset_hit = checkInsetHit(cur.x_target(), y);
-               if (inset_hit && isHighlyEditableInset(inset_hit))
-                       inset_hit->edit(cur, cur.x_target(), y);
+               // Make sure that cur gets back whatever happened to dummy(Lgb)
+               if (changed)
+                       cur = dummy;
+
+               return changed;
        }
-}
 
+       bool updateNeeded = false;
 
-void LyXText::cursorDown(bool selecting)
-{
-       LCursor & cur = bv()->cursor();
-       Row const & row = *cursorRow();
-       int x = cur.x_target();
-       int y = cursorY(cur.current()) - row.baseline() + row.height() + 1;
-       setCursorFromCoordinates(x, y);
-
-       if (!selecting) {
-               InsetBase * inset_hit = checkInsetHit(cur.x_target(), y);
-               if (inset_hit && isHighlyEditableInset(inset_hit))
-                       inset_hit->edit(cur, cur.x_target(), y);
+       if (row + 1 < int(par.rows().size())) {
+               updateNeeded |= setCursor(cur, cur.pit(),
+                                         x2pos(cur.pit(), row + 1, x));
+       } else if (cur.pit() + 1 < int(paragraphs().size())) {
+               ++cur.pit();
+               updateNeeded |= setCursor(cur, cur.pit(),
+                                         x2pos(cur.pit(), 0, x));
        }
+
+       cur.x_target() = x;
+
+       return updateNeeded;
 }
 
 
-void LyXText::cursorUpParagraph()
+bool LyXText::cursorUpParagraph(LCursor & cur)
 {
-       ParagraphList::iterator cpit = cursorPar();
-       if (cursor().pos() > 0)
-               setCursor(cpit, 0);
-       else if (cpit != paragraphs().begin())
-               setCursor(boost::prior(cpit), 0);
+       bool updated = false;
+       if (cur.pos() > 0)
+               updated = setCursor(cur, cur.pit(), 0);
+       else if (cur.pit() != 0)
+               updated = setCursor(cur, cur.pit() - 1, 0);
+       return updated;
 }
 
 
-void LyXText::cursorDownParagraph()
+bool LyXText::cursorDownParagraph(LCursor & cur)
 {
-       ParagraphList::iterator pit = cursorPar();
-       ParagraphList::iterator next_pit = boost::next(pit);
-
-       if (next_pit != paragraphs().end())
-               setCursor(next_pit, 0);
+       bool updated = false;
+       if (cur.pit() != cur.lastpit())
+               updated = setCursor(cur, cur.pit() + 1, 0);
        else
-               setCursor(pit, pit->size());
+               updated = setCursor(cur, cur.pit(), cur.lastpos());
+       return updated;
 }
 
 
@@ -1581,176 +1164,138 @@ void LyXText::cursorDownParagraph()
 // position. Called by deleteEmptyParagraphMechanism
 void LyXText::fixCursorAfterDelete(CursorSlice & cur, CursorSlice const & where)
 {
-       // if cursor is not in the paragraph where the delete occured,
-       // do nothing
-       if (cur.par() != where.par())
+       // Do nothing if cursor is not in the paragraph where the
+       // deletion occured,
+       if (cur.pit() != where.pit())
                return;
 
-       // if cursor position is after the place where the delete occured,
-       // update it
+       // If cursor position is after the deletion place update it
        if (cur.pos() > where.pos())
-               cur.pos(cur.pos()-1);
+               --cur.pos();
 
-       // check also if we don't want to set the cursor on a spot behind the
+       // Check also if we don't want to set the cursor on a spot behind the
        // pagragraph because we erased the last character.
        if (cur.pos() > cur.lastpos())
                cur.pos() = cur.lastpos();
 }
 
 
-bool LyXText::deleteEmptyParagraphMechanism(CursorSlice const & old_cursor)
+bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur, LCursor & old)
 {
-#warning Disabled as it crashes after the cursor data shift... (Andre)
-       return false;
-
        // Would be wrong to delete anything if we have a selection.
-       if (bv()->cursor().selection())
+       if (cur.selection())
                return false;
 
-       // Don't do anything if the cursor is invalid
-       if (old_cursor.par() == -1)
-               return false;
+       //lyxerr[Debug::DEBUG] << "DEPM: cur:\n" << cur << "old:\n" << old << endl;
+       // old should point to us
+       BOOST_ASSERT(old.text() == this);
+
+       Paragraph & oldpar = old.paragraph();
 
-#if 0
        // We allow all kinds of "mumbo-jumbo" when freespacing.
-       ParagraphList::iterator const old_pit = getPar(old_cursor);
-       if (old_pit->isFreeSpacing())
+       if (oldpar.isFreeSpacing())
                return false;
 
        /* Ok I'll put some comments here about what is missing.
-          I have fixed BackSpace (and thus Delete) to not delete
-          double-spaces automagically. I have also changed Cut,
-          Copy and Paste to hopefully do some sensible things.
           There are still some small problems that can lead to
           double spaces stored in the document file or space at
           the beginning of paragraphs(). This happens if you have
           the cursor between to spaces and then save. Or if you
           cut and paste and the selection have a space at the
-          beginning and then save right after the paste. I am
-          sure none of these are very hard to fix, but I will
-          put out 1.1.4pre2 with FIX_DOUBLE_SPACE defined so
-          that I can get some feedback. (Lgb)
+          beginning and then save right after the paste. (Lgb)
        */
 
-       // If old_cursor.pos() == 0 and old_cursor.pos()(1) == LineSeparator
+       // If old.pos() == 0 and old.pos()(1) == LineSeparator
        // delete the LineSeparator.
        // MISSING
 
-       // If old_cursor.pos() == 1 and old_cursor.pos()(0) == LineSeparator
+       // If old.pos() == 1 and old.pos()(0) == LineSeparator
        // delete the LineSeparator.
        // MISSING
 
-       // If the pos around the old_cursor were spaces, delete one of them.
-       if (old_cursor.par() != cursor().par()
-           || old_cursor.pos() != cursor().pos()) {
-
-               // Only if the cursor has really moved
-               if (old_cursor.pos() > 0
-                   && old_cursor.pos() < old_pit->size()
-                   && old_pit->isLineSeparator(old_cursor.pos())
-                   && old_pit->isLineSeparator(old_cursor.pos() - 1)) {
-                       bool erased = old_pit->erase(old_cursor.pos() - 1);
-                       redoParagraph(old_pit);
-
-                       if (!erased)
-                               return false;
+       bool const same_inset = &old.inset() == &cur.inset();
+       bool const same_par = same_inset && old.pit() == cur.pit();
+       bool const same_par_pos = same_par && old.pos() == cur.pos();
+
+       // If the chars around the old cursor were spaces, delete one of them.
+       if (!same_par_pos) {
+               // Only if the cursor has really moved.
+               if (old.pos() > 0
+                   && old.pos() < oldpar.size()
+                   && oldpar.isLineSeparator(old.pos())
+                   && oldpar.isLineSeparator(old.pos() - 1)
+                   && oldpar.lookupChange(old.pos() - 1) != Change::DELETED) {
+                       // We need to set the text to Change::INSERTED to
+                       // get it erased properly
+                       oldpar.setChange(old.pos() -1, Change::INSERTED);
+                       oldpar.erase(old.pos() - 1);
 #ifdef WITH_WARNINGS
 #warning This will not work anymore when we have multiple views of the same buffer
 // In this case, we will have to correct also the cursors held by
 // other bufferviews. It will probably be easier to do that in a more
 // automated way in CursorSlice code. (JMarc 26/09/2001)
 #endif
-                       // correct all cursors held by the LyXText
-                       fixCursorAfterDelete(cursor(), old_cursor);
-                       fixCursorAfterDelete(anchor(), old_cursor);
-                       return false;
+                       // correct all cursor parts
+                       if (same_par) {
+                               fixCursorAfterDelete(cur.top(), old.top());
+                               cur.resetAnchor();
+                       }
+                       return true;
                }
        }
 
-       // don't delete anything if this is the ONLY paragraph!
-       if (paragraphs().size() == 1)
+       // only do our magic if we changed paragraph
+       if (same_par)
                return false;
 
-       // Do not delete empty paragraphs with keepempty set.
-       if (old_pit->allowEmpty())
+       // don't delete anything if this is the ONLY paragraph!
+       if (old.lastpit() == 0)
                return false;
 
-       // only do our magic if we changed paragraph
-       if (old_cursor.par() == cursor().par())
+       // Do not delete empty paragraphs with keepempty set.
+       if (oldpar.allowEmpty())
                return false;
 
-       // record if we have deleted a paragraph
-       // we can't possibly have deleted a paragraph before this point
-       bool deleted = false;
-
-       if (old_pit->empty()
-           || (old_pit->size() == 1 && old_pit->isLineSeparator(0))) {
-               // ok, we will delete something
-               CursorSlice tmpcursor;
-
-               deleted = true;
-
-               bool selection_position_was_oldcursor_position =
-                       anchor().par() == old_cursor.par()
-                       && anchor().pos() == old_cursor.pos();
-
-               tmpcursor = cursor();
-               cursor() = old_cursor; // that undo can restore the right cursor position
-
-               ParagraphList::iterator endpit = boost::next(old_pit);
-               while (endpit != paragraphs().end() && endpit->getDepth())
-                       ++endpit;
-
-               recUndo(parOffset(old_pit), parOffset(endpit) - 1);
-               cursor() = tmpcursor;
-
-               // cache cursor pit
-               ParagraphList::iterator tmppit = cursorPar();
-               // delete old par
-               paragraphs().erase(old_pit);
-               // update cursor par offset
-               cursor().par(parOffset(tmppit));
-               redoParagraph();
-
-               if (selection_position_was_oldcursor_position) {
-                       // correct selection
-                       bv()->resetAnchor();
+       if (oldpar.empty() || (oldpar.size() == 1 && oldpar.isLineSeparator(0))) {
+               // Delete old par.
+               recordUndo(old, Undo::ATOMIC, old.pit());
+               ParagraphList & plist = old.text()->paragraphs();
+               plist.erase(plist.begin() + old.pit());
+
+               // see #warning above
+               if (cur.depth() >= old.depth()) {
+                       CursorSlice & curslice = cur[old.depth() - 1];
+                       if (&curslice.inset() == &old.inset() 
+                           && curslice.pit() > old.pit()) {
+                               --curslice.pit();
+                               // since a paragraph has been deleted, all the
+                               // insets after `old' have been copied and
+                               // their address has changed. Therefore we
+                               // need to `regenerate' cur. (JMarc)
+                               cur.updateInsets(&(cur.bottom().inset()));
+                               cur.resetAnchor();
+                       }
                }
-       }
-
-       if (deleted)
+               updateCounters(old.buffer());
                return true;
-
-       if (old_pit->stripLeadingSpaces()) {
-               redoParagraph(old_pit);
-               bv()->resetAnchor();
        }
-       return false;
-#endif
-}
-
-
-ParagraphList & LyXText::paragraphs() const
-{
-       return const_cast<ParagraphList &>(paragraphs_);
-}
 
+       if (oldpar.stripLeadingSpaces())
+               cur.resetAnchor();
 
-void LyXText::recUndo(paroffset_type first, paroffset_type last) const
-{
-       recordUndo(Undo::ATOMIC, this, first, last);
+       return false;
 }
 
 
-void LyXText::recUndo(lyx::paroffset_type par) const
+void LyXText::recUndo(pit_type first, pit_type last) const
 {
-       recordUndo(Undo::ATOMIC, this, par, par);
+       recordUndo(bv()->cursor(), Undo::ATOMIC, first, last);
 }
 
 
-bool LyXText::isInInset() const
+void LyXText::recUndo(pit_type par) const
 {
-       return in_inset_;
+       recordUndo(bv()->cursor(), Undo::ATOMIC, par, par);
 }