]> git.lyx.org Git - lyx.git/blobdiff - src/buffer.C
fix the <...> delimiters
[lyx.git] / src / buffer.C
index 9f4d9ef31741177bf51c98c501c7f08b26f6d14e..70aaad0b9584927f70782934654c3220c0fdb582 100644 (file)
@@ -31,7 +31,7 @@
 #include "version.h"
 #include "LaTeX.h"
 #include "Chktex.h"
-#include "LyXView.h"
+#include "frontends/LyXView.h"
 #include "debug.h"
 #include "LaTeXFeatures.h"
 #include "lyxtext.h"
 #include <locale>
 #endif
 
+#ifndef CXX_GLOBAL_CSTD
+using std::pow;
+#endif
 
 using std::ostream;
 using std::ofstream;
@@ -284,10 +287,11 @@ string last_inset_read;
 #ifndef NO_COMPABILITY
 struct ErtComp
 {
-       ErtComp() : active(false), in_tabular(false) {
+       ErtComp() : active(false), fromlayout(false), in_tabular(false) {
        }
        string contents;
        bool active;
+       bool fromlayout;
        bool in_tabular;
        LyXFont font;
 };
@@ -320,6 +324,7 @@ bool Buffer::readLyXformat2(LyXLex & lex, Paragraph * par)
 #ifndef NO_COMPABILITY
        ert_comp.contents.erase();
        ert_comp.active = false;
+       ert_comp.fromlayout = false;
        ert_comp.in_tabular = false;
 #endif
        int pos = 0;
@@ -328,12 +333,15 @@ bool Buffer::readLyXformat2(LyXLex & lex, Paragraph * par)
 
        Paragraph * first_par = 0;
        LyXFont font(LyXFont::ALL_INHERIT, params.language);
+
+#if 0
        if (file_format < 216 && params.language->lang() == "hebrew")
                font.setLanguage(default_language);
+#endif
 
        if (!par) {
                par = new Paragraph;
-               par->layout(textclasslist[params.textclass].defaultLayoutName());
+               par->layout(textclasslist[params.textclass].defaultLayout());
        } else {
                // We are inserting into an existing document
                users->text->breakParagraph(users);
@@ -401,16 +409,57 @@ bool Buffer::readLyXformat2(LyXLex & lex, Paragraph * par)
 
 
 #ifndef NO_COMPABILITY
+
+Inset * Buffer::isErtInset(Paragraph * par, int pos) const
+{
+       Inset * inset;
+       if ((par->getChar(pos) == Paragraph::META_INSET) &&
+               (inset = par->getInset(pos)) &&
+               (inset->lyxCode() == Inset::ERT_CODE))
+       {
+               return inset;
+       }
+       return 0;
+}
+
 void Buffer::insertErtContents(Paragraph * par, int & pos, bool set_inactive)
 {
-       if (!ert_comp.contents.empty()) {
+       if (ert_comp.contents.find_first_not_of(' ') != string::npos) {
+               // we only skip completely empty ERT (only spaces) otherwise
+               // we have to insert it as is.
+               string str(ert_comp.contents);
                lyxerr[Debug::INSETS] << "ERT contents:\n'"
-                                     << ert_comp.contents << "'" << endl;
-               Inset * inset = new InsetERT(params, params.language,
-                                            ert_comp.contents, true);
-               par->insertInset(pos++, inset, ert_comp.font);
-               ert_comp.contents.erase();
+                       << str << "'" << endl;
+               // check if we have already an ert inset a position earlier
+               // if this is the case then we should insert the contents
+               // inside the other ertinset as it is stupid to have various
+               // ert in a row.
+               Inset * inset;
+               if ((pos > 0) && (inset=isErtInset(par, pos-1))) {
+                       // get the last paragraph from the inset before
+                       Paragraph * last = inset->firstParagraph();
+                       while(last->next())
+                               last = last->next();
+                       // create the new paragraph after the last one
+                       Paragraph * par = new Paragraph(last);
+                       par->layout(textclasslist[params.textclass].defaultLayoutName());
+                       par->setInsetOwner(last->inInset());
+                       // set the contents
+                       LyXFont font(LyXFont::ALL_INHERIT, params.language);
+                       string::const_iterator cit = str.begin();
+                       string::const_iterator end = str.end();
+                       pos_type pos = 0;
+                       for (; cit != end; ++cit) {
+                               par->insertChar(pos++, *cit, font);
+                       }
+               } else {
+                       Inset * inset =
+                               new InsetERT(params, params.language, str, true);
+                       par->insertInset(pos++, inset, ert_comp.font);
+               }
        }
+       ert_comp.contents.erase();
+       ert_comp.fromlayout = false;
        if (set_inactive) {
                ert_comp.active = false;
        }
@@ -438,6 +487,12 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
        static Paragraph * parBeforeMinipage;
 #endif
 #endif
+
+       // The order of the tags tested may seem unnatural, but this
+       // has been done in order to reduce the number of string
+       // comparisons needed to recognize a given token. This leads
+       // on large documents like UserGuide to a reduction of a
+       // factor 5! (JMarc)
        if (token[0] != '\\') {
 #ifndef NO_COMPABILITY
                if (ert_comp.active) {
@@ -452,17 +507,33 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
 #ifndef NO_COMPABILITY
                }
 #endif
-       } else if (token == "\\i") {
-               Inset * inset = new InsetLatexAccent;
-               inset->read(this, lex);
-               par->insertInset(pos, inset, font);
-               ++pos;
        } else if (token == "\\layout") {
 #ifndef NO_COMPABILITY
+               bool old_fromlayout = ert_comp.fromlayout;
+               bool create_new_par = true;
                ert_comp.in_tabular = false;
                // Do the insetert.
-               insertErtContents(par, pos);
+               if (!par->size() && par->previous() &&
+                       (par->previous()->size() == 1) &&
+                       isErtInset(par->previous(), par->previous()->size()-1))
+               {
+                       int p = par->previous()->size();
+                       insertErtContents(par->previous(), p);
+                       create_new_par = false;
+               } else {
+                       insertErtContents(par, pos);
+               }
+#endif
+               // reset the font as we start a new layout and if the font is
+               // not ALL_INHERIT,document_language then it will be set to the
+               // right values after this tag (Jug 20020420)
+               font = LyXFont(LyXFont::ALL_INHERIT, params.language);
+
+#if 0
+               if (file_format < 216 && params.language->lang() == "hebrew")
+                       font.setLanguage(default_language);
 #endif
+
                lex.eatLine();
                string layoutname = lex.getString();
 
@@ -471,7 +542,15 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
                if (layoutname.empty()) {
                        layoutname = tclass.defaultLayoutName();
                }
-
+#ifndef NO_COMPABILITY
+               if (compare_no_case(layoutname, "latex") == 0) {
+                       ert_comp.active = true;
+                       ert_comp.fromlayout = true;
+                       ert_comp.font = font;
+                       if (old_fromlayout)
+                               create_new_par = false;
+               }
+#endif
                bool hasLayout = tclass.hasLayout(layoutname);
                if (!hasLayout) {
                        lyxerr << "Layout '" << layoutname << "' does not"
@@ -482,19 +561,6 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
                        layoutname = tclass.defaultLayoutName();
                }
 
-               // reset the font as we start a new layout and if the font is
-               // not ALL_INHERIT,document_language then it will be set to the
-               // right values after this tag (Jug 20020420)
-               font = LyXFont(LyXFont::ALL_INHERIT, params.language);
-               if (file_format < 216 && params.language->lang() == "hebrew")
-                       font.setLanguage(default_language);
-
-#ifndef NO_COMPABILITY
-               if (compare_no_case(layoutname, "latex") == 0) {
-                       ert_comp.active = true;
-                       ert_comp.font = font;
-               }
-#endif
 #ifdef USE_CAPTION
                // The is the compability reading of layout caption.
                // It can be removed in LyX version 1.3.0. (Lgb)
@@ -547,25 +613,167 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
                        par->InsertInset(pos, inset, font);
                        ++pos;
                } else {
+#endif
+#ifndef NO_COMPABILITY
+               if (create_new_par) {
 #endif
                        if (!first_par)
                                first_par = par;
                        else {
                                par = new Paragraph(par);
-                               par->layout(textclasslist[params.textclass].defaultLayoutName());
+                               par->layout(textclasslist[params.textclass].defaultLayout());
                        }
                        pos = 0;
+                       par->layout(textclasslist[params.textclass][layoutname]);
+                       // Test whether the layout is obsolete.
+                       LyXLayout_ptr const & layout = par->layout();
+                       if (!layout->obsoleted_by().empty())
+                               par->layout(textclasslist[params.textclass][layout->obsoleted_by()]);
+                       par->params().depth(depth);
+#ifndef NO_COMPABILITY
+               } else {
+                       // we duplicate code here because it's easier to remove
+                       // the code then of NO_COMPATIBILITY
                        par->layout(layoutname);
                        // Test whether the layout is obsolete.
-                       LyXLayout const & layout =
+                       LyXLayout_ptr const & layout =
                                textclasslist[params.textclass][par->layout()];
-                       if (!layout.obsoleted_by().empty())
-                               par->layout(layout.obsoleted_by());
+                       if (!layout->obsoleted_by().empty())
+                               par->layout(layout->obsoleted_by());
                        par->params().depth(depth);
+               }
+#endif
 #if USE_CAPTION
                }
 #endif
 
+       } else if (token == "\\end_inset") {
+               lyxerr << "Solitary \\end_inset. Missing \\begin_inset?.\n"
+                      << "Last inset read was: " << last_inset_read
+                      << endl;
+               // 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") {
+#ifndef NO_COMPABILITY
+               insertErtContents(par, pos, false);
+               ert_stack.push(ert_comp);
+               ert_comp = ErtComp();
+#endif
+               readInset(lex, par, pos, font);
+#ifndef NO_COMPABILITY
+               ert_comp = ert_stack.top();
+               ert_stack.pop();
+               insertErtContents(par, pos);
+#endif
+       } else if (token == "\\family") {
+               lex.next();
+               font.setLyXFamily(lex.getString());
+       } else if (token == "\\series") {
+               lex.next();
+               font.setLyXSeries(lex.getString());
+       } else if (token == "\\shape") {
+               lex.next();
+               font.setLyXShape(lex.getString());
+       } else if (token == "\\size") {
+               lex.next();
+               font.setLyXSize(lex.getString());
+#ifndef NO_COMPABILITY
+       } else if (token == "\\latex") {
+               lex.next();
+               string const tok = lex.getString();
+               if (tok == "no_latex") {
+                       // Do the insetert.
+                       insertErtContents(par, pos);
+               } else if (tok == "latex") {
+                       ert_comp.active = true;
+                       ert_comp.font = font;
+               } else if (tok == "default") {
+                       // Do the insetert.
+                       insertErtContents(par, pos);
+               } else {
+                       lex.printError("Unknown LaTeX font flag "
+                                      "`$$Token'");
+               }
+#endif
+       } else if (token == "\\lang") {
+               lex.next();
+               string const tok = lex.getString();
+               Language const * lang = languages.getLanguage(tok);
+               if (lang) {
+                       font.setLanguage(lang);
+               } else {
+                       font.setLanguage(params.language);
+                       lex.printError("Unknown language `$$Token'");
+               }
+       } else if (token == "\\numeric") {
+               lex.next();
+               font.setNumber(font.setLyXMisc(lex.getString()));
+       } else if (token == "\\emph") {
+               lex.next();
+               font.setEmph(font.setLyXMisc(lex.getString()));
+       } else if (token == "\\bar") {
+               lex.next();
+               string const tok = lex.getString();
+               // This is dirty, but gone with LyX3. (Asger)
+               if (tok == "under")
+                       font.setUnderbar(LyXFont::ON);
+               else if (tok == "no")
+                       font.setUnderbar(LyXFont::OFF);
+               else if (tok == "default")
+                       font.setUnderbar(LyXFont::INHERIT);
+               else
+                       lex.printError("Unknown bar font flag "
+                                      "`$$Token'");
+       } else if (token == "\\noun") {
+               lex.next();
+               font.setNoun(font.setLyXMisc(lex.getString()));
+       } else if (token == "\\color") {
+               lex.next();
+               font.setLyXColor(lex.getString());
+       } else if (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 (lex.isOK()) {
+                               lex.next();
+                               string next_token = lex.getString();
+                               if (next_token == "\\-") {
+                                       par->insertChar(pos, '-', font);
+                               } else if (next_token == "\\protected_separator"
+                                       || next_token == "~") {
+                                       par->insertChar(pos, ' ', font);
+                               } else {
+                                       lex.printError("Token `$$Token' "
+                                                      "is in free space "
+                                                      "paragraph layout!");
+                                       --pos;
+                               }
+                       }
+               } else {
+                       Inset * inset = new InsetSpecialChar;
+                       inset->read(this, lex);
+                       par->insertInset(pos, inset, font);
+               }
+               ++pos;
+       } else if (token == "\\i") {
+               Inset * inset = new InsetLatexAccent;
+               inset->read(this, lex);
+               par->insertInset(pos, inset, font);
+               ++pos;
+       } else if (token == "\\backslash") {
+#ifndef NO_COMPABILITY
+               if (ert_comp.active) {
+                       ert_comp.contents += "\\";
+               } else {
+#endif
+               par->insertChar(pos, '\\', font);
+               ++pos;
+#ifndef NO_COMPABILITY
+               }
+#endif
 #ifndef NO_COMPABILITY
        } else if (token == "\\begin_float") {
                insertErtContents(par, pos);
@@ -657,7 +865,6 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
                LyXLex nylex(0, 0);
                nylex.setStream(istr);
                inset->read(this, nylex);
-
                par->insertInset(pos, inset, font);
                ++pos;
                insertErtContents(par, pos);
@@ -719,6 +926,10 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
                params.fonts = lex.getString();
        } else if (token == "\\noindent") {
                par->params().noindent(true);
+       } else if (token == "\\leftindent") {
+               lex.nextToken();
+               LyXLength value(lex.getString());
+               par->params().leftIndent(value);
        } else if (token == "\\fill_top") {
                par->params().spaceTop(VSpace(VSpace::VFILL));
        } else if (token == "\\fill_bottom") {
@@ -959,71 +1170,6 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
        } else if (token == "\\float_placement") {
                lex.nextToken();
                params.float_placement = lex.getString();
-       } else if (token == "\\family") {
-               lex.next();
-               font.setLyXFamily(lex.getString());
-       } else if (token == "\\series") {
-               lex.next();
-               font.setLyXSeries(lex.getString());
-       } else if (token == "\\shape") {
-               lex.next();
-               font.setLyXShape(lex.getString());
-       } else if (token == "\\size") {
-               lex.next();
-               font.setLyXSize(lex.getString());
-#ifndef NO_COMPABILITY
-       } else if (token == "\\latex") {
-               lex.next();
-               string const tok = lex.getString();
-               if (tok == "no_latex") {
-                       // Do the insetert.
-                       insertErtContents(par, pos);
-               } else if (tok == "latex") {
-                       ert_comp.active = true;
-                       ert_comp.font = font;
-               } else if (tok == "default") {
-                       // Do the insetert.
-                       insertErtContents(par, pos);
-               } else {
-                       lex.printError("Unknown LaTeX font flag "
-                                      "`$$Token'");
-               }
-#endif
-       } else if (token == "\\lang") {
-               lex.next();
-               string const tok = lex.getString();
-               Language const * lang = languages.getLanguage(tok);
-               if (lang) {
-                       font.setLanguage(lang);
-               } else {
-                       font.setLanguage(params.language);
-                       lex.printError("Unknown language `$$Token'");
-               }
-       } else if (token == "\\numeric") {
-               lex.next();
-               font.setNumber(font.setLyXMisc(lex.getString()));
-       } else if (token == "\\emph") {
-               lex.next();
-               font.setEmph(font.setLyXMisc(lex.getString()));
-       } else if (token == "\\bar") {
-               lex.next();
-               string const tok = lex.getString();
-               // This is dirty, but gone with LyX3. (Asger)
-               if (tok == "under")
-                       font.setUnderbar(LyXFont::ON);
-               else if (tok == "no")
-                       font.setUnderbar(LyXFont::OFF);
-               else if (tok == "default")
-                       font.setUnderbar(LyXFont::INHERIT);
-               else
-                       lex.printError("Unknown bar font flag "
-                                      "`$$Token'");
-       } else if (token == "\\noun") {
-               lex.next();
-               font.setNoun(font.setLyXMisc(lex.getString()));
-       } else if (token == "\\color") {
-               lex.next();
-               font.setLyXColor(lex.getString());
        } else if (token == "\\align") {
                int tmpret = lex.findToken(string_align);
                if (tmpret == -1) ++tmpret;
@@ -1035,15 +1181,21 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
        } else if (token == "\\added_space_top") {
                lex.nextToken();
                VSpace value = VSpace(lex.getString());
+               // only add the length when value > 0 or
+               // with option keep
                if ((value.length().len().value() != 0) ||
+                   value.keep() ||
                    (value.kind() != VSpace::LENGTH))
-                   par->params().spaceTop(value);
+                       par->params().spaceTop(value);
        } else if (token == "\\added_space_bottom") {
                lex.nextToken();
                VSpace value = VSpace(lex.getString());
+               // only add the length when value > 0 or
+               // with option keep
                if ((value.length().len().value() != 0) ||
+                  value.keep() ||
                    (value.kind() != VSpace::LENGTH))
-                   par->params().spaceBottom(value);
+                       par->params().spaceBottom(value);
 #ifndef NO_COMPABILITY
 #ifndef NO_PEXTRA_REALLY
        } else if (token == "\\pextra_type") {
@@ -1070,53 +1222,6 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
                lex.eatLine();
                par->params().labelWidthString(lex.getString());
                // do not delete this token, it is still needed!
-       } else if (token == "\\end_inset") {
-               lyxerr << "Solitary \\end_inset. Missing \\begin_inset?.\n"
-                      << "Last inset read was: " << last_inset_read
-                      << endl;
-               // 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") {
-#ifndef NO_COMPABILITY
-               insertErtContents(par, pos, false);
-               ert_stack.push(ert_comp);
-               ert_comp = ErtComp();
-#endif
-               readInset(lex, par, pos, font);
-#ifndef NO_COMPABILITY
-               ert_comp = ert_stack.top();
-               ert_stack.pop();
-               insertErtContents(par, pos);
-#endif
-       } else if (token == "\\SpecialChar") {
-               LyXLayout const & layout =
-                       textclasslist[params.textclass][par->layout()];
-
-               // Insets don't make sense in a free-spacing context! ---Kayvan
-               if (layout.free_spacing || par->isFreeSpacing()) {
-                       if (lex.isOK()) {
-                               lex.next();
-                               string next_token = lex.getString();
-                               if (next_token == "\\-") {
-                                       par->insertChar(pos, '-', font);
-                               } else if (next_token == "\\protected_separator"
-                                       || next_token == "~") {
-                                       par->insertChar(pos, ' ', font);
-                               } else {
-                                       lex.printError("Token `$$Token' "
-                                                      "is in free space "
-                                                      "paragraph layout!");
-                                       --pos;
-                               }
-                       }
-               } else {
-                       Inset * inset = new InsetSpecialChar;
-                       inset->read(this, lex);
-                       par->insertInset(pos, inset, font);
-               }
-               ++pos;
        } else if (token == "\\newline") {
 #ifndef NO_COMPABILITY
                if (!ert_comp.in_tabular && ert_comp.active) {
@@ -1149,10 +1254,9 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
                // This is a backward compability thingie. (Lgb)
                // Remove it later some time...introduced with fileformat
                // 2.16. (Lgb)
-               LyXLayout const & layout =
-                       textclasslist[params.textclass][par->layout()];
+               LyXLayout_ptr const & layout = par->layout();
 
-               if (layout.free_spacing || par->isFreeSpacing()) {
+               if (layout->free_spacing || par->isFreeSpacing()) {
                        par->insertChar(pos, ' ', font);
                } else {
                        Inset * inset = new InsetSpecialChar(InsetSpecialChar::PROTECTED_SEPARATOR);
@@ -1165,17 +1269,6 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
                        par->bibkey = new InsetBibKey(p);
                }
                par->bibkey->read(this, lex);
-       } else if (token == "\\backslash") {
-#ifndef NO_COMPABILITY
-               if (ert_comp.active) {
-                       ert_comp.contents += "\\";
-               } else {
-#endif
-               par->insertChar(pos, '\\', font);
-               ++pos;
-#ifndef NO_COMPABILITY
-               }
-#endif
        } else if (token == "\\the_end") {
 #ifndef NO_COMPABILITY
                // If we still have some ert active here we have to insert
@@ -1386,6 +1479,11 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
                        }
                        parBeforeMinipage->insertInset
                                (parBeforeMinipage->size(), mini, font);
+               } else if (par->params().pextraType() == Paragraph::PEXTRA_INDENT) {
+                       par->params().leftIndent(LyXLength(par->params().pextraWidth()));
+                       if (!par->params().pextraWidthp().empty()) {
+                               par->params().leftIndent(LyXLength((par->params().pextraWidthp())+"col%"));
+                       }
                }
        }
        // End of pextra_minipage compability
@@ -1399,8 +1497,8 @@ Buffer::parseSingleLyXformat2Token(LyXLex & lex, Paragraph *& par,
 void Buffer::insertStringAsLines(Paragraph *& par, pos_type & pos,
                                 LyXFont const & fn,string const & str) const
 {
-       LyXLayout const & layout =
-               textclasslist[params.textclass][par->layout()];
+       LyXLayout_ptr const & layout = par->layout();
+
        LyXFont font = fn;
 
        par->checkInsertChar(font);
@@ -1411,9 +1509,9 @@ void Buffer::insertStringAsLines(Paragraph *& par, pos_type & pos,
        for(string::const_iterator cit = str.begin();
            cit != str.end(); ++cit) {
                if (*cit == '\n') {
-                       if (autobreakrows && (par->size() || layout.keepempty)) {
+                       if (autobreakrows && (par->size() || layout->keepempty)) {
                                par->breakParagraph(params, pos,
-                                                   layout.isEnvironment());
+                                                   layout->isEnvironment());
                                par = par->next();
                                pos = 0;
                                space_inserted = true;
@@ -1422,12 +1520,12 @@ void Buffer::insertStringAsLines(Paragraph *& par, pos_type & pos,
                        }
                        // do not insert consecutive spaces if !free_spacing
                } else if ((*cit == ' ' || *cit == '\t') &&
-                          space_inserted && !layout.free_spacing &&
+                          space_inserted && !layout->free_spacing &&
                                   !par->isFreeSpacing())
                {
                        continue;
                } else if (*cit == '\t') {
-                       if (!layout.free_spacing && !par->isFreeSpacing()) {
+                       if (!layout->free_spacing && !par->isFreeSpacing()) {
                                // tabs are like spaces here
                                par->insertChar(pos, ' ', font);
                                ++pos;
@@ -1609,13 +1707,21 @@ bool Buffer::readFile(LyXLex & lex, Paragraph * par)
                                                   _("Old LyX file format found. "
                                                     "Use LyX 0.10.x to read this!"));
                                        return false;
+                               } else if (file_format < 220) {
+                                       Alert::alert(_("ERROR!"),
+                                                    _("Old LyX file format found. "
+                                                      "Use LyX 1.2.x to read this!"));
+                                       return false;
                                }
                        }
                        bool the_end = readLyXformat2(lex, par);
                        params.setPaperStuff();
+
+#if 0
                        // the_end was added in 213
                        if (file_format < 213)
                                the_end = true;
+#endif
 
                        if (!the_end) {
                                Alert::alert(_("Warning!"),
@@ -1666,7 +1772,6 @@ bool Buffer::save() const
                */
 
                // Should probably have some more error checking here.
-               // Should be cleaned up in 0.13, at least a bit.
                // Doing it this way, also makes the inodes stay the same.
                // This is still not a very good solution, in particular we
                // might loose the owner of the backup.
@@ -1844,7 +1949,7 @@ string const Buffer::asciiParagraph(Paragraph const * par,
 #endif
 
        // First write the layout
-       string const & tmp = par->layout();
+       string const & tmp = par->layout()->name();
        if (compare_no_case(tmp, "itemize") == 0) {
                ltype = 1;
                ltype_depth = depth + 1;
@@ -2438,8 +2543,6 @@ void Buffer::makeLaTeXFile(string const & fname,
                                + params.preamble + '\n';
                }
 
-               preamble += "\\makeatother\n";
-
                // Itemize bullet settings need to be last in case the user
                // defines their own bullets that use a package included
                // in the user-defined preamble -- ARRae
@@ -2481,8 +2584,6 @@ void Buffer::makeLaTeXFile(string const & fname,
                        texrow.newline();
                }
 
-               ofs << preamble;
-
                // We try to load babel late, in case it interferes
                // with other packages.
                if (use_babel) {
@@ -2492,10 +2593,14 @@ void Buffer::makeLaTeXFile(string const & fname,
                                tmp = string("\\usepackage[") +
                                        language_options.str().c_str() +
                                        "]{babel}";
-                       ofs << tmp << "\n";
-                       texrow.newline();
+                       preamble += tmp + "\n";
+                       preamble += features.getBabelOptions();
                }
 
+               preamble += "\\makeatother\n";
+
+               ofs << preamble;
+
                // make the body.
                ofs << "\\begin{document}\n";
                texrow.newline();
@@ -2558,7 +2663,8 @@ void Buffer::makeLaTeXFile(string const & fname,
 // LaTeX all paragraphs from par to endpar, if endpar == 0 then to the end
 //
 void Buffer::latexParagraphs(ostream & ofs, Paragraph * par,
-                            Paragraph * endpar, TexRow & texrow) const
+                            Paragraph * endpar, TexRow & texrow,
+                            bool moving_arg) const
 {
        bool was_title = false;
        bool already_title = false;
@@ -2571,10 +2677,9 @@ void Buffer::latexParagraphs(ostream & ofs, Paragraph * par,
                // 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)) {
-                       LyXLayout const & layout =
-                               textclasslist[params.textclass][par->layout()];
+                       LyXLayout_ptr const & layout = par->layout();
 
-                       if (layout.intitle) {
+                       if (layout->intitle) {
                                if (already_title) {
                                        lyxerr <<"Error in latexParagraphs: You"
                                                " should not mix title layouts"
@@ -2588,13 +2693,15 @@ void Buffer::latexParagraphs(ostream & ofs, Paragraph * par,
                                was_title = false;
                        }
 
-                       if (layout.isEnvironment()) {
+                       if (layout->isEnvironment() ||
+                               !par->params().leftIndent().zero())
+                       {
                                par = par->TeXEnvironment(this, params, ofs, texrow);
                        } else {
-                               par = par->TeXOnePar(this, params, ofs, texrow, false);
+                               par = par->TeXOnePar(this, params, ofs, texrow, moving_arg);
                        }
                } else {
-                       par = par->TeXOnePar(this, params, ofs, texrow, false);
+                       par = par->TeXOnePar(this, params, ofs, texrow, moving_arg);
                }
        }
        // It might be that we only have a title in this document
@@ -2638,21 +2745,35 @@ bool Buffer::isSGML() const
 }
 
 
-void Buffer::sgmlOpenTag(ostream & os, Paragraph::depth_type,
+int Buffer::sgmlOpenTag(ostream & os, Paragraph::depth_type depth, bool mixcont,
                         string const & latexname) const
 {
-       if (!latexname.empty() && latexname != "!-- --")
-               //os << "<!-- " << depth << " -->" << "<" << latexname << ">";
+       if (!latexname.empty() && latexname != "!-- --") {
+               if (!mixcont)
+                       os << string(" ",depth);
                os << "<" << latexname << ">";
+       }
+
+       if (!mixcont)
+               os << endl;
+
+       return mixcont?0:1;
 }
 
 
-void Buffer::sgmlCloseTag(ostream & os, Paragraph::depth_type,
+int Buffer::sgmlCloseTag(ostream & os, Paragraph::depth_type depth, bool mixcont,
                          string const & latexname) const
 {
-       if (!latexname.empty() && latexname != "!-- --")
-               //os << "<!-- " << depth << " -->" << "</" << latexname << ">\n";
+       if (!latexname.empty() && latexname != "!-- --") {
+               if (!mixcont)
+                       os << endl << string(" ",depth);
                os << "</" << latexname << ">";
+       }
+
+       if (!mixcont)
+               os << endl;
+
+       return mixcont?0:1;
 }
 
 
@@ -2692,12 +2813,12 @@ void Buffer::makeLinuxDocFile(string const & fname, bool nice, bool body_only)
                ofs << ">\n\n";
 
                if (params.options.empty())
-                       sgmlOpenTag(ofs, 0, top_element);
+                       sgmlOpenTag(ofs, 0, false, top_element);
                else {
                        string top = top_element;
                        top += " ";
                        top += params.options;
-                       sgmlOpenTag(ofs, 0, top);
+                       sgmlOpenTag(ofs, 0, false, top);
                }
        }
 
@@ -2711,15 +2832,14 @@ void Buffer::makeLinuxDocFile(string const & fname, bool nice, bool body_only)
        vector<string> environment_stack(5);
 
        while (par) {
-               LyXLayout const & style = tclass[par->layout()];
-
+               LyXLayout_ptr const & style = par->layout();
                // treat <toc> as a special case for compatibility with old code
                if (par->isInset(0)) {
                        Inset * inset = par->getInset(0);
                        Inset::Code lyx_code = inset->lyxCode();
                        if (lyx_code == Inset::TOC_CODE) {
                                string const temp = "toc";
-                               sgmlOpenTag(ofs, depth, temp);
+                               sgmlOpenTag(ofs, depth, false, temp);
 
                                par = par->next();
                                continue;
@@ -2728,23 +2848,23 @@ void Buffer::makeLinuxDocFile(string const & fname, bool nice, bool body_only)
 
                // environment tag closing
                for (; depth > par->params().depth(); --depth) {
-                       sgmlCloseTag(ofs, depth, environment_stack[depth]);
+                       sgmlCloseTag(ofs, depth, false, environment_stack[depth]);
                        environment_stack[depth].erase();
                }
 
                // write opening SGML tags
-               switch (style.latextype) {
+               switch (style->latextype) {
                case LATEX_PARAGRAPH:
                        if (depth == par->params().depth()
                           && !environment_stack[depth].empty()) {
-                               sgmlCloseTag(ofs, depth, environment_stack[depth]);
+                               sgmlCloseTag(ofs, depth, false, environment_stack[depth]);
                                environment_stack[depth].erase();
                                if (depth)
                                        --depth;
                                else
                                        ofs << "</p>";
                        }
-                       sgmlOpenTag(ofs, depth, style.latexname());
+                       sgmlOpenTag(ofs, depth, false, style->latexname());
                        break;
 
                case LATEX_COMMAND:
@@ -2754,20 +2874,22 @@ void Buffer::makeLinuxDocFile(string const & fname, bool nice, bool body_only)
                                            " LatexType Command.\n"));
 
                        if (!environment_stack[depth].empty()) {
-                               sgmlCloseTag(ofs, depth,
-                                            environment_stack[depth]);
+                               sgmlCloseTag(ofs, depth, false, environment_stack[depth]);
                                ofs << "</p>";
                        }
 
                        environment_stack[depth].erase();
-                       sgmlOpenTag(ofs, depth, style.latexname());
+                       sgmlOpenTag(ofs, depth, false, style->latexname());
                        break;
 
                case LATEX_ENVIRONMENT:
                case LATEX_ITEM_ENVIRONMENT:
+               {
+                       string const & latexname = style->latexname();
+
                        if (depth == par->params().depth()
-                           && environment_stack[depth] != style.latexname()) {
-                               sgmlCloseTag(ofs, depth,
+                           && environment_stack[depth] != latexname) {
+                               sgmlCloseTag(ofs, depth, false,
                                             environment_stack[depth]);
                                environment_stack[depth].erase();
                        }
@@ -2775,31 +2897,33 @@ void Buffer::makeLinuxDocFile(string const & fname, bool nice, bool body_only)
                               depth = par->params().depth();
                               environment_stack[depth].erase();
                        }
-                       if (environment_stack[depth] != style.latexname()) {
+                       if (environment_stack[depth] != latexname) {
                                if (depth == 0) {
-                                       sgmlOpenTag(ofs, depth, "p");
+                                       sgmlOpenTag(ofs, depth, false, "p");
                                }
-                               sgmlOpenTag(ofs, depth, style.latexname());
+                               sgmlOpenTag(ofs, depth, false, latexname);
 
                                if (environment_stack.size() == depth + 1)
                                        environment_stack.push_back("!-- --");
-                               environment_stack[depth] = style.latexname();
+                               environment_stack[depth] = latexname;
                        }
 
-                       if (style.latexparam() == "CDATA")
+                       if (style->latexparam() == "CDATA")
                                ofs << "<![CDATA[";
 
-                       if (style.latextype == LATEX_ENVIRONMENT) break;
+                       if (style->latextype == LATEX_ENVIRONMENT) break;
 
-                       if (style.labeltype == LABEL_MANUAL)
+                       if (style->labeltype == LABEL_MANUAL)
                                item_name = "tag";
                        else
                                item_name = "item";
 
-                       sgmlOpenTag(ofs, depth + 1, item_name);
-                       break;
+                       sgmlOpenTag(ofs, depth + 1, false, item_name);
+               }
+               break;
+
                default:
-                       sgmlOpenTag(ofs, depth, style.latexname());
+                       sgmlOpenTag(ofs, depth, false, style->latexname());
                        break;
                }
 
@@ -2809,27 +2933,27 @@ void Buffer::makeLinuxDocFile(string const & fname, bool nice, bool body_only)
 
                ofs << "\n";
                // write closing SGML tags
-               switch (style.latextype) {
+               switch (style->latextype) {
                case LATEX_COMMAND:
                        break;
                case LATEX_ENVIRONMENT:
                case LATEX_ITEM_ENVIRONMENT:
-                       if (style.latexparam() == "CDATA")
+                       if (style->latexparam() == "CDATA")
                                ofs << "]]>";
                        break;
                default:
-                       sgmlCloseTag(ofs, depth, style.latexname());
+                       sgmlCloseTag(ofs, depth, false, style->latexname());
                        break;
                }
        }
 
        // Close open tags
-       for (int i=depth; i >= 0; --i)
-               sgmlCloseTag(ofs, depth, environment_stack[i]);
+       for (int i = depth; i >= 0; --i)
+               sgmlCloseTag(ofs, depth, false, environment_stack[i]);
 
        if (!body_only) {
                ofs << "\n\n";
-               sgmlCloseTag(ofs, 0, top_element);
+               sgmlCloseTag(ofs, 0, false, top_element);
        }
 
        ofs.close();
@@ -2900,18 +3024,18 @@ void Buffer::simpleLinuxDocOnePar(ostream & os,
        Paragraph * par,
        Paragraph::depth_type /*depth*/)
 {
-       LyXLayout const & style =
-               textclasslist[params.textclass][par->layout()];
+       LyXLayout_ptr const & style = par->layout();
+
        string::size_type char_line_count = 5;     // Heuristic choice ;-)
 
        // gets paragraph main font
        LyXFont font_old;
        bool desc_on;
-       if (style.labeltype == LABEL_MANUAL) {
-               font_old = style.labelfont;
+       if (style->labeltype == LABEL_MANUAL) {
+               font_old = style->labelfont;
                desc_on = true;
        } else {
-               font_old = style.font;
+               font_old = style->font;
                desc_on = false;
        }
 
@@ -3045,7 +3169,7 @@ void Buffer::simpleLinuxDocOnePar(ostream & os,
                        continue;
                }
 
-               if (style.latexparam() == "CDATA") {
+               if (style->latexparam() == "CDATA") {
                        // "TeX"-Mode on == > SGML-Mode on.
                        if (c != '\0')
                                os << c;
@@ -3053,7 +3177,7 @@ void Buffer::simpleLinuxDocOnePar(ostream & os,
                } else {
                        string sgml_string;
                        if (par->sgmlConvertChar(c, sgml_string)
-                           && !style.free_spacing && !par->isFreeSpacing())
+                           && !style->free_spacing && !par->isFreeSpacing())
                        {
                                // in freespacing mode, spaces are
                                // non-breaking characters
@@ -3093,12 +3217,14 @@ void Buffer::simpleLinuxDocOnePar(ostream & os,
 void Buffer::sgmlError(Paragraph * /*par*/, int /*pos*/,
        string const & /*message*/) const
 {
+#ifdef WITH_WARNINGS
 #warning This is wrong we cannot insert an inset like this!!!
        // I guess this was Jose' so I explain you more or less why this
        // is wrong. This way you insert something in the paragraph and
        // don't tell it to LyXText (row rebreaking and undo handling!!!)
        // I deactivate this code, have a look at BufferView::insertErrors
        // how you should do this correctly! (Jug 20020315)
+#endif
 #if 0
        // insert an error marker in text
        InsetError * new_inset = new InsetError(message);
@@ -3153,7 +3279,7 @@ void Buffer::makeDocBookFile(string const & fname, bool nice, bool only_body)
                top += " ";
                top += params.options;
        }
-       sgmlOpenTag(ofs, 0, top);
+       sgmlOpenTag(ofs, 0, false, top);
 
        ofs << "<!-- DocBook file was created by " << lyx_docversion
            << "\n  See http://www.lyx.org/ for more information -->\n";
@@ -3177,49 +3303,42 @@ void Buffer::makeDocBookFile(string const & fname, bool nice, bool only_body)
                string c_params;
                int desc_on = 0; // description mode
 
-               LyXLayout const & style = tclass[par->layout()];
+               LyXLayout_ptr const & style = par->layout();
 
                // environment tag closing
                for (; depth > par->params().depth(); --depth) {
                        if (environment_inner[depth] != "!-- --") {
                                item_name = "listitem";
-                               sgmlCloseTag(ofs, command_depth + depth,
-                                            item_name);
+                               sgmlCloseTag(ofs, command_depth + depth, false, item_name);
                                if (environment_inner[depth] == "varlistentry")
-                                       sgmlCloseTag(ofs, depth+command_depth,
-                                                    environment_inner[depth]);
+                                       sgmlCloseTag(ofs, depth+command_depth, false, environment_inner[depth]);
                        }
-                       sgmlCloseTag(ofs, depth + command_depth,
-                                    environment_stack[depth]);
+                       sgmlCloseTag(ofs, depth + command_depth, false, environment_stack[depth]);
                        environment_stack[depth].erase();
                        environment_inner[depth].erase();
                }
 
                if (depth == par->params().depth()
-                  && environment_stack[depth] != style.latexname()
+                  && environment_stack[depth] != style->latexname()
                   && !environment_stack[depth].empty()) {
                        if (environment_inner[depth] != "!-- --") {
                                item_name= "listitem";
-                               sgmlCloseTag(ofs, command_depth+depth,
-                                            item_name);
+                               sgmlCloseTag(ofs, command_depth+depth, false, item_name);
                                if (environment_inner[depth] == "varlistentry")
-                                       sgmlCloseTag(ofs,
-                                                    depth + command_depth,
-                                                    environment_inner[depth]);
+                                       sgmlCloseTag(ofs, depth + command_depth, false, environment_inner[depth]);
                        }
 
-                       sgmlCloseTag(ofs, depth + command_depth,
-                                    environment_stack[depth]);
+                       sgmlCloseTag(ofs, depth + command_depth, false, environment_stack[depth]);
 
                        environment_stack[depth].erase();
                        environment_inner[depth].erase();
                }
 
                // Write opening SGML tags.
-               switch (style.latextype) {
+               switch (style->latextype) {
                case LATEX_PARAGRAPH:
                        sgmlOpenTag(ofs, depth + command_depth,
-                                   style.latexname());
+                                   false, style->latexname());
                        break;
 
                case LATEX_COMMAND:
@@ -3228,21 +3347,27 @@ void Buffer::makeDocBookFile(string const & fname, bool nice, bool only_body)
                                          _("Error : Wrong depth for "
                                            "LatexType Command.\n"));
 
-                       command_name = style.latexname();
+                       command_name = style->latexname();
 
-                       sgmlparam = style.latexparam();
+                       sgmlparam = style->latexparam();
                        c_params = split(sgmlparam, c_depth,'|');
 
                        cmd_depth = lyx::atoi(c_depth);
 
                        if (command_flag) {
                                if (cmd_depth < command_base) {
-                                       for (Paragraph::depth_type j = command_depth; j >= command_base; --j)
-                                               sgmlCloseTag(ofs, j, command_stack[j]);
+                                       for (Paragraph::depth_type j = command_depth;
+                                            j >= command_base; --j) {
+                                               sgmlCloseTag(ofs, j, false, command_stack[j]);
+                                               ofs << endl;
+                                       }
                                        command_depth = command_base = cmd_depth;
                                } else if (cmd_depth <= command_depth) {
-                                       for (int j = command_depth; j >= int(cmd_depth); --j)
-                                               sgmlCloseTag(ofs, j, command_stack[j]);
+                                       for (int j = command_depth;
+                                            j >= int(cmd_depth); --j) {
+                                               sgmlCloseTag(ofs, j, false, command_stack[j]);
+                                               ofs << endl;
+                                       }
                                        command_depth = cmd_depth;
                                } else
                                        command_depth = cmd_depth;
@@ -3269,12 +3394,10 @@ void Buffer::makeDocBookFile(string const & fname, bool nice, bool only_body)
                                }
                        }
 
-                       sgmlOpenTag(ofs, depth + command_depth, command_name);
-                       if (c_params.empty())
-                               item_name = "title";
-                       else
-                               item_name = c_params;
-                       sgmlOpenTag(ofs, depth + 1 + command_depth, item_name);
+                       sgmlOpenTag(ofs, depth + command_depth, false, command_name);
+
+                       item_name = c_params.empty()?"title":c_params;
+                       sgmlOpenTag(ofs, depth + 1 + command_depth, false, item_name);
                        break;
 
                case LATEX_ENVIRONMENT:
@@ -3284,97 +3407,79 @@ void Buffer::makeDocBookFile(string const & fname, bool nice, bool only_body)
                                environment_stack[depth].erase();
                        }
 
-                       if (environment_stack[depth] != style.latexname()) {
+                       if (environment_stack[depth] != style->latexname()) {
                                if (environment_stack.size() == depth + 1) {
                                        environment_stack.push_back("!-- --");
                                        environment_inner.push_back("!-- --");
                                }
-                               environment_stack[depth] = style.latexname();
+                               environment_stack[depth] = style->latexname();
                                environment_inner[depth] = "!-- --";
-                               sgmlOpenTag(ofs, depth + command_depth,
-                                           environment_stack[depth]);
+                               sgmlOpenTag(ofs, depth + command_depth, false, environment_stack[depth]);
                        } else {
                                if (environment_inner[depth] != "!-- --") {
                                        item_name= "listitem";
-                                       sgmlCloseTag(ofs,
-                                                    command_depth + depth,
-                                                    item_name);
+                                       sgmlCloseTag(ofs, command_depth + depth, false, item_name);
                                        if (environment_inner[depth] == "varlistentry")
-                                               sgmlCloseTag(ofs,
-                                                            depth + command_depth,
-                                                            environment_inner[depth]);
+                                               sgmlCloseTag(ofs, depth + command_depth, false, environment_inner[depth]);
                                }
                        }
 
-                       if (style.latextype == LATEX_ENVIRONMENT) {
-                               if (!style.latexparam().empty()) {
-                                       if (style.latexparam() == "CDATA")
+                       if (style->latextype == LATEX_ENVIRONMENT) {
+                               if (!style->latexparam().empty()) {
+                                       if (style->latexparam() == "CDATA")
                                                ofs << "<![CDATA[";
                                        else
-                                               sgmlOpenTag(ofs, depth + command_depth,
-                                                           style.latexparam());
+                                               sgmlOpenTag(ofs, depth + command_depth, false, style->latexparam());
                                }
                                break;
                        }
 
-                       desc_on = (style.labeltype == LABEL_MANUAL);
-
-                       if (desc_on)
-                               environment_inner[depth]= "varlistentry";
-                       else
-                               environment_inner[depth]= "listitem";
+                       desc_on = (style->labeltype == LABEL_MANUAL);
 
+                       environment_inner[depth] = desc_on ? "varlistentry" : "listitem";
                        sgmlOpenTag(ofs, depth + 1 + command_depth,
-                                   environment_inner[depth]);
+                                   false, environment_inner[depth]);
 
-                       if (desc_on) {
-                               item_name= "term";
-                               sgmlOpenTag(ofs, depth + 1 + command_depth,
-                                           item_name);
-                       } else {
-                               item_name= "para";
-                               sgmlOpenTag(ofs, depth + 1 + command_depth,
-                                           item_name);
-                       }
+                       item_name = desc_on ? "term" : "para";
+                       sgmlOpenTag(ofs, depth + 1 + command_depth,
+                                   false, item_name);
                        break;
                default:
                        sgmlOpenTag(ofs, depth + command_depth,
-                                   style.latexname());
+                                   false, style->latexname());
                        break;
                }
 
-               simpleDocBookOnePar(ofs, par, desc_on, depth+1+command_depth);
+               simpleDocBookOnePar(ofs, par, desc_on,
+                                   depth + 1 + command_depth);
                par = par->next();
 
                string end_tag;
                // write closing SGML tags
-               switch (style.latextype) {
+               switch (style->latextype) {
                case LATEX_COMMAND:
-                       if (c_params.empty())
-                               end_tag = "title";
-                       else
-                               end_tag = c_params;
-                       sgmlCloseTag(ofs, depth + command_depth, end_tag);
+                       end_tag = c_params.empty() ? "title" : c_params;
+                       sgmlCloseTag(ofs, depth + command_depth,
+                                    false, end_tag);
                        break;
                case LATEX_ENVIRONMENT:
-                       if (!style.latexparam().empty()) {
-                               if (style.latexparam() == "CDATA")
+                       if (!style->latexparam().empty()) {
+                               if (style->latexparam() == "CDATA")
                                        ofs << "]]>";
                                else
-                                       sgmlCloseTag(ofs, depth + command_depth,
-                                                    style.latexparam());
+                                       sgmlCloseTag(ofs, depth + command_depth, false, style->latexparam());
                        }
                        break;
                case LATEX_ITEM_ENVIRONMENT:
                        if (desc_on == 1) break;
                        end_tag= "para";
-                       sgmlCloseTag(ofs, depth + 1 + command_depth, end_tag);
+                       sgmlCloseTag(ofs, depth + 1 + command_depth, false, end_tag);
                        break;
                case LATEX_PARAGRAPH:
-                       sgmlCloseTag(ofs, depth + command_depth, style.latexname());
+                       sgmlCloseTag(ofs, depth + command_depth, false, style->latexname());
                        break;
                default:
-                       sgmlCloseTag(ofs, depth + command_depth, style.latexname());
+                       sgmlCloseTag(ofs, depth + command_depth, false, style->latexname());
                        break;
                }
        }
@@ -3384,24 +3489,23 @@ void Buffer::makeDocBookFile(string const & fname, bool nice, bool only_body)
                if (!environment_stack[depth].empty()) {
                        if (environment_inner[depth] != "!-- --") {
                                item_name = "listitem";
-                               sgmlCloseTag(ofs, command_depth + depth,
-                                            item_name);
+                               sgmlCloseTag(ofs, command_depth + depth, false, item_name);
                               if (environment_inner[depth] == "varlistentry")
-                                      sgmlCloseTag(ofs, depth + command_depth,
-                                                   environment_inner[depth]);
+                                      sgmlCloseTag(ofs, depth + command_depth, false, environment_inner[depth]);
                        }
 
-                       sgmlCloseTag(ofs, depth + command_depth,
-                                    environment_stack[depth]);
+                       sgmlCloseTag(ofs, depth + command_depth, false, environment_stack[depth]);
                }
        }
 
        for (int j = command_depth; j >= 0 ; --j)
-               if (!command_stack[j].empty())
-                       sgmlCloseTag(ofs, j, command_stack[j]);
+               if (!command_stack[j].empty()) {
+                       sgmlCloseTag(ofs, j, false, command_stack[j]);
+                       ofs << endl;
+               }
 
        ofs << "\n\n";
-       sgmlCloseTag(ofs, 0, top_element);
+       sgmlCloseTag(ofs, 0, false, top_element);
 
        ofs.close();
        // How to check for successful close
@@ -3417,10 +3521,9 @@ void Buffer::simpleDocBookOnePar(ostream & os,
 {
        bool emph_flag = false;
 
-       LyXLayout const & style =
-               textclasslist[params.textclass][par->layout()];
+       LyXLayout_ptr const & style = par->layout();
 
-       LyXFont font_old = style.labeltype == LABEL_MANUAL ? style.labelfont : style.font;
+       LyXFont font_old = (style->labeltype == LABEL_MANUAL ? style->labelfont : style->font);
 
        int char_line_count = depth;
        //if (!style.free_spacing)
@@ -3433,10 +3536,18 @@ void Buffer::simpleDocBookOnePar(ostream & os,
                // handle <emphasis> tag
                if (font_old.emph() != font.emph()) {
                        if (font.emph() == LyXFont::ON) {
+                               if (style->latexparam() == "CDATA")
+                                       os << "]]>";
                                os << "<emphasis>";
+                               if (style->latexparam() == "CDATA")
+                                       os << "<![CDATA[";
                                emph_flag = true;
                        } else if (i) {
+                               if (style->latexparam() == "CDATA")
+                                       os << "]]>";
                                os << "</emphasis>";
+                               if (style->latexparam() == "CDATA")
+                                       os << "<![CDATA[";
                                emph_flag = false;
                        }
                }
@@ -3445,11 +3556,11 @@ void Buffer::simpleDocBookOnePar(ostream & os,
                if (par->isInset(i)) {
                        Inset * inset = par->getInset(i);
                        // don't print the inset in position 0 if desc_on == 3 (label)
-                       if ( i || desc_on != 3) {
-                               if(style.latexparam() == "CDATA")
+                       if (i || desc_on != 3) {
+                               if (style->latexparam() == "CDATA")
                                        os << "]]>";
-                               inset->docbook(this, os);
-                               if(style.latexparam() == "CDATA")
+                               inset->docbook(this, os, false);
+                               if (style->latexparam() == "CDATA")
                                        os << "<![CDATA[";
                        }
                } else {
@@ -3457,9 +3568,9 @@ void Buffer::simpleDocBookOnePar(ostream & os,
                        string sgml_string;
                        par->sgmlConvertChar(c, sgml_string);
 
-                       if (style.pass_thru) {
+                       if (style->pass_thru) {
                                os << c;
-                       } else if (style.free_spacing || par->isFreeSpacing() || c != ' ') {
+                       } else if (style->free_spacing || par->isFreeSpacing() || c != ' ') {
                                        os << sgml_string;
                        } else if (desc_on ==1) {
                                ++char_line_count;
@@ -3473,7 +3584,11 @@ void Buffer::simpleDocBookOnePar(ostream & os,
        }
 
        if (emph_flag) {
+               if (style->latexparam() == "CDATA")
+                       os << "]]>";
                os << "</emphasis>";
+               if (style->latexparam() == "CDATA")
+                       os << "<![CDATA[";
        }
 
        // resets description flag correctly
@@ -3481,11 +3596,11 @@ void Buffer::simpleDocBookOnePar(ostream & os,
                // <term> not closed...
                os << "</term>\n<listitem><para>&nbsp;</para>";
        }
-       if (style.free_spacing) os << '\n';
+       if (style->free_spacing)
+               os << '\n';
 }
 
 
-// This should be enabled when the Chktex class is implemented. (Asger)
 // chktex should be run with these flags disabled: 3, 22, 25, 30, 38(?)
 // Other flags: -wall -v0 -x
 int Buffer::runChktex()
@@ -3527,7 +3642,8 @@ int Buffer::runChktex()
        // if we removed error insets before we ran chktex or if we inserted
        // error insets after we ran chktex, this must be run:
        if (removedErrorInsets || res) {
-               users->redraw();
+#warning repaint needed here, or do you mean update() ?
+               users->repaint();
                users->fitCursor();
        }
        users->owner()->allowInput();
@@ -3640,7 +3756,8 @@ Buffer::Lists const Buffer::getLists() const
        string const layout("Caption");
 
        while (par) {
-               char const labeltype = textclass[par->layout()].labeltype;
+#warning bogus type (Lgb)
+               char const labeltype = par->layout()->labeltype;
 
                if (labeltype >= LABEL_COUNTER_CHAPTER
                    && labeltype <= LABEL_COUNTER_CHAPTER + params.tocdepth) {
@@ -3672,7 +3789,7 @@ Buffer::Lists const Buffer::getLists() const
                                        // the inset...
                                        Paragraph * tmp = il->inset.paragraph();
                                        while (tmp) {
-                                               if (tmp->layout() == layout) {
+                                               if (tmp->layout()->name() == layout) {
                                                        SingleList & item = l[type];
                                                        string const str =
                                                                tostr(item.size()+1) + ". " + tmp->asString(this, false);
@@ -3772,24 +3889,28 @@ void Buffer::markDepClean(string const & name)
 }
 
 
-bool Buffer::dispatch(string const & command)
+bool Buffer::dispatch(string const & command, bool * result)
 {
        // Split command string into command and argument
        string cmd;
        string line = frontStrip(command);
        string const arg = strip(frontStrip(split(line, cmd, ' ')));
 
-       return dispatch(lyxaction.LookupFunc(cmd), arg);
+       return dispatch(lyxaction.LookupFunc(cmd), arg, result);
 }
 
 
-bool Buffer::dispatch(int action, string const & argument)
+bool Buffer::dispatch(int action, string const & argument, bool * result)
 {
        bool dispatched = true;
+
        switch (action) {
-               case LFUN_EXPORT:
-                       Exporter::Export(this, argument, false);
+               case LFUN_EXPORT: {
+                       bool const tmp = Exporter::Export(this, argument, false);
+                       if (result)
+                               *result = tmp;
                        break;
+               }
 
                default:
                        dispatched = false;
@@ -3810,7 +3931,8 @@ void Buffer::resizeInsets(BufferView * bv)
 
 void Buffer::redraw()
 {
-       users->redraw();
+#warning repaint needed here, or do you mean update() ?
+       users->repaint();
        users->fitCursor();
 }