]> git.lyx.org Git - lyx.git/blobdiff - src/Paragraph.cpp
PrefShortcuts: list all shortcuts (bound and unbound), using a better implementation
[lyx.git] / src / Paragraph.cpp
index a07f33eca7807fe6f508916e1a7cf98fd82a1a4d..104c31dac9974c1f1c4854ec4fc78cbce81d6432 100644 (file)
 
 #include "Buffer.h"
 #include "BufferParams.h"
+#include "Changes.h"
 #include "Counters.h"
 #include "Encoding.h"
 #include "debug.h"
 #include "gettext.h"
+#include "InsetList.h"
 #include "Language.h"
 #include "LaTeXFeatures.h"
 #include "Color.h"
+#include "Layout.h"
 #include "Length.h"
 #include "Font.h"
 #include "LyXRC.h"
-#include "Row.h"
 #include "Messages.h"
 #include "OutputParams.h"
 #include "output_latex.h"
 #include "paragraph_funcs.h"
 #include "ParagraphParameters.h"
-#include "rowpainter.h"
 #include "sgml.h"
 #include "TexRow.h"
 #include "VSpace.h"
@@ -67,8 +68,10 @@ using std::ostream;
 namespace lyx {
 
 using support::contains;
+using support::prefixIs;
 using support::suffixIs;
 using support::rsplit;
+using support::rtrim;
 
 
 /////////////////////////////////////////////////////////////////////
@@ -188,6 +191,11 @@ public:
                             unsigned int & column,
                             Font const & font,
                             Layout const & style);
+       /// Output consecutive known unicode chars, belonging to the same
+       /// language as specified by \p preamble, to \p os starting from \p c.
+       /// \return the number of characters written.
+       int knownLangChars(odocstream & os, value_type c, string & preamble,
+                          Change &, Encoding const &, pos_type &);
        ///
        void simpleTeXSpecialChars(Buffer const &, BufferParams const &,
                                   odocstream &,
@@ -223,6 +231,9 @@ public:
 
        /// Who owns us?
        Paragraph * owner_;
+
+       ///
+       InsetList insetlist_;
 };
 
 
@@ -271,6 +282,8 @@ Paragraph::Pimpl::Pimpl(Pimpl const & p, Paragraph * owner)
        inset_owner = p.inset_owner;
        fontlist = p.fontlist;
        id_ = paragraph_id++;
+       insetlist_ = p.insetlist_;
+       insetlist_.clone();
 }
 
 
@@ -455,7 +468,7 @@ void Paragraph::Pimpl::insertChar(pos_type pos, value_type c, Change const & cha
        }
 
        // Update the insets
-       owner_->insetlist.increasePosAfterPos(pos);
+       insetlist_.increasePosAfterPos(pos);
 }
 
 
@@ -468,8 +481,8 @@ void Paragraph::Pimpl::insertInset(pos_type pos, Inset * inset,
        insertChar(pos, META_INSET, change);
        BOOST_ASSERT(owner_->text_[pos] == META_INSET);
 
-       // Add a new entry in the insetlist.
-       owner_->insetlist.insert(inset, pos);
+       // Add a new entry in the insetlist_.
+       insetlist_.insert(inset, pos);
 }
 
 
@@ -508,7 +521,7 @@ bool Paragraph::Pimpl::eraseChar(pos_type pos, bool trackChanges)
 
        // if it is an inset, delete the inset entry
        if (owner_->text_[pos] == Paragraph::META_INSET) {
-               owner_->insetlist.erase(pos);
+               insetlist_.erase(pos);
        }
 
        owner_->text_.erase(owner_->text_.begin() + pos);
@@ -543,8 +556,8 @@ bool Paragraph::Pimpl::eraseChar(pos_type pos, bool trackChanges)
        for (; it != fend; ++it)
                it->pos(it->pos() - 1);
 
-       // Update the insetlist
-       owner_->insetlist.decreasePosAfterPos(pos);
+       // Update the insetlist_
+       insetlist_.decreasePosAfterPos(pos);
 
        return true;
 }
@@ -591,7 +604,7 @@ bool Paragraph::Pimpl::simpleTeXBlanks(Encoding const & encoding,
        if (style.pass_thru)
                return false;
 
-       if (i < size() - 1) {
+       if (i + 1 < size()) {
                char_type next = getChar(i + 1);
                if (Encodings::isCombiningChar(next)) {
                        // This space has an accent, so we must always output it.
@@ -605,7 +618,7 @@ bool Paragraph::Pimpl::simpleTeXBlanks(Encoding const & encoding,
            && column > lyxrc.plaintext_linelen
            && i
            && getChar(i - 1) != ' '
-           && (i < size() - 1)
+           && (i + 1 < size())
            // same in FreeSpacing mode
            && !owner_->isFreeSpacing()
            // In typewriter mode, we want to avoid
@@ -628,6 +641,66 @@ bool Paragraph::Pimpl::simpleTeXBlanks(Encoding const & encoding,
 }
 
 
+int Paragraph::Pimpl::knownLangChars(odocstream & os,
+                                    value_type c,
+                                    string & preamble,
+                                    Change & runningChange,
+                                    Encoding const & encoding,
+                                    pos_type & i)
+{
+       // When the character is marked by the proper language, we simply
+       // get its code point in some encoding, otherwise we get the
+       // translation specified in the unicodesymbols file, which is
+       // something like "\textLANG{<spec>}". So, we have to retain
+       // "\textLANG{<spec>" for the first char but only "<spec>" for
+       // all subsequent chars.
+       docstring const latex1 = rtrim(encoding.latexChar(c), "}");
+       int length = latex1.length();
+       os << latex1;
+       while (i + 1 < size()) {
+               char_type next = getChar(i + 1);
+               // Stop here if next character belongs to another
+               // language or there is a change tracking status.
+               if (!Encodings::isKnownLangChar(next, preamble) ||
+                   runningChange != lookupChange(i + 1))
+                       break;
+               Font prev_font;
+               bool found = false;
+               FontList::const_iterator cit = fontlist.begin();
+               FontList::const_iterator end = fontlist.end();
+               for (; cit != end; ++cit) {
+                       if (cit->pos() >= i && !found) {
+                               prev_font = cit->font();
+                               found = true;
+                       }
+                       if (cit->pos() >= i + 1)
+                               break;
+               }
+               // Stop here if there is a font attribute change.
+               if (found && cit != end && prev_font != cit->font())
+                       break;
+               docstring const latex = rtrim(encoding.latexChar(next), "}");
+               docstring::size_type const j =
+                                       latex.find_first_of(from_ascii("{"));
+               if (j == docstring::npos) {
+                       os << latex;
+                       length += latex.length();
+               } else {
+                       os << latex.substr(j + 1);
+                       length += latex.substr(j + 1).length();
+               }
+               ++i;
+       }
+       // When the proper language is set, we are simply passed a code
+       // point, so we should not try to close the \textLANG command.
+       if (prefixIs(latex1, from_ascii("\\" + preamble))) {
+               os << '}';
+               ++length;
+       }
+       return length;
+}
+
+
 bool Paragraph::Pimpl::isTextAt(string const & str, pos_type pos) const
 {
        pos_type const len = str.length();
@@ -696,7 +769,7 @@ void Paragraph::Pimpl::simpleTeXSpecialChars(Buffer const & buf,
                        break;
 
                // FIXME: move this to InsetNewline::latex
-               if (inset->lyxCode() == Inset::NEWLINE_CODE) {
+               if (inset->lyxCode() == NEWLINE_CODE) {
                        // newlines are handled differently here than
                        // the default in simpleTeXSpecialChars().
                        if (!style.newline_allowed) {
@@ -740,9 +813,9 @@ void Paragraph::Pimpl::simpleTeXSpecialChars(Buffer const & buf,
                bool close = false;
                odocstream::pos_type const len = os.tellp();
 
-               if ((inset->lyxCode() == Inset::GRAPHICS_CODE
-                    || inset->lyxCode() == Inset::MATH_CODE
-                    || inset->lyxCode() == Inset::URL_CODE)
+               if ((inset->lyxCode() == GRAPHICS_CODE
+                    || inset->lyxCode() == MATH_CODE
+                    || inset->lyxCode() == HYPERLINK_CODE)
                    && running_font.isRightToLeft()) {
                        if (running_font.language()->lang() == "farsi")
                                os << "\\beginL{}";
@@ -751,12 +824,10 @@ void Paragraph::Pimpl::simpleTeXSpecialChars(Buffer const & buf,
                        close = true;
                }
 
-#ifdef WITH_WARNINGS
-#warning Bug: we can have an empty font change here!
+// FIXME: Bug: we can have an empty font change here!
 // if there has just been a font change, we are going to close it
 // right now, which means stupid latex code like \textsf{}. AFAIK,
 // this does not harm dvi output. A minor bug, thus (JMarc)
-#endif
                // Some insets cannot be inside a font change command.
                // However, even such insets *can* be placed in \L or \R
                // or their equivalents (for RTL language switches), so we don't
@@ -947,7 +1018,7 @@ void Paragraph::Pimpl::simpleTeXSpecialChars(Buffer const & buf,
 
                        if (pnr == phrases_nr && c != '\0') {
                                Encoding const & encoding = *(runparams.encoding);
-                               if (i < size() - 1) {
+                               if (i + 1 < size()) {
                                        char_type next = getChar(i + 1);
                                        if (Encodings::isCombiningChar(next)) {
                                                column += latexSurrogatePair(os, c, next, encoding) - 1;
@@ -955,6 +1026,14 @@ void Paragraph::Pimpl::simpleTeXSpecialChars(Buffer const & buf,
                                                break;
                                        }
                                }
+                               string preamble;
+                               if (Encodings::isKnownLangChar(c, preamble)) {
+                                       column +=
+                                               knownLangChars(os, c, preamble,
+                                                       running_change,
+                                                       encoding, i) - 1;
+                                       break;
+                               }
                                docstring const latex = encoding.latexChar(c);
                                if (latex.length() > 1 &&
                                    latex[latex.length() - 1] != '}') {
@@ -1032,13 +1111,13 @@ void Paragraph::Pimpl::validate(LaTeXFeatures & features,
                features.require("ParagraphLeftIndent");
 
        // then the insets
-       InsetList::const_iterator icit = owner_->insetlist.begin();
-       InsetList::const_iterator iend = owner_->insetlist.end();
+       InsetList::const_iterator icit = insetlist_.begin();
+       InsetList::const_iterator iend = insetlist_.end();
        for (; icit != iend; ++icit) {
                if (icit->inset) {
                        icit->inset->validate(features);
                        if (layout.needprotect &&
-                           icit->inset->lyxCode() == Inset::FOOT_CODE)
+                           icit->inset->lyxCode() == FOOT_CODE)
                                features.require("NeedLyXFootnoteCode");
                }
        }
@@ -1077,16 +1156,11 @@ Paragraph::Paragraph()
 
 
 Paragraph::Paragraph(Paragraph const & par)
-       : itemdepth(par.itemdepth), insetlist(par.insetlist),
+       : itemdepth(par.itemdepth),
        layout_(par.layout_),
        text_(par.text_), begin_of_body_(par.begin_of_body_),
        pimpl_(new Paragraph::Pimpl(*par.pimpl_, this))
 {
-       //lyxerr << "Paragraph::Paragraph(Paragraph const&)" << endl;
-       InsetList::iterator it = insetlist.begin();
-       InsetList::iterator end = insetlist.end();
-       for (; it != end; ++it)
-               it->inset = it->inset->clone().release();
 }
 
 
@@ -1095,13 +1169,6 @@ Paragraph & Paragraph::operator=(Paragraph const & par)
        // needed as we will destroy the pimpl_ before copying it
        if (&par != this) {
                itemdepth = par.itemdepth;
-
-               insetlist = par.insetlist;
-               InsetList::iterator it = insetlist.begin();
-               InsetList::iterator end = insetlist.end();
-               for (; it != end; ++it)
-                       it->inset = it->inset->clone().release();
-
                layout_ = par.layout();
                text_ = par.text_;
                begin_of_body_ = par.begin_of_body_;
@@ -1288,21 +1355,10 @@ void Paragraph::insertInset(pos_type pos, Inset * inset,
        pimpl_->insertInset(pos, inset, change);
        // Set the font/language of the inset...
        setFont(pos, font);
-       // ... as well as the font/language of the text inside the inset
-       // FIXME: This is far from perfect. It basically overrides work being done
-       // in the InsetText constructor. Also, it doesn't work for Tables 
-       // (precisely because each cell's font/language is set in the Table's 
-       // constructor, so by now it's too late). The long-term solution should
-       // be moving current_font into Cursor, and getting rid of all this...
-       // (see http://thread.gmane.org/gmane.editors.lyx.devel/88869/focus=88944)
-       if (inset->asTextInset()) {
-               inset->asTextInset()->text_.current_font = font;
-               inset->asTextInset()->text_.real_current_font = font;
-       }
 }
 
 
-bool Paragraph::insetAllowed(Inset_code code)
+bool Paragraph::insetAllowed(InsetCode code)
 {
        return !pimpl_->inset_owner || pimpl_->inset_owner->insetAllowed(code);
 }
@@ -1380,18 +1436,14 @@ Font const Paragraph::getFont(BufferParams const & bparams, pos_type pos,
 {
        BOOST_ASSERT(pos >= 0);
 
-       Layout_ptr const & lout = layout();
+       Font font = getFontSettings(bparams, pos);
 
        pos_type const body_pos = beginOfBody();
-
-       Font layoutfont;
        if (pos < body_pos)
-               layoutfont = lout->labelfont;
+               font.realize(layout_->labelfont);
        else
-               layoutfont = lout->font;
+               font.realize(layout_->font);
 
-       Font font = getFontSettings(bparams, pos);
-       font.realize(layoutfont);
        font.realize(outerfont);
        font.realize(bparams.getFont());
 
@@ -1626,7 +1678,7 @@ docstring const & Paragraph::getLabelstring() const
 // the next two functions are for the manual labels
 docstring const Paragraph::getLabelWidthString() const
 {
-       if (!params().labelWidthString().empty())
+       if (layout()->margintype == MARGIN_MANUAL)
                return params().labelWidthString();
        else
                return _("Senseless with this layout!");
@@ -1653,7 +1705,7 @@ docstring const Paragraph::translateIfPossible(docstring const & s,
 }
 
 
-docstring Paragraph::expandLabel(Layout_ptr const & layout,
+docstring Paragraph::expandLabel(LayoutPtr const & layout,
                BufferParams const & bparams, bool process_appendix) const
 {
        TextClass const & tclass = bparams.getTextClass();
@@ -1665,6 +1717,10 @@ docstring Paragraph::expandLabel(Layout_ptr const & layout,
        else
                fmt = translateIfPossible(layout->labelstring(), bparams);
 
+       if (fmt.empty() && layout->labeltype == LABEL_COUNTER 
+           && !layout->counter.empty())
+               fmt = "\\the" + layout->counter;
+
        // handle 'inherited level parts' in 'fmt',
        // i.e. the stuff between '@' in   '@Section@.\arabic{subsection}'
        size_t const i = fmt.find('@', 0);
@@ -1672,8 +1728,12 @@ docstring Paragraph::expandLabel(Layout_ptr const & layout,
                size_t const j = fmt.find('@', i + 1);
                if (j != docstring::npos) {
                        docstring parent(fmt, i + 1, j - i - 1);
-                       docstring label = expandLabel(tclass[parent], bparams);
-                       fmt = docstring(fmt, 0, i) + label + docstring(fmt, j + 1, docstring::npos);
+                       docstring label = from_ascii("??");
+                       if (tclass.hasLayout(parent))
+                               docstring label = expandLabel(tclass[parent], bparams,
+                                                     process_appendix);
+                       fmt = docstring(fmt, 0, i) + label 
+                               + docstring(fmt, j + 1, docstring::npos);
                }
        }
 
@@ -1681,7 +1741,7 @@ docstring Paragraph::expandLabel(Layout_ptr const & layout,
 }
 
 
-void Paragraph::applyLayout(Layout_ptr const & new_layout)
+void Paragraph::applyLayout(LayoutPtr const & new_layout)
 {
        layout(new_layout);
        LyXAlignment const oldAlign = params().align();
@@ -1739,8 +1799,8 @@ void Paragraph::setBeginOfBody()
 int Paragraph::getPositionOfInset(Inset const * inset) const
 {
        // Find the entry.
-       InsetList::const_iterator it = insetlist.begin();
-       InsetList::const_iterator end = insetlist.end();
+       InsetList::const_iterator it = pimpl_->insetlist_.begin();
+       InsetList::const_iterator end = pimpl_->insetlist_.end();
        for (; it != end; ++it)
                if (it->inset == inset)
                        return it->pos;
@@ -1750,9 +1810,9 @@ int Paragraph::getPositionOfInset(Inset const * inset) const
 
 InsetBibitem * Paragraph::bibitem() const
 {
-       if (!insetlist.empty()) {
-               Inset * inset = insetlist.begin()->inset;
-               if (inset->lyxCode() == Inset::BIBITEM_CODE)
+       if (!pimpl_->insetlist_.empty()) {
+               Inset * inset = pimpl_->insetlist_.begin()->inset;
+               if (inset->lyxCode() == BIBITEM_CODE)
                        return static_cast<InsetBibitem *>(inset);
        }
        return 0;
@@ -1770,9 +1830,9 @@ namespace {
 // paragraphs inside floats need different alignment tags to avoid
 // unwanted space
 
-bool noTrivlistCentering(Inset::Code code)
+bool noTrivlistCentering(InsetCode code)
 {
-       return code == Inset::FLOAT_CODE || code == Inset::WRAP_CODE;
+       return code == FLOAT_CODE || code == WRAP_CODE;
 }
 
 
@@ -1789,7 +1849,7 @@ string correction(string const & orig)
 
 
 string const corrected_env(string const & suffix, string const & env,
-       Inset::Code code)
+       InsetCode code)
 {
        string output = suffix + "{";
        if (noTrivlistCentering(code))
@@ -1958,7 +2018,7 @@ bool Paragraph::simpleTeXOnePar(Buffer const & buf,
 
        bool return_value = false;
 
-       Layout_ptr style;
+       LayoutPtr style;
 
        // well we have to check if we are in an inset with unlimited
        // length (all in one row) if that is true then we don't allow
@@ -2168,11 +2228,9 @@ bool Paragraph::simpleTeXOnePar(Buffer const & buf,
                                        runparams, basefont, basefont);
                }
 #else
-#ifdef WITH_WARNINGS
-//#warning For now we ALWAYS have to close the foreign font settings if they are
-//#warning there as we start another \selectlanguage with the next paragraph if
-//#warning we are in need of this. This should be fixed sometime (Jug)
-#endif
+//FIXME: For now we ALWAYS have to close the foreign font settings if they are
+//FIXME: there as we start another \selectlanguage with the next paragraph if
+//FIXME: we are in need of this. This should be fixed sometime (Jug)
                running_font.latexWriteEndChanges(os, bparams, runparams,
                                basefont, basefont);
 #endif
@@ -2244,14 +2302,14 @@ bool Paragraph::emptyTag() const
        for (pos_type i = 0; i < size(); ++i) {
                if (isInset(i)) {
                        Inset const * inset = getInset(i);
-                       Inset::Code lyx_code = inset->lyxCode();
-                       if (lyx_code != Inset::TOC_CODE &&
-                           lyx_code != Inset::INCLUDE_CODE &&
-                           lyx_code != Inset::GRAPHICS_CODE &&
-                           lyx_code != Inset::ERT_CODE &&
-                           lyx_code != Inset::LISTINGS_CODE &&
-                           lyx_code != Inset::FLOAT_CODE &&
-                           lyx_code != Inset::TABULAR_CODE) {
+                       InsetCode lyx_code = inset->lyxCode();
+                       if (lyx_code != TOC_CODE &&
+                           lyx_code != INCLUDE_CODE &&
+                           lyx_code != GRAPHICS_CODE &&
+                           lyx_code != ERT_CODE &&
+                           lyx_code != LISTINGS_CODE &&
+                           lyx_code != FLOAT_CODE &&
+                           lyx_code != TABULAR_CODE) {
                                return false;
                        }
                } else {
@@ -2269,8 +2327,8 @@ string Paragraph::getID(Buffer const & buf, OutputParams const & runparams) cons
        for (pos_type i = 0; i < size(); ++i) {
                if (isInset(i)) {
                        Inset const * inset = getInset(i);
-                       Inset::Code lyx_code = inset->lyxCode();
-                       if (lyx_code == Inset::LABEL_CODE) {
+                       InsetCode lyx_code = inset->lyxCode();
+                       if (lyx_code == LABEL_CODE) {
                                string const id = static_cast<InsetCommand const *>(inset)->getContents();
                                return "id='" + to_utf8(sgml::cleanID(buf, runparams, from_utf8(id))) + "'";
                        }
@@ -2324,7 +2382,7 @@ void Paragraph::simpleDocBookOnePar(Buffer const & buf,
 {
        bool emph_flag = false;
 
-       Layout_ptr const & style = layout();
+       LayoutPtr const & style = layout();
        Font font_old =
                style->labeltype == LABEL_MANUAL ? style->labelfont : style->font;
 
@@ -2371,10 +2429,17 @@ void Paragraph::simpleDocBookOnePar(Buffer const & buf,
 }
 
 
+bool Paragraph::isHfill(pos_type pos) const
+{
+       return isInset(pos)
+               && getInset(pos)->lyxCode() == HFILL_CODE;
+}
+
+
 bool Paragraph::isNewline(pos_type pos) const
 {
        return isInset(pos)
-               && getInset(pos)->lyxCode() == Inset::NEWLINE_CODE;
+               && getInset(pos)->lyxCode() == NEWLINE_CODE;
 }
 
 
@@ -2404,19 +2469,17 @@ Paragraph::getParLanguage(BufferParams const & bparams) const
 {
        if (!empty())
                return getFirstFontSettings(bparams).language();
-#ifdef WITH_WARNINGS
-#warning FIXME we should check the prev par as well (Lgb)
-#endif
+       // FIXME: we should check the prev par as well (Lgb)
        return bparams.language;
 }
 
 
-bool Paragraph::isRightToLeftPar(BufferParams const & bparams) const
+bool Paragraph::isRTL(BufferParams const & bparams) const
 {
        return lyxrc.rtl_support
                && getParLanguage(bparams)->rightToLeft()
-               && ownerCode() != Inset::ERT_CODE
-               && ownerCode() != Inset::LISTINGS_CODE;
+               && ownerCode() != ERT_CODE
+               && ownerCode() != LISTINGS_CODE;
 }
 
 
@@ -2533,13 +2596,13 @@ int Paragraph::id() const
 }
 
 
-Layout_ptr const & Paragraph::layout() const
+LayoutPtr const & Paragraph::layout() const
 {
        return layout_;
 }
 
 
-void Paragraph::layout(Layout_ptr const & new_layout)
+void Paragraph::layout(LayoutPtr const & new_layout)
 {
        layout_ = new_layout;
 }
@@ -2551,10 +2614,10 @@ Inset * Paragraph::inInset() const
 }
 
 
-Inset::Code Paragraph::ownerCode() const
+InsetCode Paragraph::ownerCode() const
 {
        return pimpl_->inset_owner
-               ? pimpl_->inset_owner->lyxCode() : Inset::NO_CODE;
+               ? pimpl_->inset_owner->lyxCode() : NO_CODE;
 }
 
 
@@ -2577,7 +2640,7 @@ bool Paragraph::isFreeSpacing() const
 
        // for now we just need this, later should we need this in some
        // other way we can always add a function to Inset too.
-       return ownerCode() == Inset::ERT_CODE || ownerCode() == Inset::LISTINGS_CODE;
+       return ownerCode() == ERT_CODE || ownerCode() == LISTINGS_CODE;
 }
 
 
@@ -2585,7 +2648,7 @@ bool Paragraph::allowEmpty() const
 {
        if (layout()->keepempty)
                return true;
-       return ownerCode() == Inset::ERT_CODE || ownerCode() == Inset::LISTINGS_CODE;
+       return ownerCode() == ERT_CODE || ownerCode() == LISTINGS_CODE;
 }
 
 
@@ -2629,42 +2692,6 @@ char_type Paragraph::transformChar(char_type c, pos_type pos) const
 }
 
 
-bool Paragraph::hfillExpansion(Row const & row, pos_type pos) const
-{
-       if (!isHfill(pos))
-               return false;
-
-       BOOST_ASSERT(pos >= row.pos() && pos < row.endpos());
-
-       // expand at the end of a row only if there is another hfill on the same row
-       if (pos == row.endpos() - 1) {
-               for (pos_type i = row.pos(); i < pos; i++) {
-                       if (isHfill(i))
-                               return true;
-               }
-               return false;
-       }
-
-       // expand at the beginning of a row only if it is the first row of a paragraph
-       if (pos == row.pos()) {
-               return pos == 0;
-       }
-
-       // do not expand in some labels
-       if (layout()->margintype != MARGIN_MANUAL && pos < beginOfBody())
-               return false;
-
-       // if there is anything between the first char of the row and
-       // the specified position that is neither a newline nor an hfill,
-       // the hfill will be expanded, otherwise it won't
-       for (pos_type i = row.pos(); i < pos; i++) {
-               if (!isNewline(i) && !isHfill(i))
-                       return true;
-       }
-       return false;
-}
-
-
 int Paragraph::checkBiblio(bool track_changes)
 {
        //FIXME From JS:
@@ -2675,10 +2702,10 @@ int Paragraph::checkBiblio(bool track_changes)
        if (layout()->labeltype != LABEL_BIBLIO)
                return 0;
 
-       bool hasbibitem = !insetlist.empty()
+       bool hasbibitem = !pimpl_->insetlist_.empty()
                // Insist on it being in pos 0
                && getChar(0) == Paragraph::META_INSET
-               && insetlist.begin()->inset->lyxCode() == Inset::BIBITEM_CODE;
+               && pimpl_->insetlist_.begin()->inset->lyxCode() == BIBITEM_CODE;
 
        docstring oldkey;
        docstring oldlabel;
@@ -2689,10 +2716,10 @@ int Paragraph::checkBiblio(bool track_changes)
        // we're assuming there's only one of these, which there
        // should be.
        int erasedInsetPosition = -1;
-       InsetList::iterator it = insetlist.begin();
-       InsetList::iterator end = insetlist.end();
+       InsetList::iterator it = pimpl_->insetlist_.begin();
+       InsetList::iterator end = pimpl_->insetlist_.end();
        for (; it != end; ++it)
-               if (it->inset->lyxCode() == Inset::BIBITEM_CODE
+               if (it->inset->lyxCode() == BIBITEM_CODE
                    && it->pos > 0) {
                        InsetBibitem * olditem = static_cast<InsetBibitem *>(it->inset);
                        oldkey = olditem->getParam("key");
@@ -2711,7 +2738,7 @@ int Paragraph::checkBiblio(bool track_changes)
        //erase one. So we give its properties to the beginning inset.
        if (hasbibitem) {
                InsetBibitem * inset =
-                       static_cast<InsetBibitem *>(insetlist.begin()->inset);
+                       static_cast<InsetBibitem *>(pimpl_->insetlist_.begin()->inset);
                if (!oldkey.empty())
                        inset->setParam("key", oldkey);
                inset->setParam("label", oldlabel);
@@ -2737,4 +2764,49 @@ void Paragraph::checkAuthors(AuthorList const & authorList)
        pimpl_->changes_.checkAuthors(authorList);
 }
 
+
+bool Paragraph::isUnchanged(pos_type pos) const
+{
+       return lookupChange(pos).type == Change::UNCHANGED;
+}
+
+
+bool Paragraph::isInserted(pos_type pos) const
+{
+       return lookupChange(pos).type == Change::INSERTED;
+}
+
+
+bool Paragraph::isDeleted(pos_type pos) const
+{
+       return lookupChange(pos).type == Change::DELETED;
+}
+
+
+InsetList const & Paragraph::insetList() const
+{
+       return pimpl_->insetlist_;
+}
+
+
+Inset * Paragraph::releaseInset(pos_type pos)
+{
+       Inset * inset = pimpl_->insetlist_.release(pos);
+       /// does not honour change tracking!
+       eraseChar(pos, false);
+       return inset;
+}
+
+
+Inset * Paragraph::getInset(pos_type pos)
+{
+       return pimpl_->insetlist_.get(pos);
+}
+
+
+Inset const * Paragraph::getInset(pos_type pos) const
+{
+       return pimpl_->insetlist_.get(pos);
+}
+
 } // namespace lyx