]> git.lyx.org Git - lyx.git/blobdiff - src/paragraph_funcs.C
Point fix, earlier forgotten
[lyx.git] / src / paragraph_funcs.C
index 694c3a9a2bd30884323b8d04ee683d0f43af9e41..c13bf6e706ee48188bda7c738fdb1db24e894b57 100644 (file)
@@ -5,7 +5,7 @@
  *
  * \author Lars Gullik Bjønnes
  *
- * Full author contact details are available in file CREDITS
+ * Full author contact details are available in file CREDITS.
  */
 
 #include <config.h>
 #include "lyxlex.h"
 #include "factory.h"
 #include "Lsstream.h"
+
 #include "support/lstrings.h"
+#include "support/LAssert.h"
+
 #include "insets/insetoptarg.h"
 #include "insets/insetcommandparams.h"
 #include "insets/insetbibitem.h"
 #include "insets/insetlatexaccent.h"
 #include "insets/insettabular.h"
 #include "insets/insethfill.h"
-#include "insets/inseterror.h"
 #include "insets/insetnewline.h"
 
-extern string bibitemWidest(Buffer const *);
+extern string bibitemWidest(Buffer const &);
+
+using namespace lyx::support;
 
 using lyx::pos_type;
 //using lyx::layout_type;
@@ -43,6 +47,35 @@ using std::endl;
 using std::ostream;
 
 
+namespace {
+
+bool moveItem(Paragraph & from, Paragraph & to,
+       BufferParams const & params, pos_type i, pos_type j)
+{
+       char const tmpchar = from.getChar(i);
+       LyXFont tmpfont = from.getFontSettings(params, i);
+
+       if (tmpchar == Paragraph::META_INSET) {
+               InsetOld * tmpinset = 0;
+               if (from.getInset(i)) {
+                       // the inset is not in a paragraph anymore
+                       tmpinset = from.insetlist.release(i);
+               }
+
+               if (!to.insetAllowed(tmpinset->lyxCode()))
+                       return false;
+               to.insertInset(j, tmpinset, tmpfont);
+       } else {
+               if (!to.checkInsertChar(tmpfont))
+                       return false;
+               to.insertChar(j, tmpchar, tmpfont);
+       }
+       return true;
+}
+
+}
+
+
 void breakParagraph(BufferParams const & bparams,
                    ParagraphList & paragraphs,
                    ParagraphList::iterator par,
@@ -71,7 +104,7 @@ void breakParagraph(BufferParams const & bparams,
                tmp->setLabelWidthString(par->params().labelWidthString());
        }
 
-       bool const isempty = (par->layout()->keepempty && par->empty());
+       bool const isempty = (par->allowEmpty() && par->empty());
 
        if (!isempty && (par->size() > pos || par->empty() || flag == 2)) {
                tmp->layout(par->layout());
@@ -103,16 +136,15 @@ void breakParagraph(BufferParams const & bparams,
                pos_type j = pos;
 
                for (; i <= pos_end; ++i) {
-                       Change::Type change(par->lookupChange(i));
-                       par->cutIntoMinibuffer(bparams, i);
-                       if (tmp->insertFromMinibuffer(j - pos)) {
+                       Change::Type change = par->lookupChange(i);
+                       if (moveItem(*par, *tmp, bparams, i, j - pos)) {
                                tmp->setChange(j - pos, change);
                                ++j;
                        }
                }
-               for (i = pos_end; i >= pos; --i) {
+
+               for (i = pos_end; i >= pos; --i)
                        par->eraseIntern(i);
-               }
        }
 
        if (pos)
@@ -160,15 +192,12 @@ void breakParagraphConservative(BufferParams const & bparams,
                // paragraph
                pos_type pos_end = par->size() - 1;
 
-               for (pos_type i = pos, j = pos; i <= pos_end; ++i) {
-                       par->cutIntoMinibuffer(bparams, i);
-                       if (tmp->insertFromMinibuffer(j - pos))
+               for (pos_type i = pos, j = pos; i <= pos_end; ++i)
+                       if (moveItem(*par, *tmp, bparams, i, j - pos))
                                ++j;
-               }
 
-               for (pos_type k = pos_end; k >= pos; --k) {
+               for (pos_type k = pos_end; k >= pos; --k)
                        par->erase(k);
-               }
        }
 }
 
@@ -188,11 +217,9 @@ void mergeParagraph(BufferParams const & bparams,
        pos_type pos_insert = par->size();
 
        // ok, now copy the paragraph
-       for (pos_type i = 0, j = 0; i <= pos_end; ++i) {
-               the_next->cutIntoMinibuffer(bparams, i);
-               if (par->insertFromMinibuffer(pos_insert + j))
+       for (pos_type i = 0, j = 0; i <= pos_end; ++i)
+               if (moveItem(*the_next, *par, bparams, i, pos_insert + j))
                        ++j;
-       }
 
        paragraphs.erase(the_next);
 }
@@ -203,12 +230,12 @@ ParagraphList::iterator depthHook(ParagraphList::iterator pit,
                                  Paragraph::depth_type depth)
 {
        ParagraphList::iterator newpit = pit;
-       ParagraphList::iterator beg = plist.begin();
+       ParagraphList::iterator beg = const_cast<ParagraphList&>(plist).begin();
 
        if (newpit != beg)
                --newpit;
 
-       while (newpit !=  beg && newpit->getDepth() > depth) {
+       while (newpit != beg && newpit->getDepth() > depth) {
                --newpit;
        }
 
@@ -223,7 +250,7 @@ ParagraphList::iterator outerHook(ParagraphList::iterator pit,
                                  ParagraphList const & plist)
 {
        if (!pit->getDepth())
-               return plist.end();
+               return const_cast<ParagraphList&>(plist).end();
        return depthHook(pit, plist,
                         Paragraph::depth_type(pit->getDepth() - 1));
 }
@@ -243,12 +270,12 @@ int getEndLabel(ParagraphList::iterator p, ParagraphList const & plist)
 {
        ParagraphList::iterator pit = p;
        Paragraph::depth_type par_depth = p->getDepth();
-       while (pit != plist.end()) {
+       while (pit != const_cast<ParagraphList&>(plist).end()) {
                LyXLayout_ptr const & layout = pit->layout();
                int const endlabeltype = layout->endlabeltype;
 
                if (endlabeltype != END_LABEL_NO_LABEL) {
-                       if (boost::next(p) == plist.end())
+                       if (boost::next(p) == const_cast<ParagraphList&>(plist).end())
                                return endlabeltype;
 
                        Paragraph::depth_type const next_depth = boost::next(p)->getDepth();
@@ -261,7 +288,7 @@ int getEndLabel(ParagraphList::iterator p, ParagraphList const & plist)
                if (par_depth == 0)
                        break;
                pit = outerHook(pit, plist);
-               if (pit != plist.end())
+               if (pit != const_cast<ParagraphList&>(plist).end())
                        par_depth = pit->getDepth();
        }
        return END_LABEL_NO_LABEL;
@@ -271,14 +298,14 @@ int getEndLabel(ParagraphList::iterator p, ParagraphList const & plist)
 namespace {
 
 ParagraphList::iterator
-TeXEnvironment(Buffer const * buf,
+TeXEnvironment(Buffer const & buf,
               ParagraphList const & paragraphs,
               ParagraphList::iterator pit,
               ostream & os, TexRow & texrow,
               LatexRunParams const & runparams);
 
 ParagraphList::iterator
-TeXOnePar(Buffer const * buf,
+TeXOnePar(Buffer const & buf,
          ParagraphList const & paragraphs,
          ParagraphList::iterator pit,
          ostream & os, TexRow & texrow,
@@ -287,7 +314,7 @@ TeXOnePar(Buffer const * buf,
 
 
 ParagraphList::iterator
-TeXDeeper(Buffer const * buf,
+TeXDeeper(Buffer const & buf,
          ParagraphList const & paragraphs,
          ParagraphList::iterator pit,
          ostream & os, TexRow & texrow,
@@ -296,7 +323,7 @@ TeXDeeper(Buffer const * buf,
        lyxerr[Debug::LATEX] << "TeXDeeper...     " << &*pit << endl;
        ParagraphList::iterator par = pit;
 
-       while (par != paragraphs.end() &&
+       while (par != const_cast<ParagraphList&>(paragraphs).end() &&
                     par->params().depth() == pit->params().depth()) {
                if (par->layout()->isEnvironment()) {
                        par = TeXEnvironment(buf, paragraphs, par,
@@ -313,7 +340,7 @@ TeXDeeper(Buffer const * buf,
 
 
 ParagraphList::iterator
-TeXEnvironment(Buffer const * buf,
+TeXEnvironment(Buffer const & buf,
               ParagraphList const & paragraphs,
               ParagraphList::iterator pit,
               ostream & os, TexRow & texrow,
@@ -321,14 +348,14 @@ TeXEnvironment(Buffer const * buf,
 {
        lyxerr[Debug::LATEX] << "TeXEnvironment...     " << &*pit << endl;
 
-       BufferParams const & bparams = buf->params;
+       BufferParams const & bparams = buf.params;
 
        LyXLayout_ptr const & style = pit->layout();
 
        Language const * language = pit->getParLanguage(bparams);
        Language const * doc_language = bparams.language;
        Language const * previous_language =
-               (pit != paragraphs.begin())
+               (pit != const_cast<ParagraphList&>(paragraphs).begin())
                ? boost::prior(pit)->getParLanguage(bparams)
                : doc_language;
        if (language->babel() != previous_language->babel()) {
@@ -379,7 +406,7 @@ TeXEnvironment(Buffer const * buf,
        do {
                par = TeXOnePar(buf, paragraphs, par, os, texrow, runparams);
 
-               if (par != paragraphs.end()&& par->params().depth() > pit->params().depth()) {
+               if (par != const_cast<ParagraphList&>(paragraphs).end() && par->params().depth() > pit->params().depth()) {
                            if (par->layout()->isParagraph()) {
 
                            // Thinko!
@@ -402,7 +429,7 @@ TeXEnvironment(Buffer const * buf,
                        par = TeXDeeper(buf, paragraphs, par, os, texrow,
                                        runparams);
                }
-       } while (par != paragraphs.end()
+       } while (par != const_cast<ParagraphList&>(paragraphs).end()
                 && par->layout() == pit->layout()
                 && par->params().depth() == pit->params().depth()
                 && par->params().leftIndent() == pit->params().leftIndent());
@@ -425,11 +452,11 @@ TeXEnvironment(Buffer const * buf,
 InsetOptArg * optArgInset(Paragraph const & par)
 {
        // Find the entry.
-       InsetList::iterator it = par.insetlist.begin();
-       InsetList::iterator end = par.insetlist.end();
+       InsetList::const_iterator it = par.insetlist.begin();
+       InsetList::const_iterator end = par.insetlist.end();
        for (; it != end; ++it) {
-               Inset * ins = it.getInset();
-               if (ins->lyxCode() == Inset::OPTARG_CODE) {
+               InsetOld * ins = it->inset;
+               if (ins->lyxCode() == InsetOld::OPTARG_CODE) {
                        return static_cast<InsetOptArg *>(ins);
                }
        }
@@ -438,26 +465,26 @@ InsetOptArg * optArgInset(Paragraph const & par)
 
 
 ParagraphList::iterator
-TeXOnePar(Buffer const * buf,
+TeXOnePar(Buffer const & buf,
          ParagraphList const & paragraphs,
          ParagraphList::iterator pit,
          ostream & os, TexRow & texrow,
          LatexRunParams const & runparams,
          string const & everypar)
 {
-       lyxerr[Debug::LATEX] << "TeXOnePar...     " << &*pit << " '" << everypar
-<< "'" << endl;
-       BufferParams const & bparams = buf->params;
+       lyxerr[Debug::LATEX] << "TeXOnePar...     " << &*pit << " '"
+               << everypar << "'" << endl;
+       BufferParams const & bparams = buf.params;
 
-       Inset const * in = pit->inInset();
+       InsetOld const * in = pit->inInset();
        bool further_blank_line = false;
        LyXLayout_ptr style;
 
        // well we have to check if we are in an inset with unlimited
-       // lenght (all in one row) if that is true then we don't allow
+       // length (all in one row) if that is true then we don't allow
        // any special options in the paragraph and also we don't allow
        // any environment other then "Standard" to be valid!
-       if ((in == 0) || !in->forceDefaultParagraphs(in)) {
+       if (in == 0 || !in->forceDefaultParagraphs(in)) {
                style = pit->layout();
 
                if (pit->params().startOfAppendix()) {
@@ -466,7 +493,7 @@ TeXOnePar(Buffer const * buf,
                }
 
                if (!pit->params().spacing().isDefault()
-                       && (pit == paragraphs.begin() || !boost::prior(pit)->hasSameLayout(*pit))) {
+                       && (pit == const_cast<ParagraphList&>(paragraphs).begin() || !boost::prior(pit)->hasSameLayout(*pit))) {
                        os << pit->params().spacing().writeEnvirBegin() << '\n';
                        texrow.newline();
                }
@@ -504,14 +531,14 @@ TeXOnePar(Buffer const * buf,
        Language const * language = pit->getParLanguage(bparams);
        Language const * doc_language = bparams.language;
        Language const * previous_language =
-               (pit != paragraphs.begin())
+               (pit != const_cast<ParagraphList&>(paragraphs).begin())
                ? boost::prior(pit)->getParLanguage(bparams)
                : doc_language;
 
        if (language->babel() != previous_language->babel()
            // check if we already put language command in TeXEnvironment()
            && !(style->isEnvironment()
-                && (pit == paragraphs.begin() ||
+                && (pit == const_cast<ParagraphList&>(paragraphs).begin() ||
                     (boost::prior(pit)->layout() != pit->layout() &&
                      boost::prior(pit)->getDepth() <= pit->getDepth())
                     || boost::prior(pit)->getDepth() < pit->getDepth())))
@@ -591,7 +618,7 @@ TeXOnePar(Buffer const * buf,
        bool is_command = style->isCommand();
 
        if (style->resfont.size() != font.size()
-           && boost::next(pit) != paragraphs.end()
+           && boost::next(pit) != const_cast<ParagraphList&>(paragraphs).end()
            && !is_command) {
                if (!need_par)
                        os << '{';
@@ -604,29 +631,33 @@ TeXOnePar(Buffer const * buf,
        switch (style->latextype) {
        case LATEX_ITEM_ENVIRONMENT:
        case LATEX_LIST_ENVIRONMENT:
-               if (boost::next(pit) != paragraphs.end()
+               if (boost::next(pit) != const_cast<ParagraphList&>(paragraphs).end()
                    && (pit->params().depth() < boost::next(pit)->params().depth())) {
                        os << '\n';
                        texrow.newline();
                }
                break;
-       case LATEX_ENVIRONMENT:
+       case LATEX_ENVIRONMENT: {
                // if its the last paragraph of the current environment
                // skip it otherwise fall through
-               if (boost::next(pit) != paragraphs.end()
-                   && (boost::next(pit)->layout() != pit->layout()
-                       || boost::next(pit)->params().depth() != pit->params().depth()))
+               ParagraphList::iterator next = boost::next(pit);
+
+               if (next != const_cast<ParagraphList&>(paragraphs).end()
+                   && (next->layout() != pit->layout()
+                       || next->params().depth() != pit->params().depth()))
                        break;
+       }
+
                // fall through possible
        default:
                // we don't need it for the last paragraph!!!
-               if (boost::next(pit) != paragraphs.end()) {
+               if (boost::next(pit) != const_cast<ParagraphList&>(paragraphs).end()) {
                        os << '\n';
                        texrow.newline();
                }
        }
 
-       if ((in == 0) || !in->forceDefaultParagraphs(in)) {
+       if (in == 0 || !in->forceDefaultParagraphs(in)) {
                further_blank_line = false;
                if (pit->params().lineBottom()) {
                        os << "\\lyxline{\\" << font.latexSize() << '}';
@@ -649,14 +680,14 @@ TeXOnePar(Buffer const * buf,
                }
 
                if (!pit->params().spacing().isDefault()
-                       && (boost::next(pit) == paragraphs.end()|| !boost::next(pit)->hasSameLayout(*pit))) {
+                       && (boost::next(pit) == const_cast<ParagraphList&>(paragraphs).end()|| !boost::next(pit)->hasSameLayout(*pit))) {
                        os << pit->params().spacing().writeEnvirEnd() << '\n';
                        texrow.newline();
                }
        }
 
        // we don't need it for the last paragraph!!!
-       if (boost::next(pit) != paragraphs.end()) {
+       if (boost::next(pit) != const_cast<ParagraphList&>(paragraphs).end()) {
                os << '\n';
                texrow.newline();
        } else {
@@ -689,7 +720,7 @@ TeXOnePar(Buffer const * buf,
 //
 // LaTeX all paragraphs from par to endpar, if endpar == 0 then to the end
 //
-void latexParagraphs(Buffer const * buf,
+void latexParagraphs(Buffer const & buf,
                     ParagraphList const & paragraphs,
                     ostream & os,
                     TexRow & texrow,
@@ -698,18 +729,18 @@ void latexParagraphs(Buffer const * buf,
 {
        bool was_title = false;
        bool already_title = false;
-       LyXTextClass const & tclass = buf->params.getLyXTextClass();
-       ParagraphList::iterator par = paragraphs.begin();
-       ParagraphList::iterator endpar = paragraphs.end();
+       LyXTextClass const & tclass = buf.params.getLyXTextClass();
+       ParagraphList::iterator par = const_cast<ParagraphList&>(paragraphs).begin();
+       ParagraphList::iterator endpar = const_cast<ParagraphList&>(paragraphs).end();
 
        // if only_body
        while (par != endpar) {
-               Inset * in = par->inInset();
+               InsetOld * in = par->inInset();
                // 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
                // any special options in the paragraph and also we don't allow
                // any environment other then "Standard" to be valid!
-               if ((in == 0) || !in->forceDefaultParagraphs(in)) {
+               if (in == 0 || !in->forceDefaultParagraphs(in)) {
                        LyXLayout_ptr const & layout = par->layout();
 
                        if (layout->intitle) {
@@ -786,7 +817,7 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                for (; cit != token.end(); ++cit) {
                        par.insertChar(par.size(), (*cit), font, change);
                }
-       } else if (token == "\\layout") {
+       } else if (token == "\\begin_layout") {
                lex.eatLine();
                string layoutname = lex.getString();
 
@@ -810,81 +841,24 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                        layoutname = tclass.defaultLayoutName();
                }
 
-#ifdef USE_CAPTION
-               // The is the compability reading of layout caption.
-               if (compare_ascii_no_case(layoutname, "caption") == 0) {
-                       // We expect that the par we are now working on is
-                       // really inside a InsetText inside a InsetFloat.
-                       // We also know that captions can only be
-                       // one paragraph. (Lgb)
-
-                       // We should now read until the next "\layout"
-                       // is reached.
-                       // This is probably not good enough, what if the
-                       // caption is the last par in the document (Lgb)
-                       istream & ist = lex.getStream();
-                       stringstream ss;
-                       string line;
-                       int begin = 0;
-                       while (true) {
-                               getline(ist, line);
-                               if (prefixIs(line, "\\layout")) {
-                                       lex.pushToken(line);
-                                       break;
-                               }
-                               if (prefixIs(line, "\\begin_inset"))
-                                       ++begin;
-                               if (prefixIs(line, "\\end_inset")) {
-                                       if (begin)
-                                               --begin;
-                                       else {
-                                               lex.pushToken(line);
-                                               break;
-                                       }
-                               }
-
-                               ss << line << '\n';
-                       }
-
-                       // Now we should have the whole layout in ss
-                       // we should now be able to give this to the
-                       // caption inset.
-                       ss << "\\end_inset\n";
-
-                       // This seems like a bug in stringstream.
-                       // We really should be able to use ss
-                       // directly. (Lgb)
-                       istringstream is(ss.str());
-                       LyXLex tmplex(0, 0);
-                       tmplex.setStream(is);
-                       Inset * inset = new InsetCaption;
-                       inset->Read(this, tmplex);
-                       par.insertInset(pos, inset, font);
-                       ++pos;
-               } else {
-#endif
-                       par.layout(bp.getLyXTextClass()[layoutname]);
+               par.layout(bp.getLyXTextClass()[layoutname]);
 
-                       // Test whether the layout is obsolete.
-                       LyXLayout_ptr const & layout = par.layout();
-                       if (!layout->obsoleted_by().empty())
-                               par.layout(bp.getLyXTextClass()[layout->obsoleted_by()]);
+               // Test whether the layout is obsolete.
+               LyXLayout_ptr const & layout = par.layout();
+               if (!layout->obsoleted_by().empty())
+                       par.layout(bp.getLyXTextClass()[layout->obsoleted_by()]);
 
-                       par.params().read(lex);
-#if USE_CAPTION
-               }
-#endif
+               par.params().read(lex);
 
+       } else if (token == "\\end_layout") {
+               lyxerr << "Solitary \\end_layout in line " << lex.getLineNo() << "\n"
+                      << "Missing \\begin_layout?.\n";
        } else if (token == "\\end_inset") {
                lyxerr << "Solitary \\end_inset in line " << lex.getLineNo() << "\n"
                       << "Missing \\begin_inset?.\n";
-               // Simply ignore this. The insets do not have
-               // to read this.
-               // But insets should read it, it is a part of
-               // the inset isn't it? Lgb.
        } else if (token == "\\begin_inset") {
-               Inset * i = readInset(lex, buf);
-               par.insertInset(par.size(), i, font, change);
+               InsetOld * inset = readInset(lex, buf);
+               par.insertInset(par.size(), inset, font, change);
        } else if (token == "\\family") {
                lex.next();
                font.setLyXFamily(lex.getString());
@@ -933,10 +907,9 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                lex.next();
                font.setLyXColor(lex.getString());
        } else if (token == "\\InsetSpace" || token == "\\SpecialChar") {
-               LyXLayout_ptr const & layout = par.layout();
 
                // Insets don't make sense in a free-spacing context! ---Kayvan
-               if (layout->free_spacing || par.isFreeSpacing()) {
+               if (par.isFreeSpacing()) {
                        if (token == "\\InsetSpace")
                                par.insertChar(par.size(), ' ', font, change);
                        else if (lex.isOK()) {
@@ -951,35 +924,35 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                                }
                        }
                } else {
-                       Inset * inset = 0;
+                       InsetOld * inset = 0;
                        if (token == "\\SpecialChar" )
                                inset = new InsetSpecialChar;
                        else
                                inset = new InsetSpace;
-                       inset->read(&buf, lex);
+                       inset->read(buf, lex);
                        par.insertInset(par.size(), inset, font, change);
                }
        } else if (token == "\\i") {
-               Inset * inset = new InsetLatexAccent;
-               inset->read(&buf, lex);
+               InsetOld * inset = new InsetLatexAccent;
+               inset->read(buf, lex);
                par.insertInset(par.size(), inset, font, change);
        } else if (token == "\\backslash") {
                par.insertChar(par.size(), '\\', font, change);
        } else if (token == "\\newline") {
-               Inset * inset = new InsetNewline;
-               inset->read(&buf, lex);
+               InsetOld * inset = new InsetNewline;
+               inset->read(buf, lex);
                par.insertInset(par.size(), inset, font, change);
        } else if (token == "\\LyXTable") {
-               Inset * inset = new InsetTabular(buf);
-               inset->read(&buf, lex);
+               InsetOld * inset = new InsetTabular(buf);
+               inset->read(buf, lex);
                par.insertInset(par.size(), inset, font, change);
        } else if (token == "\\bibitem") {
                InsetCommandParams p("bibitem", "dummy");
                InsetBibitem * inset = new InsetBibitem(p);
-               inset->read(&buf, lex);
+               inset->read(buf, lex);
                par.insertInset(par.size(), inset, font, change);
        } else if (token == "\\hfill") {
-               par.insertInset(par.size(), new InsetHFill(), font, change);
+               par.insertInset(par.size(), new InsetHFill, font, change);
        } else if (token == "\\change_unchanged") {
                // Hack ! Needed for empty paragraphs :/
                // FIXME: is it still ??
@@ -1004,9 +977,9 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                lex.eatLine();
                string const s = bformat(_("Unknown token: %1$s %2$s\n"),
                        token, lex.getString());
-               // we can do this here this way because we're actually reading
-               // the buffer and don't care about LyXText right now.
-               par.insertInset(par.size(), new InsetError(s), font);
+
+               buf.error(ErrorItem(_("Unknown token"), s,
+                                   par.id(), 0, par.size()));
                return 1;
        }
        return 0;
@@ -1032,16 +1005,20 @@ int readParagraph(Buffer & buf, Paragraph & par, LyXLex & lex)
                if (token.empty())
                        continue;
 
+               if (token == "\\end_layout") {
+                       //Ok, paragraph finished
+                       break;
+               }
+
                lyxerr[Debug::PARSER] << "Handling paragraph token: `"
                                      << token << '\'' << endl;
-
-               // reached the next paragraph. FIXME: really we should
-               // change the file format to indicate the end of a par
-               // clearly, but for now, this hack will do
-               if (token == "\\layout" || token == "\\the_end"
+               if (token == "\\begin_layout" || token == "\\end_document"
                    || token == "\\end_inset" || token == "\\begin_deeper"
                    || token == "\\end_deeper") {
                        lex.pushToken(token);
+                       lyxerr << "Paragraph ended in line "
+                              << lex.getLineNo() << "\n"
+                              << "Missing \\end_layout.\n";
                        break;
                }
        }
@@ -1057,10 +1034,10 @@ LyXFont const outerFont(ParagraphList::iterator pit,
        LyXFont tmpfont(LyXFont::ALL_INHERIT);
 
        // Resolve against environment font information
-       while (pit != plist.end() &&
+       while (pit != const_cast<ParagraphList&>(plist).end() &&
               par_depth && !tmpfont.resolved()) {
                pit = outerHook(pit, plist);
-               if (pit != plist.end()) {
+               if (pit != const_cast<ParagraphList&>(plist).end()) {
                        tmpfont.realize(pit->layout()->font);
                        par_depth = pit->getDepth();
                }
@@ -1070,11 +1047,48 @@ LyXFont const outerFont(ParagraphList::iterator pit,
 }
 
 
-LyXFont const realizeFont(LyXFont const & font,
-                         BufferParams const & params)
+ParagraphList::iterator outerPar(Buffer const & buf, InsetOld const * inset)
 {
-       LyXTextClass const & tclass = params.getLyXTextClass();
-       LyXFont tmpfont(font);
-       tmpfont.realize(tclass.defaultfont());
-       return tmpfont;
+       ParIterator pit = const_cast<Buffer &>(buf).par_iterator_begin();
+       ParIterator end = const_cast<Buffer &>(buf).par_iterator_end();
+       for ( ; pit != end; ++pit) {
+
+               ParagraphList * plist;
+               // the second '=' below is intentional
+               for (int i = 0; (plist = inset->getParagraphs(i)); ++i)
+                       if (plist == &pit.plist())
+                               return pit.outerPar();
+
+               InsetList::iterator ii = pit->insetlist.begin();
+               InsetList::iterator iend = pit->insetlist.end();
+               for ( ; ii != iend; ++ii)
+                       if (ii->inset == inset)
+                               return pit.outerPar();
+       }
+       lyxerr << "outerPar: should not happen" << endl;
+       Assert(false);
+       return const_cast<Buffer &>(buf).paragraphs.end(); // shut up compiler
+}
+
+
+Paragraph const & ownerPar(Buffer const & buf, InsetOld const * inset)
+{
+       ParConstIterator pit = buf.par_iterator_begin();
+       ParConstIterator end = buf.par_iterator_end();
+       for ( ; pit != end; ++pit) {
+               ParagraphList * plist;
+               // the second '=' below is intentional
+               for (int i = 0; (plist = inset->getParagraphs(i)); ++i)
+                       if (plist == &pit.plist())
+                               return *pit.pit();
+
+               InsetList::const_iterator ii = pit->insetlist.begin();
+               InsetList::const_iterator iend = pit->insetlist.end();
+               for ( ; ii != iend; ++ii)
+                       if (ii->inset == inset)
+                               return *pit.pit();
+       }
+       lyxerr << "ownerPar: should not happen" << endl;
+       Assert(false);
+       return buf.paragraphs.front(); // shut up compiler
 }