]> git.lyx.org Git - lyx.git/blobdiff - src/text2.C
Alfredo's second patch
[lyx.git] / src / text2.C
index f8b6104d841dee5ca202ca21e53920e7996de9fb..c7babeb609ecf9f262e0211f224fd09dc28293e4 100644 (file)
 #include "language.h"
 #include "ParagraphParameters.h"
 #include "counters.h"
+#include "lyxrow_funcs.h"
+#include "paragraph_funcs.h"
 
-#include "insets/inseterror.h"
-#include "insets/insetbib.h"
-#include "insets/insetspecialchar.h"
-#include "insets/insettext.h"
+#include "insets/insetbibitem.h"
 #include "insets/insetfloat.h"
-#include "insets/insetwrap.h"
 
 #include "support/LAssert.h"
 #include "support/textutils.h"
 #include "support/lstrings.h"
 
-#include "BoostFormat.h"
+#include "support/BoostFormat.h"
+#include <boost/tuple/tuple.hpp>
 
 using std::vector;
 using std::copy;
@@ -52,92 +51,52 @@ using lyx::pos_type;
 
 
 LyXText::LyXText(BufferView * bv)
-       : height(0), width(0), first_y(0),
-         inset_owner(0), the_locking_inset(0), need_break_row(0), 
-         refresh_y(0), refresh_row(0), bv_owner(bv), 
-         status_(LyXText::UNCHANGED), firstrow(0), lastrow(0)
-{}
-
-
-LyXText::LyXText(InsetText * inset)
-       :  height(0), width(0), first_y(0),
-          inset_owner(inset), the_locking_inset(0), need_break_row(0), 
-          refresh_y(0), refresh_row(0), bv_owner(0), 
-          status_(LyXText::UNCHANGED), firstrow(0), lastrow(0)
-{}
-
-
-void LyXText::init(BufferView * bview, bool reinit)
+       : height(0), width(0), anchor_row_offset_(0),
+         inset_owner(0), the_locking_inset(0), bv_owner(bv)
 {
-       if (reinit) {
-               // Delete all rows, this does not touch the paragraphs!
-               Row * tmprow = firstrow;
-               while (firstrow) {
-                       tmprow = firstrow->next();
-                       delete firstrow;
-                       firstrow = tmprow;
-               }
+       anchor_row_ = rows().end();
+       need_break_row = rows().end();
+       refresh_row = rows().end();
 
-               lastrow = 0;
-               refresh_row = 0;
-               need_break_row = 0;
-               width = height = 0;
-               copylayouttype.erase();
-               first_y = refresh_y = 0;
-               status_ = LyXText::UNCHANGED;
-       } else if (firstrow)
-               return;
+       clearPaint();
+}
 
-       Paragraph * par = ownerParagraph();
-       current_font = getFont(bview->buffer(), par, 0);
 
-       while (par) {
-               insertParagraph(bview, par, lastrow);
-               par = par->next();
-       }
-       setCursorIntern(bview, firstrow->par(), 0);
-       selection.cursor = cursor;
+LyXText::LyXText(BufferView * bv, InsetText * inset)
+       : height(0), width(0), anchor_row_offset_(0),
+         inset_owner(inset), the_locking_inset(0), bv_owner(bv)
+{
+       anchor_row_ = rows().end();
+       need_break_row = rows().end();
+       refresh_row = rows().end();
 
-       updateCounters(bview);
+       clearPaint();
 }
 
 
-LyXText::~LyXText()
+void LyXText::init(BufferView * bview, bool reinit)
 {
-       // Delete all rows, this does not touch the paragraphs!
-       Row * tmprow = firstrow;
-       while (firstrow) {
-               tmprow = firstrow->next();
-               delete firstrow;
-               firstrow = tmprow;
-       }
-}
+       if (reinit) {
+               rowlist_.clear();
+               need_break_row = rows().end();
+               width = height = 0;
+               top_y(0);
+               clearPaint();
+       } else if (!rowlist_.empty())
+               return;
 
+       ParagraphList::iterator pit = ownerParagraphs().begin();
+       ParagraphList::iterator end = ownerParagraphs().end();
 
-namespace {
+       current_font = getFont(bview->buffer(), pit, 0);
 
-LyXFont const realizeFont(LyXFont const & font,
-                         Buffer const * buf,
-                         Paragraph * par)
-{
-       LyXTextClass const & tclass = buf->params.getLyXTextClass();
-       LyXFont tmpfont(font);
-       Paragraph::depth_type par_depth = par->getDepth();
-
-       // Resolve against environment font information
-       while (par && par_depth && !tmpfont.resolved()) {
-               par = par->outerHook();
-               if (par) {
-                       tmpfont.realize(par->layout()->font);
-                       par_depth = par->getDepth();
-               }
+       for (; pit != end; ++pit) {
+               insertParagraph(pit, rowlist_.end());
        }
+       setCursorIntern(rowlist_.begin()->par(), 0);
+       selection.cursor = cursor;
 
-       tmpfont.realize(tclass.defaultfont());
-
-       return tmpfont;
-}
-
+       updateCounters();
 }
 
 
@@ -148,26 +107,26 @@ LyXFont const realizeFont(LyXFont const & font,
 // smaller. (Asger)
 // If position is -1, we get the layout font of the paragraph.
 // If position is -2, we get the font of the manual label of the paragraph.
-LyXFont const LyXText::getFont(Buffer const * buf, Paragraph * par,
+LyXFont const LyXText::getFont(Buffer const * buf, ParagraphList::iterator pit,
                               pos_type pos) const
 {
        lyx::Assert(pos >= 0);
 
-       LyXLayout_ptr const & layout = par->layout();
+       LyXLayout_ptr const & layout = pit->layout();
 
        // We specialize the 95% common case:
-       if (!par->getDepth()) {
+       if (!pit->getDepth()) {
                if (layout->labeltype == LABEL_MANUAL
-                   && pos < par->beginningOfMainBody()) {
+                   && pos < pit->beginningOfBody()) {
                        // 1% goes here
-                       LyXFont f = par->getFontSettings(buf->params, pos);
-                       if (par->inInset())
-                               par->inInset()->getDrawFont(f);
+                       LyXFont f = pit->getFontSettings(buf->params, pos);
+                       if (pit->inInset())
+                               pit->inInset()->getDrawFont(f);
                        return f.realize(layout->reslabelfont);
                } else {
-                       LyXFont f = par->getFontSettings(buf->params, pos);
-                       if (par->inInset())
-                               par->inInset()->getDrawFont(f);
+                       LyXFont f = pit->getFontSettings(buf->params, pos);
+                       if (pit->inInset())
+                               pit->inInset()->getDrawFont(f);
                        return f.realize(layout->resfont);
                }
        }
@@ -176,7 +135,7 @@ LyXFont const LyXText::getFont(Buffer const * buf, Paragraph * par,
 
        LyXFont layoutfont;
 
-       if (pos < par->beginningOfMainBody()) {
+       if (pos < pit->beginningOfBody()) {
                // 1% goes here
                layoutfont = layout->labelfont;
        } else {
@@ -184,84 +143,99 @@ LyXFont const LyXText::getFont(Buffer const * buf, Paragraph * par,
                layoutfont = layout->font;
        }
 
-       LyXFont tmpfont = par->getFontSettings(buf->params, pos);
+       LyXFont tmpfont = pit->getFontSettings(buf->params, pos);
        tmpfont.realize(layoutfont);
 
-       if (par->inInset())
-               par->inInset()->getDrawFont(tmpfont);
+       if (pit->inInset())
+               pit->inInset()->getDrawFont(tmpfont);
+
+       // Realize with the fonts of lesser depth.
+       tmpfont.realize(outerFont(pit, ownerParagraphs()));
 
-       return realizeFont(tmpfont, buf, par);
+       return realizeFont(tmpfont, buf->params);
 }
 
 
-LyXFont const LyXText::getLayoutFont(Buffer const * buf, Paragraph * par) const
+LyXFont const LyXText::getLayoutFont(Buffer const * buf,
+                                    ParagraphList::iterator pit) const
 {
-       LyXLayout_ptr const & layout = par->layout();
+       LyXLayout_ptr const & layout = pit->layout();
 
-       if (!par->getDepth()) {
+       if (!pit->getDepth()) {
                return layout->resfont;
        }
 
-       return realizeFont(layout->font, buf, par);
+       LyXFont font(layout->font);
+       // Realize with the fonts of lesser depth.
+       font.realize(outerFont(pit, ownerParagraphs()));
+
+       return realizeFont(font, buf->params);
 }
 
 
-LyXFont const LyXText::getLabelFont(Buffer const * buf, Paragraph * par) const
+LyXFont const LyXText::getLabelFont(Buffer const * buf,
+                                   ParagraphList::iterator pit) const
 {
-       LyXLayout_ptr const & layout = par->layout();
+       LyXLayout_ptr const & layout = pit->layout();
 
-       if (!par->getDepth()) {
+       if (!pit->getDepth()) {
                return layout->reslabelfont;
        }
 
-       return realizeFont(layout->labelfont, buf, par);
+       LyXFont font(layout->labelfont);
+       // Realize with the fonts of lesser depth.
+       font.realize(outerFont(pit, ownerParagraphs()));
+
+       return realizeFont(layout->labelfont, buf->params);
 }
 
 
-void LyXText::setCharFont(BufferView * bv, Paragraph * par,
+void LyXText::setCharFont(ParagraphList::iterator pit,
                          pos_type pos, LyXFont const & fnt,
                          bool toggleall)
 {
-       Buffer const * buf = bv->buffer();
-       LyXFont font = getFont(buf, par, pos);
+       Buffer const * buf = bv()->buffer();
+       LyXFont font = getFont(buf, pit, pos);
        font.update(fnt, buf->params.language, toggleall);
        // Let the insets convert their font
-       if (par->isInset(pos)) {
-               Inset * inset = par->getInset(pos);
+       if (pit->isInset(pos)) {
+               Inset * inset = pit->getInset(pos);
                if (isEditableInset(inset)) {
                        UpdatableInset * uinset =
                                static_cast<UpdatableInset *>(inset);
-                       uinset->setFont(bv, fnt, toggleall, true);
+                       uinset->setFont(bv(), fnt, toggleall, true);
                }
        }
 
        // Plug thru to version below:
-       setCharFont(buf, par, pos, font);
+       setCharFont(buf, pit, pos, font);
 }
 
 
-void LyXText::setCharFont(Buffer const * buf, Paragraph * par,
+void LyXText::setCharFont(Buffer const * buf, ParagraphList::iterator pit,
                          pos_type pos, LyXFont const & fnt)
 {
        LyXFont font(fnt);
 
        LyXTextClass const & tclass = buf->params.getLyXTextClass();
-       LyXLayout_ptr const & layout = par->layout();
+       LyXLayout_ptr const & layout = pit->layout();
 
        // Get concrete layout font to reduce against
        LyXFont layoutfont;
 
-       if (pos < par->beginningOfMainBody())
+       if (pos < pit->beginningOfBody())
                layoutfont = layout->labelfont;
        else
                layoutfont = layout->font;
 
        // Realize against environment font information
-       if (par->getDepth()) {
-               Paragraph * tp = par;
-               while (!layoutfont.resolved() && tp && tp->getDepth()) {
-                       tp = tp->outerHook();
-                       if (tp)
+       if (pit->getDepth()) {
+               ParagraphList::iterator tp = pit;
+               while (!layoutfont.resolved() &&
+                      tp != ownerParagraphs().end() &&
+                      tp->getDepth()) {
+                       tp = outerHook(tp, ownerParagraphs());
+                       if (tp != ownerParagraphs().end())
                                layoutfont.realize(tp->layout()->font);
                }
        }
@@ -271,112 +245,80 @@ void LyXText::setCharFont(Buffer const * buf, Paragraph * par,
        // Now, reduce font against full layout font
        font.reduce(layoutfont);
 
-       par->setFont(pos, font);
-}
-
-
-// inserts a new row behind the specified row, increments
-// the touched counters
-void LyXText::insertRow(Row * row, Paragraph * par,
-                       pos_type pos) const
-{
-       Row * tmprow = new Row;
-       if (!row) {
-               tmprow->previous(0);
-               tmprow->next(firstrow);
-               firstrow = tmprow;
-       } else {
-               tmprow->previous(row);
-               tmprow->next(row->next());
-               row->next(tmprow);
-       }
-
-       if (tmprow->next())
-               tmprow->next()->previous(tmprow);
-
-       if (tmprow->previous())
-               tmprow->previous()->next(tmprow);
-
-
-       tmprow->par(par);
-       tmprow->pos(pos);
-
-       if (row == lastrow)
-               lastrow = tmprow;
+       pit->setFont(pos, font);
 }
 
 
 // removes the row and reset the touched counters
-void LyXText::removeRow(Row * row) const
+void LyXText::removeRow(RowList::iterator rit)
 {
-       Row * row_prev = row->previous();
-       if (row->next())
-               row->next()->previous(row_prev);
-       if (!row_prev) {
-               firstrow = row->next();
-//             lyx::Assert(firstrow);
-       } else  {
-               row_prev->next(row->next());
-       }
-       if (row == lastrow) {
-               lyx::Assert(!row->next());
-               lastrow = row_prev;
+       /* FIXME: when we cache the bview, this should just
+        * become a postPaint(), I think */
+       if (refresh_row == rit) {
+               if (rit == rows().begin())
+                       refresh_row = boost::next(rit);
+               else
+                       refresh_row = boost::prior(rit);
+
+               // what about refresh_y
        }
-       if (refresh_row == row) {
-               refresh_row = row_prev ? row_prev : row->next();
-               // what about refresh_y, refresh_height
+
+       if (anchor_row_ == rit) {
+               if (rit != rows().begin()) {
+                       anchor_row_ = boost::prior(rit);
+                       anchor_row_offset_ += anchor_row_->height();
+               } else {
+                       anchor_row_ = boost::next(rit);
+                       anchor_row_offset_ -= rit->height();
+               }
        }
 
-       height -= row->height(); // the text becomes smaller
+       // the text becomes smaller
+       height -= rit->height();
 
-       delete row;
+       rowlist_.erase(rit);
 }
 
 
 // remove all following rows of the paragraph of the specified row.
-void LyXText::removeParagraph(Row * row) const
+void LyXText::removeParagraph(RowList::iterator rit)
 {
-       Paragraph * tmppar = row->par();
-       row = row->next();
+       ParagraphList::iterator tmppit = rit->par();
+       ++rit;
 
-       Row * tmprow;
-       while (row && row->par() == tmppar) {
-               tmprow = row->next();
-               removeRow(row);
-               row = tmprow;
+       while (rit != rows().end() && rit->par() == tmppit) {
+               RowList::iterator tmprit = boost::next(rit);
+               removeRow(rit);
+               rit = tmprit;
        }
 }
 
 
-// insert the specified paragraph behind the specified row
-void LyXText::insertParagraph(BufferView * bview, Paragraph * par,
-                             Row * row) const
+void LyXText::insertParagraph(ParagraphList::iterator pit,
+                             RowList::iterator rowit)
 {
        // insert a new row, starting at position 0
-       insertRow(row, par, 0);
+       Row newrow(pit, 0);
+       RowList::iterator rit = rowlist_.insert(rowit, newrow);
 
-       // and now append the whole paragraph behind the new row
-       if (!row) {
-               firstrow->height(0);
-               appendParagraph(bview, firstrow);
-       } else {
-               row->next()->height(0);
-               appendParagraph(bview, row->next());
-       }
+       // and now append the whole paragraph before the new row
+       appendParagraph(rit);
 }
 
 
 Inset * LyXText::getInset() const
 {
-       if (cursor.pos() < cursor.par()->size()
-                  && cursor.par()->isInset(cursor.pos())) {
-               return cursor.par()->getInset(cursor.pos());
+       ParagraphList::iterator pit = cursor.par();
+       pos_type const pos = cursor.pos();
+
+       if (pos < pit->size() && pit->isInset(pos)) {
+               return pit->getInset(pos);
        }
        return 0;
 }
 
 
-void LyXText::toggleInset(BufferView * bview)
+void LyXText::toggleInset()
 {
        Inset * inset = getInset();
        // is there an editable inset at cursor position?
@@ -384,100 +326,102 @@ void LyXText::toggleInset(BufferView * bview)
                // No, try to see if we are inside a collapsable inset
                if (inset_owner && inset_owner->owner()
                    && inset_owner->owner()->isOpen()) {
-                       bview->unlockInset(static_cast<UpdatableInset *>(inset_owner->owner()));
-                       inset_owner->owner()->close(bview);
-                       bview->getLyXText()->cursorRight(bview);
+                       bv()->unlockInset(static_cast<UpdatableInset *>(inset_owner->owner()));
+                       inset_owner->owner()->close(bv());
+                       bv()->getLyXText()->cursorRight(bv());
                }
                return;
        }
-       //bview->owner()->message(inset->editMessage());
+       //bv()->owner()->message(inset->editMessage());
 
        // do we want to keep this?? (JMarc)
        if (!isHighlyEditableInset(inset))
-               setCursorParUndo(bview);
+               setCursorParUndo(bv());
 
        if (inset->isOpen()) {
-               inset->close(bview);
+               inset->close(bv());
        } else {
-               inset->open(bview);
+               inset->open(bv());
        }
-#if 0
-       inset->open(bview, !inset->isOpen());
-#endif
+
+       bv()->updateInset(inset);
 }
 
 
 /* used in setlayout */
 // Asger is not sure we want to do this...
-void LyXText::makeFontEntriesLayoutSpecific(Buffer const * buf,
-                                           Paragraph * par)
+void LyXText::makeFontEntriesLayoutSpecific(Buffer const & buf,
+                                           Paragraph & par)
 {
-       LyXLayout_ptr const & layout = par->layout();
+       LyXLayout_ptr const & layout = par.layout();
+       pos_type const psize = par.size();
 
        LyXFont layoutfont;
-       for (pos_type pos = 0; pos < par->size(); ++pos) {
-               if (pos < par->beginningOfMainBody())
+       for (pos_type pos = 0; pos < psize; ++pos) {
+               if (pos < par.beginningOfBody())
                        layoutfont = layout->labelfont;
                else
                        layoutfont = layout->font;
 
-               LyXFont tmpfont = par->getFontSettings(buf->params, pos);
+               LyXFont tmpfont = par.getFontSettings(buf.params, pos);
                tmpfont.reduce(layoutfont);
-               par->setFont(pos, tmpfont);
+               par.setFont(pos, tmpfont);
        }
 }
 
 
-Paragraph * LyXText::setLayout(BufferView * bview,
-                              LyXCursor & cur, LyXCursor & sstart_cur,
-                              LyXCursor & send_cur,
-                              string const & layout)
+ParagraphList::iterator
+LyXText::setLayout(LyXCursor & cur, LyXCursor & sstart_cur,
+                  LyXCursor & send_cur,
+                  string const & layout)
 {
-       Paragraph * endpar = send_cur.par()->next();
-       Paragraph * undoendpar = endpar;
+       ParagraphList::iterator endpit = boost::next(send_cur.par());
+       ParagraphList::iterator undoendpit = endpit;
+       ParagraphList::iterator pars_end = ownerParagraphs().end();
 
-       if (endpar && endpar->getDepth()) {
-               while (endpar && endpar->getDepth()) {
-                       endpar = endpar->next();
-                       undoendpar = endpar;
+       if (endpit != pars_end && endpit->getDepth()) {
+               while (endpit != pars_end && endpit->getDepth()) {
+                       ++endpit;
+                       undoendpit = endpit;
                }
-       } else if (endpar) {
-               endpar = endpar->next(); // because of parindents etc.
+       } else if (endpit != pars_end) {
+               // because of parindents etc.
+               ++endpit;
        }
 
-       setUndo(bview, Undo::EDIT, sstart_cur.par(), undoendpar);
+       setUndo(bv(), Undo::EDIT, sstart_cur.par(), undoendpit);
 
        // ok we have a selection. This is always between sstart_cur
        // and sel_end cursor
        cur = sstart_cur;
-       Paragraph * par = sstart_cur.par();
-       Paragraph * epar = send_cur.par()->next();
+       ParagraphList::iterator pit = sstart_cur.par();
+       ParagraphList::iterator epit = boost::next(send_cur.par());
 
        LyXLayout_ptr const & lyxlayout =
-               bview->buffer()->params.getLyXTextClass()[layout];
+               bv()->buffer()->params.getLyXTextClass()[layout];
 
        do {
-               par->applyLayout(lyxlayout);
-               makeFontEntriesLayoutSpecific(bview->buffer(), par);
-               Paragraph * fppar = par;
-               fppar->params().spaceTop(lyxlayout->fill_top ?
+               pit->applyLayout(lyxlayout);
+               makeFontEntriesLayoutSpecific(*bv()->buffer(), *pit);
+               ParagraphList::iterator fppit = pit;
+               fppit->params().spaceTop(lyxlayout->fill_top ?
                                         VSpace(VSpace::VFILL)
                                         : VSpace(VSpace::NONE));
-               fppar->params().spaceBottom(lyxlayout->fill_bottom ?
+               fppit->params().spaceBottom(lyxlayout->fill_bottom ?
                                            VSpace(VSpace::VFILL)
                                            : VSpace(VSpace::NONE));
                if (lyxlayout->margintype == MARGIN_MANUAL)
-                       par->setLabelWidthString(lyxlayout->labelstring());
-               cur.par(par);
-               par = par->next();
-       } while (par != epar);
+                       pit->setLabelWidthString(lyxlayout->labelstring());
+               cur.par(pit);
+               ++pit;
+       } while (pit != epit);
 
-       return endpar;
+       return endpit;
 }
 
 
 // set layout over selection and make a total rebreak of those paragraphs
-void LyXText::setLayout(BufferView * bview, string const & layout)
+void LyXText::setLayout(string const & layout)
 {
        LyXCursor tmpcursor = cursor;  /* store the current cursor  */
 
@@ -487,161 +431,124 @@ void LyXText::setLayout(BufferView * bview, string const & layout)
                selection.start = cursor;  // dummy selection
                selection.end = cursor;
        }
-       Paragraph * endpar = setLayout(bview, cursor, selection.start,
-                                      selection.end, layout);
-       redoParagraphs(bview, selection.start, endpar);
+       ParagraphList::iterator endpit = setLayout(cursor, selection.start,
+                                                  selection.end, layout);
+       redoParagraphs(selection.start, endpit);
 
        // we have to reset the selection, because the
        // geometry could have changed
-       setCursor(bview, selection.start.par(),
+       setCursor(selection.start.par(),
                  selection.start.pos(), false);
        selection.cursor = cursor;
-       setCursor(bview, selection.end.par(), selection.end.pos(), false);
-       updateCounters(bview);
+       setCursor(selection.end.par(), selection.end.pos(), false);
+       updateCounters();
        clearSelection();
-       setSelection(bview);
-       setCursor(bview, tmpcursor.par(), tmpcursor.pos(), true);
+       setSelection();
+       setCursor(tmpcursor.par(), tmpcursor.pos(), true);
 }
 
 
-// increment depth over selection and
-// make a total rebreak of those paragraphs
-void  LyXText::incDepth(BufferView * bview)
+bool LyXText::changeDepth(bv_funcs::DEPTH_CHANGE type, bool test_only)
 {
-       // If there is no selection, just use the current paragraph
-       if (!selection.set()) {
-               selection.start = cursor; // dummy selection
-               selection.end = cursor;
-       }
-
-       // We end at the next paragraph with depth 0
-       Paragraph * endpar = selection.end.par()->next();
+       ParagraphList::iterator pit(cursor.par());
+       ParagraphList::iterator end(cursor.par());
+       ParagraphList::iterator start = pit;
 
-       Paragraph * undoendpar = endpar;
-
-       if (endpar && endpar->getDepth()) {
-               while (endpar && endpar->getDepth()) {
-                       endpar = endpar->next();
-                       undoendpar = endpar;
-               }
-       } else if (endpar) {
-               endpar = endpar->next(); // because of parindents etc.
+       if (selection.set()) {
+               pit = selection.start.par();
+               end = selection.end.par();
+               start = pit;
        }
 
-       setUndo(bview, Undo::EDIT,
-               selection.start.par(), undoendpar);
+       ParagraphList::iterator pastend = boost::next(end);
 
-       LyXCursor tmpcursor = cursor; // store the current cursor
+       if (!test_only)
+               setUndo(bv(), Undo::EDIT, start, pastend);
 
-       // ok we have a selection. This is always between sel_start_cursor
-       // and sel_end cursor
-       cursor = selection.start;
+       bool changed = false;
+
+       int prev_after_depth = 0;
+#warning parlist ... could be nicer ?
+       if (start != ownerParagraphs().begin()) {
+               prev_after_depth = boost::prior(start)->getMaxDepthAfter();
+       }
 
        while (true) {
-               // NOTE: you can't change the depth of a bibliography entry
-               if (cursor.par()->layout()->labeltype != LABEL_BIBLIO) {
-                       Paragraph * prev = cursor.par()->previous();
-
-                       if (prev) {
-                               if (cursor.par()->getDepth()
-                                   < prev->getMaxDepthAfter()) {
-                                       cursor.par()->params().depth(cursor.par()->getDepth() + 1);
+               int const depth = pit->params().depth();
+               if (type == bv_funcs::INC_DEPTH) {
+                       if (depth < prev_after_depth
+                           && pit->layout()->labeltype != LABEL_BIBLIO) {
+                               changed = true;
+                               if (!test_only) {
+                                       pit->params().depth(depth + 1);
                                }
+
                        }
+               } else if (depth) {
+                       changed = true;
+                       if (!test_only)
+                               pit->params().depth(depth - 1);
                }
-               if (cursor.par() == selection.end.par())
-                       break;
-               cursor.par(cursor.par()->next());
-       }
 
-       redoParagraphs(bview, selection.start, endpar);
-
-       // we have to reset the selection, because the
-       // geometry could have changed
-       setCursor(bview, selection.start.par(), selection.start.pos());
-       selection.cursor = cursor;
-       setCursor(bview, selection.end.par(), selection.end.pos());
-       updateCounters(bview);
-       clearSelection();
-       setSelection(bview);
-       setCursor(bview, tmpcursor.par(), tmpcursor.pos());
-}
+               prev_after_depth = pit->getMaxDepthAfter();
 
+               if (pit == end) {
+                       break;
+               }
 
-// decrement depth over selection and
-// make a total rebreak of those paragraphs
-void  LyXText::decDepth(BufferView * bview)
-{
-       // if there is no selection just set the layout
-       // of the current paragraph
-       if (!selection.set()) {
-               selection.start = cursor; // dummy selection
-               selection.end = cursor;
+               ++pit;
        }
-       Paragraph * endpar = selection.end.par()->next();
-       Paragraph * undoendpar = endpar;
 
-       if (endpar && endpar->getDepth()) {
-               while (endpar && endpar->getDepth()) {
-                       endpar = endpar->next();
-                       undoendpar = endpar;
-               }
-       } else if (endpar) {
-               endpar = endpar->next(); // because of parindents etc.
-       }
+       if (test_only)
+               return changed;
 
-       setUndo(bview, Undo::EDIT,
-               selection.start.par(), undoendpar);
+       // Wow, redoParagraphs is stupid.
+       LyXCursor tmpcursor;
+       setCursor(tmpcursor, &(*start), 0);
 
-       LyXCursor tmpcursor = cursor; // store the current cursor
+       //redoParagraphs(tmpcursor, &(*pastend));
+       redoParagraphs(tmpcursor, &(*pastend));
 
-       // ok we have a selection. This is always between sel_start_cursor
-       // and sel_end cursor
-       cursor = selection.start;
+       // We need to actually move the text->cursor. I don't
+       // understand why ...
+       tmpcursor = cursor;
 
-       while (true) {
-               if (cursor.par()->params().depth()) {
-                       cursor.par()->params()
-                               .depth(cursor.par()->params().depth() - 1);
-               }
-               if (cursor.par() == selection.end.par()) {
-                       break;
-               }
-               cursor.par(cursor.par()->next());
+       // we have to reset the visual selection because the
+       // geometry could have changed
+       if (selection.set()) {
+               setCursor(selection.start.par(), selection.start.pos());
+               selection.cursor = cursor;
+               setCursor(selection.end.par(), selection.end.pos());
        }
 
-       redoParagraphs(bview, selection.start, endpar);
+       // this handles the counter labels, and also fixes up
+       // depth values for follow-on (child) paragraphs
+       updateCounters();
 
-       // we have to reset the selection, because the
-       // geometry could have changed
-       setCursor(bview, selection.start.par(),
-                 selection.start.pos());
-       selection.cursor = cursor;
-       setCursor(bview, selection.end.par(), selection.end.pos());
-       updateCounters(bview);
-       clearSelection();
-       setSelection(bview);
-       setCursor(bview, tmpcursor.par(), tmpcursor.pos());
+       setSelection();
+       setCursor(tmpcursor.par(), tmpcursor.pos());
+
+       return changed;
 }
 
 
 // set font over selection and make a total rebreak of those paragraphs
-void LyXText::setFont(BufferView * bview, LyXFont const & font, bool toggleall)
+void LyXText::setFont(LyXFont const & font, bool toggleall)
 {
        // if there is no selection just set the current_font
        if (!selection.set()) {
                // Determine basis font
                LyXFont layoutfont;
-               if (cursor.pos() < cursor.par()->beginningOfMainBody()) {
-                       layoutfont = getLabelFont(bview->buffer(),
+               if (cursor.pos() < cursor.par()->beginningOfBody()) {
+                       layoutfont = getLabelFont(bv()->buffer(),
                                                  cursor.par());
                } else {
-                       layoutfont = getLayoutFont(bview->buffer(),
+                       layoutfont = getLayoutFont(bv()->buffer(),
                                                   cursor.par());
                }
                // Update current font
                real_current_font.update(font,
-                                        bview->buffer()->params.language,
+                                        bv()->buffer()->params.language,
                                         toggleall);
 
                // Reduce to implicit settings
@@ -658,8 +565,8 @@ void LyXText::setFont(BufferView * bview, LyXFont const & font, bool toggleall)
        // ok we have a selection. This is always between sel_start_cursor
        // and sel_end cursor
 
-       setUndo(bview, Undo::EDIT,
-               selection.start.par(), selection.end.par()->next());
+       setUndo(bv(), Undo::EDIT,
+               selection.start.par(), boost::next(selection.end.par()));
        freezeUndo();
        cursor = selection.start;
        while (cursor.par() != selection.end.par() ||
@@ -667,172 +574,143 @@ void LyXText::setFont(BufferView * bview, LyXFont const & font, bool toggleall)
        {
                if (cursor.pos() < cursor.par()->size()) {
                        // an open footnote should behave like a closed one
-                       setCharFont(bview, cursor.par(), cursor.pos(),
+                       setCharFont(cursor.par(), cursor.pos(),
                                    font, toggleall);
                        cursor.pos(cursor.pos() + 1);
                } else {
                        cursor.pos(0);
-                       cursor.par(cursor.par()->next());
+                       cursor.par(boost::next(cursor.par()));
                }
        }
        unFreezeUndo();
 
-       redoParagraphs(bview, selection.start, selection.end.par()->next());
+       redoParagraphs(selection.start, boost::next(selection.end.par()));
 
        // we have to reset the selection, because the
        // geometry could have changed, but we keep
        // it for user convenience
-       setCursor(bview, selection.start.par(), selection.start.pos());
+       setCursor(selection.start.par(), selection.start.pos());
        selection.cursor = cursor;
-       setCursor(bview, selection.end.par(), selection.end.pos());
-       setSelection(bview);
-       setCursor(bview, tmpcursor.par(), tmpcursor.pos(), true,
+       setCursor(selection.end.par(), selection.end.pos());
+       setSelection();
+       setCursor(tmpcursor.par(), tmpcursor.pos(), true,
                  tmpcursor.boundary());
 }
 
 
-void LyXText::redoHeightOfParagraph(BufferView * bview, LyXCursor const & cur)
+void LyXText::redoHeightOfParagraph()
 {
-       Row * tmprow = cur.row();
-       int y = cur.y() - tmprow->baseline();
+       RowList::iterator tmprow = cursor.row();
+       int y = cursor.y() - tmprow->baseline();
 
-       setHeightOfRow(bview, tmprow);
+       setHeightOfRow(tmprow);
 
-       while (tmprow->previous()
-              && tmprow->previous()->par() == tmprow->par()) {
-               tmprow = tmprow->previous();
+       while (tmprow != rows().begin()
+              && boost::prior(tmprow)->par() == tmprow->par()) {
+               --tmprow;
                y -= tmprow->height();
-               setHeightOfRow(bview, tmprow);
+               setHeightOfRow(tmprow);
        }
 
-       // we can set the refreshing parameters now
-       status(bview, LyXText::NEED_MORE_REFRESH);
-       refresh_y = y;
-       refresh_row = tmprow;
-       setCursor(bview, cur.par(), cur.pos(), false, cursor.boundary());
+       postPaint(y);
+
+       setCursor(cursor.par(), cursor.pos(), false, cursor.boundary());
 }
 
 
-void LyXText::redoDrawingOfParagraph(BufferView * bview, LyXCursor const & cur)
+void LyXText::redoDrawingOfParagraph(LyXCursor const & cur)
 {
-       Row * tmprow = cur.row();
+       RowList::iterator tmprow = cur.row();
 
        int y = cur.y() - tmprow->baseline();
-       setHeightOfRow(bview, tmprow);
+       setHeightOfRow(tmprow);
 
-       while (tmprow->previous()
-              && tmprow->previous()->par() == tmprow->par())  {
-               tmprow = tmprow->previous();
+       while (tmprow != rows().begin()
+              && boost::prior(tmprow)->par() == tmprow->par())  {
+               --tmprow;
                y -= tmprow->height();
        }
 
-       // we can set the refreshing parameters now
-       if (status_ == LyXText::UNCHANGED || y < refresh_y) {
-               refresh_y = y;
-               refresh_row = tmprow;
-       }
-       status(bview, LyXText::NEED_MORE_REFRESH);
-       setCursor(bview, cur.par(), cur.pos());
+       postPaint(y);
+       setCursor(cur.par(), cur.pos());
 }
 
 
 // deletes and inserts again all paragaphs between the cursor
 // and the specified par
 // This function is needed after SetLayout and SetFont etc.
-void LyXText::redoParagraphs(BufferView * bview, LyXCursor const & cur,
-                            Paragraph const * endpar) const
-{
-       Row * tmprow2;
-       Paragraph * tmppar = 0;
-       Paragraph * first_phys_par = 0;
-
-       Row * tmprow = cur.row();
-
-       int y = cur.y() - tmprow->baseline();
-
-       if (!tmprow->previous()) {
-               // a trick/hack for UNDO
-               // This is needed because in an UNDO/REDO we could have changed
-               // the ownerParagrah() so the paragraph inside the row is NOT
-               // my really first par anymore. Got it Lars ;) (Jug 20011206)
-               first_phys_par = ownerParagraph();
+void LyXText::redoParagraphs(LyXCursor const & cur,
+                            ParagraphList::iterator endpit)
+{
+       RowList::iterator tmprit = cur.row();
+       int y = cur.y() - tmprit->baseline();
+
+       ParagraphList::iterator first_phys_pit;
+       RowList::iterator prevrit;
+       if (tmprit == rows().begin()) {
+               // A trick/hack for UNDO.
+               // This is needed because in an UNDO/REDO we could have
+               // changed the ownerParagrah() so the paragraph inside
+               // the row is NOT my really first par anymore.
+               // Got it Lars ;) (Jug 20011206)
+               first_phys_pit = ownerParagraphs().begin();
+               prevrit = rows().end();
        } else {
-               first_phys_par = tmprow->par();
-               while (tmprow->previous()
-                      && tmprow->previous()->par() == first_phys_par)
+               first_phys_pit = tmprit->par();
+               while (tmprit != rows().begin()
+                      && boost::prior(tmprit)->par() == first_phys_pit)
                {
-                       tmprow = tmprow->previous();
-                       y -= tmprow->height();
+                       --tmprit;
+                       y -= tmprit->height();
                }
+               prevrit = boost::prior(tmprit);
        }
 
-       // we can set the refreshing parameters now
-       status(bview, LyXText::NEED_MORE_REFRESH);
-       refresh_y = y;
-       refresh_row = tmprow->previous();        /* the real refresh row will
-                                               be deleted, so I store
-                                               the previous here */
        // remove it
-       if (tmprow->next())
-               tmppar = tmprow->next()->par();
-       else
-               tmppar = 0;
-       while (tmprow->next() && tmppar != endpar) {
-               removeRow(tmprow->next());
-               if (tmprow->next()) {
-                       tmppar = tmprow->next()->par();
-               } else {
-                       tmppar = 0;
-               }
+       while (tmprit != rows().end() && tmprit->par() != endpit) {
+               RowList::iterator tmprit2 = tmprit++;
+               removeRow(tmprit2);
        }
 
-       // remove the first one
-       tmprow2 = tmprow;     /* this is because tmprow->previous()
-                                can be 0 */
-       tmprow = tmprow->previous();
-       removeRow(tmprow2);
+       // Reinsert the paragraphs.
+       ParagraphList::iterator tmppit = first_phys_pit;
 
-       tmppar = first_phys_par;
-
-       do {
-               if (tmppar) {
-                       insertParagraph(bview, tmppar, tmprow);
-                       if (!tmprow) {
-                               tmprow = firstrow;
-                       }
-                       while (tmprow->next()
-                              && tmprow->next()->par() == tmppar) {
-                               tmprow = tmprow->next();
-                       }
-                       tmppar = tmppar->next();
+       while (tmppit != ownerParagraphs().end()) {
+               insertParagraph(tmppit, tmprit);
+               while (tmprit != rows().end()
+                      && tmprit->par() == tmppit) {
+                       ++tmprit;
                }
-       } while (tmppar && tmppar != endpar);
-
-       // this is because of layout changes
-       if (refresh_row) {
-               refresh_y -= refresh_row->height();
-               setHeightOfRow(bview, refresh_row);
+               ++tmppit;
+               if (tmppit == endpit)
+                       break;
+       }
+       if (prevrit != rows().end()) {
+               setHeightOfRow(prevrit);
+               const_cast<LyXText *>(this)->postPaint(y - prevrit->height());
        } else {
-               refresh_row = firstrow;
-               refresh_y = 0;
-               setHeightOfRow(bview, refresh_row);
+               setHeightOfRow(rows().begin());
+               const_cast<LyXText *>(this)->postPaint(0);
        }
+       if (tmprit != rows().end())
+               setHeightOfRow(tmprit);
 
-       if (tmprow && tmprow->next())
-               setHeightOfRow(bview, tmprow->next());
-       updateCounters(bview);
+       updateCounters();
 }
 
 
-void LyXText::fullRebreak(BufferView * bview)
+void LyXText::fullRebreak()
 {
-       if (!firstrow) {
-               init(bview);
+       if (rows().empty()) {
+               init(bv());
                return;
        }
-       if (need_break_row) {
-               breakAgain(bview, need_break_row);
-               need_break_row = 0;
+
+       RowList::iterator rows_end = rows().end();
+
+       if (need_break_row != rows_end) {
+               breakAgain(need_break_row);
+               need_break_row = rows_end;
                return;
        }
 }
@@ -846,7 +724,7 @@ void LyXText::fullRebreak(BufferView * bview)
 // They also delete the corresponding row
 
 // need the selection cursor:
-void LyXText::setSelection(BufferView * bview)
+void LyXText::setSelection()
 {
        bool const lsel = selection.set();
 
@@ -898,7 +776,7 @@ void LyXText::setSelection(BufferView * bview)
                selection.set(false);
 
        if (inset_owner && (selection.set() || lsel))
-               inset_owner->setUpdateStatus(bview, InsetText::SELECTION);
+               inset_owner->setUpdateStatus(bv(), InsetText::SELECTION);
 }
 
 
@@ -908,38 +786,34 @@ string const LyXText::selectionAsString(Buffer const * buffer,
        if (!selection.set()) return string();
 
        // should be const ...
-       Paragraph * startpar(selection.start.par());
-       Paragraph * endpar(selection.end.par());
+       ParagraphList::iterator startpit = selection.start.par();
+       ParagraphList::iterator endpit = selection.end.par();
        pos_type const startpos(selection.start.pos());
        pos_type const endpos(selection.end.pos());
 
-       if (startpar == endpar) {
-               return startpar->asString(buffer, startpos, endpos, label);
+       if (startpit == endpit) {
+               return startpit->asString(buffer, startpos, endpos, label);
        }
 
        string result;
 
        // First paragraph in selection
-       result += startpar->asString(buffer, startpos, startpar->size(), label) + "\n\n";
+       result += startpit->asString(buffer, startpos, startpit->size(), label) + "\n\n";
 
        // The paragraphs in between (if any)
-       LyXCursor tmpcur(selection.start);
-       tmpcur.par(tmpcur.par()->next());
-       while (tmpcur.par() != endpar) {
-               result += tmpcur.par()->asString(buffer, 0,
-                                                tmpcur.par()->size(),
-                                                label) + "\n\n";
-               tmpcur.par(tmpcur.par()->next());
+       ParagraphList::iterator pit = boost::next(startpit);
+       for (; pit != endpit; ++pit) {
+               result += pit->asString(buffer, 0, pit->size(), label) + "\n\n";
        }
 
        // Last paragraph in selection
-       result += endpar->asString(buffer, 0, endpos, label);
+       result += endpit->asString(buffer, 0, endpos, label);
 
        return result;
 }
 
 
-void LyXText::clearSelection() const
+void LyXText::clearSelection()
 {
        selection.set(false);
        selection.mark(false);
@@ -950,53 +824,58 @@ void LyXText::clearSelection() const
 }
 
 
-void LyXText::cursorHome(BufferView * bview) const
+void LyXText::cursorHome()
 {
-       setCursor(bview, cursor.par(), cursor.row()->pos());
+       setCursor(cursor.par(), cursor.row()->pos());
 }
 
 
-void LyXText::cursorEnd(BufferView * bview) const
+void LyXText::cursorEnd()
 {
-       if (!cursor.row()->next()
-           || cursor.row()->next()->par() != cursor.row()->par()) {
-               setCursor(bview, cursor.par(), cursor.row()->lastPos() + 1);
+       if (cursor.par()->empty())
+               return;
+
+       RowList::iterator rit = cursor.row();
+       RowList::iterator next_rit = boost::next(rit);
+       ParagraphList::iterator pit = rit->par();
+       pos_type last_pos = lastPos(*this, rit);
+
+       if (next_rit == rows().end() || next_rit->par() != pit) {
+               ++last_pos;
        } else {
-               if (!cursor.par()->empty() &&
-                   (cursor.par()->getChar(cursor.row()->lastPos()) == ' '
-                    || cursor.par()->isNewline(cursor.row()->lastPos()))) {
-                       setCursor(bview, cursor.par(), cursor.row()->lastPos());
-               } else {
-                       setCursor(bview,cursor.par(),
-                                 cursor.row()->lastPos() + 1);
+               if (pit->empty() ||
+                   (pit->getChar(last_pos) != ' ' && !pit->isNewline(last_pos))) {
+                       ++last_pos;
                }
        }
+
+       setCursor(pit, last_pos);
 }
 
 
-void LyXText::cursorTop(BufferView * bview) const
+void LyXText::cursorTop()
 {
-       while (cursor.par()->previous())
-               cursor.par(cursor.par()->previous());
-       setCursor(bview, cursor.par(), 0);
+       setCursor(ownerParagraphs().begin(), 0);
 }
 
 
-void LyXText::cursorBottom(BufferView * bview) const
+void LyXText::cursorBottom()
 {
-       while (cursor.par()->next())
-               cursor.par(cursor.par()->next());
-       setCursor(bview, cursor.par(), cursor.par()->size());
+#warning FIXME
+       // This is how it should be:
+       // ParagraphList::iterator lastpit = boost::prior(ownerParagraphs().end());
+       ParagraphList::iterator lastpit = &ownerParagraphs().back();
+       int pos = lastpit->size();
+       setCursor(lastpit, pos);
 }
 
 
-void LyXText::toggleFree(BufferView * bview,
-                        LyXFont const & font, bool toggleall)
+void LyXText::toggleFree(LyXFont const & font, bool toggleall)
 {
        // If the mask is completely neutral, tell user
        if (font == LyXFont(LyXFont::ALL_IGNORE)) {
                // Could only happen with user style
-               bview->owner()->message(_("No font change defined. Use Character under the Layout menu to define font change."));
+               bv()->owner()->message(_("No font change defined. Use Character under the Layout menu to define font change."));
                return;
        }
 
@@ -1006,43 +885,43 @@ void LyXText::toggleFree(BufferView * bview,
        LyXCursor resetCursor = cursor;
        bool implicitSelection = (font.language() == ignore_language
                                  && font.number() == LyXFont::IGNORE)
-               ? selectWordWhenUnderCursor(bview, WHOLE_WORD_STRICT) : false;
+               ? selectWordWhenUnderCursor(WHOLE_WORD_STRICT) : false;
 
        // Set font
-       setFont(bview, font, toggleall);
+       setFont(font, toggleall);
 
        // Implicit selections are cleared afterwards
        //and cursor is set to the original position.
        if (implicitSelection) {
                clearSelection();
                cursor = resetCursor;
-               setCursor(bview, cursor.par(), cursor.pos());
+               setCursor(cursor.par(), cursor.pos());
                selection.cursor = cursor;
        }
        if (inset_owner)
-               inset_owner->setUpdateStatus(bview, InsetText::CURSOR_PAR);
+               inset_owner->setUpdateStatus(bv(), InsetText::CURSOR_PAR);
 }
 
 
-string LyXText::getStringToIndex(BufferView * bview)
+string LyXText::getStringToIndex()
 {
        // Try implicit word selection
        // If there is a change in the language the implicit word selection
        // is disabled.
        LyXCursor const reset_cursor = cursor;
-       bool const implicitSelection = selectWordWhenUnderCursor(bview, PREVIOUS_WORD);
+       bool const implicitSelection = selectWordWhenUnderCursor(PREVIOUS_WORD);
 
        string idxstring;
        if (!selection.set())
-               bview->owner()->message(_("Nothing to index!"));
+               bv()->owner()->message(_("Nothing to index!"));
        else if (selection.start.par() != selection.end.par())
-               bview->owner()->message(_("Cannot index more than one paragraph!"));
+               bv()->owner()->message(_("Cannot index more than one paragraph!"));
        else
-               idxstring = selectionAsString(bview->buffer(), false);
+               idxstring = selectionAsString(bv()->buffer(), false);
 
        // Reset cursors to their original position.
        cursor = reset_cursor;
-       setCursor(bview, cursor.par(), cursor.pos());
+       setCursor(cursor.par(), cursor.pos());
        selection.cursor = cursor;
 
        // Clear the implicit selection.
@@ -1059,14 +938,13 @@ string LyXText::getStringToIndex(BufferView * bview)
 // they do not duplicate themself and you cannnot make dirty things with
 // them!
 
-void LyXText::setParagraph(BufferView * bview,
-                          bool line_top, bool line_bottom,
+void LyXText::setParagraph(bool line_top, bool line_bottom,
                           bool pagebreak_top, bool pagebreak_bottom,
                           VSpace const & space_top,
                           VSpace const & space_bottom,
                           Spacing const & spacing,
                           LyXAlignment align,
-                          string labelwidthstring,
+                          string const & labelwidthstring,
                           bool noindent)
 {
        LyXCursor tmpcursor = cursor;
@@ -1076,85 +954,88 @@ void LyXText::setParagraph(BufferView * bview,
        }
 
        // make sure that the depth behind the selection are restored, too
-       Paragraph * endpar = selection.end.par()->next();
-       Paragraph * undoendpar = endpar;
-
-       if (endpar && endpar->getDepth()) {
-               while (endpar && endpar->getDepth()) {
-                       endpar = endpar->next();
-                       undoendpar = endpar;
+       ParagraphList::iterator endpit = boost::next(selection.end.par());
+       ParagraphList::iterator undoendpit = endpit;
+       ParagraphList::iterator pars_end = ownerParagraphs().end();
+
+       if (endpit != pars_end && endpit->getDepth()) {
+               while (endpit != pars_end && endpit->getDepth()) {
+                       ++endpit;
+                       undoendpit = endpit;
                }
-       }
-       else if (endpar) {
+       } else if (endpit!= pars_end) {
                // because of parindents etc.
-               endpar = endpar->next();
+               ++endpit;
        }
 
-       setUndo(bview, Undo::EDIT, selection.start.par(), undoendpar);
+       setUndo(bv(), Undo::EDIT, selection.start.par(), undoendpit);
+
+
+       ParagraphList::iterator tmppit = selection.end.par();
 
+       while (tmppit != boost::prior(selection.start.par())) {
+               setCursor(tmppit, 0);
+               postPaint(cursor.y() - cursor.row()->baseline());
 
-       Paragraph * tmppar = selection.end.par();
+               ParagraphList::iterator pit = cursor.par();
+               ParagraphParameters & params = pit->params();
 
-       while (tmppar != selection.start.par()->previous()) {
-               setCursor(bview, tmppar, 0);
-               status(bview, LyXText::NEED_MORE_REFRESH);
-               refresh_row = cursor.row();
-               refresh_y = cursor.y() - cursor.row()->baseline();
-               cursor.par()->params().lineTop(line_top);
-               cursor.par()->params().lineBottom(line_bottom);
-               cursor.par()->params().pagebreakTop(pagebreak_top);
-               cursor.par()->params().pagebreakBottom(pagebreak_bottom);
-               cursor.par()->params().spaceTop(space_top);
-               cursor.par()->params().spaceBottom(space_bottom);
-               cursor.par()->params().spacing(spacing);
+               params.lineTop(line_top);
+               params.lineBottom(line_bottom);
+               params.pagebreakTop(pagebreak_top);
+               params.pagebreakBottom(pagebreak_bottom);
+               params.spaceTop(space_top);
+               params.spaceBottom(space_bottom);
+               params.spacing(spacing);
                // does the layout allow the new alignment?
-               LyXLayout_ptr const & layout = cursor.par()->layout();
+               LyXLayout_ptr const & layout = pit->layout();
 
                if (align == LYX_ALIGN_LAYOUT)
                        align = layout->align;
                if (align & layout->alignpossible) {
                        if (align == layout->align)
-                               cursor.par()->params().align(LYX_ALIGN_LAYOUT);
+                               params.align(LYX_ALIGN_LAYOUT);
                        else
-                               cursor.par()->params().align(align);
+                               params.align(align);
                }
-               cursor.par()->setLabelWidthString(labelwidthstring);
-               cursor.par()->params().noindent(noindent);
-               tmppar = cursor.par()->previous();
+               pit->setLabelWidthString(labelwidthstring);
+               params.noindent(noindent);
+               tmppit = boost::prior(pit);
        }
 
-       redoParagraphs(bview, selection.start, endpar);
+       redoParagraphs(selection.start, endpit);
 
        clearSelection();
-       setCursor(bview, selection.start.par(), selection.start.pos());
+       setCursor(selection.start.par(), selection.start.pos());
        selection.cursor = cursor;
-       setCursor(bview, selection.end.par(), selection.end.pos());
-       setSelection(bview);
-       setCursor(bview, tmpcursor.par(), tmpcursor.pos());
+       setCursor(selection.end.par(), selection.end.pos());
+       setSelection();
+       setCursor(tmpcursor.par(), tmpcursor.pos());
        if (inset_owner)
-               bview->updateInset(inset_owner, true);
+               bv()->updateInset(inset_owner);
 }
 
 
 // set the counter of a paragraph. This includes the labels
-void LyXText::setCounter(Buffer const * buf, Paragraph * par) const
+void LyXText::setCounter(Buffer const * buf, ParagraphList::iterator pit)
 {
        LyXTextClass const & textclass = buf->params.getLyXTextClass();
-       LyXLayout_ptr const & layout = par->layout();
+       LyXLayout_ptr const & layout = pit->layout();
 
-       if (par->previous()) {
+       if (pit != ownerParagraphs().begin()) {
 
-               par->params().appendix(par->previous()->params().appendix());
-               if (!par->params().appendix() && par->params().startOfAppendix()) {
-                       par->params().appendix(true);
+               pit->params().appendix(boost::prior(pit)->params().appendix());
+               if (!pit->params().appendix() &&
+                   pit->params().startOfAppendix()) {
+                       pit->params().appendix(true);
                        textclass.counters().reset();
                }
-               par->enumdepth = par->previous()->enumdepth;
-               par->itemdepth = par->previous()->itemdepth;
+               pit->enumdepth = boost::prior(pit)->enumdepth;
+               pit->itemdepth = boost::prior(pit)->itemdepth;
        } else {
-               par->params().appendix(par->params().startOfAppendix());
-               par->enumdepth = 0;
-               par->itemdepth = 0;
+               pit->params().appendix(pit->params().startOfAppendix());
+               pit->enumdepth = 0;
+               pit->itemdepth = 0;
        }
 
        /* Maybe we have to increment the enumeration depth.
@@ -1164,31 +1045,32 @@ void LyXText::setCounter(Buffer const * buf, Paragraph * par) const
         * AND, bibliographies can't have their depth changed ie. they
         *      are always of depth 0
         */
-       if (par->previous()
-           && par->previous()->getDepth() < par->getDepth()
-           && par->previous()->layout()->labeltype == LABEL_COUNTER_ENUMI
-           && par->enumdepth < 3
+       if (pit != ownerParagraphs().begin()
+           && boost::prior(pit)->getDepth() < pit->getDepth()
+           && boost::prior(pit)->layout()->labeltype == LABEL_COUNTER_ENUMI
+           && pit->enumdepth < 3
            && layout->labeltype != LABEL_BIBLIO) {
-               par->enumdepth++;
+               pit->enumdepth++;
        }
 
        // Maybe we have to decrement the enumeration depth, see note above
-       if (par->previous()
-           && par->previous()->getDepth() > par->getDepth()
+       if (pit != ownerParagraphs().begin()
+           && boost::prior(pit)->getDepth() > pit->getDepth()
            && layout->labeltype != LABEL_BIBLIO) {
-               par->enumdepth = par->depthHook(par->getDepth())->enumdepth;
+               pit->enumdepth = depthHook(pit, ownerParagraphs(),
+                                          pit->getDepth())->enumdepth;
        }
 
-       if (!par->params().labelString().empty()) {
-               par->params().labelString(string());
+       if (!pit->params().labelString().empty()) {
+               pit->params().labelString(string());
        }
 
        if (layout->margintype == MARGIN_MANUAL) {
-               if (par->params().labelWidthString().empty()) {
-                       par->setLabelWidthString(layout->labelstring());
+               if (pit->params().labelWidthString().empty()) {
+                       pit->setLabelWidthString(layout->labelstring());
                }
        } else {
-               par->setLabelWidthString(string());
+               pit->setLabelWidthString(string());
        }
 
        // is it a layout that has an automatic label?
@@ -1204,29 +1086,30 @@ void LyXText::setCounter(Buffer const * buf, Paragraph * par) const
                        textclass.counters().step(layout->latexname());
 
                        // Is there a label? Useful for Chapter layout
-                       if (!par->params().appendix()) {
-                               s << layout->labelstring();
+                       if (!pit->params().appendix()) {
+                               s << buf->B_(layout->labelstring());
                        } else {
-                               s << layout->labelstring_appendix();
+                               s << buf->B_(layout->labelstring_appendix());
                        }
 
                        // Use of an integer is here less than elegant. For now.
                        int head = textclass.maxcounter() - LABEL_COUNTER_CHAPTER;
-                       if (!par->params().appendix()) {
+                       if (!pit->params().appendix()) {
                                numbertype = "sectioning";
                        } else {
                                numbertype = "appendix";
-                               if (par->isRightToLeftPar(buf->params))
+                               if (pit->isRightToLeftPar(buf->params))
                                        langtype = "hebrew";
                                else
                                        langtype = "latin";
                        }
 
-                       s << textclass.counters()
+                       s << " "
+                         << textclass.counters()
                                .numberLabel(layout->latexname(),
                                             numbertype, langtype, head);
 
-                       par->params().labelString(STRCONV(s.str()));
+                       pit->params().labelString(STRCONV(s.str()));
 
                        // reset enum counters
                        textclass.counters().reset("enum");
@@ -1238,7 +1121,7 @@ void LyXText::setCounter(Buffer const * buf, Paragraph * par) const
                        // (Lgb)
                        string enumcounter("enum");
 
-                       switch (par->enumdepth) {
+                       switch (pit->enumdepth) {
                        case 2:
                                enumcounter += 'i';
                        case 1:
@@ -1258,37 +1141,34 @@ void LyXText::setCounter(Buffer const * buf, Paragraph * par) const
 
                        s << textclass.counters()
                                .numberLabel(enumcounter, "enumeration");
-                       par->params().labelString(STRCONV(s.str()));
+                       pit->params().labelString(STRCONV(s.str()));
                }
        } else if (layout->labeltype == LABEL_BIBLIO) {// ale970302
                textclass.counters().step("bibitem");
                int number = textclass.counters().value("bibitem");
-               //if (!par->bibkey()) {
-               //      Inset * inset = new InsetBibKey(InsetCommandParams("bibitem"));
-               //      //par->insertInset(0, inset);
-               //}
-               if (par->bibkey()) {
-                       par->bibkey()->setCounter(number);
-                       par->params().labelString(layout->labelstring());
+               if (pit->bibitem()) {
+                       pit->bibitem()->setCounter(number);
+                       pit->params().labelString(layout->labelstring());
                }
                // In biblio should't be following counters but...
        } else {
-               string s = layout->labelstring();
+               string s = buf->B_(layout->labelstring());
 
                // the caption hack:
                if (layout->labeltype == LABEL_SENSITIVE) {
-                       Paragraph * tmppar = par;
+                       ParagraphList::iterator tmppit = pit;
                        Inset * in = 0;
                        bool isOK = false;
-                       while (tmppar && tmppar->inInset()
+                       while (tmppit != ownerParagraphs().end() &&
+                              tmppit->inInset()
                               // the single '=' is intended below
-                              && (in = tmppar->inInset()->owner())) {
+                              && (in = tmppit->inInset()->owner())) {
                                if (in->lyxCode() == Inset::FLOAT_CODE ||
                                    in->lyxCode() == Inset::WRAP_CODE) {
                                        isOK = true;
                                        break;
                                } else {
-                                       tmppar = in->parOwner();
+                                       tmppit = in->parOwner();
                                }
                        }
 
@@ -1299,16 +1179,15 @@ void LyXText::setCounter(Buffer const * buf, Paragraph * par) const
                                textclass.counters().step(fl.type());
 
                                // Doesn't work... yet.
-#warning use boost.format
 #if USE_BOOST_FORMAT
-                               s = boost::io::str(boost::format(_("%1$s #:")) % fl.name());
+                               s = boost::io::str(boost::format(_("%1$s #:")) % buf->B_(fl.name()));
                                // s << boost::format(_("%1$s %1$d:")
                                //        % fl.name()
                                //        % buf->counters().value(fl.name());
 #else
                                ostringstream o;
                                //o << fl.name() << ' ' << buf->counters().value(fl.name()) << ":";
-                               o << fl.name() << " #:";
+                               o << buf->B_(fl.name()) << " #:";
                                s = STRCONV(o.str());
 #endif
                        } else {
@@ -1317,13 +1196,13 @@ void LyXText::setCounter(Buffer const * buf, Paragraph * par) const
                                s = _("Senseless: ");
                        }
                }
-               par->params().labelString(s);
+               pit->params().labelString(s);
 
                // reset the enumeration counter. They are always reset
                // when there is any other layout between
                // Just fall-through between the cases so that all
                // enum counters deeper than enumdepth is also reset.
-               switch (par->enumdepth) {
+               switch (pit->enumdepth) {
                case 0:
                        textclass.counters().reset("enumi");
                case 1:
@@ -1338,72 +1217,64 @@ void LyXText::setCounter(Buffer const * buf, Paragraph * par) const
 
 
 // Updates all counters. Paragraphs with changed label string will be rebroken
-void LyXText::updateCounters(BufferView * bview) const
+void LyXText::updateCounters()
 {
-       Row * row = firstrow;
-       Paragraph * par = row->par();
+       RowList::iterator rowit = rows().begin();
+       ParagraphList::iterator pit = rowit->par();
 
        // CHECK if this is really needed. (Lgb)
-       bview->buffer()->params.getLyXTextClass().counters().reset();
+       bv()->buffer()->params.getLyXTextClass().counters().reset();
+
+       for (; pit != ownerParagraphs().end(); ++pit) {
+               while (rowit->par() != pit)
+                       ++rowit;
+
+               string const oldLabel = pit->params().labelString();
 
-       while (par) {
-               while (row->par() != par)
-                       row = row->next();
+               int maxdepth = 0;
+               if (pit != ownerParagraphs().begin())
+                       maxdepth = boost::prior(pit)->getMaxDepthAfter();
 
-               string const oldLabel = par->params().labelString();
+               if (pit->params().depth() > maxdepth)
+                       pit->params().depth(maxdepth);
 
                // setCounter can potentially change the labelString.
-               setCounter(bview->buffer(), par);
+               setCounter(bv()->buffer(), pit);
 
-               string const & newLabel = par->params().labelString();
+               string const & newLabel = pit->params().labelString();
 
                if (oldLabel.empty() && !newLabel.empty()) {
-                       removeParagraph(row);
-                       appendParagraph(bview, row);
+                       removeParagraph(rowit);
+                       appendParagraph(rowit);
                }
-
-               par = par->next();
        }
 }
 
 
-void LyXText::insertInset(BufferView * bview, Inset * inset)
+void LyXText::insertInset(Inset * inset)
 {
        if (!cursor.par()->insetAllowed(inset->lyxCode()))
                return;
-       setUndo(bview, Undo::FINISH, cursor.par(), cursor.par()->next());
+       setUndo(bv(), Undo::FINISH, cursor.par(),
+               boost::next(cursor.par()));
        freezeUndo();
        cursor.par()->insertInset(cursor.pos(), inset);
        // Just to rebreak and refresh correctly.
        // The character will not be inserted a second time
-       insertChar(bview, Paragraph::META_INSET);
+       insertChar(Paragraph::META_INSET);
        // If we enter a highly editable inset the cursor should be to before
        // the inset. This couldn't happen before as Undo was not handled inside
        // inset now after the Undo LyX tries to call inset->Edit(...) again
        // and cannot do this as the cursor is behind the inset and GetInset
        // does not return the inset!
        if (isHighlyEditableInset(inset)) {
-               cursorLeft(bview, true);
+               cursorLeft(true);
        }
        unFreezeUndo();
 }
 
 
-void LyXText::copyEnvironmentType()
-{
-       copylayouttype = cursor.par()->layout()->name();
-}
-
-
-void LyXText::pasteEnvironmentType(BufferView * bview)
-{
-       // do nothing if there has been no previous copyEnvironmentType()
-       if (!copylayouttype.empty())
-               setLayout(bview, copylayouttype);
-}
-
-
-void LyXText::cutSelection(BufferView * bview, bool doclear, bool realcut)
+void LyXText::cutSelection(bool doclear, bool realcut)
 {
        // Stuff what we got on the clipboard. Even if there is no selection.
 
@@ -1413,7 +1284,7 @@ void LyXText::cutSelection(BufferView * bview, bool doclear, bool realcut)
        // 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)
-       bview->stuffClipboard(selectionAsString(bview->buffer(), true));
+       bv()->stuffClipboard(selectionAsString(bv()->buffer(), true));
 
        // This doesn't make sense, if there is no selection
        if (!selection.set())
@@ -1423,70 +1294,64 @@ void LyXText::cutSelection(BufferView * bview, bool doclear, bool realcut)
        // and selection.end
 
        // make sure that the depth behind the selection are restored, too
-       Paragraph * endpar = selection.end.par()->next();
-       Paragraph * undoendpar = endpar;
-
-       if (endpar && endpar->getDepth()) {
-               while (endpar && endpar->getDepth()) {
-                       endpar = endpar->next();
-                       undoendpar = endpar;
+       ParagraphList::iterator endpit = boost::next(selection.end.par());
+       ParagraphList::iterator undoendpit = endpit;
+       ParagraphList::iterator pars_end = ownerParagraphs().end();
+
+       if (endpit != pars_end && endpit->getDepth()) {
+               while (endpit != pars_end && endpit->getDepth()) {
+                       ++endpit;
+                       undoendpit = endpit;
                }
-       } else if (endpar) {
-               endpar = endpar->next(); // because of parindents etc.
-       }
-
-       setUndo(bview, Undo::DELETE,
-               selection.start.par(), undoendpar);
-
-       // there are two cases: cut only within one paragraph or
-       // more than one paragraph
-       if (selection.start.par() == selection.end.par()) {
-               // only within one paragraph
-               endpar = selection.end.par();
-               int pos = selection.end.pos();
-               CutAndPaste::cutSelection(selection.start.par(), &endpar,
-                                         selection.start.pos(), pos,
-                                         bview->buffer()->params.textclass,
-                                         doclear, realcut);
-               selection.end.pos(pos);
-       } else {
-               endpar = selection.end.par();
-               int pos = selection.end.pos();
-               CutAndPaste::cutSelection(selection.start.par(), &endpar,
-                                         selection.start.pos(), pos,
-                                         bview->buffer()->params.textclass,
-                                         doclear, realcut);
-               cursor.par(endpar);
-               selection.end.par(endpar);
-               selection.end.pos(pos);
-               cursor.pos(selection.end.pos());
-       }
-       endpar = endpar->next();
+       } else if (endpit != pars_end) {
+               // because of parindents etc.
+               ++endpit;
+       }
+
+       setUndo(bv(), Undo::DELETE, selection.start.par(), undoendpit);
 
+
+       endpit = selection.end.par();
+       int endpos = selection.end.pos();
+
+       boost::tie(endpit, endpos) = realcut ?
+               CutAndPaste::cutSelection(ownerParagraphs(),
+                                         selection.start.par(), endpit,
+                                         selection.start.pos(), endpos,
+                                         bv()->buffer()->params.textclass,
+                                         doclear)
+               : CutAndPaste::eraseSelection(ownerParagraphs(),
+                                             selection.start.par(), endpit,
+                                             selection.start.pos(), endpos,
+                                             doclear);
        // sometimes necessary
        if (doclear)
                selection.start.par()->stripLeadingSpaces();
 
-       redoParagraphs(bview, selection.start, endpar);
-
+       redoParagraphs(selection.start, boost::next(endpit));
+#warning FIXME latent bug
+       // endpit will be invalidated on redoParagraphs once ParagraphList
+       // becomes a std::list? There are maybe other places on which this
+       // can happend? (Ab)
        // cutSelection can invalidate the cursor so we need to set
        // it anew. (Lgb)
        // we prefer the end for when tracking changes
-       cursor = selection.end;
+       cursor.pos(endpos);
+       cursor.par(endpit);
 
        // need a valid cursor. (Lgb)
        clearSelection();
 
-       setCursor(bview, cursor.par(), cursor.pos());
+       setCursor(cursor.par(), cursor.pos());
        selection.cursor = cursor;
-       updateCounters(bview);
+       updateCounters();
 }
 
 
-void LyXText::copySelection(BufferView * bview)
+void LyXText::copySelection()
 {
        // stuff the selection onto the X clipboard, from an explicit copy request
-       bview->stuffClipboard(selectionAsString(bview->buffer(), true));
+       bv()->stuffClipboard(selectionAsString(bv()->buffer(), true));
 
        // this doesnt make sense, if there is no selection
        if (!selection.set())
@@ -1502,57 +1367,58 @@ void LyXText::copySelection(BufferView * bview)
                   || selection.start.pos() < selection.end.pos()))
                selection.start.pos(selection.start.pos() + 1);
 
-       CutAndPaste::copySelection(selection.start.par(), selection.end.par(),
+       CutAndPaste::copySelection(selection.start.par(),
+                                  selection.end.par(),
                                   selection.start.pos(), selection.end.pos(),
-                                  bview->buffer()->params.textclass);
+                                  bv()->buffer()->params.textclass);
 }
 
 
-void LyXText::pasteSelection(BufferView * bview)
+void LyXText::pasteSelection()
 {
        // this does not make sense, if there is nothing to paste
-       if (!CutAndPaste::checkPastePossible(cursor.par()))
+       if (!CutAndPaste::checkPastePossible())
                return;
 
-       setUndo(bview, Undo::INSERT,
-               cursor.par(), cursor.par()->next());
-
-       Paragraph * endpar;
-       Paragraph * actpar = cursor.par();
-       int pos = cursor.pos();
+       setUndo(bv(), Undo::INSERT,
+               cursor.par(), boost::next(cursor.par()));
 
-       CutAndPaste::pasteSelection(&actpar, &endpar, pos,
-                                   bview->buffer()->params.textclass);
+       ParagraphList::iterator endpit;
+       PitPosPair ppp;
 
-       redoParagraphs(bview, cursor, endpar);
+       boost::tie(ppp, endpit) = 
+               CutAndPaste::pasteSelection(ownerParagraphs(), 
+                                           cursor.par(), cursor.pos(), 
+                                           bv()->buffer()->params.textclass);
+       
+       redoParagraphs(cursor, endpit);
 
-       setCursor(bview, cursor.par(), cursor.pos());
+       setCursor(cursor.par(), cursor.pos());
        clearSelection();
 
        selection.cursor = cursor;
-       setCursor(bview, actpar, pos);
-       setSelection(bview);
-       updateCounters(bview);
+       setCursor(ppp.first, ppp.second);
+       setSelection();
+       updateCounters();
 }
 
 
-void LyXText::setSelectionRange(BufferView * bview, lyx::pos_type length)
+void LyXText::setSelectionRange(lyx::pos_type length)
 {
        if (!length)
                return;
 
        selection.cursor = cursor;
        while (length--)
-               cursorRight(bview);
-       setSelection(bview);
+               cursorRight(bv());
+       setSelection();
 }
 
 
 // simple replacing. The font of the first selected character is used
-void LyXText::replaceSelectionWithString(BufferView * bview,
-                                        string const & str)
+void LyXText::replaceSelectionWithString(string const & str)
 {
-       setCursorParUndo(bview);
+       setCursorParUndo(bv());
        freezeUndo();
 
        if (!selection.set()) { // create a dummy selection
@@ -1563,7 +1429,7 @@ void LyXText::replaceSelectionWithString(BufferView * bview,
        // Get font setting before we cut
        pos_type pos = selection.end.pos();
        LyXFont const font = selection.start.par()
-               ->getFontSettings(bview->buffer()->params,
+               ->getFontSettings(bv()->buffer()->params,
                                  selection.start.pos());
 
        // Insert the new string
@@ -1573,37 +1439,37 @@ void LyXText::replaceSelectionWithString(BufferView * bview,
        }
 
        // Cut the selection
-       cutSelection(bview, true, false);
+       cutSelection(true, false);
 
        unFreezeUndo();
 }
 
 
 // needed to insert the selection
-void LyXText::insertStringAsLines(BufferView * bview, string const & str)
+void LyXText::insertStringAsLines(string const & str)
 {
-       Paragraph * par = cursor.par();
+       ParagraphList::iterator pit = cursor.par();
        pos_type pos = cursor.pos();
-       Paragraph * endpar = cursor.par()->next();
+       ParagraphList::iterator endpit = boost::next(cursor.par());
 
-       setCursorParUndo(bview);
+       setCursorParUndo(bv());
 
        // only to be sure, should not be neccessary
        clearSelection();
 
-       bview->buffer()->insertStringAsLines(par, pos, current_font, str);
+       bv()->buffer()->insertStringAsLines(pit, pos, current_font, str);
 
-       redoParagraphs(bview, cursor, endpar);
-       setCursor(bview, cursor.par(), cursor.pos());
+       redoParagraphs(cursor, endpit);
+       setCursor(cursor.par(), cursor.pos());
        selection.cursor = cursor;
-       setCursor(bview, par, pos);
-       setSelection(bview);
+       setCursor(pit, pos);
+       setSelection();
 }
 
 
 // turns double-CR to single CR, others where converted into one
 // blank. Then InsertStringAsLines is called
-void LyXText::insertStringAsParagraphs(BufferView * bview, string const & str)
+void LyXText::insertStringAsParagraphs(string const & str)
 {
        string linestr(str);
        bool newline_inserted = false;
@@ -1623,34 +1489,33 @@ void LyXText::insertStringAsParagraphs(BufferView * bview, string const & str)
                        newline_inserted = false;
                }
        }
-       insertStringAsLines(bview, linestr);
+       insertStringAsLines(linestr);
 }
 
 
-void LyXText::checkParagraph(BufferView * bview, Paragraph * par,
-                            pos_type pos)
+void LyXText::checkParagraph(ParagraphList::iterator pit, pos_type pos)
 {
        LyXCursor tmpcursor;
 
        int y = 0;
        pos_type z;
-       Row * row = getRow(par, pos, y);
+       RowList::iterator row = getRow(pit, pos, y);
+       RowList::iterator beg = rows().begin();
 
        // is there a break one row above
-       if (row->previous() && row->previous()->par() == row->par()) {
-               z = nextBreakPoint(bview, row->previous(), workWidth(bview));
+       if (row != beg
+           && boost::prior(row)->par() == row->par()) {
+               z = rowBreakPoint(*boost::prior(row));
                if (z >= row->pos()) {
                        // set the dimensions of the row above
-                       y -= row->previous()->height();
-                       refresh_y = y;
-                       refresh_row = row->previous();
-                       status(bview, LyXText::NEED_MORE_REFRESH);
+                       y -= boost::prior(row)->height();
+                       postPaint(y);
 
-                       breakAgain(bview, row->previous());
+                       breakAgain(boost::prior(row));
 
                        // set the cursor again. Otherwise
                        // dangling pointers are possible
-                       setCursor(bview, cursor.par(), cursor.pos(),
+                       setCursor(cursor.par(), cursor.pos(),
                                  false, cursor.boundary());
                        selection.cursor = cursor;
                        return;
@@ -1658,25 +1523,24 @@ void LyXText::checkParagraph(BufferView * bview, Paragraph * par,
        }
 
        int const tmpheight = row->height();
-       pos_type const tmplast = row->lastPos();
-       refresh_y = y;
-       refresh_row = row;
+       pos_type const tmplast = lastPos(*this, row);
 
-       breakAgain(bview, row);
-       if (row->height() == tmpheight && row->lastPos() == tmplast)
-               status(bview, LyXText::NEED_VERY_LITTLE_REFRESH);
-       else
-               status(bview, LyXText::NEED_MORE_REFRESH);
+       breakAgain(row);
+       if (row->height() == tmpheight && lastPos(*this, row) == tmplast) {
+               postRowPaint(row, y);
+       } else {
+               postPaint(y);
+       }
 
        // check the special right address boxes
-       if (par->layout()->margintype == MARGIN_RIGHT_ADDRESS_BOX) {
-               tmpcursor.par(par);
+       if (pit->layout()->margintype == MARGIN_RIGHT_ADDRESS_BOX) {
+               tmpcursor.par(pit);
                tmpcursor.row(row);
                tmpcursor.y(y);
                tmpcursor.x(0);
                tmpcursor.x_fix(0);
                tmpcursor.pos(pos);
-               redoDrawingOfParagraph(bview, tmpcursor);
+               redoDrawingOfParagraph(tmpcursor);
        }
 
        // set the cursor again. Otherwise dangling pointers are possible
@@ -1684,91 +1548,94 @@ void LyXText::checkParagraph(BufferView * bview, Paragraph * par,
 
        if (selection.set()) {
                tmpcursor = cursor;
-               setCursorIntern(bview, selection.cursor.par(), selection.cursor.pos(),
+               setCursorIntern(selection.cursor.par(), selection.cursor.pos(),
                                false, selection.cursor.boundary());
                selection.cursor = cursor;
-               setCursorIntern(bview, selection.start.par(),
+               setCursorIntern(selection.start.par(),
                                selection.start.pos(),
                                false, selection.start.boundary());
                selection.start = cursor;
-               setCursorIntern(bview, selection.end.par(),
+               setCursorIntern(selection.end.par(),
                                selection.end.pos(),
                                false, selection.end.boundary());
                selection.end = cursor;
-               setCursorIntern(bview, last_sel_cursor.par(),
+               setCursorIntern(last_sel_cursor.par(),
                                last_sel_cursor.pos(),
                                false, last_sel_cursor.boundary());
                last_sel_cursor = cursor;
                cursor = tmpcursor;
        }
-       setCursorIntern(bview, cursor.par(), cursor.pos(),
+       setCursorIntern(cursor.par(), cursor.pos(),
                        false, cursor.boundary());
 }
 
 
 // returns false if inset wasn't found
-bool LyXText::updateInset(BufferView * bview, Inset * inset)
+bool LyXText::updateInset(Inset * inset)
 {
        // first check the current paragraph
        int pos = cursor.par()->getPositionOfInset(inset);
        if (pos != -1) {
-               checkParagraph(bview, cursor.par(), pos);
+               checkParagraph(cursor.par(), pos);
                return true;
        }
 
        // check every paragraph
 
-       Paragraph * par = ownerParagraph();
-       do {
+       ParagraphList::iterator par = ownerParagraphs().begin();
+       ParagraphList::iterator end = ownerParagraphs().end();
+       for (; par != end; ++par) {
                pos = par->getPositionOfInset(inset);
                if (pos != -1) {
-                       checkParagraph(bview, par, pos);
+                       checkParagraph(par, pos);
                        return true;
                }
-               par = par->next();
-       } while (par);
+       };
 
        return false;
 }
 
 
-bool LyXText::setCursor(BufferView * bview, Paragraph * par,
+bool LyXText::setCursor(ParagraphList::iterator pit,
                        pos_type pos,
-                       bool setfont, bool boundary) const
+                       bool setfont, bool boundary)
 {
        LyXCursor old_cursor = cursor;
-       setCursorIntern(bview, par, pos, setfont, boundary);
-       return deleteEmptyParagraphMechanism(bview, old_cursor);
+       setCursorIntern(pit, pos, setfont, boundary);
+       return deleteEmptyParagraphMechanism(old_cursor);
 }
 
 
-void LyXText::setCursor(BufferView * bview, LyXCursor & cur, Paragraph * par,
-                       pos_type pos, bool boundary) const
+void LyXText::setCursor(LyXCursor & cur, ParagraphList::iterator pit,
+                       pos_type pos, bool boundary)
 {
-       lyx::Assert(par);
-       lyx::Assert(bview);
+       lyx::Assert(pit != ownerParagraphs().end());
 
-       cur.par(par);
+       cur.par(pit);
        cur.pos(pos);
        cur.boundary(boundary);
 
        // get the cursor y position in text
        int y = 0;
-       Row * row = getRow(par, pos, y);
-       Row * old_row = row;
+       RowList::iterator row = getRow(pit, pos, y);
+       RowList::iterator beg = rows().begin();
+
+       RowList::iterator old_row = row;
        cur.irow(row);
        // if we are before the first char of this row and are still in the
        // same paragraph and there is a previous row then put the cursor on
        // the end of the previous row
        cur.iy(y + row->baseline());
-       Inset * ins;
-       if (row->previous() && pos &&
-               row->previous()->par() == row->par() &&
-               par->getChar(pos) == Paragraph::META_INSET &&
-               (ins=par->getInset(pos)) && (ins->needFullRow() || ins->display()))
-       {
-               row = row->previous();
-               y -= row->height();
+       if (row != beg &&
+           pos &&
+           boost::prior(row)->par() == row->par() &&
+           pos < pit->size() &&
+           pit->getChar(pos) == Paragraph::META_INSET) {
+               Inset * ins = pit->getInset(pos);
+               if (ins && (ins->needFullRow() || ins->display())) {
+                       --row;
+                       y -= row->height();
+               }
        }
 
        cur.row(row);
@@ -1777,35 +1644,49 @@ void LyXText::setCursor(BufferView * bview, LyXCursor & cur, Paragraph * par,
        // y is now the cursor baseline
        cur.y(y);
 
-       pos_type last = old_row->lastPrintablePos();
+       pos_type last = lastPrintablePos(*this, old_row);
 
        // None of these should happen, but we're scaredy-cats
-       if (pos > par->size()) {
+       if (pos > pit->size()) {
+               lyxerr << "dont like 1 please report" << endl;
                pos = 0;
                cur.pos(0);
        } else if (pos > last + 1) {
+               lyxerr << "dont like 2 please report" << endl;
                // This shouldn't happen.
                pos = last + 1;
                cur.pos(pos);
        } else if (pos < row->pos()) {
+               lyxerr << "dont like 3 please report" << endl;
                pos = row->pos();
                cur.pos(pos);
        }
 
        // now get the cursors x position
-       float x = getCursorX(bview, row, pos, last, boundary);
+       float x = getCursorX(row, pos, last, boundary);
        cur.x(int(x));
        cur.x_fix(cur.x());
        if (old_row != row) {
-               x = getCursorX(bview, old_row, pos, last, boundary);
+               x = getCursorX(old_row, pos, last, boundary);
                cur.ix(int(x));
        } else
                cur.ix(cur.x());
+/* We take out this for the time being because 1) the redraw code is not
+   prepared to this yet and 2) because some good policy has yet to be decided
+   while editting: for instance how to act on rows being created/deleted
+   because of DEPM.
+*/
+#if 0
+       //if the cursor is in a visible row, anchor to it
+       int topy = top_y();
+       if (topy < y && y < topy + bv()->workHeight())
+               anchor_row(row);
+#endif
 }
 
 
-float LyXText::getCursorX(BufferView * bview, Row * row,
-                                                 pos_type pos, pos_type last, bool boundary) const
+float LyXText::getCursorX(RowList::iterator rit,
+                         pos_type pos, pos_type last, bool boundary) const
 {
        pos_type cursor_vpos = 0;
        float x;
@@ -1813,16 +1694,18 @@ float LyXText::getCursorX(BufferView * bview, Row * row,
        float fill_hfill;
        float fill_label_hfill;
        // This call HAS to be here because of the BidiTables!!!
-       prepareToPrint(bview, row, x, fill_separator, fill_hfill,
+       prepareToPrint(rit, x, fill_separator, fill_hfill,
                       fill_label_hfill);
 
-       if (last < row->pos())
-               cursor_vpos = row->pos();
+       ParagraphList::iterator rit_par = rit->par();
+       pos_type const rit_pos = rit->pos();
+
+       if (last < rit_pos)
+               cursor_vpos = rit_pos;
        else if (pos > last && !boundary)
-               cursor_vpos = (row->par()->isRightToLeftPar(bview->buffer()->params))
-                       ? row->pos() : last + 1;
-       else if (pos > row->pos() &&
-                (pos > last || boundary))
+               cursor_vpos = (rit_par->isRightToLeftPar(bv()->buffer()->params))
+                       ? rit_pos : last + 1;
+       else if (pos > rit_pos && (pos > last || boundary))
                /// Place cursor after char at (logical) position pos - 1
                cursor_vpos = (bidi_level(pos - 1) % 2 == 0)
                        ? log2vis(pos - 1) + 1 : log2vis(pos - 1);
@@ -1831,45 +1714,43 @@ float LyXText::getCursorX(BufferView * bview, Row * row,
                cursor_vpos = (bidi_level(pos) % 2 == 0)
                        ? log2vis(pos) : log2vis(pos) + 1;
 
-       pos_type main_body = row->par()->beginningOfMainBody();
-       if ((main_body > 0) &&
-           ((main_body-1 > last) ||
-            !row->par()->isLineSeparator(main_body-1)))
-               main_body = 0;
+       pos_type body_pos = rit_par->beginningOfBody();
+       if ((body_pos > 0) &&
+           ((body_pos - 1 > last) || !rit_par->isLineSeparator(body_pos - 1)))
+               body_pos = 0;
 
-       for (pos_type vpos = row->pos(); vpos < cursor_vpos; ++vpos) {
+       for (pos_type vpos = rit_pos; vpos < cursor_vpos; ++vpos) {
                pos_type pos = vis2log(vpos);
-               if (main_body > 0 && pos == main_body - 1) {
+               if (body_pos > 0 && pos == body_pos - 1) {
                        x += fill_label_hfill +
                                font_metrics::width(
-                                       row->par()->layout()->labelsep,
-                                       getLabelFont(bview->buffer(),
-                                                    row->par()));
-                       if (row->par()->isLineSeparator(main_body - 1))
-                               x -= singleWidth(bview,
-                                                row->par(), main_body - 1);
+                                       rit_par->layout()->labelsep,
+                                       getLabelFont(bv()->buffer(), rit_par));
+                       if (rit_par->isLineSeparator(body_pos - 1))
+                               x -= singleWidth(rit_par, body_pos - 1);
                }
-               if (row->hfillExpansion(pos)) {
-                       x += singleWidth(bview, row->par(), pos);
-                       if (pos >= main_body)
+
+               if (hfillExpansion(*this, rit, pos)) {
+                       x += singleWidth(rit_par, pos);
+                       if (pos >= body_pos)
                                x += fill_hfill;
                        else
                                x += fill_label_hfill;
-               } else if (row->par()->isSeparator(pos)) {
-                       x += singleWidth(bview, row->par(), pos);
-                       if (pos >= main_body)
+               } else if (rit_par->isSeparator(pos)) {
+                       x += singleWidth(rit_par, pos);
+                       if (pos >= body_pos)
                                x += fill_separator;
                } else
-                       x += singleWidth(bview, row->par(), pos);
+                       x += singleWidth(rit_par, pos);
        }
        return x;
 }
 
 
-void LyXText::setCursorIntern(BufferView * bview, Paragraph * par,
-                             pos_type pos, bool setfont, bool boundary) const
+void LyXText::setCursorIntern(ParagraphList::iterator pit,
+                             pos_type pos, bool setfont, bool boundary)
 {
-       InsetText * it = static_cast<InsetText *>(par->inInset());
+       InsetText * it = static_cast<InsetText *>(pit->inInset());
        if (it) {
                if (it != inset_owner) {
                        lyxerr[Debug::INSETS] << "InsetText   is " << it
@@ -1890,46 +1771,48 @@ void LyXText::setCursorIntern(BufferView * bview, Paragraph * par,
                        // I moved the lyxerr stuff in here so we can see if
                        // this is actually really needed and where!
                        // (Jug)
-                       // it->getLyXText(bview)->setCursorIntern(bview, par, pos, setfont, boundary);
+                       // it->getLyXText(bv())->setCursorIntern(bv(), par, pos, setfont, boundary);
                        return;
                }
        }
 
-       setCursor(bview, cursor, par, pos, boundary);
+       setCursor(cursor, pit, pos, boundary);
        if (setfont)
-               setCurrentFont(bview);
+               setCurrentFont();
 }
 
 
-void LyXText::setCurrentFont(BufferView * bview) const
+void LyXText::setCurrentFont()
 {
        pos_type pos = cursor.pos();
+       ParagraphList::iterator pit = cursor.par();
+
        if (cursor.boundary() && pos > 0)
                --pos;
 
        if (pos > 0) {
-               if (pos == cursor.par()->size())
+               if (pos == pit->size())
                        --pos;
                else // potentional bug... BUG (Lgb)
-                       if (cursor.par()->isSeparator(pos)) {
+                       if (pit->isSeparator(pos)) {
                                if (pos > cursor.row()->pos() &&
                                    bidi_level(pos) % 2 ==
                                    bidi_level(pos - 1) % 2)
                                        --pos;
-                               else if (pos + 1 < cursor.par()->size())
+                               else if (pos + 1 < pit->size())
                                        ++pos;
                        }
        }
 
        current_font =
-               cursor.par()->getFontSettings(bview->buffer()->params, pos);
-       real_current_font = getFont(bview->buffer(), cursor.par(), pos);
+               pit->getFontSettings(bv()->buffer()->params, pos);
+       real_current_font = getFont(bv()->buffer(), pit, pos);
 
-       if (cursor.pos() == cursor.par()->size() &&
-           isBoundary(bview->buffer(), cursor.par(), cursor.pos()) &&
+       if (cursor.pos() == pit->size() &&
+           isBoundary(bv()->buffer(), *pit, cursor.pos()) &&
            !cursor.boundary()) {
                Language const * lang =
-                       cursor.par()->getParLanguage(bview->buffer()->params);
+                       pit->getParLanguage(bv()->buffer()->params);
                current_font.setLanguage(lang);
                current_font.setNumber(LyXFont::OFF);
                real_current_font.setLanguage(lang);
@@ -1938,13 +1821,127 @@ void LyXText::setCurrentFont(BufferView * bview) const
 }
 
 
-void LyXText::setCursorFromCoordinates(BufferView * bview, int x, int y) const
+// 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(RowList::iterator rit, int & x, bool & boundary) const
+{
+       float tmpx = 0.0;
+       float fill_separator;
+       float fill_hfill;
+       float fill_label_hfill;
+
+       prepareToPrint(rit, tmpx, fill_separator,
+                      fill_hfill, fill_label_hfill);
+
+       pos_type vc = rit->pos();
+       pos_type last = lastPrintablePos(*this, rit);
+       pos_type c = 0;
+
+       ParagraphList::iterator rit_par = rit->par();
+       LyXLayout_ptr const & layout = rit->par()->layout();
+
+       bool left_side = false;
+
+       pos_type body_pos = rit_par->beginningOfBody();
+       float last_tmpx = tmpx;
+
+       if (body_pos > 0 &&
+           (body_pos - 1 > last ||
+            !rit_par->isLineSeparator(body_pos - 1)))
+               body_pos = 0;
+
+       // check for empty row
+       if (!rit_par->size()) {
+               x = int(tmpx);
+               return 0;
+       }
+
+       while (vc <= last && tmpx <= x) {
+               c = vis2log(vc);
+               last_tmpx = tmpx;
+               if (body_pos > 0 && c == body_pos - 1) {
+                       tmpx += fill_label_hfill +
+                               font_metrics::width(layout->labelsep,
+                                              getLabelFont(bv()->buffer(), &*rit_par));
+                       if (rit_par->isLineSeparator(body_pos - 1))
+                               tmpx -= singleWidth(rit_par, body_pos - 1);
+               }
+
+               if (hfillExpansion(*this, rit, c)) {
+                       tmpx += singleWidth(rit_par, c);
+                       if (c >= body_pos)
+                               tmpx += fill_hfill;
+                       else
+                               tmpx += fill_label_hfill;
+               } else if (rit_par->isSeparator(c)) {
+                       tmpx += singleWidth(rit_par, c);
+                       if (c >= body_pos)
+                               tmpx+= fill_separator;
+               } else {
+                       tmpx += singleWidth(rit_par, c);
+               }
+               ++vc;
+       }
+
+       if ((tmpx + last_tmpx) / 2 > x) {
+               tmpx = last_tmpx;
+               left_side = true;
+       }
+
+       if (vc > last + 1)  // This shouldn't happen.
+               vc = last + 1;
+
+       boundary = false;
+       // This (rtl_support test) is not needed, but gives
+       // some speedup if rtl_support=false
+       bool const lastrow = lyxrc.rtl_support &&
+               (boost::next(rit) == rowlist_.end() ||
+                boost::next(rit)->par() != rit_par);
+       // If lastrow is false, we don't need to compute
+       // the value of rtl.
+       bool const rtl = (lastrow)
+               ? rit_par->isRightToLeftPar(bv()->buffer()->params)
+               : false;
+       if (lastrow &&
+                ((rtl &&  left_side && vc == rit->pos() && x < tmpx - 5) ||
+                  (!rtl && !left_side && vc == last + 1   && x > tmpx + 5)))
+               c = last + 1;
+       else if (vc == rit->pos()) {
+               c = vis2log(vc);
+               if (bidi_level(c) % 2 == 1)
+                       ++c;
+       } else {
+               c = vis2log(vc - 1);
+               bool const rtl = (bidi_level(c) % 2 == 1);
+               if (left_side == rtl) {
+                       ++c;
+                       boundary = isBoundary(bv()->buffer(), *rit_par, c);
+               }
+       }
+
+       if (rit->pos() <= last && c > last
+           && rit_par->isNewline(last)) {
+               if (bidi_level(last) % 2 == 0)
+                       tmpx -= singleWidth(rit_par, last);
+               else
+                       tmpx += singleWidth(rit_par, last);
+               c = last;
+       }
+
+       c -= rit->pos();
+       x = int(tmpx);
+       return c;
+}
+
+
+void LyXText::setCursorFromCoordinates(int x, int y)
 {
        LyXCursor old_cursor = cursor;
 
-       setCursorFromCoordinates(bview, cursor, x, y);
-       setCurrentFont(bview);
-       deleteEmptyParagraphMechanism(bview, old_cursor);
+       setCursorFromCoordinates(cursor, x, y);
+       setCurrentFont();
+       deleteEmptyParagraphMechanism(old_cursor);
 }
 
 
@@ -1955,14 +1952,17 @@ namespace {
         * and the next row is filled by an inset that spans an entire
         * row.
         */
-       bool beforeFullRowInset(Row & row, LyXCursor & cur) {
-               if (!row.next())
+       bool beforeFullRowInset(LyXText & lt, LyXCursor const & cur) {
+               RowList::iterator row = cur.row();
+               if (boost::next(row) == lt.rows().end())
                        return false;
-               Row const & next = *row.next();
+               Row const & next = *boost::next(row);
 
                if (next.pos() != cur.pos() || next.par() != cur.par())
                        return false;
-               if (!cur.par()->isInset(cur.pos()))
+
+               if (cur.pos() == cur.par()->size()
+                   || !cur.par()->isInset(cur.pos()))
                        return false;
                Inset const * inset = cur.par()->getInset(cur.pos());
                if (inset->needFullRow() || inset->display())
@@ -1972,26 +1972,25 @@ namespace {
 }
 
 
-void LyXText::setCursorFromCoordinates(BufferView * bview, LyXCursor & cur,
-                                      int x, int y) const
+void LyXText::setCursorFromCoordinates(LyXCursor & cur, int x, int y)
 {
        // Get the row first.
 
-       Row * row = getRowNearY(y);
+       RowList::iterator row = getRowNearY(y);
        bool bound = false;
-       pos_type const column = getColumnNearX(bview, row, x, bound);
+       pos_type const column = getColumnNearX(row, x, bound);
        cur.par(row->par());
        cur.pos(row->pos() + column);
        cur.x(x);
        cur.y(y + row->baseline());
        cur.row(row);
 
-       if (beforeFullRowInset(*row, cur)) {
-               pos_type last = row->lastPrintablePos();
-               float x = getCursorX(bview, row->next(), cur.pos(), last, bound);
+       if (beforeFullRowInset(*this, cur)) {
+               pos_type last = lastPrintablePos(*this, row);
+               float x = getCursorX(boost::next(row), cur.pos(), last, bound);
                cur.ix(int(x));
-               cur.iy(y + row->height() + row->next()->baseline());
-               cur.irow(row->next());
+               cur.iy(y + row->height() + boost::next(row)->baseline());
+               cur.irow(boost::next(row));
        } else {
                cur.iy(cur.y());
                cur.ix(cur.x());
@@ -2001,107 +2000,111 @@ void LyXText::setCursorFromCoordinates(BufferView * bview, LyXCursor & cur,
 }
 
 
-void LyXText::cursorLeft(BufferView * bview, bool internal) const
+void LyXText::cursorLeft(bool internal)
 {
        if (cursor.pos() > 0) {
                bool boundary = cursor.boundary();
-               setCursor(bview, cursor.par(), cursor.pos() - 1, true, false);
+               setCursor(cursor.par(), cursor.pos() - 1, true, false);
                if (!internal && !boundary &&
-                   isBoundary(bview->buffer(), cursor.par(), cursor.pos() + 1))
-                       setCursor(bview, cursor.par(), cursor.pos() + 1, true, true);
-       } else if (cursor.par()->previous()) { // steps into the above paragraph.
-               Paragraph * par = cursor.par()->previous();
-               setCursor(bview, par, par->size());
+                   isBoundary(bv()->buffer(), *cursor.par(), cursor.pos() + 1))
+                       setCursor(cursor.par(), cursor.pos() + 1, true, true);
+       } else if (cursor.par() != ownerParagraphs().begin()) { // steps into the above paragraph.
+               ParagraphList::iterator pit = boost::prior(cursor.par());
+               setCursor(pit, pit->size());
        }
 }
 
 
-void LyXText::cursorRight(BufferView * bview, bool internal) const
+void LyXText::cursorRight(bool internal)
 {
-       if (!internal && cursor.boundary() &&
-           !cursor.par()->isNewline(cursor.pos()))
-               setCursor(bview, cursor.par(), cursor.pos(), true, false);
-       else if (cursor.pos() < cursor.par()->size()) {
-               setCursor(bview, cursor.par(), cursor.pos() + 1, true, false);
+       bool const at_end = (cursor.pos() == cursor.par()->size());
+       bool const at_newline = !at_end &&
+               cursor.par()->isNewline(cursor.pos());
+
+       if (!internal && cursor.boundary() && !at_newline)
+               setCursor(cursor.par(), cursor.pos(), true, false);
+       else if (!at_end) {
+               setCursor(cursor.par(), cursor.pos() + 1, true, false);
                if (!internal &&
-                   isBoundary(bview->buffer(), cursor.par(), cursor.pos()))
-                       setCursor(bview, cursor.par(), cursor.pos(), true, true);
-       } else if (cursor.par()->next())
-               setCursor(bview, cursor.par()->next(), 0);
+                   isBoundary(bv()->buffer(), *cursor.par(), cursor.pos()))
+                       setCursor(cursor.par(), cursor.pos(), true, true);
+       } else if (boost::next(cursor.par()) != ownerParagraphs().end())
+               setCursor(boost::next(cursor.par()), 0);
 }
 
 
-void LyXText::cursorUp(BufferView * bview, bool selecting) const
+void LyXText::cursorUp(bool selecting)
 {
 #if 1
        int x = cursor.x_fix();
        int y = cursor.y() - cursor.row()->baseline() - 1;
-       setCursorFromCoordinates(bview, x, y);
+       setCursorFromCoordinates(x, y);
        if (!selecting) {
-               int y1 = cursor.iy() - first_y;
+               int topy = top_y();
+               int y1 = cursor.iy() - topy;
                int y2 = y1;
-               y -= first_y;
-               Inset * inset_hit = checkInsetHit(bview, x, y1);
+               y -= topy;
+               Inset * inset_hit = checkInsetHit(x, y1);
                if (inset_hit && isHighlyEditableInset(inset_hit)) {
-                       inset_hit->edit(bview, x, y - (y2 - y1), mouse_button::none);
+                       inset_hit->edit(bv(), x, y - (y2 - y1), mouse_button::none);
                }
        }
 #else
-       setCursorFromCoordinates(bview, cursor.x_fix(),
+       setCursorFromCoordinates(bv(), cursor.x_fix(),
                                 cursor.y() - cursor.row()->baseline() - 1);
 #endif
 }
 
 
-void LyXText::cursorDown(BufferView * bview, bool selecting) const
+void LyXText::cursorDown(bool selecting)
 {
 #if 1
        int x = cursor.x_fix();
        int y = cursor.y() - cursor.row()->baseline() +
                cursor.row()->height() + 1;
-       setCursorFromCoordinates(bview, x, y);
+       setCursorFromCoordinates(x, y);
        if (!selecting && cursor.row() == cursor.irow()) {
-               int y1 = cursor.iy() - first_y;
+               int topy = top_y();
+               int y1 = cursor.iy() - topy;
                int y2 = y1;
-               y -= first_y;
-               Inset * inset_hit = checkInsetHit(bview, x, y1);
+               y -= topy;
+               Inset * inset_hit = checkInsetHit(x, y1);
                if (inset_hit && isHighlyEditableInset(inset_hit)) {
-                       inset_hit->edit(bview, x, y - (y2 - y1), mouse_button::none);
+                       inset_hit->edit(bv(), x, y - (y2 - y1), mouse_button::none);
                }
        }
 #else
-       setCursorFromCoordinates(bview, cursor.x_fix(),
+       setCursorFromCoordinates(bv(), cursor.x_fix(),
                                 cursor.y() - cursor.row()->baseline()
                                 + cursor.row()->height() + 1);
 #endif
 }
 
 
-void LyXText::cursorUpParagraph(BufferView * bview) const
+void LyXText::cursorUpParagraph()
 {
        if (cursor.pos() > 0) {
-               setCursor(bview, cursor.par(), 0);
+               setCursor(cursor.par(), 0);
        }
-       else if (cursor.par()->previous()) {
-               setCursor(bview, cursor.par()->previous(), 0);
+       else if (cursor.par() != ownerParagraphs().begin()) {
+               setCursor(boost::prior(cursor.par()), 0);
        }
 }
 
 
-void LyXText::cursorDownParagraph(BufferView * bview) const
+void LyXText::cursorDownParagraph()
 {
-       if (cursor.par()->next()) {
-               setCursor(bview, cursor.par()->next(), 0);
+       if (boost::next(cursor.par()) != ownerParagraphs().end()) {
+               setCursor(boost::next(cursor.par()), 0);
        } else {
-               setCursor(bview, cursor.par(), cursor.par()->size());
+               setCursor(cursor.par(), cursor.par()->size());
        }
 }
 
 // fix the cursor `cur' after a characters has been deleted at `where'
 // position. Called by deleteEmptyParagraphMechanism
-void LyXText::fixCursorAfterDelete(BufferView * bview,
-                                  LyXCursor & cur,
-                                  LyXCursor const & where) const
+void LyXText::fixCursorAfterDelete(LyXCursor & cur,
+                                  LyXCursor const & where)
 {
        // if cursor is not in the paragraph where the delete occured,
        // do nothing
@@ -2119,12 +2122,11 @@ void LyXText::fixCursorAfterDelete(BufferView * bview,
                cur.pos(cur.par()->size());
 
        // recompute row et al. for this cursor
-       setCursor(bview, cur, cur.par(), cur.pos(), cur.boundary());
+       setCursor(cur, cur.par(), cur.pos(), cur.boundary());
 }
 
 
-bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
-                                           LyXCursor const & old_cursor) const
+bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
 {
        // Would be wrong to delete anything if we have a selection.
        if (selection.set())
@@ -2169,7 +2171,7 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
                    && old_cursor.par()->isLineSeparator(old_cursor.pos())
                    && old_cursor.par()->isLineSeparator(old_cursor.pos() - 1)) {
                        old_cursor.par()->erase(old_cursor.pos() - 1);
-                       redoParagraphs(bview, old_cursor, old_cursor.par()->next());
+                       redoParagraphs(old_cursor, boost::next(old_cursor.par()));
 
 #ifdef WITH_WARNINGS
 #warning This will not work anymore when we have multiple views of the same buffer
@@ -2178,23 +2180,23 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
 // automated way in LyXCursor code. (JMarc 26/09/2001)
 #endif
                        // correct all cursors held by the LyXText
-                       fixCursorAfterDelete(bview, cursor, old_cursor);
-                       fixCursorAfterDelete(bview, selection.cursor,
+                       fixCursorAfterDelete(cursor, old_cursor);
+                       fixCursorAfterDelete(selection.cursor,
                                             old_cursor);
-                       fixCursorAfterDelete(bview, selection.start,
+                       fixCursorAfterDelete(selection.start,
                                             old_cursor);
-                       fixCursorAfterDelete(bview, selection.end, old_cursor);
-                       fixCursorAfterDelete(bview, last_sel_cursor,
+                       fixCursorAfterDelete(selection.end, old_cursor);
+                       fixCursorAfterDelete(last_sel_cursor,
                                             old_cursor);
-                       fixCursorAfterDelete(bview, toggle_cursor, old_cursor);
-                       fixCursorAfterDelete(bview, toggle_end_cursor,
+                       fixCursorAfterDelete(toggle_cursor, old_cursor);
+                       fixCursorAfterDelete(toggle_end_cursor,
                                             old_cursor);
                        return false;
                }
        }
 
        // don't delete anything if this is the ONLY paragraph!
-       if (!old_cursor.par()->next() && !old_cursor.par()->previous())
+       if (ownerParagraphs().size() == 1)
                return false;
 
        // Do not delete empty paragraphs with keepempty set.
@@ -2209,96 +2211,93 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
        // we can't possibly have deleted a paragraph before this point
        bool deleted = false;
 
-       if ((old_cursor.par()->empty()
-            || (old_cursor.par()->size() == 1
-                && old_cursor.par()->isLineSeparator(0)))) {
+       if (old_cursor.par()->empty() ||
+           (old_cursor.par()->size() == 1 &&
+            old_cursor.par()->isLineSeparator(0))) {
                // ok, we will delete anything
                LyXCursor tmpcursor;
 
-               // make sure that you do not delete any environments
-               status(bview, LyXText::NEED_MORE_REFRESH);
                deleted = true;
 
-               if (old_cursor.row()->previous()) {
-                       refresh_row = old_cursor.row()->previous();
-                       refresh_y = old_cursor.y() - old_cursor.row()->baseline() - refresh_row->height();
+               bool selection_position_was_oldcursor_position = (
+                       selection.cursor.par()  == old_cursor.par()
+                       && selection.cursor.pos() == old_cursor.pos());
+
+               if (old_cursor.row() != rows().begin()) {
+                       RowList::iterator
+                               prevrow = boost::prior(old_cursor.row());
+                       const_cast<LyXText *>(this)->postPaint(old_cursor.y() - old_cursor.row()->baseline() - prevrow->height());
                        tmpcursor = cursor;
                        cursor = old_cursor; // that undo can restore the right cursor position
-                       Paragraph * endpar = old_cursor.par()->next();
-                       if (endpar && endpar->getDepth()) {
-                               while (endpar && endpar->getDepth()) {
-                                       endpar = endpar->next();
-                               }
+                       #warning FIXME. --end() iterator is usable here
+                       ParagraphList::iterator endpit = boost::next(old_cursor.par());
+                       while (endpit != ownerParagraphs().end() &&
+                              endpit->getDepth()) {
+                               ++endpit;
                        }
-                       setUndo(bview, Undo::DELETE, old_cursor.par(), endpar);
+
+                       setUndo(bv(), Undo::DELETE, old_cursor.par(), endpit);
                        cursor = tmpcursor;
 
                        // delete old row
                        removeRow(old_cursor.row());
-                       if (ownerParagraph() == old_cursor.par()) {
-                               ownerParagraph(ownerParagraph()->next());
-                       }
                        // delete old par
-                       delete old_cursor.par();
+                       ownerParagraphs().erase(old_cursor.par());
 
                        /* Breakagain the next par. Needed because of
                         * the parindent that can occur or dissappear.
                         * The next row can change its height, if
                         * there is another layout before */
-                       if (refresh_row->next()) {
-                               breakAgain(bview, refresh_row->next());
-                               updateCounters(bview);
+                       if (boost::next(prevrow) != rows().end()) {
+                               breakAgain(boost::next(prevrow));
+                               updateCounters();
                        }
-                       setHeightOfRow(bview, refresh_row);
+                       setHeightOfRow(prevrow);
                } else {
-                       refresh_row = old_cursor.row()->next();
-                       refresh_y = old_cursor.y() - old_cursor.row()->baseline();
+                       RowList::iterator nextrow = boost::next(old_cursor.row());
+                       const_cast<LyXText *>(this)->postPaint(
+                               old_cursor.y() - old_cursor.row()->baseline());
 
                        tmpcursor = cursor;
                        cursor = old_cursor; // that undo can restore the right cursor position
-                       Paragraph * endpar = old_cursor.par()->next();
-                       if (endpar && endpar->getDepth()) {
-                               while (endpar && endpar->getDepth()) {
-                                       endpar = endpar->next();
-                               }
+#warning FIXME. --end() iterator is usable here
+                       ParagraphList::iterator endpit = boost::next(old_cursor.par());
+                       while (endpit != ownerParagraphs().end() &&
+                              endpit->getDepth()) {
+                               ++endpit;
                        }
-                       setUndo(bview, Undo::DELETE, old_cursor.par(), endpar);
+
+                       setUndo(bv(), Undo::DELETE, old_cursor.par(), endpit);
                        cursor = tmpcursor;
 
                        // delete old row
                        removeRow(old_cursor.row());
                        // delete old par
-                       if (ownerParagraph() == old_cursor.par()) {
-                               ownerParagraph(ownerParagraph()->next());
-                       }
-
-                       delete old_cursor.par();
+                       ownerParagraphs().erase(old_cursor.par());
 
                        /* Breakagain the next par. Needed because of
                           the parindent that can occur or dissappear.
                           The next row can change its height, if
                           there is another layout before */
-                       if (refresh_row) {
-                               breakAgain(bview, refresh_row);
-                               updateCounters(bview);
+                       if (nextrow != rows().end()) {
+                               breakAgain(nextrow);
+                               updateCounters();
                        }
                }
 
                // correct cursor y
-               setCursorIntern(bview, cursor.par(), cursor.pos());
+               setCursorIntern(cursor.par(), cursor.pos());
 
-               if (selection.cursor.par()  == old_cursor.par()
-                   && selection.cursor.pos() == old_cursor.pos()) {
+               if (selection_position_was_oldcursor_position) {
                        // correct selection
                        selection.cursor = cursor;
                }
        }
        if (!deleted) {
                if (old_cursor.par()->stripLeadingSpaces()) {
-                       redoParagraphs(bview, old_cursor,
-                                      old_cursor.par()->next());
+                       redoParagraphs(old_cursor, boost::next(old_cursor.par()));
                        // correct cursor y
-                       setCursorIntern(bview, cursor.par(), cursor.pos());
+                       setCursorIntern(cursor.par(), cursor.pos());
                        selection.cursor = cursor;
                }
        }
@@ -2306,74 +2305,83 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
 }
 
 
-Paragraph * LyXText::ownerParagraph() const
+ParagraphList & LyXText::ownerParagraphs() const
 {
        if (inset_owner) {
-               return inset_owner->paragraph();
+               return inset_owner->paragraphs;
        }
-       return &*(bv_owner->buffer()->paragraphs.begin());
+       return bv_owner->buffer()->paragraphs;
 }
 
 
-void LyXText::ownerParagraph(Paragraph * p) const
+LyXText::refresh_status LyXText::refreshStatus() const
 {
-       if (inset_owner) {
-               inset_owner->paragraph(p);
-       } else {
-               bv_owner->buffer()->paragraphs.set(p);
-       }
+       return refresh_status_;
 }
 
 
-void LyXText::ownerParagraph(int id, Paragraph * p) const
+void LyXText::clearPaint()
 {
-       Paragraph * op = bv_owner->buffer()->getParFromID(id);
-       if (op && op->inInset()) {
-               static_cast<InsetText *>(op->inInset())->paragraph(p);
-       } else {
-               ownerParagraph(p);
-       }
+       refresh_status_ = REFRESH_NONE;
+       refresh_row = rows().end();
+       refresh_y = 0;
 }
 
 
-LyXText::text_status LyXText::status() const
+void LyXText::postPaint(int start_y)
 {
-       return status_;
+       refresh_status old = refresh_status_;
+
+       refresh_status_ = REFRESH_AREA;
+       refresh_row = rows().end();
+
+       if (old != REFRESH_NONE && refresh_y < start_y)
+               return;
+
+       refresh_y = start_y;
+
+       if (!inset_owner)
+               return;
+
+       // We are an inset's lyxtext. Tell the top-level lyxtext
+       // it needs to update the row we're in.
+       LyXText * t = bv()->text;
+       t->postRowPaint(t->cursor.row(), t->cursor.y() - t->cursor.row()->baseline());
 }
 
 
-void LyXText::status(BufferView * bview, LyXText::text_status st) const
+// FIXME: we should probably remove this y parameter,
+// make refresh_y be 0, and use row->y etc.
+void LyXText::postRowPaint(RowList::iterator rit, int start_y)
 {
-       LyXText * t = bview->text;
+       if (refresh_status_ != REFRESH_NONE && refresh_y < start_y) {
+               refresh_status_ = REFRESH_AREA;
+               return;
+       } else {
+               refresh_y = start_y;
+       }
 
-       // We should only go up with refreshing code so this means that if
-       // we have a MORE refresh we should never set it to LITTLE if we still
-       // didn't handle it (and then it will be UNCHANGED. Now as long as
-       // we stay inside one LyXText this may work but we need to tell the
-       // outermost LyXText that it should REALLY draw us if there is some
-       // change in a Inset::LyXText. So you see that when we are inside a
-       // inset's LyXText we give the LITTLE to the outermost LyXText to
-       // tell'em that it should redraw the actual row (where the inset
-       // resides! Capito?!
+       if (refresh_status_ == REFRESH_AREA)
+               return;
 
-       if (status_ != NEED_MORE_REFRESH || st != NEED_VERY_LITTLE_REFRESH) {
-               status_ = st;
-               if (inset_owner && st != UNCHANGED) {
-                       t->status(bview, NEED_VERY_LITTLE_REFRESH);
-                       if (!t->refresh_row) {
-                               t->refresh_row = t->cursor.row();
-                               t->refresh_y = t->cursor.y() -
-                                       t->cursor.row()->baseline();
-                       }
-               }
-       }
+       refresh_status_ = REFRESH_ROW;
+       refresh_row = rit;
+
+       if (!inset_owner)
+               return;
+
+       // We are an inset's lyxtext. Tell the top-level lyxtext
+       // it needs to update the row we're in.
+       LyXText * t = bv()->text;
+       t->postRowPaint(t->cursor.row(), t->cursor.y() - t->cursor.row()->baseline());
 }
 
 
-bool LyXText::isTopLevel() const
+bool LyXText::isInInset() const
 {
-       /// only the top-level lyxtext has a non-null bv owner
-       return bv_owner;
+       // Sub-level has non-null bv owner and
+       // non-null inset owner.
+       return inset_owner != 0 && bv_owner != 0;
 }