]> git.lyx.org Git - lyx.git/blobdiff - src/text.C
* Painter.h:
[lyx.git] / src / text.C
index 975c658a48695cc6bd5dd1243482ace5de9a625e..9b806455a2d52dfbd807a2df1540e7ac80a9f855 100644 (file)
@@ -41,7 +41,6 @@
 #include "lyxlex.h"
 #include "lyxrc.h"
 #include "lyxrow.h"
-#include "lyxrow_funcs.h"
 #include "metricsinfo.h"
 #include "paragraph.h"
 #include "paragraph_funcs.h"
 
 #include <sstream>
 
-using lyx::docstring;
-using lyx::char_type;
-using lyx::pit_type;
-using lyx::pos_type;
-using lyx::word_location;
 
-using lyx::support::bformat;
-using lyx::support::contains;
-using lyx::support::lowercase;
-using lyx::support::split;
-using lyx::support::uppercase;
+namespace lyx {
 
-using lyx::cap::cutSelection;
-using lyx::cap::pasteParagraphList;
+using support::bformat;
+using support::contains;
+using support::lowercase;
+using support::split;
+using support::uppercase;
 
-using lyx::frontend::FontMetrics;
+using cap::cutSelection;
+using cap::pasteParagraphList;
+
+using frontend::FontMetrics;
 
 using std::auto_ptr;
 using std::advance;
@@ -169,9 +165,9 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                for (; cit != token.end(); ++cit)
                        par.insertChar(par.size(), (*cit), font, change);
 #else
-               lyx::docstring dstr = lex.getDocString();
-               lyx::docstring::const_iterator cit = dstr.begin();
-               lyx::docstring::const_iterator cend = dstr.end();
+               docstring dstr = lex.getDocString();
+               docstring::const_iterator cit = dstr.begin();
+               docstring::const_iterator cend = dstr.end();
                for (; cit != cend; ++cit)
                        par.insertChar(par.size(), *cit, font, change);
 #endif
@@ -180,7 +176,7 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                string layoutname = lex.getString();
 
                font = LyXFont(LyXFont::ALL_INHERIT, bp.language);
-               change = Change();
+               change = Change(Change::UNCHANGED);
 
                LyXTextClass const & tclass = bp.getLyXTextClass();
 
@@ -193,7 +189,7 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                if (!hasLayout) {
                        errorList.push_back(ErrorItem(_("Unknown layout"),
                        bformat(_("Layout '%1$s' does not exist in textclass '%2$s'\nTrying to use the default instead.\n"),
-                       lyx::from_utf8(layoutname), lyx::from_utf8(tclass.name())), par.id(), 0, par.size()));
+                       from_utf8(layoutname), from_utf8(tclass.name())), par.id(), 0, par.size()));
                        layoutname = tclass.defaultLayoutName();
                }
 
@@ -222,7 +218,7 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                        par.insertInset(par.size(), inset, font, change);
                else {
                        lex.eatLine();
-                       docstring line = lyx::from_utf8(lex.getString());
+                       docstring line = from_utf8(lex.getString());
                        errorList.push_back(ErrorItem(_("Unknown Inset"), line,
                                            par.id(), 0, par.size()));
                }
@@ -314,11 +310,6 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                auto_ptr<InsetBase> inset(new InsetTabular(buf));
                inset->read(buf, lex);
                par.insertInset(par.size(), inset.release(), font, change);
-       } else if (token == "\\bibitem") {
-               InsetCommandParams p("bibitem", "dummy");
-               auto_ptr<InsetBibitem> inset(new InsetBibitem(p));
-               inset->read(buf, lex);
-               par.insertInset(par.size(), inset.release(), font, change);
        } else if (token == "\\hfill") {
                par.insertInset(par.size(), new InsetHFill, font, change);
        } else if (token == "\\lyxline") {
@@ -334,13 +325,12 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                lex.eatLine();
                std::istringstream is(lex.getString());
                unsigned int aid;
-               lyx::time_type ct;
+               time_type ct;
                is >> aid >> ct;
                if (aid >= bp.author_map.size()) {
                        errorList.push_back(ErrorItem(_("Change tracking error"),
                                            bformat(_("Unknown author index for insertion: %1$d\n"), aid),
                                            par.id(), 0, par.size()));
-
                        change = Change(Change::UNCHANGED);
                } else
                        change = Change(Change::INSERTED, bp.author_map[aid], ct);
@@ -348,35 +338,34 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                lex.eatLine();
                std::istringstream is(lex.getString());
                unsigned int aid;
-               lyx::time_type ct;
+               time_type ct;
                is >> aid >> ct;
                if (aid >= bp.author_map.size()) {
                        errorList.push_back(ErrorItem(_("Change tracking error"),
                                            bformat(_("Unknown author index for deletion: %1$d\n"), aid),
                                            par.id(), 0, par.size()));
-
                        change = Change(Change::UNCHANGED);
                } else
                        change = Change(Change::DELETED, bp.author_map[aid], ct);
        } else {
                lex.eatLine();
                errorList.push_back(ErrorItem(_("Unknown token"),
-                       bformat(_("Unknown token: %1$s %2$s\n"), lyx::from_utf8(token),
-                       lyx::from_utf8(lex.getString())),
+                       bformat(_("Unknown token: %1$s %2$s\n"), from_utf8(token),
+                       from_utf8(lex.getString())),
                        par.id(), 0, par.size()));
        }
 }
 
 
-void readParagraph(Buffer const & buf, Paragraph & par, LyXLex & lex, ErrorList & errorList)
+void readParagraph(Buffer const & buf, Paragraph & par, LyXLex & lex,
+       ErrorList & errorList)
 {
        lex.nextToken();
        string token = lex.getString();
        LyXFont font;
-       Change change;
+       Change change(Change::UNCHANGED);
 
        while (lex.isOK()) {
-
                readParToken(buf, par, lex, token, font, change, errorList);
 
                lex.nextToken();
@@ -501,9 +490,9 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
        if (isMainText())
                l_margin += changebarMargin();
 
-       string leftm = tclass.leftmargin();
-       docstring dleft(leftm.begin(), leftm.end());
-       l_margin += theFontMetrics(params.getFont()).signedWidth(dleft);
+       // FIXME UNICODE
+       docstring leftm = from_utf8(tclass.leftmargin());
+       l_margin += theFontMetrics(params.getFont()).signedWidth(leftm);
 
        if (par.getDepth() != 0) {
                // find the next level paragraph
@@ -534,45 +523,41 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
        switch (layout->margintype) {
        case MARGIN_DYNAMIC:
                if (!layout->leftmargin.empty()) {
-                       string leftm = layout->leftmargin;
-                       docstring dleft(leftm.begin(), leftm.end());
-                       l_margin += theFontMetrics(params.getFont()).signedWidth(dleft);
+                       // FIXME UNICODE
+                       docstring leftm = from_utf8(layout->leftmargin);
+                       l_margin += theFontMetrics(params.getFont()).signedWidth(leftm);
                }
                if (!par.getLabelstring().empty()) {
-                       string labin = layout->labelindent;
-                       docstring dlabin(labin.begin(), labin.end());
-                       l_margin += labelfont_metrics.signedWidth(dlabin);
-                       string labstr = par.getLabelstring();
-                       docstring dlabstr(labstr.begin(), labstr.end());
-                       l_margin += labelfont_metrics.width(dlabstr);
-                       string labsep = layout->labelsep;
-                       docstring dlabsep(labsep.begin(), labsep.end());
-                       l_margin += labelfont_metrics.width(dlabsep);
+                       // FIXME UNICODE
+                       docstring labin = from_utf8(layout->labelindent);
+                       l_margin += labelfont_metrics.signedWidth(labin);
+                       docstring labstr = par.getLabelstring();
+                       l_margin += labelfont_metrics.width(labstr);
+                       docstring labsep = from_utf8(layout->labelsep);
+                       l_margin += labelfont_metrics.width(labsep);
                }
                break;
 
        case MARGIN_MANUAL: {
-               string labin = layout->labelindent;
-               docstring dlabin(labin.begin(), labin.end());
-               l_margin += labelfont_metrics.signedWidth(dlabin);
+               // FIXME UNICODE
+               docstring labin = from_utf8(layout->labelindent);
+               l_margin += labelfont_metrics.signedWidth(labin);
                // The width of an empty par, even with manual label, should be 0
                if (!par.empty() && pos >= par.beginOfBody()) {
                        if (!par.getLabelWidthString().empty()) {
-                               string labstr = par.getLabelWidthString();
-                               docstring dlabstr(labstr.begin(), labstr.end());
-                               l_margin += labelfont_metrics.width(dlabstr);
-                               string labsep = layout->labelsep;
-                               docstring dlabsep(labsep.begin(), labsep.end());
-                               l_margin += labelfont_metrics.width(dlabsep);
+                               docstring labstr = par.getLabelWidthString();
+                               l_margin += labelfont_metrics.width(labstr);
+                               docstring labsep = from_utf8(layout->labelsep);
+                               l_margin += labelfont_metrics.width(labsep);
                        }
                }
                break;
        }
 
        case MARGIN_STATIC: {
-               string leftm = layout->leftmargin;
-               docstring dleft(leftm.begin(), leftm.end());
-               l_margin += theFontMetrics(params.getFont()).signedWidth(dleft)
+               // FIXME UNICODE
+               docstring leftm = from_utf8(layout->leftmargin);
+               l_margin += theFontMetrics(params.getFont()).signedWidth(leftm)
                        * 4     / (par.getDepth() + 4);
                break;
        }
@@ -580,13 +565,13 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
        case MARGIN_FIRST_DYNAMIC:
                if (layout->labeltype == LABEL_MANUAL) {
                        if (pos >= par.beginOfBody()) {
-                               string leftm = layout->leftmargin;
-                               docstring dleft(leftm.begin(), leftm.end());
-                               l_margin += labelfont_metrics.signedWidth(dleft);
+                               // FIXME UNICODE
+                               l_margin += labelfont_metrics.signedWidth(
+                                       from_utf8(layout->leftmargin));
                        } else {
-                               string labin = layout->labelindent;
-                               docstring dlabin(labin.begin(), labin.end());
-                               l_margin += labelfont_metrics.signedWidth(dlabin);
+                               // FIXME UNICODE
+                               l_margin += labelfont_metrics.signedWidth(
+                                       from_utf8(layout->labelindent));
                        }
                } else if (pos != 0
                           // Special case to fix problems with
@@ -594,22 +579,15 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
                           || (layout->labeltype == LABEL_STATIC
                               && layout->latextype == LATEX_ENVIRONMENT
                               && !isFirstInSequence(pit, pars_))) {
-                       string leftm = layout->leftmargin;
-                       docstring dleft(leftm.begin(), leftm.end());
-                       l_margin += labelfont_metrics.signedWidth(dleft);
+                       // FIXME UNICODE
+                       l_margin += labelfont_metrics.signedWidth(from_utf8(layout->leftmargin));
                } else if (layout->labeltype != LABEL_TOP_ENVIRONMENT
                           && layout->labeltype != LABEL_BIBLIO
                           && layout->labeltype !=
                           LABEL_CENTERED_TOP_ENVIRONMENT) {
-                       string labin = layout->labelindent;
-                       docstring dlabin(labin.begin(), labin.end());
-                       l_margin += labelfont_metrics.signedWidth(dlabin);
-                       string labsep = layout->labelsep;
-                       docstring dlabsep(labsep.begin(), labsep.end());
-                       l_margin += labelfont_metrics.width(dlabsep);
-                       string labstr = par.getLabelstring();
-                       docstring dlabstr(labstr.begin(), labstr.end());
-                       l_margin += labelfont_metrics.width(dlabstr);
+                       l_margin += labelfont_metrics.signedWidth(from_utf8(layout->labelindent));
+                       l_margin += labelfont_metrics.width(from_utf8(layout->labelsep));
+                       l_margin += labelfont_metrics.width(par.getLabelstring());
                }
                break;
 
@@ -665,7 +643,7 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
                || bv()->buffer()->params().paragraph_separation ==
                   BufferParams::PARSEP_INDENT))
        {
-               docstring din(parindent.begin(), parindent.end());
+               docstring din = from_utf8(parindent);
                l_margin += theFontMetrics(params.getFont()).signedWidth(din);
        }
 
@@ -684,15 +662,13 @@ int LyXText::rightMargin(Paragraph const & par) const
 
        BufferParams const & params = bv()->buffer()->params();
        LyXTextClass const & tclass = params.getLyXTextClass();
-       string trmarg = tclass.rightmargin();
-       docstring dtrmarg(trmarg.begin(), trmarg.end());
-       string lrmarg = par.layout()->rightmargin;
-       docstring dlrmarg(lrmarg.begin(), lrmarg.end());
+       docstring trmarg = from_utf8(tclass.rightmargin());
+       docstring lrmarg = from_utf8(par.layout()->rightmargin);
        FontMetrics const & fm = theFontMetrics(params.getFont());
        int const r_margin =
-               ::rightMargin()
-               + fm.signedWidth(dtrmarg)
-               + fm.signedWidth(dlrmarg)
+               lyx::rightMargin()
+               + fm.signedWidth(trmarg)
+               + fm.signedWidth(lrmarg)
                * 4 / (par.getDepth() + 4);
 
        return r_margin;
@@ -773,9 +749,8 @@ void LyXText::rowBreakPoint(pit_type const pit, Row & row) const
 
                // add the auto-hfill from label end to the body
                if (body_pos && i == body_pos) {
-                       string lsep = layout->labelsep;
-                       docstring dlsep(lsep.begin(), lsep.end());
-                       int add = fm.width(dlsep);
+                       docstring lsep = from_utf8(layout->labelsep);
+                       int add = fm.width(lsep);
                        if (par.isLineSeparator(i - 1))
                                add -= singleWidth(par, i - 1);
 
@@ -848,8 +823,7 @@ void LyXText::setRowWidth(pit_type const pit, Row & row) const
        pos_type const end = row.endpos();
 
        Paragraph const & par = pars_[pit];
-       string const & labelsep = par.layout()->labelsep;
-       docstring dlsep(labelsep.begin(), labelsep.end());
+       docstring const labelsep = from_utf8(par.layout()->labelsep);
        int w = leftMargin(pit, row.pos());
 
        pos_type const body_pos = par.beginOfBody();
@@ -861,7 +835,7 @@ void LyXText::setRowWidth(pit_type const pit, Row & row) const
                FontIterator fi = FontIterator(*this, par, i);
                for ( ; i < end; ++i, ++fi) {
                        if (body_pos > 0 && i == body_pos) {
-                               w += fm.width(dlsep);
+                               w += fm.width(labelsep);
                                if (par.isLineSeparator(i - 1))
                                        w -= singleWidth(par, i - 1);
                                w = max(w, labelEnd(pit));
@@ -872,7 +846,7 @@ void LyXText::setRowWidth(pit_type const pit, Row & row) const
        }
 
        if (body_pos > 0 && body_pos >= end) {
-               w += fm.width(dlsep);
+               w += fm.width(labelsep);
                if (end > 0 && par.isLineSeparator(end - 1))
                        w -= singleWidth(par, end - 1);
                w = max(w, labelEnd(pit));
@@ -900,15 +874,13 @@ int LyXText::labelFill(Paragraph const & par, Row const & row) const
        for (pos_type i = row.pos(); i <= last; ++i)
                w += singleWidth(par, i);
 
-       string const & label = par.params().labelWidthString();
+       docstring const & label = par.params().labelWidthString();
        if (label.empty())
                return 0;
 
-       docstring dlab(label.begin(), label.end());
-
        FontMetrics const & fm = theFontMetrics(getLabelFont(par));
 
-       return max(0, fm.width(dlab) - w);
+       return max(0, fm.width(label) - w);
 }
 
 
@@ -1125,7 +1097,8 @@ void LyXText::breakParagraph(LCursor & cur, bool keep_layout)
        // Always break behind a space
        // It is better to erase the space (Dekel)
        if (cur.pos() != cur.lastpos() && cpar.isLineSeparator(cur.pos()))
-               cpar.erase(cur.pos());
+               // FIXME: change tracking (MG)
+               cpar.eraseChar(cur.pos(), cur.buffer().params().trackChanges);
 
        // How should the layout for the new paragraph be?
        int preserve_layout = 0;
@@ -1141,7 +1114,7 @@ void LyXText::breakParagraph(LCursor & cur, bool keep_layout)
        // we need to set this before we insert the paragraph.
        bool const isempty = cpar.allowEmpty() && cpar.empty();
 
-       ::breakParagraph(cur.buffer().params(), paragraphs(), cpit,
+       lyx::breakParagraph(cur.buffer().params(), paragraphs(), cpit,
                         cur.pos(), preserve_layout);
 
        // After this, neither paragraph contains any rows!
@@ -1160,7 +1133,8 @@ void LyXText::breakParagraph(LCursor & cur, bool keep_layout)
        }
 
        while (!pars_[next_par].empty() && pars_[next_par].isNewline(0))
-               pars_[next_par].erase(0);
+               // FIXME: change tracking (MG)
+               pars_[next_par].eraseChar(0, cur.buffer().params().trackChanges);
 
        ParIterator current_it(cur);
        ParIterator last_it(cur);
@@ -1173,7 +1147,7 @@ void LyXText::breakParagraph(LCursor & cur, bool keep_layout)
        if (cur.buffer().params().trackChanges) {
                // FIXME: Change tracking (MG)
                cur.paragraph().setChange(cur.paragraph().size(),
-                       Change::INSERTED);
+                       Change(Change::INSERTED));
        }
 
        // This check is necessary. Otherwise the new empty paragraph will
@@ -1202,9 +1176,9 @@ void LyXText::insertChar(LCursor & cur, char_type c)
                par.isFreeSpacing();
 
        if (lyxrc.auto_number) {
-               static docstring const number_operators = lyx::from_ascii("+-/*");
-               static docstring const number_unary_operators = lyx::from_ascii("+-");
-               static docstring const number_seperators = lyx::from_ascii(".,:");
+               static docstring const number_operators = from_ascii("+-/*");
+               static docstring const number_unary_operators = from_ascii("+-");
+               static docstring const number_seperators = from_ascii(".,:");
 
                if (current_font.number() == LyXFont::ON) {
                        if (!isDigit(c) && !contains(number_operators, c) &&
@@ -1255,9 +1229,8 @@ void LyXText::insertChar(LCursor & cur, char_type c)
                        return;
                }
                BOOST_ASSERT(cur.pos() > 0);
-               if ((par.isLineSeparator(cur.pos() - 1)
-                   || par.isNewline(cur.pos() - 1))
-                   && par.lookupChange(cur.pos() - 1) != Change::DELETED) {
+               if ((par.isLineSeparator(cur.pos() - 1) || par.isNewline(cur.pos() - 1))
+                   && !par.isDeleted(cur.pos() - 1)) {
                        static bool sent_space_message = false;
                        if (!sent_space_message) {
                                cur.message(_("You cannot type two spaces this way. "
@@ -1268,7 +1241,7 @@ void LyXText::insertChar(LCursor & cur, char_type c)
                }
        }
 
-       par.insertChar(cur.pos(), c, current_font);
+       par.insertChar(cur.pos(), c, current_font, cur.buffer().params().trackChanges);
        setCursor(cur, cur.pit(), cur.pos() + 1, false, cur.boundary());
        charInserted();
 }
@@ -1391,9 +1364,8 @@ LyXText::computeRowMetrics(pit_type const pit, Row const & row) const
                if (body_pos > 0
                    && (body_pos > end || !par.isLineSeparator(body_pos - 1)))
                {
-                       string lsep = layout->labelsep;
-                       docstring dlsep(lsep.begin(), lsep.end());
-                       result.x += theFontMetrics(getLabelFont(par)).width(dlsep);
+                       docstring const lsep = from_utf8(layout->labelsep);
+                       result.x += theFontMetrics(getLabelFont(par)).width(lsep);
                        if (body_pos <= end)
                                result.x += result.label_hfill;
                }
@@ -1487,22 +1459,22 @@ void LyXText::acceptChange(LCursor & cur)
        DocIterator it = cur.selectionBegin();
        DocIterator et = cur.selectionEnd();
        pit_type pit = it.pit();
-       Change::Type const type = pars_[pit].lookupChange(it.pos()).type;
+       bool isDeleted = pars_[pit].isDeleted(it.pos());
        for (; pit <= et.pit(); ++pit) {
                pos_type left  = ( pit == it.pit() ? it.pos() : 0 );
                pos_type right =
                    ( pit == et.pit() ? et.pos() : pars_[pit].size() + 1 );
-               pars_[pit].acceptChange(left, right);
+               pars_[pit].acceptChanges(left, right);
        }
-       if (type == Change::DELETED) {
+       if (isDeleted) {
                ParagraphList & plist = paragraphs();
                if (it.pit() + 1 < et.pit())
                        pars_.erase(boost::next(plist.begin(), it.pit() + 1),
                                    boost::next(plist.begin(), et.pit()));
 
                // Paragraph merge if appropriate:
-               if (pars_[it.pit()].lookupChange(pars_[it.pit()].size())
-                       == Change::DELETED) {
+               // FIXME: change tracking (MG)
+               if (pars_[it.pit()].isDeleted(pars_[it.pit()].size())) {
                        setCursorIntern(cur, it.pit() + 1, 0);
                        backspacePos0(cur);
                }
@@ -1524,21 +1496,21 @@ void LyXText::rejectChange(LCursor & cur)
        DocIterator it = cur.selectionBegin();
        DocIterator et = cur.selectionEnd();
        pit_type pit = it.pit();
-       Change::Type const type = pars_[pit].lookupChange(it.pos()).type;
+       bool isInserted = pars_[pit].isInserted(it.pos());
        for (; pit <= et.pit(); ++pit) {
                pos_type left  = ( pit == it.pit() ? it.pos() : 0 );
                pos_type right =
                    ( pit == et.pit() ? et.pos() : pars_[pit].size() + 1 );
-               pars_[pit].rejectChange(left, right);
+               pars_[pit].rejectChanges(left, right);
        }
-       if (type == Change::INSERTED) {
+       if (isInserted) {
                ParagraphList & plist = paragraphs();
                if (it.pit() + 1 < et.pit())
                        pars_.erase(boost::next(plist.begin(), it.pit() + 1),
                                    boost::next(plist.begin(), et.pit()));
                // Paragraph merge if appropriate:
-               if (pars_[it.pit()].lookupChange(pars_[it.pit()].size())
-                       == Change::INSERTED) {
+               // FIXME: change tracking (MG)
+               if (pars_[it.pit()].isInserted(pars_[it.pit()].size())) {
                        setCursorIntern(cur, it.pit() + 1, 0);
                        backspacePos0(cur);
                }
@@ -1613,7 +1585,7 @@ void LyXText::changeCase(LCursor & cur, LyXText::TextCase action)
                to = cur.selEnd();
        } else {
                from = cur.top();
-               getWord(from, to, lyx::PARTIAL_WORD);
+               getWord(from, to, PARTIAL_WORD);
                cursorRightOneWord(cur);
        }
 
@@ -1644,10 +1616,11 @@ void LyXText::changeCase(LCursor & cur, LyXText::TextCase action)
                                break;
                        }
                }
-#ifdef WITH_WARNINGS
-#warning changes
-#endif
-               pars_[pit].setChar(pos, c);
+
+               // FIXME: change tracking (MG)
+               // sorry but we are no longer allowed to set a single character directly
+               // we have to rewrite this method in terms of erase&insert operations
+               //pars_[pit].setChar(pos, c);
                ++pos;
        }
 }
@@ -1662,7 +1635,8 @@ bool LyXText::erase(LCursor & cur)
                recordUndo(cur, Undo::DELETE, cur.pit());
                setCursorIntern(cur, cur.pit(), cur.pos() + 1, false, cur.boundary());
                needsUpdate = backspace(cur);
-               if (cur.paragraph().lookupChange(cur.pos()) == Change::DELETED)
+               // FIXME: change tracking (MG)
+               if (cur.paragraph().isDeleted(cur.pos()))
                        cur.posRight();
        } else if (cur.pit() != cur.lastpit()) {
                LCursor scur = cur;
@@ -1675,7 +1649,8 @@ bool LyXText::erase(LCursor & cur)
                                // FIXME: Change tracking (MG)
                                // move forward after the paragraph break is DELETED
                                Paragraph & par = cur.paragraph();
-                               if (par.lookupChange(par.size()) == Change::DELETED)
+                               // FIXME: change tracking (MG)
+                               if (par.isDeleted(par.size()))
                                        setCursorIntern(cur, cur.pit() + 1, 0);
                                }
                } else {
@@ -1782,8 +1757,10 @@ bool LyXText::backspace(LCursor & cur)
                        // deleted:
                        Paragraph & par = pars_[cur.pit() - 1];
                        // Take care of a just inserted para break:
-                       if (par.lookupChange(par.size()) != Change::INSERTED) {
-                               par.setChange(par.size(), Change::DELETED);
+                       // FIXME: change tracking (MG)
+                       if (!par.isInserted(par.size())) {
+                               // FIXME: change tracking (MG)
+                               par.setChange(par.size(), Change(Change::DELETED));
                                setCursorIntern(cur, cur.pit() - 1, par.size());
                                return true;
                        }
@@ -1801,7 +1778,8 @@ bool LyXText::backspace(LCursor & cur)
                // without the dreaded mechanism. (JMarc)
                setCursorIntern(cur, cur.pit(), cur.pos() - 1,
                                false, cur.boundary());
-               cur.paragraph().erase(cur.pos());
+               // FIXME: change tracking (MG)
+               cur.paragraph().eraseChar(cur.pos(), cur.buffer().params().trackChanges);
        }
 
        if (cur.pos() == cur.lastpos())
@@ -1822,8 +1800,8 @@ bool LyXText::dissolveInset(LCursor & cur) {
        recordUndoInset(cur);
        cur.selHandle(false);
        // save position
-       lyx::pos_type spos = cur.pos();
-       lyx::pit_type spit = cur.pit();
+       pos_type spos = cur.pos();
+       pit_type spit = cur.pit();
        ParagraphList plist;
        if (cur.lastpit() != 0 || cur.lastpos() != 0)
                plist = paragraphs();
@@ -1832,9 +1810,10 @@ bool LyXText::dissolveInset(LCursor & cur) {
        if (spit == 0)
                spos += cur.pos();
        spit += cur.pit();
-       cur.paragraph().erase(cur.pos());
+       Buffer & b = cur.buffer();
+       // FIXME: change tracking (MG)
+       cur.paragraph().eraseChar(cur.pos(), b.params().trackChanges);
        if (!plist.empty()) {
-               Buffer & b = cur.buffer();
                pasteParagraphList(cur, plist, b.params().textclass,
                                   b.errorList("Paste"));
                // restore position
@@ -1871,7 +1850,8 @@ bool LyXText::redoParagraph(pit_type const pit)
                if (!hasbibitem) {
                        InsetBibitem * inset(new
                                InsetBibitem(InsetCommandParams("bibitem")));
-                       par.insertInset(0, static_cast<InsetBase *>(inset));
+                       // FIXME: change tracking (MG)
+                       par.insertInset(0, static_cast<InsetBase *>(inset), Change(Change::INSERTED));
                        bv()->cursor().posRight();
                }
        }
@@ -2054,17 +2034,19 @@ void LyXText::drawSelection(PainterInfo & pi, int x, int) const
        DocIterator beg = cur.selectionBegin();
        DocIterator end = cur.selectionEnd();
 
+       BufferView * bv = pi.base.bv;
+
        // the selection doesn't touch the visible screen
-       if (bv_funcs::status(pi.base.bv, beg) == bv_funcs::CUR_BELOW
-           || bv_funcs::status(pi.base.bv, end) == bv_funcs::CUR_ABOVE)
+       if (bv_funcs::status(bv, beg) == bv_funcs::CUR_BELOW
+           || bv_funcs::status(bv, end) == bv_funcs::CUR_ABOVE)
                return;
 
        Paragraph const & par1 = pars_[beg.pit()];
        Paragraph const & par2 = pars_[end.pit()];
 
-       bool const above = (bv_funcs::status(pi.base.bv, beg)
+       bool const above = (bv_funcs::status(bv, beg)
                            == bv_funcs::CUR_ABOVE);
-       bool const below = (bv_funcs::status(pi.base.bv, end)
+       bool const below = (bv_funcs::status(bv, end)
                            == bv_funcs::CUR_BELOW);
        int y1,y2,x1,x2;
        if (above) {
@@ -2074,7 +2056,7 @@ void LyXText::drawSelection(PainterInfo & pi, int x, int) const
                x2 = dim_.wid;
        } else {
                Row const & row1 = par1.getRow(beg.pos(), beg.boundary());
-               y1 = bv_funcs::getPos(beg, beg.boundary()).y_ - row1.ascent();
+               y1 = bv_funcs::getPos(*bv, beg, beg.boundary()).y_ - row1.ascent();
                y2 = y1 + row1.height();
                int const startx = cursorX(beg.top(), beg.boundary());
                x1 = !isRTL(par1) ? startx : 0;
@@ -2083,13 +2065,13 @@ void LyXText::drawSelection(PainterInfo & pi, int x, int) const
 
        int Y1,Y2,X1,X2;
        if (below) {
-               Y1 = pi.base.bv->workHeight();
-               Y2 = pi.base.bv->workHeight();
+               Y1 = bv->workHeight();
+               Y2 = bv->workHeight();
                X1 = 0;
                X2 = dim_.wid;
        } else {
                Row const & row2 = par2.getRow(end.pos(), end.boundary());
-               Y1 = bv_funcs::getPos(end, end.boundary()).y_ - row2.ascent();
+               Y1 = bv_funcs::getPos(*bv, end, end.boundary()).y_ - row2.ascent();
                Y2 = Y1 + row2.height();
                int const endx = cursorX(end.top(), end.boundary());
                X1 = !isRTL(par2) ? 0 : endx;
@@ -2140,7 +2122,7 @@ void LyXText::getWord(CursorSlice & from, CursorSlice & to,
 {
        Paragraph const & from_par = pars_[from.pit()];
        switch (loc) {
-       case lyx::WHOLE_WORD_STRICT:
+       case WHOLE_WORD_STRICT:
                if (from.pos() == 0 || from.pos() == from_par.size()
                    || !from_par.isLetter(from.pos())
                    || !from_par.isLetter(from.pos() - 1)) {
@@ -2149,22 +2131,22 @@ void LyXText::getWord(CursorSlice & from, CursorSlice & to,
                }
                // no break here, we go to the next
 
-       case lyx::WHOLE_WORD:
+       case WHOLE_WORD:
                // If we are already at the beginning of a word, do nothing
                if (!from.pos() || !from_par.isLetter(from.pos() - 1))
                        break;
                // no break here, we go to the next
 
-       case lyx::PREVIOUS_WORD:
+       case PREVIOUS_WORD:
                // always move the cursor to the beginning of previous word
                while (from.pos() && from_par.isLetter(from.pos() - 1))
                        --from.pos();
                break;
-       case lyx::NEXT_WORD:
+       case NEXT_WORD:
                lyxerr << "LyXText::getWord: NEXT_WORD not implemented yet"
                       << endl;
                break;
-       case lyx::PARTIAL_WORD:
+       case PARTIAL_WORD:
                // no need to move the 'from' cursor
                break;
        }
@@ -2179,7 +2161,7 @@ void LyXText::write(Buffer const & buf, std::ostream & os) const
 {
        ParagraphList::const_iterator pit = paragraphs().begin();
        ParagraphList::const_iterator end = paragraphs().end();
-       Paragraph::depth_type dth = 0;
+       depth_type dth = 0;
        for (; pit != end; ++pit)
                pit->write(buf, os, buf.params(), dth);
 }
@@ -2187,7 +2169,7 @@ void LyXText::write(Buffer const & buf, std::ostream & os) const
 
 bool LyXText::read(Buffer const & buf, LyXLex & lex, ErrorList & errorList)
 {
-       Paragraph::depth_type depth = 0;
+       depth_type depth = 0;
 
        while (lex.isOK()) {
                lex.nextToken();
@@ -2196,21 +2178,17 @@ bool LyXText::read(Buffer const & buf, LyXLex & lex, ErrorList & errorList)
                if (token.empty())
                        continue;
 
-               if (token == "\\end_inset") {
+               if (token == "\\end_inset")
                        break;
-               }
 
-               if (token == "\\end_body") {
+               if (token == "\\end_body")
                        continue;
-               }
 
-               if (token == "\\begin_body") {
+               if (token == "\\begin_body")
                        continue;
-               }
 
-               if (token == "\\end_document") {
+               if (token == "\\end_document")
                        return false;
-               }
 
                if (token == "\\begin_layout") {
                        lex.pushToken(token);
@@ -2222,7 +2200,7 @@ bool LyXText::read(Buffer const & buf, LyXLex & lex, ErrorList & errorList)
 
                        // FIXME: goddamn InsetTabular makes us pass a Buffer
                        // not BufferParams
-                       ::readParagraph(buf, pars_.back(), lex, errorList);
+                       lyx::readParagraph(buf, pars_.back(), lex, errorList);
 
                } else if (token == "\\begin_deeper") {
                        ++depth;
@@ -2302,9 +2280,9 @@ int LyXText::cursorX(CursorSlice const & sl, bool boundary) const
        for (pos_type vpos = row_pos; vpos < cursor_vpos; ++vpos) {
                pos_type pos = bidi.vis2log(vpos);
                if (body_pos > 0 && pos == body_pos - 1) {
-                       string lsep = par.layout()->labelsep;
-                       docstring dlsep(lsep.begin(), lsep.end());
-                       x += m.label_hfill + labelfm.width(dlsep);
+                       // FIXME UNICODE
+                       docstring const lsep = from_utf8(par.layout()->labelsep);
+                       x += m.label_hfill + labelfm.width(lsep);
                        if (par.isLineSeparator(body_pos - 1))
                                x -= singleWidth(par, body_pos - 1);
                }
@@ -2317,7 +2295,7 @@ int LyXText::cursorX(CursorSlice const & sl, bool boundary) const
 
                x += singleWidth(par, pos, par.getChar(pos), font);
 
-               if (hfillExpansion(par, row, pos))
+               if (par.hfillExpansion(row, pos))
                        x += (pos >= body_pos) ? m.hfill : m.label_hfill;
                else if (par.isSeparator(pos) && pos >= body_pos)
                        x += m.separator;
@@ -2372,19 +2350,17 @@ string LyXText::currentState(LCursor & cur)
        Paragraph const & par = cur.paragraph();
        std::ostringstream os;
 
-       bool const show_change = par.lookupChange(cur.pos()) != Change::UNCHANGED;
-
        if (buf.params().trackChanges)
                os << "[C] ";
 
-       if (show_change) {
-               Change change = par.lookupChange(cur.pos());
+       Change change = par.lookupChange(cur.pos());
+
+       if (change.type != Change::UNCHANGED) {
                Author const & a = buf.params().authors().get(change.author);
-               os << lyx::to_utf8(_("Change: ")) << a.name();
+               os << to_utf8(_("Change: ")) << a.name();
                if (!a.email().empty())
                        os << " (" << a.email() << ")";
-               if (change.changetime)
-                       os << lyx::to_utf8(_(" at ")) << ctime(&change.changetime);
+               os << to_utf8(_(" at ")) << ctime(&change.changetime);
                os << " : ";
        }
 
@@ -2394,34 +2370,34 @@ string LyXText::currentState(LCursor & cur)
        LyXFont font = real_current_font;
        font.reduce(buf.params().getFont());
 
-       // avoid lyx::to_utf8(_(...)) re-entrance problem
+       // avoid to_utf8(_(...)) re-entrance problem
        string const s = font.stateText(&buf.params());
-       os << lyx::to_utf8(bformat(_("Font: %1$s"), lyx::from_utf8(s)));
+       os << to_utf8(bformat(_("Font: %1$s"), from_utf8(s)));
 
-       // os << lyx::to_utf8(bformat(_("Font: %1$s"), font.stateText(&buf.params)));
+       // os << to_utf8(bformat(_("Font: %1$s"), font.stateText(&buf.params)));
 
        // The paragraph depth
        int depth = cur.paragraph().getDepth();
        if (depth > 0)
-               os << lyx::to_utf8(bformat(_(", Depth: %1$d"), depth));
+               os << to_utf8(bformat(_(", Depth: %1$d"), depth));
 
        // The paragraph spacing, but only if different from
        // buffer spacing.
        Spacing const & spacing = par.params().spacing();
        if (!spacing.isDefault()) {
-               os << lyx::to_utf8(_(", Spacing: "));
+               os << to_utf8(_(", Spacing: "));
                switch (spacing.getSpace()) {
                case Spacing::Single:
-                       os << lyx::to_utf8(_("Single"));
+                       os << to_utf8(_("Single"));
                        break;
                case Spacing::Onehalf:
-                       os << lyx::to_utf8(_("OneHalf"));
+                       os << to_utf8(_("OneHalf"));
                        break;
                case Spacing::Double:
-                       os << lyx::to_utf8(_("Double"));
+                       os << to_utf8(_("Double"));
                        break;
                case Spacing::Other:
-                       os << lyx::to_utf8(_("Other (")) << spacing.getValueAsString() << ')';
+                       os << to_utf8(_("Other (")) << spacing.getValueAsString() << ')';
                        break;
                case Spacing::Default:
                        // should never happen, do nothing
@@ -2430,11 +2406,11 @@ string LyXText::currentState(LCursor & cur)
        }
 
 #ifdef DEVEL_VERSION
-       os << lyx::to_utf8(_(", Inset: ")) << &cur.inset();
-       os << lyx::to_utf8(_(", Paragraph: ")) << cur.pit();
-       os << lyx::to_utf8(_(", Id: ")) << par.id();
-       os << lyx::to_utf8(_(", Position: ")) << cur.pos();
-       os << lyx::to_utf8(_(", Boundary: ")) << cur.boundary();
+       os << to_utf8(_(", Inset: ")) << &cur.inset();
+       os << to_utf8(_(", Paragraph: ")) << cur.pit();
+       os << to_utf8(_(", Id: ")) << par.id();
+       os << to_utf8(_(", Position: ")) << cur.pos();
+       os << to_utf8(_(", Boundary: ")) << cur.boundary();
 //     Row & row = cur.textRow();
 //     os << bformat(_(", Row b:%1$d e:%2$d"), row.pos(), row.endpos());
 #endif
@@ -2461,13 +2437,11 @@ string LyXText::getPossibleLabel(LCursor & cur) const
        // for captions, we want the abbreviation of the float type
        if (layout->labeltype == LABEL_SENSITIVE) {
                // Search for the first float or wrap inset in the iterator
-               size_t i = cur.depth();
-               while (i > 0) {
-                       --i;
+               for (int i = cur.depth(); --i >= 0; ) {
                        InsetBase * const in = &cur[i].inset();
                        if (in->lyxCode() == InsetBase::FLOAT_CODE
                            || in->lyxCode() == InsetBase::WRAP_CODE) {
-                               name = in->getInsetName();
+                               name = to_utf8(in->getInsetName());
                                break;
                        }
                }
@@ -2482,7 +2456,7 @@ string LyXText::getPossibleLabel(LCursor & cur) const
                text.erase();
 
        // FIXME UNICODE
-       string par_text = lyx::to_utf8(pars_[pit].asString(cur.buffer(), false));
+       string par_text = to_utf8(pars_[pit].asString(cur.buffer(), false));
        for (int i = 0; i < lyxrc.label_init_length; ++i) {
                if (par_text.empty())
                        break;
@@ -2539,7 +2513,7 @@ bool LyXText::setCursorFromCoordinates(LCursor & cur, int const x, int const y)
 {
        BOOST_ASSERT(this == cur.text());
        pit_type pit = getPitNearY(y);
-       int yy = theCoords.get(this, pit).y_ - pars_[pit].ascent();
+       int yy = cur.bv().coordCache().get(this, pit).y_ - pars_[pit].ascent();
        lyxerr[Debug::DEBUG]
                << BOOST_CURRENT_FUNCTION
                << ": x: " << x
@@ -2577,3 +2551,6 @@ bool LyXText::setCursorFromCoordinates(LCursor & cur, int const x, int const y)
 
        return setCursor(cur, pit, pos, true, bound);
 }
+
+
+} // namespace lyx