]> git.lyx.org Git - lyx.git/blobdiff - src/text2.C
fix some C++ parsing bugs
[lyx.git] / src / text2.C
index 3575e7acc8a69276d891d1d8277ec483533b6d98..2f83d8679761970a259f82512fce01bce3fa6dc4 100644 (file)
@@ -52,65 +52,51 @@ using lyx::pos_type;
 
 
 LyXText::LyXText(BufferView * bv)
-       : height(0), width(0), top_row_(0), top_row_offset_(0),
+       : height(0), width(0), anchor_row_(0), anchor_row_offset_(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)
-{}
+         bv_owner(bv)
+{
+       refresh_row = 0;
+       clearPaint();
+}
 
 
-LyXText::LyXText(InsetText * inset)
-       : height(0), width(0), top_row_(0), top_row_offset_(0),
+LyXText::LyXText(BufferView * bv, InsetText * inset)
+       : height(0), width(0), anchor_row_(0), anchor_row_offset_(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)
-{}
+         bv_owner(bv)
+{
+       refresh_row = 0;
+       clearPaint();
+}
 
 
 void LyXText::init(BufferView * bview, bool reinit)
 {
        if (reinit) {
-               // Delete all rows, this does not touch the paragraphs!
-               Row * tmprow = firstrow;
-               while (firstrow) {
-                       tmprow = firstrow->next();
-                       delete firstrow;
-                       firstrow = tmprow;
-               }
-
-               lastrow = 0;
-               refresh_row = 0;
+               rowlist_.clear();
                need_break_row = 0;
                width = height = 0;
                copylayouttype.erase();
-               top_y(refresh_y = 0);
-               status_ = LyXText::UNCHANGED;
-       } else if (firstrow)
+               top_y(0);
+               clearPaint();
+       } else if (firstRow())
                return;
 
        Paragraph * par = ownerParagraph();
        current_font = getFont(bview->buffer(), par, 0);
 
        while (par) {
-               insertParagraph(bview, par, lastrow);
+               if (rowlist_.empty())
+                       insertParagraph(par, 0);
+               else
+                       insertParagraph(par, lastRow());
                par = par->next();
        }
-       setCursorIntern(bview, firstrow->par(), 0);
+       setCursorIntern(firstRow()->par(), 0);
        selection.cursor = cursor;
 
-       updateCounters(bview);
-}
-
-
-LyXText::~LyXText()
-{
-       // Delete all rows, this does not touch the paragraphs!
-       Row * tmprow = firstrow;
-       while (firstrow) {
-               tmprow = firstrow->next();
-               delete firstrow;
-               firstrow = tmprow;
-       }
+       updateCounters();
 }
 
 
@@ -218,11 +204,11 @@ LyXFont const LyXText::getLabelFont(Buffer const * buf, Paragraph * par) const
 }
 
 
-void LyXText::setCharFont(BufferView * bv, Paragraph * par,
+void LyXText::setCharFont(Paragraph * par,
                          pos_type pos, LyXFont const & fnt,
                          bool toggleall)
 {
-       Buffer const * buf = bv->buffer();
+       Buffer const * buf = bv()->buffer();
        LyXFont font = getFont(buf, par, pos);
        font.update(fnt, buf->params.language, toggleall);
        // Let the insets convert their font
@@ -231,7 +217,7 @@ void LyXText::setCharFont(BufferView * bv, Paragraph * par,
                if (isEditableInset(inset)) {
                        UpdatableInset * uinset =
                                static_cast<UpdatableInset *>(inset);
-                       uinset->setFont(bv, fnt, toggleall, true);
+                       uinset->setFont(bv(), fnt, toggleall, true);
                }
        }
 
@@ -278,72 +264,55 @@ void LyXText::setCharFont(Buffer const * buf, Paragraph * par,
 // inserts a new row before the specified row, increments
 // the touched counters
 void LyXText::insertRow(Row * row, Paragraph * par,
-                       pos_type pos) const
+                       pos_type pos)
 {
        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;
+       if (!row) {
+               rowlist_.insert(rowlist_.begin(), tmprow);
+       } else {
+               rowlist_.insert(row->next(), tmprow);
+       }
 }
 
 
 // removes the row and reset the touched counters
-void LyXText::removeRow(Row * row) const
+void LyXText::removeRow(Row * row)
 {
+       lyx::Assert(row);
+
        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;
-       }
+       Row * row_next = row->next();
+       int const row_height = row->height();
+
+       /* FIXME: when we cache the bview, this should just
+        * become a postPaint(), I think */
        if (refresh_row == row) {
-               refresh_row = row_prev ? row_prev : row->next();
-               // what about refresh_y, refresh_height
+               refresh_row = row_prev ? row_prev : row_next;
+               // what about refresh_y
        }
-       if (top_row_ == row) {
-               if (row->next()) {
-                       top_row_ = row->next();
-                       top_row_offset_ -= row->height();
+
+       if (anchor_row_ == row) {
+               if (row_prev) {
+                       anchor_row_ = row_prev;
+                       anchor_row_offset_ += row_prev->height();
                } else {
-                       top_row_ = row_prev;
-                       top_row_offset_ = 0;
+                       anchor_row_ = row_next;
+                       anchor_row_offset_ -= row_height;
                }
        }
 
-       height -= row->height(); // the text becomes smaller
+       // the text becomes smaller
+       height -= row_height;
 
-       delete row;
+       rowlist_.erase(row);
 }
 
 
 // remove all following rows of the paragraph of the specified row.
-void LyXText::removeParagraph(Row * row) const
+void LyXText::removeParagraph(Row * row)
 {
        Paragraph * tmppar = row->par();
        row = row->next();
@@ -357,20 +326,16 @@ void LyXText::removeParagraph(Row * row) const
 }
 
 
-// insert the specified paragraph behind the specified row
-void LyXText::insertParagraph(BufferView * bview, Paragraph * par,
-                             Row * row) const
+void LyXText::insertParagraph(Paragraph * par, Row * row)
 {
        // insert a new row, starting at position 0
        insertRow(row, par, 0);
 
        // and now append the whole paragraph before the new row
        if (!row) {
-               firstrow->height(0);
-               appendParagraph(bview, firstrow);
+               appendParagraph(firstRow());
        } else {
-               row->next()->height(0);
-               appendParagraph(bview, row->next());
+               appendParagraph(row->next());
        }
 }
 
@@ -385,7 +350,7 @@ Inset * LyXText::getInset() const
 }
 
 
-void LyXText::toggleInset(BufferView * bview)
+void LyXText::toggleInset()
 {
        Inset * inset = getInset();
        // is there an editable inset at cursor position?
@@ -393,26 +358,25 @@ 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);
 }
 
 
@@ -437,8 +401,7 @@ void LyXText::makeFontEntriesLayoutSpecific(Buffer const & buf,
 }
 
 
-Paragraph * LyXText::setLayout(BufferView * bview,
-                              LyXCursor & cur, LyXCursor & sstart_cur,
+Paragraph * LyXText::setLayout(LyXCursor & cur, LyXCursor & sstart_cur,
                               LyXCursor & send_cur,
                               string const & layout)
 {
@@ -454,7 +417,7 @@ Paragraph * LyXText::setLayout(BufferView * bview,
                endpar = endpar->next(); // because of parindents etc.
        }
 
-       setUndo(bview, Undo::EDIT, sstart_cur.par(), undoendpar);
+       setUndo(bv(), Undo::EDIT, sstart_cur.par(), undoendpar);
 
        // ok we have a selection. This is always between sstart_cur
        // and sel_end cursor
@@ -463,11 +426,11 @@ Paragraph * LyXText::setLayout(BufferView * bview,
        Paragraph * epar = send_cur.par()->next();
 
        LyXLayout_ptr const & lyxlayout =
-               bview->buffer()->params.getLyXTextClass()[layout];
+               bv()->buffer()->params.getLyXTextClass()[layout];
 
        do {
                par->applyLayout(lyxlayout);
-               makeFontEntriesLayoutSpecific(*bview->buffer(), *par);
+               makeFontEntriesLayoutSpecific(*bv()->buffer(), *par);
                Paragraph * fppar = par;
                fppar->params().spaceTop(lyxlayout->fill_top ?
                                         VSpace(VSpace::VFILL)
@@ -486,7 +449,7 @@ Paragraph * LyXText::setLayout(BufferView * bview,
 
 
 // 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  */
 
@@ -496,26 +459,26 @@ void LyXText::setLayout(BufferView * bview, string const & layout)
                selection.start = cursor;  // dummy selection
                selection.end = cursor;
        }
-       Paragraph * endpar = setLayout(bview, cursor, selection.start,
+       Paragraph * endpar = setLayout(cursor, selection.start,
                                       selection.end, layout);
-       redoParagraphs(bview, selection.start, endpar);
+       redoParagraphs(selection.start, endpar);
 
        // 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)
+void  LyXText::incDepth()
 {
        // If there is no selection, just use the current paragraph
        if (!selection.set()) {
@@ -537,7 +500,7 @@ void LyXText::incDepth(BufferView * bview)
                endpar = endpar->next(); // because of parindents etc.
        }
 
-       setUndo(bview, Undo::EDIT,
+       setUndo(bv(), Undo::EDIT,
                selection.start.par(), undoendpar);
 
        LyXCursor tmpcursor = cursor; // store the current cursor
@@ -563,23 +526,23 @@ void LyXText::incDepth(BufferView * bview)
                cursor.par(cursor.par()->next());
        }
 
-       redoParagraphs(bview, selection.start, endpar);
+       redoParagraphs(selection.start, endpar);
 
        // we have to reset the selection, because the
        // geometry could have changed
-       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());
-       updateCounters(bview);
+       setCursor(selection.end.par(), selection.end.pos());
+       updateCounters();
        clearSelection();
-       setSelection(bview);
-       setCursor(bview, tmpcursor.par(), tmpcursor.pos());
+       setSelection();
+       setCursor(tmpcursor.par(), tmpcursor.pos());
 }
 
 
 // decrement depth over selection and
 // make a total rebreak of those paragraphs
-void  LyXText::decDepth(BufferView * bview)
+void  LyXText::decDepth()
 {
        // if there is no selection just set the layout
        // of the current paragraph
@@ -599,7 +562,7 @@ void  LyXText::decDepth(BufferView * bview)
                endpar = endpar->next(); // because of parindents etc.
        }
 
-       setUndo(bview, Undo::EDIT,
+       setUndo(bv(), Undo::EDIT,
                selection.start.par(), undoendpar);
 
        LyXCursor tmpcursor = cursor; // store the current cursor
@@ -619,38 +582,38 @@ void  LyXText::decDepth(BufferView * bview)
                cursor.par(cursor.par()->next());
        }
 
-       redoParagraphs(bview, selection.start, endpar);
+       redoParagraphs(selection.start, endpar);
 
        // we have to reset the selection, because the
        // geometry could have changed
-       setCursor(bview, selection.start.par(),
+       setCursor(selection.start.par(),
                  selection.start.pos());
        selection.cursor = cursor;
-       setCursor(bview, selection.end.par(), selection.end.pos());
-       updateCounters(bview);
+       setCursor(selection.end.par(), selection.end.pos());
+       updateCounters();
        clearSelection();
-       setSelection(bview);
-       setCursor(bview, tmpcursor.par(), tmpcursor.pos());
+       setSelection();
+       setCursor(tmpcursor.par(), tmpcursor.pos());
 }
 
 
 // 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()->beginningOfBody()) {
-                       layoutfont = getLabelFont(bview->buffer(),
+                       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
@@ -667,7 +630,7 @@ 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,
+       setUndo(bv(), Undo::EDIT,
                selection.start.par(), selection.end.par()->next());
        freezeUndo();
        cursor = selection.start;
@@ -676,7 +639,7 @@ 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 {
@@ -686,48 +649,46 @@ void LyXText::setFont(BufferView * bview, LyXFont const & font, bool toggleall)
        }
        unFreezeUndo();
 
-       redoParagraphs(bview, selection.start, selection.end.par()->next());
+       redoParagraphs(selection.start, selection.end.par()->next());
 
        // 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)
+void LyXText::redoHeightOfParagraph()
 {
        Row * 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();
                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, cursor.par(), cursor.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();
 
        int y = cur.y() - tmprow->baseline();
-       setHeightOfRow(bview, tmprow);
+       setHeightOfRow(tmprow);
 
        while (tmprow->previous()
               && tmprow->previous()->par() == tmprow->par())  {
@@ -735,38 +696,35 @@ void LyXText::redoDrawingOfParagraph(BufferView * bview, LyXCursor const & cur)
                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
+void LyXText::redoParagraphs(LyXCursor const & cur,
+                            Paragraph const * endpar)
 {
-       Row * tmprow2;
-       Paragraph * tmppar = 0;
-       Paragraph * first_phys_par = 0;
-
        Row * tmprow = cur.row();
 
        int y = cur.y() - tmprow->baseline();
 
+       Paragraph * first_phys_par = 0;
        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();
+               lyxerr << "ownerParagraph" << endl;
+
        } else {
                first_phys_par = tmprow->par();
+               lyxerr << "tmprow->par()" << endl;
+
+               // Find first row of this paragraph.
                while (tmprow->previous()
                       && tmprow->previous()->par() == first_phys_par)
                {
@@ -775,17 +733,12 @@ void LyXText::redoParagraphs(BufferView * bview, LyXCursor const & cur,
                }
        }
 
-       // 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
+       Row * prevrow = tmprow->previous();
+
+       // Remove all the rows until we reach endpar
+       Paragraph * tmppar = 0;
        if (tmprow->next())
                tmppar = tmprow->next()->par();
-       else
-               tmppar = 0;
        while (tmprow->next() && tmppar != endpar) {
                removeRow(tmprow->next());
                if (tmprow->next()) {
@@ -795,19 +748,19 @@ void LyXText::redoParagraphs(BufferView * bview, LyXCursor const & cur,
                }
        }
 
-       // remove the first one
-       tmprow2 = tmprow;     /* this is because tmprow->previous()
-                                can be 0 */
+       // Remove the first of the paragraphs rows.
+       // This is because tmprow->previous() can be 0
+       Row * tmprow2 = tmprow;
        tmprow = tmprow->previous();
        removeRow(tmprow2);
 
+       // Reinsert the paragraphs.
        tmppar = first_phys_par;
-
        do {
                if (tmppar) {
-                       insertParagraph(bview, tmppar, tmprow);
+                       insertParagraph(tmppar, tmprow);
                        if (!tmprow) {
-                               tmprow = firstrow;
+                               tmprow = firstRow();
                        }
                        while (tmprow->next()
                               && tmprow->next()->par() == tmppar) {
@@ -818,29 +771,28 @@ void LyXText::redoParagraphs(BufferView * bview, LyXCursor const & cur,
        } while (tmppar && tmppar != endpar);
 
        // this is because of layout changes
-       if (refresh_row) {
-               refresh_y -= refresh_row->height();
-               setHeightOfRow(bview, refresh_row);
+       if (prevrow) {
+               setHeightOfRow(prevrow);
+               const_cast<LyXText *>(this)->postPaint(y - prevrow->height());
        } else {
-               refresh_row = firstrow;
-               refresh_y = 0;
-               setHeightOfRow(bview, refresh_row);
+               setHeightOfRow(firstRow());
+               const_cast<LyXText *>(this)->postPaint(0);
        }
 
        if (tmprow && tmprow->next())
-               setHeightOfRow(bview, tmprow->next());
-       updateCounters(bview);
+               setHeightOfRow(tmprow->next());
+       updateCounters();
 }
 
 
-void LyXText::fullRebreak(BufferView * bview)
+void LyXText::fullRebreak()
 {
-       if (!firstrow) {
-               init(bview);
+       if (!firstRow()) {
+               init(bv());
                return;
        }
        if (need_break_row) {
-               breakAgain(bview, need_break_row);
+               breakAgain(need_break_row);
                need_break_row = 0;
                return;
        }
@@ -855,7 +807,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();
 
@@ -907,7 +859,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);
 }
 
 
@@ -948,7 +900,7 @@ string const LyXText::selectionAsString(Buffer const * buffer,
 }
 
 
-void LyXText::clearSelection() const
+void LyXText::clearSelection()
 {
        selection.set(false);
        selection.mark(false);
@@ -959,53 +911,52 @@ 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);
+               setCursor(cursor.par(), cursor.row()->lastPos() + 1);
        } else {
                if (!cursor.par()->empty() &&
                    (cursor.par()->getChar(cursor.row()->lastPos()) == ' '
                     || cursor.par()->isNewline(cursor.row()->lastPos()))) {
-                       setCursor(bview, cursor.par(), cursor.row()->lastPos());
+                       setCursor(cursor.par(), cursor.row()->lastPos());
                } else {
-                       setCursor(bview,cursor.par(),
+                       setCursor(cursor.par(),
                                  cursor.row()->lastPos() + 1);
                }
        }
 }
 
 
-void LyXText::cursorTop(BufferView * bview) const
+void LyXText::cursorTop()
 {
        while (cursor.par()->previous())
                cursor.par(cursor.par()->previous());
-       setCursor(bview, cursor.par(), 0);
+       setCursor(cursor.par(), 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());
+       setCursor(cursor.par(), cursor.par()->size());
 }
 
 
-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;
        }
 
@@ -1015,43 +966,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.
@@ -1068,14 +1019,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;
@@ -1099,16 +1049,14 @@ void LyXText::setParagraph(BufferView * bview,
                endpar = endpar->next();
        }
 
-       setUndo(bview, Undo::EDIT, selection.start.par(), undoendpar);
+       setUndo(bv(), Undo::EDIT, selection.start.par(), undoendpar);
 
 
        Paragraph * tmppar = selection.end.par();
 
        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();
+               setCursor(tmppar, 0);
+               postPaint(cursor.y() - cursor.row()->baseline());
                cursor.par()->params().lineTop(line_top);
                cursor.par()->params().lineBottom(line_bottom);
                cursor.par()->params().pagebreakTop(pagebreak_top);
@@ -1132,21 +1080,21 @@ void LyXText::setParagraph(BufferView * bview,
                tmppar = cursor.par()->previous();
        }
 
-       redoParagraphs(bview, selection.start, endpar);
+       redoParagraphs(selection.start, endpar);
 
        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, Paragraph * par)
 {
        LyXTextClass const & textclass = buf->params.getLyXTextClass();
        LyXLayout_ptr const & layout = par->layout();
@@ -1343,13 +1291,13 @@ 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;
+       Row * row = firstRow();
        Paragraph * par = row->par();
 
        // CHECK if this is really needed. (Lgb)
-       bview->buffer()->params.getLyXTextClass().counters().reset();
+       bv()->buffer()->params.getLyXTextClass().counters().reset();
 
        while (par) {
                while (row->par() != par)
@@ -1358,13 +1306,13 @@ void LyXText::updateCounters(BufferView * bview) const
                string const oldLabel = par->params().labelString();
 
                // setCounter can potentially change the labelString.
-               setCounter(bview->buffer(), par);
+               setCounter(bv()->buffer(), par);
 
                string const & newLabel = par->params().labelString();
 
                if (oldLabel.empty() && !newLabel.empty()) {
                        removeParagraph(row);
-                       appendParagraph(bview, row);
+                       appendParagraph(row);
                }
 
                par = par->next();
@@ -1372,23 +1320,23 @@ void LyXText::updateCounters(BufferView * bview) const
 }
 
 
-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(), cursor.par()->next());
        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();
 }
@@ -1400,15 +1348,15 @@ void LyXText::copyEnvironmentType()
 }
 
 
-void LyXText::pasteEnvironmentType(BufferView * bview)
+void LyXText::pasteEnvironmentType()
 {
        // do nothing if there has been no previous copyEnvironmentType()
        if (!copylayouttype.empty())
-               setLayout(bview, copylayouttype);
+               setLayout(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.
 
@@ -1418,7 +1366,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())
@@ -1440,7 +1388,7 @@ void LyXText::cutSelection(BufferView * bview, bool doclear, bool realcut)
                endpar = endpar->next(); // because of parindents etc.
        }
 
-       setUndo(bview, Undo::DELETE,
+       setUndo(bv(), Undo::DELETE,
                selection.start.par(), undoendpar);
 
        // there are two cases: cut only within one paragraph or
@@ -1451,7 +1399,7 @@ void LyXText::cutSelection(BufferView * bview, bool doclear, bool realcut)
                int pos = selection.end.pos();
                CutAndPaste::cutSelection(selection.start.par(), &endpar,
                                          selection.start.pos(), pos,
-                                         bview->buffer()->params.textclass,
+                                         bv()->buffer()->params.textclass,
                                          doclear, realcut);
                selection.end.pos(pos);
        } else {
@@ -1459,7 +1407,7 @@ void LyXText::cutSelection(BufferView * bview, bool doclear, bool realcut)
                int pos = selection.end.pos();
                CutAndPaste::cutSelection(selection.start.par(), &endpar,
                                          selection.start.pos(), pos,
-                                         bview->buffer()->params.textclass,
+                                         bv()->buffer()->params.textclass,
                                          doclear, realcut);
                cursor.par(endpar);
                selection.end.par(endpar);
@@ -1472,7 +1420,7 @@ void LyXText::cutSelection(BufferView * bview, bool doclear, bool realcut)
        if (doclear)
                selection.start.par()->stripLeadingSpaces();
 
-       redoParagraphs(bview, selection.start, endpar);
+       redoParagraphs(selection.start, endpar);
 
        // cutSelection can invalidate the cursor so we need to set
        // it anew. (Lgb)
@@ -1482,16 +1430,16 @@ void LyXText::cutSelection(BufferView * bview, bool doclear, bool realcut)
        // 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())
@@ -1509,17 +1457,17 @@ void LyXText::copySelection(BufferView * bview)
 
        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())
                return;
 
-       setUndo(bview, Undo::INSERT,
+       setUndo(bv(), Undo::INSERT,
                cursor.par(), cursor.par()->next());
 
        Paragraph * endpar;
@@ -1527,37 +1475,36 @@ void LyXText::pasteSelection(BufferView * bview)
        int pos = cursor.pos();
 
        CutAndPaste::pasteSelection(&actpar, &endpar, pos,
-                                   bview->buffer()->params.textclass);
+                                   bv()->buffer()->params.textclass);
 
-       redoParagraphs(bview, cursor, endpar);
+       redoParagraphs(cursor, endpar);
 
-       setCursor(bview, cursor.par(), cursor.pos());
+       setCursor(cursor.par(), cursor.pos());
        clearSelection();
 
        selection.cursor = cursor;
-       setCursor(bview, actpar, pos);
-       setSelection(bview);
-       updateCounters(bview);
+       setCursor(actpar, pos);
+       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
@@ -1568,7 +1515,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
@@ -1578,37 +1525,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();
        pos_type pos = cursor.pos();
        Paragraph * endpar = cursor.par()->next();
 
-       setCursorParUndo(bview);
+       setCursorParUndo(bv());
 
        // only to be sure, should not be neccessary
        clearSelection();
 
-       bview->buffer()->insertStringAsLines(par, pos, current_font, str);
+       bv()->buffer()->insertStringAsLines(par, pos, current_font, str);
 
-       redoParagraphs(bview, cursor, endpar);
-       setCursor(bview, cursor.par(), cursor.pos());
+       redoParagraphs(cursor, endpar);
+       setCursor(cursor.par(), cursor.pos());
        selection.cursor = cursor;
-       setCursor(bview, par, pos);
-       setSelection(bview);
+       setCursor(par, 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;
@@ -1628,11 +1575,11 @@ void LyXText::insertStringAsParagraphs(BufferView * bview, string const & str)
                        newline_inserted = false;
                }
        }
-       insertStringAsLines(bview, linestr);
+       insertStringAsLines(linestr);
 }
 
 
-void LyXText::checkParagraph(BufferView * bview, Paragraph * par,
+void LyXText::checkParagraph(Paragraph * par,
                             pos_type pos)
 {
        LyXCursor tmpcursor;
@@ -1643,19 +1590,17 @@ void LyXText::checkParagraph(BufferView * bview, Paragraph * par,
 
        // is there a break one row above
        if (row->previous() && row->previous()->par() == row->par()) {
-               z = rowBreakPoint(*bview, *row->previous());
+               z = rowBreakPoint(*row->previous());
                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);
+                       postPaint(y);
 
-                       breakAgain(bview, row->previous());
+                       breakAgain(row->previous());
 
                        // 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;
@@ -1664,14 +1609,13 @@ void LyXText::checkParagraph(BufferView * bview, Paragraph * par,
 
        int const tmpheight = row->height();
        pos_type const tmplast = row->lastPos();
-       refresh_y = y;
-       refresh_row = 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 && row->lastPos() == tmplast) {
+               postRowPaint(row, y);
+       } else {
+               postPaint(y);
+       }
 
        // check the special right address boxes
        if (par->layout()->margintype == MARGIN_RIGHT_ADDRESS_BOX) {
@@ -1681,7 +1625,7 @@ void LyXText::checkParagraph(BufferView * bview, Paragraph * par,
                tmpcursor.x(0);
                tmpcursor.x_fix(0);
                tmpcursor.pos(pos);
-               redoDrawingOfParagraph(bview, tmpcursor);
+               redoDrawingOfParagraph(tmpcursor);
        }
 
        // set the cursor again. Otherwise dangling pointers are possible
@@ -1689,35 +1633,35 @@ 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;
        }
 
@@ -1727,7 +1671,7 @@ bool LyXText::updateInset(BufferView * bview, Inset * inset)
        do {
                pos = par->getPositionOfInset(inset);
                if (pos != -1) {
-                       checkParagraph(bview, par, pos);
+                       checkParagraph(par, pos);
                        return true;
                }
                par = par->next();
@@ -1737,21 +1681,20 @@ bool LyXText::updateInset(BufferView * bview, Inset * inset)
 }
 
 
-bool LyXText::setCursor(BufferView * bview, Paragraph * par,
+bool LyXText::setCursor(Paragraph * par,
                        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(par, 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, Paragraph * par,
+                       pos_type pos, bool boundary)
 {
        lyx::Assert(par);
-       lyx::Assert(bview);
 
        cur.par(par);
        cur.pos(pos);
@@ -1802,19 +1745,23 @@ void LyXText::setCursor(BufferView * bview, LyXCursor & cur, Paragraph * par,
        }
 
        // 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());
+       //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);
 }
 
 
-float LyXText::getCursorX(BufferView * bview, Row * row,
-                                                 pos_type pos, pos_type last, bool boundary) const
+float LyXText::getCursorX(Row * row,
+                         pos_type pos, pos_type last, bool boundary) const
 {
        pos_type cursor_vpos = 0;
        float x;
@@ -1822,13 +1769,13 @@ 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(row, x, fill_separator, fill_hfill,
                       fill_label_hfill);
 
        if (last < row->pos())
                cursor_vpos = row->pos();
        else if (pos > last && !boundary)
-               cursor_vpos = (row->par()->isRightToLeftPar(bview->buffer()->params))
+               cursor_vpos = (row->par()->isRightToLeftPar(bv()->buffer()->params))
                        ? row->pos() : last + 1;
        else if (pos > row->pos() &&
                 (pos > last || boundary))
@@ -1843,7 +1790,7 @@ float LyXText::getCursorX(BufferView * bview, Row * row,
        pos_type body_pos = row->par()->beginningOfBody();
        if ((body_pos > 0) &&
            ((body_pos-1 > last) ||
-            !row->par()->isLineSeparator(body_pos-1)))
+            !row->par()->isLineSeparator(body_pos - 1)))
                body_pos = 0;
 
        for (pos_type vpos = row->pos(); vpos < cursor_vpos; ++vpos) {
@@ -1852,31 +1799,31 @@ float LyXText::getCursorX(BufferView * bview, Row * row,
                        x += fill_label_hfill +
                                font_metrics::width(
                                        row->par()->layout()->labelsep,
-                                       getLabelFont(bview->buffer(),
+                                       getLabelFont(bv()->buffer(),
                                                     row->par()));
                        if (row->par()->isLineSeparator(body_pos - 1))
-                               x -= singleWidth(bview,
+                               x -= singleWidth(
                                                 row->par(), body_pos - 1);
                }
                if (row->hfillExpansion(pos)) {
-                       x += singleWidth(bview, row->par(), pos);
+                       x += singleWidth(row->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);
+                       x += singleWidth(row->par(), pos);
                        if (pos >= body_pos)
                                x += fill_separator;
                } else
-                       x += singleWidth(bview, row->par(), pos);
+                       x += singleWidth(row->par(), pos);
        }
        return x;
 }
 
 
-void LyXText::setCursorIntern(BufferView * bview, Paragraph * par,
-                             pos_type pos, bool setfont, bool boundary) const
+void LyXText::setCursorIntern(Paragraph * par,
+                             pos_type pos, bool setfont, bool boundary)
 {
        InsetText * it = static_cast<InsetText *>(par->inInset());
        if (it) {
@@ -1899,18 +1846,18 @@ 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, par, pos, boundary);
        if (setfont)
-               setCurrentFont(bview);
+               setCurrentFont();
 }
 
 
-void LyXText::setCurrentFont(BufferView * bview) const
+void LyXText::setCurrentFont()
 {
        pos_type pos = cursor.pos();
        if (cursor.boundary() && pos > 0)
@@ -1931,14 +1878,14 @@ void LyXText::setCurrentFont(BufferView * bview) const
        }
 
        current_font =
-               cursor.par()->getFontSettings(bview->buffer()->params, pos);
-       real_current_font = getFont(bview->buffer(), cursor.par(), pos);
+               cursor.par()->getFontSettings(bv()->buffer()->params, pos);
+       real_current_font = getFont(bv()->buffer(), cursor.par(), pos);
 
        if (cursor.pos() == cursor.par()->size() &&
-           isBoundary(bview->buffer(), cursor.par(), cursor.pos()) &&
+           isBoundary(bv()->buffer(), cursor.par(), cursor.pos()) &&
            !cursor.boundary()) {
                Language const * lang =
-                       cursor.par()->getParLanguage(bview->buffer()->params);
+                       cursor.par()->getParLanguage(bv()->buffer()->params);
                current_font.setLanguage(lang);
                current_font.setNumber(LyXFont::OFF);
                real_current_font.setLanguage(lang);
@@ -1950,7 +1897,7 @@ void LyXText::setCurrentFont(BufferView * bview) 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(BufferView * bview, Row * row, int & x,
+LyXText::getColumnNearX(Row * row, int & x,
                        bool & boundary) const
 {
        float tmpx = 0.0;
@@ -1958,7 +1905,7 @@ LyXText::getColumnNearX(BufferView * bview, Row * row, int & x,
        float fill_hfill;
        float fill_label_hfill;
 
-       prepareToPrint(bview, row, tmpx, fill_separator,
+       prepareToPrint(row, tmpx, fill_separator,
                       fill_hfill, fill_label_hfill);
 
        pos_type vc = row->pos();
@@ -1989,23 +1936,23 @@ LyXText::getColumnNearX(BufferView * bview, Row * row, int & x,
                if (body_pos > 0 && c == body_pos-1) {
                        tmpx += fill_label_hfill +
                                font_metrics::width(layout->labelsep,
-                                              getLabelFont(bview->buffer(), row->par()));
+                                              getLabelFont(bv()->buffer(), row->par()));
                        if (row->par()->isLineSeparator(body_pos - 1))
-                               tmpx -= singleWidth(bview, row->par(), body_pos-1);
+                               tmpx -= singleWidth(row->par(), body_pos-1);
                }
 
                if (row->hfillExpansion(c)) {
-                       tmpx += singleWidth(bview, row->par(), c);
+                       tmpx += singleWidth(row->par(), c);
                        if (c >= body_pos)
                                tmpx += fill_hfill;
                        else
                                tmpx += fill_label_hfill;
                } else if (row->par()->isSeparator(c)) {
-                       tmpx += singleWidth(bview, row->par(), c);
+                       tmpx += singleWidth(row->par(), c);
                        if (c >= body_pos)
                                tmpx+= fill_separator;
                } else {
-                       tmpx += singleWidth(bview, row->par(), c);
+                       tmpx += singleWidth(row->par(), c);
                }
                ++vc;
        }
@@ -2023,7 +1970,7 @@ LyXText::getColumnNearX(BufferView * bview, Row * row, int & x,
                                         // some speedup if rtl_support=false
                && (!row->next() || row->next()->par() != row->par());
        bool const rtl = (lastrow)
-               ? row->par()->isRightToLeftPar(bview->buffer()->params)
+               ? row->par()->isRightToLeftPar(bv()->buffer()->params)
                : false; // If lastrow is false, we don't need to compute
                         // the value of rtl.
 
@@ -2040,16 +1987,16 @@ LyXText::getColumnNearX(BufferView * bview, Row * row, int & x,
                bool const rtl = (bidi_level(c) % 2 == 1);
                if (left_side == rtl) {
                        ++c;
-                       boundary = isBoundary(bview->buffer(), row->par(), c);
+                       boundary = isBoundary(bv()->buffer(), row->par(), c);
                }
        }
 
        if (row->pos() <= last && c > last
            && row->par()->isNewline(last)) {
                if (bidi_level(last) % 2 == 0)
-                       tmpx -= singleWidth(bview, row->par(), last);
+                       tmpx -= singleWidth(row->par(), last);
                else
-                       tmpx += singleWidth(bview, row->par(), last);
+                       tmpx += singleWidth(row->par(), last);
                c = last;
        }
 
@@ -2059,13 +2006,13 @@ LyXText::getColumnNearX(BufferView * bview, Row * row, int & x,
 }
 
 
-void LyXText::setCursorFromCoordinates(BufferView * bview, int x, int y) const
+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);
 }
 
 
@@ -2093,14 +2040,14 @@ 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);
        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);
@@ -2109,7 +2056,7 @@ void LyXText::setCursorFromCoordinates(BufferView * bview, LyXCursor & cur,
 
        if (beforeFullRowInset(*row, cur)) {
                pos_type last = row->lastPrintablePos();
-               float x = getCursorX(bview, row->next(), cur.pos(), last, bound);
+               float x = getCursorX(row->next(), cur.pos(), last, bound);
                cur.ix(int(x));
                cur.iy(y + row->height() + row->next()->baseline());
                cur.irow(row->next());
@@ -2122,109 +2069,108 @@ 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);
+                   isBoundary(bv()->buffer(), cursor.par(), cursor.pos() + 1))
+                       setCursor(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());
+               setCursor(par, par->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);
+               setCursor(cursor.par(), cursor.pos(), true, false);
        else if (cursor.pos() < cursor.par()->size()) {
-               setCursor(bview, cursor.par(), cursor.pos() + 1, true, false);
+               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);
+                   isBoundary(bv()->buffer(), cursor.par(), cursor.pos()))
+                       setCursor(cursor.par(), cursor.pos(), true, true);
        } else if (cursor.par()->next())
-               setCursor(bview, cursor.par()->next(), 0);
+               setCursor(cursor.par()->next(), 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 topy = top_y();
                int y1 = cursor.iy() - topy;
                int y2 = y1;
                y -= topy;
-               Inset * inset_hit = checkInsetHit(bview, x, y1);
+               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 topy = top_y();
                int y1 = cursor.iy() - topy;
                int y2 = y1;
                y -= topy;
-               Inset * inset_hit = checkInsetHit(bview, x, y1);
+               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);
+               setCursor(cursor.par()->previous(), 0);
        }
 }
 
 
-void LyXText::cursorDownParagraph(BufferView * bview) const
+void LyXText::cursorDownParagraph()
 {
        if (cursor.par()->next()) {
-               setCursor(bview, cursor.par()->next(), 0);
+               setCursor(cursor.par()->next(), 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
@@ -2242,12 +2188,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())
@@ -2292,7 +2237,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, old_cursor.par()->next());
 
 #ifdef WITH_WARNINGS
 #warning This will not work anymore when we have multiple views of the same buffer
@@ -2301,16 +2246,16 @@ 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;
                }
@@ -2338,13 +2283,11 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
                // 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();
+                       const_cast<LyXText *>(this)->postPaint(old_cursor.y() - old_cursor.row()->baseline()
+                                 - old_cursor.row()->previous()->height());
                        tmpcursor = cursor;
                        cursor = old_cursor; // that undo can restore the right cursor position
                        Paragraph * endpar = old_cursor.par()->next();
@@ -2353,7 +2296,7 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
                                        endpar = endpar->next();
                                }
                        }
-                       setUndo(bview, Undo::DELETE, old_cursor.par(), endpar);
+                       setUndo(bv(), Undo::DELETE, old_cursor.par(), endpar);
                        cursor = tmpcursor;
 
                        // delete old row
@@ -2368,14 +2311,17 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
                         * 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 (refresh_row) {
+                               if (refresh_row->next()) {
+                                       breakAgain(refresh_row->next());
+                                       updateCounters();
+                               }
+                               setHeightOfRow(refresh_row);
                        }
-                       setHeightOfRow(bview, refresh_row);
                } else {
-                       refresh_row = old_cursor.row()->next();
-                       refresh_y = old_cursor.y() - old_cursor.row()->baseline();
+                       Row * nextrow = old_cursor.row()->next();
+                       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
@@ -2385,7 +2331,7 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
                                        endpar = endpar->next();
                                }
                        }
-                       setUndo(bview, Undo::DELETE, old_cursor.par(), endpar);
+                       setUndo(bv(), Undo::DELETE, old_cursor.par(), endpar);
                        cursor = tmpcursor;
 
                        // delete old row
@@ -2401,14 +2347,14 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
                           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) {
+                               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()) {
@@ -2418,10 +2364,10 @@ bool LyXText::deleteEmptyParagraphMechanism(BufferView * bview,
        }
        if (!deleted) {
                if (old_cursor.par()->stripLeadingSpaces()) {
-                       redoParagraphs(bview, old_cursor,
+                       redoParagraphs(old_cursor,
                                       old_cursor.par()->next());
                        // correct cursor y
-                       setCursorIntern(bview, cursor.par(), cursor.pos());
+                       setCursorIntern(cursor.par(), cursor.pos());
                        selection.cursor = cursor;
                }
        }
@@ -2459,51 +2405,74 @@ void LyXText::ownerParagraph(int id, Paragraph * p) const
 }
 
 
-LyXText::text_status LyXText::status() const
+LyXText::refresh_status LyXText::refreshStatus() const
 {
-       return status_;
+       return refresh_status_;
 }
 
 
-void LyXText::status(BufferView * bview, LyXText::text_status new_status) const
+void LyXText::clearPaint()
 {
-       // We should not lose information from previous status
-       // sets, or we'll forget to repaint the other bits
-       // covered by the NEED_MORE_REFRESH
-       if (new_status == NEED_VERY_LITTLE_REFRESH
-           && status_ == NEED_MORE_REFRESH)
-               return;
+       refresh_status_ = REFRESH_NONE;
+       refresh_row = 0;
+       refresh_y = 0;
+}
+
+
+void LyXText::postPaint(int start_y)
+{
+       refresh_status old = refresh_status_;
 
-       status_ = new_status;
+       refresh_status_ = REFRESH_AREA;
+       refresh_row = 0;
 
-       if (new_status == UNCHANGED)
+       if (old != REFRESH_NONE && refresh_y < start_y)
                return;
 
-       if (!inset_owner)
-              return;
+       refresh_y = start_y;
 
-       LyXText * t = bview->text;
+       if (!inset_owner)
+               return;
 
        // We are an inset's lyxtext. Tell the top-level lyxtext
        // it needs to update the row we're in.
-       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();
-       }
+       LyXText * t = bv()->text;
+       t->postRowPaint(t->cursor.row(), t->cursor.y() - t->cursor.row()->baseline());
 }
 
 
-bool LyXText::isTopLevel() const
+// FIXME: we should probably remove this y parameter,
+// make refresh_y be 0, and use row->y etc.
+void LyXText::postRowPaint(Row * row, int start_y)
 {
-       /// only the top-level lyxtext has a non-null bv owner
-       return bv_owner;
+       if (refresh_status_ != REFRESH_NONE && refresh_y < start_y) {
+               refresh_status_ = REFRESH_AREA;
+               return;
+       } else {
+               refresh_y = start_y;
+       }
+
+       if (refresh_status_ == REFRESH_AREA)
+               return;
+
+       refresh_status_ = REFRESH_ROW;
+       refresh_row = row;
+
+       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::isInInset() const
 {
-       return inset_owner;
+       // Sub-level has non-null bv owner and
+       // non-null inset owner.
+       return inset_owner != 0 && bv_owner != 0;
 }