]> git.lyx.org Git - lyx.git/blobdiff - src/paragraph.C
initialize footnotekind int LyXParagraph()
[lyx.git] / src / paragraph.C
index 84f97e7f07a982cf38ac35f8a8dd36ab7cf202ec..b54f9b0f23c6d91244f4c224abb67794753985bf 100644 (file)
 #pragma implementation "lyxparagraph.h"
 #endif
 
+#include <fstream>
+using std::fstream;
+using std::ios;
+
 #include "lyxparagraph.h"
 #include "support/textutils.h"
 #include "lyxrc.h"
@@ -21,6 +25,7 @@
 #include "tex-strings.h"
 #include "bufferparams.h"
 #include "support/FileInfo.h"
+#include "support/LAssert.h"
 #include "debug.h"
 #include "LaTeXFeatures.h"
 #include "insets/insetinclude.h"
@@ -30,7 +35,6 @@
 
 
 extern void addNewlineAndDepth(string & file, int const depth); // Jug 990923
-extern unsigned char GetCurrentTextClass(); // this should be fixed/removed
 int tex_code_break_column = 72;  // needs non-zero initialization. set later.
 // this is a bad idea, but how can LyXParagraph find its buffer to get
 // parameters? (JMarc)
@@ -41,7 +45,7 @@ extern LyXRC * lyxrc;
 // ale970405
 extern string bibitemWidthest();
 
-/* this is a minibuffer */
+// this is a minibuffer
 static char minibuffer_char;
 static LyXFont minibuffer_font;
 static Inset * minibuffer_inset;
@@ -62,12 +66,15 @@ LyXParagraph::LyXParagraph()
        itemdepth = 0;
        next = 0;
        previous = 0;
-#ifndef NEW_TABLE
+#ifndef NEW_FONTTABLE // OK
        fonttable = 0;
+#endif
+#ifndef NEW_INSETTABLE
        insettable = 0;
 #endif
        footnoteflag = LyXParagraph::NO_FOOTNOTE;
-
+       footnotekind = LyXParagraph::FOOTNOTE; // should not be needed
+       
        align = LYX_ALIGN_BLOCK;
 
        /* table stuff -- begin*/ 
@@ -79,22 +86,27 @@ LyXParagraph::LyXParagraph()
 }
 
 
-/* this konstruktor inserts the new paragraph in a list */ 
+// This konstruktor inserts the new paragraph in a list.
 LyXParagraph::LyXParagraph(LyXParagraph * par)
 {
-#warning we also need a reserve here
-#warning this would be a nice place to shrink par
+       text.reserve(500);
+       par->text.resize(par->text.size());
+
        for (int i = 0; i < 10; ++i) setCounter(i, 0);
        appendix = false;
        enumdepth = 0;
        itemdepth = 0;
+       // double linked list begin
        next = par->next;
        if (next)
                next->previous = this;
        previous = par;
        previous->next = this;
-#ifndef NEW_TABLE
+       // end
+#ifndef NEW_FONTTABLE // OK
        fonttable = 0;
+#endif
+#ifndef NEW_INSETTABLE
        insettable = 0;
 #endif
        footnoteflag = LyXParagraph::NO_FOOTNOTE;
@@ -111,7 +123,7 @@ LyXParagraph::LyXParagraph(LyXParagraph * par)
 }
 
 
-void LyXParagraph::writeFile(FILE * file, BufferParams & params,
+void LyXParagraph::writeFile(ostream & os, BufferParams & params,
                             char footflag, char dth)
 {
        LyXFont font1, font2;
@@ -124,73 +136,72 @@ void LyXParagraph::writeFile(FILE * file, BufferParams & params,
            || !previous
            || previous->footnoteflag == LyXParagraph::NO_FOOTNOTE){
                
-               /* The beginning or the end of a footnote environment? */ 
+               // The beginning or the end of a footnote environment?
                if (footflag != footnoteflag) {
                        footflag = footnoteflag;
                        if (footflag) {
-                               fprintf(file, "\n\\begin_float %s ", 
-                                       string_footnotekinds[footnotekind]);
-                       }
-                       else {
-                               fprintf(file, "\n\\end_float ");
+                               os << "\n\\begin_float "
+                                  << string_footnotekinds[footnotekind]
+                                  << " ";
+                       else {
+                               os << "\n\\end_float ";
                        }
                }
 
-               /* The beginning or end of a deeper (i.e. nested) area? */
+               // The beginning or end of a deeper (i.e. nested) area?
                if (dth != depth) {
                        if (depth > dth) {
                                while (depth > dth) {
-                                       fprintf(file, "\n\\begin_deeper ");
-                                       dth++;
+                                       os << "\n\\begin_deeper ";
+                                       ++dth;
                                }
-                       }
-                       else {
+                       } else {
                                while (depth < dth) {
-                                       fprintf(file, "\n\\end_deeper ");
-                                       dth--;
+                                       os << "\n\\end_deeper ";
+                                       --dth;
                                }
                        }
                }
 
-               /* First write the layout */ 
-               fprintf(file, "\n\\layout %s\n",
-                       textclasslist.NameOfLayout(params.textclass, layout)
-                       .c_str());
+               // First write the layout
+               os << "\n\\layout "
+                  << textclasslist.NameOfLayout(params.textclass, layout)
+                  << "\n";
 
-               /* maybe some vertical spaces */ 
+               // Maybe some vertical spaces.
                if (added_space_top.kind() != VSpace::NONE)
-                       fprintf(file, "\\added_space_top %s ", 
-                               added_space_top.asLyXCommand().c_str());
+                       os << "\\added_space_top "
+                          << added_space_top.asLyXCommand() << " ";
                if (added_space_bottom.kind() != VSpace::NONE)
-                       fprintf(file, "\\added_space_bottom %s ",
-                               added_space_bottom.asLyXCommand().c_str());
+                       os << "\\added_space_bottom "
+                          << added_space_bottom.asLyXCommand() << " ";
                        
-               /* The labelwidth string used in lists */
+               // The labelwidth string used in lists.
                if (!labelwidthstring.empty())
-                       fprintf(file, "\\labelwidthstring %s\n",
-                               labelwidthstring.c_str());
+                       os << "\\labelwidthstring "
+                          << labelwidthstring << '\n';
 
-               /* Lines above or below? */
+               // Lines above or below?
                if (line_top)
-                       fprintf(file, "\\line_top ");
+                       os << "\\line_top ";
                if (line_bottom)
-                       fprintf(file, "\\line_bottom ");
+                       os << "\\line_bottom ";
 
-               /* Pagebreaks above or below? */
+               // Pagebreaks above or below?
                if (pagebreak_top)
-                       fprintf(file, "\\pagebreak_top ");
+                       os << "\\pagebreak_top ";
                if (pagebreak_bottom)
-                       fprintf(file, "\\pagebreak_bottom ");
+                       os << "\\pagebreak_bottom ";
                        
-               /* Start of appendix? */
+               // Start of appendix?
                if (start_of_appendix)
-                       fprintf(file, "\\start_of_appendix ");
+                       os << "\\start_of_appendix ";
 
-               /* Noindent? */
+               // Noindent?
                if (noindent)
-                       fprintf(file, "\\noindent ");
+                       os << "\\noindent ";
                        
-               /* Alignment? */
+               // Alignment?
                if (align != LYX_ALIGN_LAYOUT) {
                        switch (align) {
                        case LYX_ALIGN_LEFT: h = 1; break;
@@ -198,61 +209,58 @@ void LyXParagraph::writeFile(FILE * file, BufferParams & params,
                        case LYX_ALIGN_CENTER: h = 3; break;
                        default: h = 0; break;
                        }
-                       fprintf(file, "\\align %s ", string_align[h]);
+                       os << "\\align " << string_align[h] << " ";
                }
                 if (pextra_type != PEXTRA_NONE) {
-                        fprintf(file, "\\pextra_type %d", pextra_type);
+                        os << "\\pextra_type " << pextra_type;
                         if (pextra_type == PEXTRA_MINIPAGE) {
-                               fprintf(file, " \\pextra_alignment %d",
-                                       pextra_alignment);
+                               os << " \\pextra_alignment "
+                                  << pextra_alignment;
                                if (pextra_hfill)
-                                       fprintf(file, " \\pextra_hfill %d",
-                                               pextra_hfill);
+                                       os << " \\pextra_hfill "
+                                          << pextra_hfill;
                                if (pextra_start_minipage)
-                                       fprintf(file,
-                                               " \\pextra_start_minipage %d",
-                                               pextra_start_minipage);
+                                       os << " \\pextra_start_minipage "
+                                          << pextra_start_minipage;
                         }
                         if (!pextra_width.empty()) {
-                               fprintf(file, " \\pextra_width %s",
-                                        VSpace(pextra_width)
-                                       .asLyXCommand().c_str());
+                               os << " \\pextra_width "
+                                  << VSpace(pextra_width).asLyXCommand();
                         } else if (!pextra_widthp.empty()) {
-                               fprintf(file, " \\pextra_widthp %s",
-                                       pextra_widthp.c_str());
+                               os << " \\pextra_widthp "
+                                  << pextra_widthp;
                         }
-                        fprintf(file, "\n");
+                       os << '\n';
                 }
-       }
-       else {
-               /* Dummy layout. This means that a footnote ended */
-               fprintf(file, "\n\\end_float ");
+       } else {
+               // Dummy layout. This means that a footnote ended.
+               os << "\n\\end_float ";
                footflag = LyXParagraph::NO_FOOTNOTE;
        }
                
-       /* It might be a table */ 
+       // It might be a table.
        if (table){
-               fprintf(file, "\\LyXTable\n");
-               table->Write(file);
+               os << "\\LyXTable\n";
+               table->Write(os);
        }
 
        // bibitem  ale970302
        if (bibkey)
-               bibkey->Write(file);
+               bibkey->Write(os);
 
        font1 = LyXFont(LyXFont::ALL_INHERIT);
 
        column = 0;
-       for (size_type i = 0; i < size(); i++) {
-               if (!i){
-                       fprintf(file, "\n");
+       for (size_type i = 0; i < size(); ++i) {
+               if (!i) {
+                       os << "\n";
                        column = 0;
                }
                
                // Write font changes
                font2 = GetFontSettings(i);
                if (font2 != font1) {
-                       font2.lyxWriteChanges(font1, file);
+                       font2.lyxWriteChanges(font1, os);
                        column = 0;
                        font1 = font2;
                }
@@ -266,67 +274,67 @@ void LyXParagraph::writeFile(FILE * file, BufferParams & params,
                                        // international char, let it write
                                        // code directly so it's shorter in
                                        // the file
-                                       inset->Write(file);
+                                       inset->Write(os);
                                } else {
-                                       fprintf(file, "\n\\begin_inset ");
-                                       inset->Write(file);
-                                       fprintf(file, "\n\\end_inset \n");
-                                       fprintf(file, "\n");
+                                       os << "\n\\begin_inset ";
+                                       inset->Write(os);
+                                       os << "\n\\end_inset \n\n";
                                        column = 0;
                                }
                        break;
                case META_NEWLINE: 
-                       fprintf(file, "\n\\newline \n");
+                       os << "\n\\newline \n";
                        column = 0;
                        break;
                case META_HFILL: 
-                       fprintf(file, "\n\\hfill \n");
+                       os << "\n\\hfill \n";
                        column = 0;
                        break;
                case META_PROTECTED_SEPARATOR: 
-                       fprintf(file, "\n\\protected_separator \n");
+                       os << "\n\\protected_separator \n";
                        column = 0;
                        break;
-               case '\\': 
-                       fprintf(file, "\n\\backslash \n");
+               case '\\':
+                       os << "\n\\backslash \n";
                        column = 0;
                        break;
                case '.':
                        if (i + 1 < size() && GetChar(i + 1) == ' ') {
-                               fprintf(file, ".\n");
+                               os << ".\n";
                                column = 0;
                        } else
-                               fprintf(file, ".");
+                               os << ".";
                        break;
                default:
                        if ((column > 70 && c == ' ')
                            || column > 79){
-                               fprintf(file, "\n");
+                               os << "\n";
                                column = 0;
                        }
                        // this check is to amend a bug. LyX sometimes
                        // inserts '\0' this could cause problems.
                        if (c != '\0')
-                               fprintf(file, "%c", c);
+                               os << c;
                        else
                                lyxerr << "ERROR (LyXParagraph::writeFile):"
                                        " NULL char in structure." << endl;
-                       column++;
+                       ++column;
                        break;
                }
        }
 
        // now write the next paragraph
        if (next)
-               next->writeFile(file, params, footflag, dth);
+               next->writeFile(os, params, footflag, dth);
 }
 
 
 void LyXParagraph::validate(LaTeXFeatures & features)
 {
        // this will be useful later
-       LyXLayout const & layout = textclasslist.Style(GetCurrentTextClass(), 
-                                          GetLayout());
+       LyXLayout const & layout =
+               textclasslist.Style(current_view->buffer()->params.textclass, 
+                                   GetLayout());
        
        // check the params.
        if (line_top || line_bottom)
@@ -335,7 +343,8 @@ void LyXParagraph::validate(LaTeXFeatures & features)
        // then the layouts
        features.layout[GetLayout()] = true;
 
-#ifdef NEW_TABLE
+       // then the fonts
+#ifdef NEW_FONTTABLE // OK, but does not affect structure anyway
        for (FontList::const_iterator cit = fontlist.begin();
             cit != fontlist.end(); ++cit) {
                if ((*cit).font.noun() == LyXFont::ON) {
@@ -350,7 +359,8 @@ void LyXParagraph::validate(LaTeXFeatures & features)
                switch ((*cit).font.color()) {
                case LyXFont::NONE:
                case LyXFont::INHERIT_COLOR:
-               case LyXFont::IGNORE_COLOR:  break;
+               case LyXFont::IGNORE_COLOR:
+                       break;
                default:
                        features.color = true;
                        lyxerr[Debug::LATEX] << "Color enabled. Font: "
@@ -359,7 +369,6 @@ void LyXParagraph::validate(LaTeXFeatures & features)
                }
        }
 #else
-       // then the fonts
        FontTable * tmpfonttable = fonttable;
        while (tmpfonttable) {
                if (tmpfonttable->font.noun() == LyXFont::ON) {
@@ -385,13 +394,14 @@ void LyXParagraph::validate(LaTeXFeatures & features)
                tmpfonttable = tmpfonttable->next;
        }
 #endif
-#ifdef NEW_TABLE
+       // then the insets
+#ifdef NEW_INSETTABLE
        for (InsetList::const_iterator cit = insetlist.begin();
             cit != insetlist.end(); ++cit) {
-               (*cit).inset->Validate(features);
+               if ((*cit).inset)
+                       (*cit).inset->Validate(features);
        }
 #else
-       // then the insets
        InsetTable * tmpinsettable = insettable;
        while (tmpinsettable) {
                if (tmpinsettable->inset) {
@@ -400,6 +410,7 @@ void LyXParagraph::validate(LaTeXFeatures & features)
                tmpinsettable = tmpinsettable->next;
        }
 #endif
+
         if (table && table->IsLongTable())
                features.longtable = true;
         if (pextra_type == PEXTRA_INDENT)
@@ -419,7 +430,7 @@ void LyXParagraph::validate(LaTeXFeatures & features)
 }
 
 
-/* first few functions needed for cut and paste and paragraph breaking */
+// First few functions needed for cut and paste and paragraph breaking.
 void LyXParagraph::CopyIntoMinibuffer(LyXParagraph::size_type pos) const
 {
        minibuffer_char = GetChar(pos);
@@ -436,6 +447,7 @@ void LyXParagraph::CopyIntoMinibuffer(LyXParagraph::size_type pos) const
        }
 }
 
+
 void LyXParagraph::CutIntoMinibuffer(LyXParagraph::size_type pos)
 {
        minibuffer_char = GetChar(pos);
@@ -447,21 +459,22 @@ void LyXParagraph::CutIntoMinibuffer(LyXParagraph::size_type pos)
                        // This is a little hack since I want exactly
                        // the inset, not just a clone. Otherwise
                        // the inset would be deleted when calling Erase(pos)
-#ifdef NEW_TABLE
+                       // find the entry
+#ifdef NEW_INSETTABLE
                        for (InsetList::iterator it = insetlist.begin();
                             it != insetlist.end(); ++it) {
                                if ((*it).pos == pos) {
                                        (*it).inset = 0;
                                        break;
                                }
+                                       
                        }
 #else
-                       /* find the entry */ 
                        InsetTable * tmpi = insettable;
                        while (tmpi && tmpi->pos != pos) {
                                tmpi= tmpi->next;
                        }
-                       if (tmpi) {  /* This should always be true */
+                       if (tmpi) {  // This should always be true.
                                tmpi->inset = 0;
                        }
 #endif
@@ -473,7 +486,7 @@ void LyXParagraph::CutIntoMinibuffer(LyXParagraph::size_type pos)
 
        }
 
-       /* Erase(pos); now the caller is responsible for that*/
+       // Erase(pos); now the caller is responsible for that.
 }
 
 
@@ -485,7 +498,7 @@ void LyXParagraph::InsertFromMinibuffer(LyXParagraph::size_type pos)
                InsertInset(pos, minibuffer_inset);
 }
 
-/* end of minibuffer */ 
+// end of minibuffer
 
 
 
@@ -520,7 +533,7 @@ void LyXParagraph::Clear()
 }
 
 
-/* the destructor removes the new paragraph from the list */ 
+// the destructor removes the new paragraph from the list
 LyXParagraph::~LyXParagraph()
 {
        if (previous)
@@ -528,26 +541,32 @@ LyXParagraph::~LyXParagraph()
        if (next)
                next->previous = previous;
 
-#ifndef NEW_TABLE
+#ifdef NEW_INSETTABLE
+       for (InsetList::iterator it = insetlist.begin();
+            it != insetlist.end(); ++it) {
+               delete (*it).inset;
+       }
+#else
        InsetTable * tmpinset;
        while (insettable) {
                tmpinset = insettable;
                insettable = insettable->next;
-               if (tmpinset->inset)
+               if (tmpinset->inset) // DEL LINE
                        delete tmpinset->inset;
                delete tmpinset;
                if (insettable && insettable->next == insettable) {
                        // somehow this recursion appears occasionally
                        // but I can't find where.  This bandaid
                        // helps but isn't the best fix. (ARRae)
-                       if (insettable->inset) {
+                       if (insettable->inset) { // DEL LINE
                                delete insettable->inset;
                        }
                        delete insettable;
                        break;
                }
        }
-
+#endif
+#ifndef NEW_FONTTABLE // OK
        FontTable * tmpfont;
        while (fonttable) {
                tmpfont = fonttable;
@@ -557,20 +576,20 @@ LyXParagraph::~LyXParagraph()
 #endif
 
        /* table stuff -- begin*/ 
-       if (table)
+       if (table) // DEL LINE
                delete table;
        /* table stuff -- end*/ 
 
         // ale970302
-       if (bibkey)
+       if (bibkey) // DEL LINE
                delete bibkey;
 }
 
 
 void LyXParagraph::Erase(LyXParagraph::size_type pos)
 {
-       /* > because last is the next unused position, and you can 
-        * use it if you want  */
+       // > because last is the next unused position, and you can 
+       // use it if you want
        if (pos > size()) {
                if (next && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) 
                        NextAfterFootnote()->Erase(pos - text.size() - 1);
@@ -580,31 +599,27 @@ void LyXParagraph::Erase(LyXParagraph::size_type pos)
                return;
        }
        if (pos < size()) { // last is free for insertation, but should be empty
-#ifdef NEW_TABLE
-               /* if it is an inset, delete the inset entry */ 
+               // if it is an inset, delete the inset entry 
                if (text[pos] == LyXParagraph::META_INSET) {
-                       /* find the entry */
-                       for(InsetList::iterator it = insetlist.begin();
-                           it != insetlist.end(); ++it) {
+                       // find the entry
+#ifdef NEW_INSETTABLE
+                       for (InsetList::iterator it = insetlist.begin();
+                            it != insetlist.end(); ++it) {
                                if ((*it).pos == pos) {
                                        delete (*it).inset;
                                        insetlist.erase(it);
                                        break;
                                }
                        }
-               }
 #else
-               /* if it is an inset, delete the inset entry */ 
-               if (text[pos] == LyXParagraph::META_INSET) {
-                       /* find the entry */ 
                        InsetTable *tmpi = insettable;
                        InsetTable *tmpi2 = tmpi;
                        while (tmpi && tmpi->pos != pos) {
-                               tmpi2= tmpi;
-                               tmpi= tmpi->next;
+                               tmpi2 = tmpi;
+                               tmpi = tmpi->next;
                        }
                        if (tmpi) {     // this should always be true
-                               if (tmpi->inset) // delete the inset if it exists
+                               if (tmpi->inset) // delete the inset if it exists // DEL LINE
                                        delete tmpi->inset;
                                if (tmpi == insettable)
                                        insettable = tmpi->next;
@@ -612,38 +627,33 @@ void LyXParagraph::Erase(LyXParagraph::size_type pos)
                                        tmpi2->next = tmpi->next;
                                delete tmpi;
                        }
-               }
 #endif
+               }
                text.erase(text.begin() + pos);
-#ifdef NEW_TABLE
-               /* erase entries in the tables */
-               for(FontList::iterator it = fontlist.begin();
-                   it != fontlist.end(); ++it) {
+               // Erase entries in the tables.
+#ifdef NEW_FONTTABLE // Seems OK
+               for (FontList::iterator it = fontlist.begin();
+                    it != fontlist.end(); ++it) {
                        if (pos >= (*it).pos && pos <= (*it).pos_end) {
                                if ((*it).pos == (*it).pos_end) {
+                                       // If it is a multi-character font
+                                       // entry, we just make it smaller
+                                       // (see update below), otherwise we
+                                       // should delete it.
                                        fontlist.erase(it);
                                        break;
                                }
                        }
                }
-
-               /* update all other entries */
-               for(FontList::iterator it = fontlist.begin();
-                   it != fontlist.end(); ++it) {
+               // Update all other entries.
+               for (FontList::iterator it = fontlist.begin();
+                    it != fontlist.end(); ++it) {
                        if ((*it).pos > pos)
                                (*it).pos--;
                        if ((*it).pos_end >= pos)
                                (*it).pos_end--;
                }
-      
-               /* update the inset table */
-               for(InsetList::iterator it = insetlist.begin();
-                   it != insetlist.end(); ++it) {
-                       if ((*it).pos > pos)
-                               (*it).pos--;
-               }
 #else
-               /* erase entries in the tables */ 
                int found = 0;
                FontTable * tmp = fonttable;
                FontTable * prev = 0;
@@ -657,9 +667,9 @@ void LyXParagraph::Erase(LyXParagraph::size_type pos)
                }
       
                if (found && tmp->pos == tmp->pos_end) {  
-                       /* if it is a multi-character font entry, we just make 
-                        * it smaller (see update below), otherwise we should 
-                        * delete it */
+                       // if it is a multi-character font entry, we just make 
+                       // it smaller (see update below), otherwise we should 
+                       // delete it
                        if (prev)
                                prev->next = tmp->next;
                        else
@@ -668,7 +678,7 @@ void LyXParagraph::Erase(LyXParagraph::size_type pos)
                        delete tmp;
                }
 
-               /* update all other entries */
+               // Update all other entries.
 
                tmp = fonttable;
                while (tmp) {
@@ -678,15 +688,22 @@ void LyXParagraph::Erase(LyXParagraph::size_type pos)
                                tmp->pos_end--;
                        tmp = tmp->next;
                }
-      
-               /* update the inset table */ 
+#endif      
+               // Update the inset table.
+#ifdef NEW_INSETTABLE
+               for (InsetList::iterator it = insetlist.begin();
+                    it != insetlist.end(); ++it) {
+                       if ((*it).pos > pos)
+                               (*it).pos--;
+               }
+#else
                InsetTable * tmpi = insettable;
                while (tmpi) {
                        if (tmpi->pos > pos)
                                tmpi->pos--;
                        tmpi= tmpi->next;
                }
-#endif      
+#endif
        } else {
                lyxerr << "ERROR (LyXParagraph::Erase): "
                        "can't erase non-existant char." << endl;
@@ -696,8 +713,8 @@ void LyXParagraph::Erase(LyXParagraph::size_type pos)
 
 void LyXParagraph::InsertChar(LyXParagraph::size_type pos, char c)
 {
-       /* > because last is the next unused position, and you can 
-        * use it if you want  */
+       // > because last is the next unused position, and you can 
+       // use it if you want
        if (pos > size()) {
                if (next
                    && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) 
@@ -709,23 +726,16 @@ void LyXParagraph::InsertChar(LyXParagraph::size_type pos, char c)
                return;
        }
        text.insert(text.begin() + pos, c);
-#ifdef NEW_TABLE
-       // update the font table
-       for(FontList::iterator it = fontlist.begin();
-           it != fontlist.end(); ++it) {
+       // Update the font table.
+#ifdef NEW_FONTTABLE // Seems OK
+       for (FontList::iterator it = fontlist.begin();
+            it != fontlist.end(); ++it) {
                if ((*it).pos >= pos)
                        (*it).pos++;
                if ((*it).pos_end >= pos)
                        (*it).pos_end++;
        }
-       // update the inset table
-       for(InsetList::iterator it = insetlist.begin();
-           it != insetlist.end(); ++it) {
-               if ((*it).pos >= pos)
-                       (*it).pos++;
-       }
 #else
-       /* update the font table */ 
        FontTable * tmp = fonttable;
        while (tmp) {
                if (tmp->pos >= pos)
@@ -734,8 +744,16 @@ void LyXParagraph::InsertChar(LyXParagraph::size_type pos, char c)
                        tmp->pos_end++;
                tmp = tmp->next;
        }
+#endif
    
-       /* update the inset table */ 
+       // Update the inset table.
+#ifdef NEW_INSETTABLE
+       for (InsetList::iterator it = insetlist.begin();
+            it != insetlist.end(); ++it) {
+               if ((*it).pos >= pos)
+                       (*it).pos++;
+       }
+#else
        InsetTable * tmpi = insettable;
        while (tmpi) {
                if (tmpi->pos >= pos)
@@ -749,8 +767,8 @@ void LyXParagraph::InsertChar(LyXParagraph::size_type pos, char c)
 void LyXParagraph::InsertInset(LyXParagraph::size_type pos,
                               Inset * inset)
 {
-       /* > because last is the next unused position, and you can 
-        * use it if you want  */
+       // > because last is the next unused position, and you can 
+       // use it if you want
        if (pos > size()) {
                if (next
                    && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) 
@@ -767,24 +785,22 @@ void LyXParagraph::InsertInset(LyXParagraph::size_type pos,
                return;
        }
 
-#ifdef NEW_TABLE
        if (inset) {
-               InsetTable tmp;
-               tmp.pos = pos;
-               tmp.inset = inset;
-               insetlist.push_back(tmp);
-       }
-               
+               // Add a new entry in the inset table.
+#ifdef NEW_INSETTABLE
+               InsetTable tmpi;
+               InsetList::iterator it =
+                       insetlist.insert(insetlist.begin(), tmpi);
+               (*it).inset = inset;
+               (*it).pos = pos;
 #else
-       if (inset) {
-               /* add a new entry in the inset table */ 
                InsetTable * tmpi = new InsetTable;
                tmpi->pos = pos;
                tmpi->inset = inset;
                tmpi->next = insettable;
                insettable = tmpi;
-       }
 #endif
+       }
 }
 
 
@@ -802,24 +818,23 @@ Inset * LyXParagraph::GetInset(LyXParagraph::size_type pos)
                }
                return 0;
        }
-#ifdef NEW_TABLE
-       /* find the inset */
-       for(InsetList::iterator it = insetlist.begin();
-           it != insetlist.end(); ++it) {
+       // Find the inset.
+#ifdef NEW_INSETTABLE
+       for (InsetList::iterator it = insetlist.begin();
+            it != insetlist.end(); ++it) {
                if ((*it).pos == pos) {
                        return (*it).inset;
                }
        }
        lyxerr << "ERROR (LyXParagraph::GetInset): "
                "Inset does not exist: " << pos << endl;
-       text[pos] = ' '; /// WHY!!! does this set the pos to ' '????
+       text[pos] = ' '; // WHY!!! does this set the pos to ' '????
        // Did this commenting out introduce a bug? So far I have not
-       // seen any, please enlighten me. (Lgb)
+       // see any, please enlighten me. (Lgb)
        // My guess is that since the inset does not exist, we might
-       // as well replace it with a space to prevent crashes. (Asger)
+       // as well replace it with a space to prevent craches. (Asger)
        return 0;
 #else
-       /* find the inset */ 
        InsetTable * tmpi = insettable;
 
        while (tmpi && tmpi->pos != pos)
@@ -855,24 +870,23 @@ Inset const * LyXParagraph::GetInset(LyXParagraph::size_type pos) const
                }
                return 0;
        }
-#ifdef NEW_TABLE
-       /* find the inset */
-       for(InsetList::const_iterator cit = insetlist.begin();
-           cit != insetlist.end(); ++cit) {
+       // Find the inset.
+#ifdef NEW_INSETTABLE
+       for (InsetList::const_iterator cit = insetlist.begin();
+            cit != insetlist.end(); ++cit) {
                if ((*cit).pos == pos) {
                        return (*cit).inset;
                }
        }
        lyxerr << "ERROR (LyXParagraph::GetInset): "
                "Inset does not exist: " << pos << endl;
-       text[pos] = ' '; /// WHY!!! does this set the pos to ' '????
+       //text[pos] = ' '; // WHY!!! does this set the pos to ' '????
        // Did this commenting out introduce a bug? So far I have not
-       // seen any, please enlighten me. (Lgb)
+       // see any, please enlighten me. (Lgb)
        // My guess is that since the inset does not exist, we might
-       // as well replace it with a space to prevent crashes. (Asger)
+       // as well replace it with a space to prevent craches. (Asger)
        return 0;
 #else
-       /* find the inset */ 
        InsetTable * tmpi = insettable;
 
        while (tmpi && tmpi->pos != pos)
@@ -900,11 +914,11 @@ Inset const * LyXParagraph::GetInset(LyXParagraph::size_type pos) const
 LyXFont LyXParagraph::GetFontSettings(LyXParagraph::size_type pos) const
 {
        if (pos < size()) {
-#ifdef NEW_TABLE
-               for(FontList::iterator it = fontlist.begin();
-                   it != fontlist.end(); ++it) {
-                       if (pos >= (*it).pos && pos <= (*it).pos_end)
-                               return (*it).font;
+#ifdef NEW_FONTTABLE // Seems OK
+               for (FontList::const_iterator cit = fontlist.begin();
+                    cit != fontlist.end(); ++cit) {
+                       if (pos >= (*cit).pos && pos <= (*cit).pos_end)
+                               return (*cit).font;
                }
 #else
                FontTable * tmp = fonttable;
@@ -915,8 +929,8 @@ LyXFont LyXParagraph::GetFontSettings(LyXParagraph::size_type pos) const
                }
 #endif
        }
-       /* > because last is the next unused position, and you can 
-        * use it if you want  */
+       // > because last is the next unused position, and you can 
+       // use it if you want
        else if (pos > size()) {
                if (next
                    && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) 
@@ -925,13 +939,13 @@ LyXFont LyXParagraph::GetFontSettings(LyXParagraph::size_type pos) const
                else {
                        // Why is it an error to ask for the font of a
                        // position that does not exist? Would it be
-                       // enough for this to be anable on debug?
+                       // enough for this to be enabled on debug?
                        // We want strict error checking, but it's ok to only
                        // have it when debugging. (Asger)
                        lyxerr << "ERROR (LyXParagraph::GetFontSettings): "
                                "position does not exist. "
                               << pos << " (" << static_cast<int>(pos)
-                              << ")\n";
+                              << ")" << endl;
                }
        } else if (pos) {
                return GetFontSettings(pos - 1);
@@ -950,8 +964,9 @@ LyXFont LyXParagraph::GetFontSettings(LyXParagraph::size_type pos) const
 LyXFont LyXParagraph::getFont(LyXParagraph::size_type pos) const
 {
        LyXFont tmpfont;
-       LyXLayout const & layout = textclasslist.Style(GetCurrentTextClass(), 
-                                          GetLayout());
+       LyXLayout const & layout =
+               textclasslist.Style(current_view->buffer()->params.textclass, 
+                                   GetLayout());
        LyXParagraph::size_type main_body = 0;
        if (layout.labeltype == LABEL_MANUAL)
                main_body = BeginningOfMainBody();
@@ -979,24 +994,28 @@ LyXFont LyXParagraph::getFont(LyXParagraph::size_type pos) const
                par = par->DepthHook(par_depth - 1);
                if (par) {
                        tmpfont.realize(textclasslist.
-                                       Style(GetCurrentTextClass(),
+                                       Style(current_view->buffer()->params.textclass,
                                              par->GetLayout()).font);
                        par_depth = par->GetDepth();
                }
        }
 
-       tmpfont.realize(textclasslist.TextClass(GetCurrentTextClass()).defaultfont());
+       tmpfont.realize(textclasslist
+                       .TextClass(current_view->buffer()->params.textclass)
+                       .defaultfont());
        return tmpfont;
 }
 
 
 /// Returns the height of the highest font in range
-LyXFont::FONT_SIZE LyXParagraph::HighestFontInRange(LyXParagraph::size_type startpos, LyXParagraph::size_type endpos) const
+LyXFont::FONT_SIZE
+LyXParagraph::HighestFontInRange(LyXParagraph::size_type startpos,
+                                LyXParagraph::size_type endpos) const
 {
        LyXFont::FONT_SIZE maxsize = LyXFont::SIZE_TINY;
-#ifdef NEW_TABLE
-       for(FontList::const_iterator cit = fontlist.begin();
-           cit != fontlist.end(); ++cit) {
+#ifdef NEW_FONTTABLE // Seems OK
+       for (FontList::const_iterator cit = fontlist.begin();
+            cit != fontlist.end(); ++cit) {
                if (startpos <= (*cit).pos_end && endpos >= (*cit).pos) {
                        LyXFont::FONT_SIZE size = (*cit).font.size();
                        if (size > maxsize && size <= LyXFont::SIZE_HUGER)
@@ -1020,22 +1039,13 @@ LyXFont::FONT_SIZE LyXParagraph::HighestFontInRange(LyXParagraph::size_type star
 
 char LyXParagraph::GetChar(LyXParagraph::size_type pos)
 {
-#ifdef DEVEL_VERSION
-       /* a workaround to 'fix' some bugs in text-class */
-       if (pos < 0) {
-               // This function is important. It should not work around bugs.
-               // Let's find the bugs instead and fix them. (Asger)
-               lyxerr << "FATAL ERROR (LyXParagraph::GetChar):"
-                       " bad position "  << pos << endl;
-               abort();
-       }
-#endif
+       Assert(pos >= 0);
 
        if (pos < size()) {
                return text[pos];
        }
-       /* > because last is the next unused position, and you can 
-        * use it if you want  */
+       // > because last is the next unused position, and you can 
+       // use it if you want
        else if (pos > size()) {
                if (next && next->footnoteflag != LyXParagraph::NO_FOOTNOTE) 
                        return NextAfterFootnote()
@@ -1045,10 +1055,11 @@ char LyXParagraph::GetChar(LyXParagraph::size_type pos)
                                "position does not exist."
                               << pos << " (" << static_cast<int>(pos)
                               << ")\n";
+                       Assert(false);
                }
                return '\0';
        } else {
-               /* we should have a footnote environment */ 
+               // We should have a footnote environment.
                if (!next || next->footnoteflag == LyXParagraph::NO_FOOTNOTE) {
                        // Notice that LyX does request the
                        // last char from time to time. (Asger)
@@ -1077,35 +1088,27 @@ char LyXParagraph::GetChar(LyXParagraph::size_type pos)
 
 char LyXParagraph::GetChar(LyXParagraph::size_type pos) const
 {
-#ifdef DEVEL_VERSION
-       /* a workaround to 'fix' some bugs in text-class */
-       if (pos < 0) {
-               // This function is important. It should not work around bugs.
-               // Let's find the bugs instead and fix them. (Asger)
-               lyxerr << "FATAL ERROR (LyXParagraph::GetChar):"
-                       " bad position "  << pos << endl;
-               abort();
-       }
-#endif
+       Assert(pos >= 0);
 
        if (pos < size()) {
                return text[pos];
        }
-       /* > because last is the next unused position, and you can 
-        * use it if you want  */
+       // > because last is the next unused position, and you can 
+       // use it if you want
        else if (pos > size()) {
                if (next && next->footnoteflag != LyXParagraph::NO_FOOTNOTE) 
                        return NextAfterFootnote()
                                ->GetChar(pos - text.size() - 1);
                else {
-                       lyxerr << "ERROR (LyXParagraph::GetChar): "
+                       lyxerr << "ERROR (LyXParagraph::GetChar const): "
                                "position does not exist."
                               << pos << " (" << static_cast<int>(pos)
                               << ")\n";
+                       Assert(false);
                }
                return '\0';
        } else {
-               /* we should have a footnote environment */ 
+               // We should have a footnote environment.
                if (!next || next->footnoteflag == LyXParagraph::NO_FOOTNOTE) {
                        // Notice that LyX does request the
                        // last char from time to time. (Asger)
@@ -1132,40 +1135,20 @@ char LyXParagraph::GetChar(LyXParagraph::size_type pos) const
 }
 
 
+// return an string of the current word, and the end of the word in lastpos.
 string LyXParagraph::GetWord(LyXParagraph::size_type & lastpos) const
-  //Added 98/9/21 by REH
-  // return an string of the current word, and the end of the word
-  // in lastpos.
-
-  // the current word is defined as starting at the first character from
-  // the immediate left of lastpospos which meets the definition of IsLetter(),
-  // continuing to the last character to the right of this meeting
-  // IsLetter.
-
-
-  // i just left this in from GetChar()
 {
-#ifdef DEVEL_VERSION
-       /* a workaround to 'fix' some bugs in text-class */
-       if (lastpos < 0) {
-               // This function is important. It should not work around bugs.
-               // Let's find the bugs instead and fix them. (Asger)
-               lyxerr << "FATAL ERROR (LyXParagraph::GetWord):"
-                       " bad position " << lastpos << endl;
-               abort();
-       }
-#endif
+       Assert(lastpos>=0);
+
+       // the current word is defined as starting at the first character
+       // from the immediate left of lastpospos which meets the definition
+       // of IsLetter(), continuing to the last character to the right
+       // of this meeting IsLetter.
 
        string theword;
 
        // grab a word
-       
-       
-       //i think the devcode aborts before this, but why not be
-       // versatile?
-       if (lastpos < 0) lastpos= 0; 
-
-       
+               
        // move back until we have a letter
 
        //there's no real reason to have firstpos & lastpos as
@@ -1176,16 +1159,16 @@ string LyXParagraph::GetWord(LyXParagraph::size_type & lastpos) const
        int firstpos = lastpos;
        
        while ((firstpos >= 0) && !IsLetter(firstpos))
-               firstpos--;
+               --firstpos;
 
        // now find the beginning by looking for a nonletter
        
        while ((firstpos>= 0) && IsLetter(firstpos))
-               firstpos--;
+               --firstpos;
 
        // the above is now pointing to the preceeding non-letter
-       firstpos++;
-       lastpos= firstpos;
+       ++firstpos;
+       lastpos = firstpos;
 
        // so copy characters into theword  until we get a nonletter
        // note that this can easily exceed lastpos, wich means
@@ -1194,7 +1177,7 @@ string LyXParagraph::GetWord(LyXParagraph::size_type & lastpos) const
 
        while (IsLetter(lastpos)) theword += GetChar(lastpos++);
        
-       return  theword;
+       return theword;
 
 }
 
@@ -1203,8 +1186,8 @@ LyXParagraph::size_type LyXParagraph::Last() const
 {
        if (next && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE)
                return text.size() + NextAfterFootnote()->Last() + 1;
-                                                  /* the 1 is the symbol
-                                                     for the footnote */
+       // the 1 is the symbol
+       // for the footnote
        else
                return text.size();
 }
@@ -1212,8 +1195,8 @@ LyXParagraph::size_type LyXParagraph::Last() const
 
 LyXParagraph * LyXParagraph::ParFromPos(LyXParagraph::size_type pos)
 {
-       /* > because last is the next unused position, and you can 
-        * use it if you want  */
+       // > because last is the next unused position, and you can 
+       // use it if you want
        if (pos > size()) {
                if (next
                    && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) 
@@ -1223,16 +1206,15 @@ LyXParagraph * LyXParagraph::ParFromPos(LyXParagraph::size_type pos)
                        lyxerr << "ERROR (LyXParagraph::ParFromPos): "
                                "position does not exist." << endl;
                return this;
-       }
-       else
+       } else
                return this;
 }
 
 
 int LyXParagraph::PositionInParFromPos(LyXParagraph::size_type pos) const
 {
-       /* > because last is the next unused position, and you can 
-        * use it if you want  */
+       // > because last is the next unused position, and you can 
+       // use it if you want
        if (pos > size()) {
                if (next
                    && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) 
@@ -1252,10 +1234,11 @@ int LyXParagraph::PositionInParFromPos(LyXParagraph::size_type pos) const
 void LyXParagraph::SetFont(LyXParagraph::size_type pos,
                           LyXFont const & font)
 {
-       /* > because last is the next unused position, and you can 
-        * use it if you want  */
+       // > because last is the next unused position, and you can 
+       // use it if you want
        if (pos > size()) {
-               if (next && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) {
+               if (next &&
+                   next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) {
                        NextAfterFootnote()->SetFont(pos - text.size() - 1,
                                                     font);
                } else {
@@ -1266,99 +1249,116 @@ void LyXParagraph::SetFont(LyXParagraph::size_type pos,
        }
        LyXFont patternfont(LyXFont::ALL_INHERIT);
 
-// First, reduce font against layout/label font
-// Update: The SetCharFont() routine in text2.C already reduces font, so
-// we don't need to do that here. (Asger)
-// No need to simplify this because it will disappear in a new kernel. (Asger)
-#ifdef NEW_TABLE
+       // First, reduce font against layout/label font
+       // Update: The SetCharFont() routine in text2.C already
+       // reduces font, so we don't need to do that here. (Asger)
+       // No need to simplify this because it will disappear
+       // in a new kernel. (Asger)
        // Next search font table
-       for(FontList::iterator it = fontlist.begin();
-           it != fontlist.end(); ++it) {
-               if (pos >= (*it).pos && pos <= (*it).pos_end) {
-                       // found it
-                       // we found a font entry. maybe we have to
-                       // split it and create a new one 
-                       
-                       if ((*it).pos != (*it).pos_end) {
-                               // more than one character
-                               if (pos == (*it).pos) {
-                                       // maybe we could enlarge
-                                       // the left fonttable
-                                       for(FontList::iterator fit = fontlist.begin();
-                                           fit != fontlist.end(); ++fit) {
-                                               if (pos - 1 >= (*fit).pos
-                                                   && pos - 1 <= (*fit).pos_end
-                                                   && (*fit).font == font) {
-                                                       // put the position
-                                                       // under the font
-                                                       (*fit).pos_end++;
-                                                       (*it).pos++;
-                                                       return;
-                                               }
-                                       }
-                                       // Add a new entry in the 
-                                       // fonttable for the position
-                                       FontTable tmp;
-                                       tmp.pos = pos + 1;
-                                       tmp.pos_end = (*it).pos_end;
-                                       tmp.font = (*it).font;
-                                       (*it).pos_end = pos;
-                                       fontlist.push_back(tmp);
-                               } else if (pos == (*it).pos_end) {
-                                       // Add a new entry in the 
-                                       // fonttable for the position
-                                       FontTable tmp;
-                                       tmp.pos = (*it).pos;
-                                       tmp.pos_end = (*it).pos_end - 1;
-                                       tmp.font = (*it).font;
-                                       (*it).pos = (*it).pos_end;
-                                       fontlist.push_back(tmp);
-                               } else {
-                                       // Add a new entry in the 
-                                       // fonttable for the position
-                                       FontTable tmp;
-                                       tmp.pos = (*it).pos;
-                                       tmp.pos_end = pos - 1;
-                                       tmp.font = (*it).font;
-                                       fontlist.push_back(tmp);
-
-                                       tmp.pos = pos + 1;
-                                       tmp.pos_end = (*it).pos_end;
-                                       tmp.font = (*it).font;
-                                       fontlist.push_back(tmp);
-                                       
-                                       (*it).pos = pos;
-                                       (*it).pos_end = pos;
-                               }
-                       }
-                       (*it).font = font;
-                       return;
+#ifdef NEW_FONTTABLE
+       FontList::iterator tmp = fontlist.begin();
+       for (; tmp != fontlist.end(); ++tmp) {
+               if (pos >= (*tmp).pos && pos <= (*tmp).pos_end) {
+                       break;
                }
        }
-       
-       // if we did not find a font entry, but if the font at hand
-       // is the same as default, we just forget it
-       if (font == patternfont) return;
-
-       // ok, we did not find a font entry. But maybe there is exactly
-       // the needed font entry one position left
-       for(FontList::iterator it = fontlist.begin();
-           it != fontlist.end(); ++it) {
-               if (pos - 1 >= (*it).pos && pos - 1 <= (*it).pos_end
-                   && (*it).font == font) {
-                       (*it).pos_end++;
-                       return;
+       if (tmp == fontlist.end()) { // !found
+               // if we did not find a font entry, but if the font at hand
+               // is the same as default, we just forget it
+               if (font == patternfont) return;
+
+               // ok, we did not find a font entry. But maybe there is exactly
+               // the needed font ientry one position left
+               FontList::iterator tmp2 = fontlist.begin();
+               for (; tmp2 != fontlist.end(); ++tmp2) {
+                       if (pos - 1 >= (*tmp2).pos
+                           && pos - 1 <= (*tmp2).pos_end)
+                               break;
+               }
+               if (tmp2 != fontlist.end()) {
+                       // ok there is one, maybe it is exactly
+                       // the needed font
+                       if ((*tmp2).font == font) {
+                               // put the position under the font
+                               (*tmp2).pos_end++;
+                               return;
+                       }
                }
+               // Add a new entry in the
+               // fontlist for the position
+               FontTable ft;
+               ft.pos = pos;
+               ft.pos_end = pos;
+               ft.font = font; // or patternfont
+               // It seems that using font instead of patternfont here
+               // fixes all the problems. This also surfaces a "bug" in
+               // the old code.
+               fontlist.insert(fontlist.begin(), ft);
+       } else if ((*tmp).pos != (*tmp).pos_end) { // we found a font entry. maybe we have to split it and create a new one.
+
+// more than one character
+               if (pos == (*tmp).pos) {
+                       // maybe we should enlarge the left fonttable
+                       FontList::iterator tmp2 = fontlist.begin();
+                       for (; tmp2 != fontlist.end(); ++tmp2) {
+                               if (pos - 1 >= (*tmp2).pos
+                                   && pos - 1 <= (*tmp2).pos_end)
+                                       break;
+                       }
+                       // Is there is one, and is it exactly
+                       // the needed font?
+                       if (tmp2 != fontlist.end() &&
+                           (*tmp2).font == font) {
+                               // Put the position under the font
+                               (*tmp2).pos_end++;
+                               (*tmp).pos++;
+                               return;
+                       }
+
+                       // Add a new entry in the
+                       // fontlist for the position
+                       FontTable ft;
+                       ft.pos = pos + 1;
+                       ft.pos_end = (*tmp).pos_end;
+                       ft.font = (*tmp).font;
+                       (*tmp).pos_end = pos;
+                       (*tmp).font = font;
+                       fontlist.insert(fontlist.begin(), ft);
+               } else if (pos == (*tmp).pos_end) {
+                       // Add a new entry in the
+                       // fontlist for the position
+                       FontTable ft;
+                       ft.pos = (*tmp).pos;
+                       ft.pos_end = (*tmp).pos_end - 1;
+                       ft.font = (*tmp).font;
+                       (*tmp).pos = (*tmp).pos_end;
+                       (*tmp).font = font;
+                       fontlist.insert(fontlist.begin(), ft);
+               } else {
+                       // Add a new entry in the
+                       // fontlist for the position
+                       FontTable ft;
+                       ft.pos = (*tmp).pos;
+                       ft.pos_end = pos - 1;
+                       ft.font = (*tmp).font;
+                       
+                       FontTable ft2;
+                       ft2.pos = pos + 1;
+                       ft2.pos_end = (*tmp).pos_end;
+                       ft2.font = (*tmp).font;
+                       
+                       (*tmp).pos = pos;
+                       (*tmp).pos_end = pos;
+                       (*tmp).font = font;
+                       
+                       fontlist.insert(fontlist.begin(), ft);
+                       fontlist.insert(fontlist.begin(), ft2);
+               }
+       } else {
+               (*tmp).font = font;
        }
-       // Add a new entry in the 
-       // fonttable for the position
-       FontTable tmp;
-       tmp.pos = pos;
-       tmp.pos_end = pos;
-       tmp.font = patternfont;
-       fontlist.push_back(tmp);
+       
 #else
-       // Next search font table
        FontTable * tmp2;
 
        bool found = false;
@@ -1371,13 +1371,13 @@ void LyXParagraph::SetFont(LyXParagraph::size_type pos,
        }
 
        if (!found) {
-               /* if we did not find a font entry, but if the font at hand
-                * is the same as default, we just forget it */
+               // if we did not find a font entry, but if the font at hand
+               // is the same as default, we just forget it
                if (font == patternfont)
                        return;
 
-               /* ok, we did not find a font entry. But maybe there is exactly
-                * the needed font entry one position left */ 
+               // ok, we did not find a font entry. But maybe there is exactly
+               // the needed font entry one position left
                found = false;
                tmp2 = fonttable;
                while (tmp2 && !found) {
@@ -1387,30 +1387,33 @@ void LyXParagraph::SetFont(LyXParagraph::size_type pos,
                                tmp2 = tmp2->next;
                }
                if (found) {
-                       /* ok there is one. maybe it is exactly the needed font */
+                       // ok there is one. maybe it is exactly
+                       // the needed font
                        if (tmp2->font == font) {
-                               /* put the position under the font */ 
+                               // put the position under the font
                                tmp2->pos_end++;
                                return;
                        }
                }
-               /* Add a new entry in the 
-                * fonttable for the position */
+               // Add a new entry in the
+               // fonttable for the position
                tmp = new FontTable;
                tmp->pos = pos;
                tmp->pos_end = pos;
-               tmp->font = patternfont;
+               tmp->font = patternfont; // It seems that is actually totally
+               // wrong to use patternfont here, the correct should be font
+               // lockily at the end of this function we have
+               // tmp->font = font, so this one setting it to patternfont
+               // is negated.
                tmp->next = fonttable;
                fonttable = tmp;
        } else {
-               /* we found a font entry. maybe we have to split it and create
-                * a new one */ 
-
-               if (tmp->pos != tmp->pos_end) {  /* more than one character  */
+               // we found a font entry. maybe we have
+               // to split it and create a new one
 
+               if (tmp->pos != tmp->pos_end) {  // more than one character
                        if (pos == tmp->pos) {
-                               /* maybe we could enlarge the left fonttable */ 
-
+                               // maybe we could enlarge the left fonttable
                                found = false;
                                tmp2 = fonttable;
                                while (tmp2 && !found) {
@@ -1420,16 +1423,17 @@ void LyXParagraph::SetFont(LyXParagraph::size_type pos,
                                                tmp2 = tmp2->next;
                                }
 
-                               /* Is there is one, and is it exactly the needed font? */
+                               // Is there is one, and is it exactly
+                               // the needed font?
                                if (found && tmp2->font == font) {
-                                       /* put the position under the font */ 
+                                       // Put the position under the font
                                        tmp2->pos_end++;
                                        tmp->pos++;
                                        return;
                                }
 
-                               /* Add a new entry in the 
-                                * fonttable for the position */
+                               // Add a new entry in the 
+                               // fonttable for the position
                                tmp2 = new FontTable;
                                tmp2->pos = pos + 1;
                                tmp2->pos_end = tmp->pos_end;
@@ -1439,8 +1443,8 @@ void LyXParagraph::SetFont(LyXParagraph::size_type pos,
                                fonttable = tmp2;
                        }
                        else if (pos == tmp->pos_end) {
-                               /* Add a new entry in the 
-                                * fonttable for the position */
+                               // Add a new entry in the
+                               // fonttable for the position
                                tmp2 = new FontTable;
                                tmp2->pos = tmp->pos;
                                tmp2->pos_end = tmp->pos_end - 1;
@@ -1450,8 +1454,8 @@ void LyXParagraph::SetFont(LyXParagraph::size_type pos,
                                fonttable = tmp2;
                        }
                        else {
-                               /* Add a new entry in the 
-                                * fonttable for the position */
+                               // Add a new entry in the 
+                               // fonttable for the position
                                tmp2 = new FontTable;
                                tmp2->pos = tmp->pos;
                                tmp2->pos_end = pos - 1;
@@ -1476,7 +1480,7 @@ void LyXParagraph::SetFont(LyXParagraph::size_type pos,
 }
 
    
-/* this function is able to hide closed footnotes */
+// This function is able to hide closed footnotes.
 LyXParagraph * LyXParagraph::Next()
 {
        if (next && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) {
@@ -1489,9 +1493,8 @@ LyXParagraph * LyXParagraph::Next()
                                               footnote in a logical
                                               paragraph */
                else
-                       return next;    /* this should never happen! */
-       }
-       else
+                       return next;  // This should never happen!
+       } else
                return next;
 }
 
@@ -1506,9 +1509,8 @@ LyXParagraph * LyXParagraph::NextAfterFootnote()
                        return tmp;   /* there can be more than one footnote
                                         in a logical paragraph */
                else
-                       return next;     /* this should never happen! */
-       }
-       else
+                       return next;  // This should never happen!
+       } else
                return next;
 }
 
@@ -1523,9 +1525,8 @@ LyXParagraph const * LyXParagraph::NextAfterFootnote() const
                        return tmp;   /* there can be more than one footnote
                                         in a logical paragraph */
                else
-                       return next;     /* this should never happen! */
-       }
-       else
+                       return next;  // This should never happen!
+       } else
                return next;
 }
 
@@ -1541,9 +1542,8 @@ LyXParagraph * LyXParagraph::PreviousBeforeFootnote()
                        return tmp;    /* there can be more than one footnote
                                          in a logical paragraph */
                else
-                       return previous;   /* this should never happen! */
-       }
-       else
+                       return previous;  // This should never happen!
+       } else
                return previous;
 }
 
@@ -1569,13 +1569,14 @@ LyXParagraph * LyXParagraph::FirstPhysicalPar()
                return this;
        LyXParagraph * tmppar = this;
 
-       while (tmppar && (tmppar->IsDummy()
-                         || tmppar->footnoteflag != LyXParagraph::NO_FOOTNOTE))
+       while (tmppar &&
+              (tmppar->IsDummy()
+               || tmppar->footnoteflag != LyXParagraph::NO_FOOTNOTE))
                tmppar = tmppar->previous;
    
-       if (!tmppar)
-               return this;           /* this should never happen!  */
-       else
+       if (!tmppar) {
+               return this;
+       else
                return tmppar;
 }
 
@@ -1586,18 +1587,19 @@ LyXParagraph const * LyXParagraph::FirstPhysicalPar() const
                return this;
        LyXParagraph const * tmppar = this;
 
-       while (tmppar && (tmppar->IsDummy()
-                         || tmppar->footnoteflag != LyXParagraph::NO_FOOTNOTE))
+       while (tmppar &&
+              (tmppar->IsDummy()
+               || tmppar->footnoteflag != LyXParagraph::NO_FOOTNOTE))
                tmppar = tmppar->previous;
    
-       if (!tmppar)
-               return this;           /* this should never happen!  */
-       else
+       if (!tmppar) {
+               return this;
+       else
                return tmppar;
 }
 
 
-/* this function is able to hide closed footnotes */
+// This function is able to hide closed footnotes.
 LyXParagraph * LyXParagraph::Previous()
 {
        LyXParagraph * tmp = previous;
@@ -1615,13 +1617,12 @@ LyXParagraph * LyXParagraph::Previous()
 
                else
                        return previous; 
-       }
-       else
+       } else
                return previous;
 }
 
 
-/* this function is able to hide closed footnotes */
+// This function is able to hide closed footnotes.
 LyXParagraph const * LyXParagraph::Previous() const
 {
        LyXParagraph * tmp = previous;
@@ -1639,8 +1640,7 @@ LyXParagraph const * LyXParagraph::Previous() const
 
                else
                        return previous; 
-       }
-       else
+       } else
                return previous;
 }
 
@@ -1649,19 +1649,19 @@ void LyXParagraph::BreakParagraph(LyXParagraph::size_type pos,
                                  int flag)
 {
        size_type i, pos_end, pos_first;
-       /* create a new paragraph */
+       // create a new paragraph
        LyXParagraph * par = ParFromPos(pos);
        LyXParagraph * firstpar = FirstPhysicalPar();
    
        LyXParagraph * tmp = new LyXParagraph(par);
-   
+       
        tmp->footnoteflag = footnoteflag;
        tmp->footnotekind = footnotekind;
    
-       /* this is an idea for a more userfriendly layout handling, I will
-        * see what the users say */
+       // this is an idea for a more userfriendly layout handling, I will
+       // see what the users say
    
-       /* layout stays the same with latex-environments */ 
+       // layout stays the same with latex-environments
        if (flag) {
                tmp->SetOnlyLayout(firstpar->layout);
                tmp->SetLabelWidthString(firstpar->labelwidthstring);
@@ -1682,35 +1682,36 @@ void LyXParagraph::BreakParagraph(LyXParagraph::size_type pos,
                tmp->depth = firstpar->depth;
                tmp->noindent = firstpar->noindent;
    
-               /* copy everything behind the break-position
-                  to the new paragraph
-               */
+               // copy everything behind the break-position
+               // to the new paragraph
                pos_first = 0;
                while (ParFromPos(pos_first) != par)
-                       pos_first++;
+                       ++pos_first;
 
                pos_end = pos_first + par->text.size() - 1;
-               tmp->text.reserve(pos_end - pos);
+               // The constructor has already reserved 500 elements
+               //if (pos_end > pos)
+               //      tmp->text.reserve(pos_end - pos);
 
-               for (i = pos; i <= pos_end; i++) {
+               for (i = pos; i <= pos_end; ++i) {
                        par->CutIntoMinibuffer(i - pos_first);
                        tmp->InsertFromMinibuffer(i - pos);
                }
-
-               for (i = pos_end; i >= pos; i--)
+               tmp->text.resize(tmp->text.size());
+               for (i = pos_end; i >= pos; --i)
                        par->Erase(i - pos_first);
 
                par->text.resize(par->text.size());
        }
 
-       /* just an idea of me */ 
+       // just an idea of me
        if (!pos) {
                tmp->line_top = firstpar->line_top;
                tmp->pagebreak_top = firstpar->pagebreak_top;
                tmp->added_space_top = firstpar->added_space_top;
                tmp->bibkey = firstpar->bibkey;
                firstpar->Clear();
-               /* layout stays the same with latex-environments */ 
+               // layout stays the same with latex-environments
                if (flag) {
                        firstpar->SetOnlyLayout(tmp->layout);
                        firstpar->SetLabelWidthString(tmp->labelwidthstring);
@@ -1761,7 +1762,7 @@ LyXParagraph * LyXParagraph::FirstSelfrowPar()
                tmppar = tmppar->previous;
    
        if (!tmppar)
-               return this;           /* this should never happen!  */
+               return this;  // This should never happen!
        else
                return tmppar;
 }
@@ -1769,13 +1770,13 @@ LyXParagraph * LyXParagraph::FirstSelfrowPar()
 
 LyXParagraph * LyXParagraph::Clone() const
 {
-       /* create a new paragraph */
+       // create a new paragraph
        LyXParagraph * result = new LyXParagraph;
    
        result->MakeSameLayout(this);
 
-       /* this is because of the dummy layout of the paragraphs that
-          follow footnotes */
+       // this is because of the dummy layout of the paragraphs that
+       // follow footnotes
        result->layout = layout;
    
        /* table stuff -- begin*/ 
@@ -1789,9 +1790,9 @@ LyXParagraph * LyXParagraph::Clone() const
         result->bibkey = (bibkey) ? new InsetBibKey(bibkey): 0;
                
     
-       /* copy everything behind the break-position to the new paragraph */
+       // copy everything behind the break-position to the new paragraph
    
-       for (size_type i = 0; i < size(); i++) {
+       for (size_type i = 0; i < size(); ++i) {
                CopyIntoMinibuffer(i);
                result->InsertFromMinibuffer(i);
        }
@@ -1836,35 +1837,38 @@ bool LyXParagraph::HasSameLayout(LyXParagraph const * par)
 
 void LyXParagraph::BreakParagraphConservative(LyXParagraph::size_type pos)
 {
-       size_type i, pos_end, pos_first;
-       
-       /* create a new paragraph */
+       // create a new paragraph
        LyXParagraph * par = ParFromPos(pos);
 
        LyXParagraph * tmp = new LyXParagraph(par);
    
        tmp->MakeSameLayout(par);
-   
-       if (Last() > pos) {   
-               /* copy everything behind the break-position to the new
-                  paragraph */
-               pos_first = 0;
+
+       // When can pos < Last()?
+       // I guess pos == Last() is possible.
+       if (Last() > pos) {
+               // copy everything behind the break-position to the new
+               // paragraph
+               size_type pos_first = 0;
                while (ParFromPos(pos_first) != par)
-                       pos_first++;
-               pos_end = pos_first + par->text.size() - 1;
-               /* make shure there is enough memory for the now larger
-                  paragraph. This is not neccessary, because
-                  InsertFromMinibuffer will enlarge the memory (it uses
-                  InsertChar of course). But doing it by hand
-                  is MUCH faster! (only one time, not thousend times!!) */
-               tmp->text.reserve(pos_end - pos);
-
-               for (i = pos; i <= pos_end; i++) {
-      
+                       ++pos_first;
+               size_type pos_end = pos_first + par->text.size() - 1;
+               // make sure there is enough memory for the now larger
+               // paragraph. This is not neccessary, because
+               // InsertFromMinibuffer will enlarge the memory (it uses
+               // InsertChar of course). But doing it by hand
+               // is MUCH faster! (only one time, not thousend times!!)
+               // Not needed since the constructor aleady have
+               // reserved 500 elements in text.
+               //if (pos_end > pos)
+               //      tmp->text.reserve(pos_end - pos);
+
+               for (size_type i = pos; i <= pos_end; ++i) {
                        par->CutIntoMinibuffer(i - pos_first);
                        tmp->InsertFromMinibuffer(i - pos);
                }
-               for (i = pos_end; i >= pos; i--)
+               tmp->text.resize(tmp->text.size());
+               for (size_type i = pos_end; i >= pos; --i)
                        par->Erase(i - pos_first);
 
                par->text.resize(par->text.size());
@@ -1872,30 +1876,29 @@ void LyXParagraph::BreakParagraphConservative(LyXParagraph::size_type pos)
 }
    
 
-/* be carefull, this does not make any check at all */ 
+// Be carefull, this does not make any check at all.
 void LyXParagraph::PasteParagraph()
 {
-       /* copy the next paragraph to this one */
+       // copy the next paragraph to this one
        LyXParagraph * the_next = Next();
    
        LyXParagraph * firstpar = FirstPhysicalPar();
    
-       /* first the DTP-stuff */ 
+       // first the DTP-stuff
        firstpar->line_bottom = the_next->line_bottom;
        firstpar->added_space_bottom = the_next->added_space_bottom;
        firstpar->pagebreak_bottom = the_next->pagebreak_bottom;
 
        size_type pos_end = the_next->text.size() - 1;
        size_type pos_insert = Last();
-       size_type i;
 
-       /* ok, now copy the paragraph */ 
-       for (i = 0; i <= pos_end; i++) {
+       // ok, now copy the paragraph
+       for (size_type i = 0; i <= pos_end; ++i) {
                the_next->CutIntoMinibuffer(i);
                InsertFromMinibuffer(pos_insert + i);
        }
    
-       /* delete the next paragraph */
+       // delete the next paragraph
        delete the_next;
 }
 
@@ -1952,7 +1955,7 @@ int LyXParagraph::GetFirstCounter(int i) const
 }
 
 
-/* the next two functions are for the manual labels */ 
+// the next two functions are for the manual labels
 string LyXParagraph::GetLabelWidthString() const
 {
        if (!FirstPhysicalPar()->labelwidthstring.empty())
@@ -2065,12 +2068,12 @@ void LyXParagraph::SetLayout(LyXTextClass::LayoutList::size_type new_layout)
 }
 
 
-/* if the layout of a paragraph contains a manual label, the beginning of the 
-* main body is the beginning of the second word. This is what the par-
-* function returns. If the layout does not contain a label, the main
-* body always starts with position 0. This differentiation is necessary,
-* because there cannot be a newline or a blank <= the beginning of the 
-* main body in TeX. */ 
+// if the layout of a paragraph contains a manual label, the beginning of the 
+// main body is the beginning of the second word. This is what the par-
+// function returns. If the layout does not contain a label, the main
+// body always starts with position 0. This differentiation is necessary,
+// because there cannot be a newline or a blank <= the beginning of the 
+// main body in TeX.
 
 int LyXParagraph::BeginningOfMainBody() const
 {
@@ -2087,7 +2090,7 @@ int LyXParagraph::BeginningOfMainBody() const
                char previous_char, temp;
                if (i < size()
                    && (previous_char = GetChar(i)) != LyXParagraph::META_NEWLINE) {
-                       // Yes, this  ^ is supposed to be "= " not "== "
+                       // Yes, this  ^ is supposed to be "= " not "=="
                        ++i;
                        while (i < size()
                               && previous_char != ' '
@@ -2101,7 +2104,7 @@ int LyXParagraph::BeginningOfMainBody() const
        if (i == 0 && i == size() &&
            !(footnoteflag == LyXParagraph::NO_FOOTNOTE
              && next && next->footnoteflag != LyXParagraph::NO_FOOTNOTE))
-               i++;                           /* the cursor should not jump  
+               ++i;                           /* the cursor should not jump  
                                                * to the main body if there
                                                * is nothing in! */
        return i;
@@ -2152,15 +2155,20 @@ LyXParagraph const * LyXParagraph::DepthHook(int deth) const
 
 int LyXParagraph::AutoDeleteInsets()
 {
-#ifdef NEW_TABLE
+#ifdef NEW_INSETTABLE
+       vector<size_type> tmpvec;
        int i = 0;
        for (InsetList::iterator it = insetlist.begin();
             it != insetlist.end(); ++it) {
-               if ((*it).inset->AutoDelete()) {
+               if ((*it).inset && (*it).inset->AutoDelete()) {
+                       tmpvec.push_back((*it).pos);
                        ++i;
-                       Erase((*it).pos);
                }
        }
+       for (vector<size_type>::const_iterator cit = tmpvec.begin();
+            cit != tmpvec.end(); ++cit) {
+               Erase((*cit));
+       }
        return i;
 #else
        InsetTable * tmpi = insettable;
@@ -2169,14 +2177,15 @@ int LyXParagraph::AutoDeleteInsets()
        while (tmpi) {
                tmpi2 = tmpi;
                tmpi = tmpi->next;
-               if (tmpi2->inset)
+               if (tmpi2->inset) {
                        if (tmpi2->inset->AutoDelete()) {
-                               i++;
+                               ++i;
                                Erase(tmpi2->pos);
-                       } else {}
-               else
+                       }
+               } else {
                        lyxerr << "ERROR (LyXParagraph::AutoDeleteInsets): "
                                "cannot auto-delete insets" << endl;
+               }
        }
        return i;
 #endif
@@ -2185,20 +2194,20 @@ int LyXParagraph::AutoDeleteInsets()
 
 Inset * LyXParagraph::ReturnNextInsetPointer(LyXParagraph::size_type & pos)
 {
-#ifdef NEW_TABLE
-       InsetTable * tmp = 0;
+#ifdef NEW_INSETTABLE
+       InsetList::iterator it2 = insetlist.end();
        for (InsetList::iterator it = insetlist.begin();
             it != insetlist.end(); ++it) {
-               if ((*it).pos >= pos && (!tmp || (*it).pos < tmp->pos)) {
-                       tmp = &(*it);
+               if ((*it).pos >= pos) {
+                       if (it2 == insetlist.end() || (*it).pos < (*it2).pos)
+                               it2 = it;
                }
        }
-       if (tmp) {
-               pos = tmp->pos;
-               return tmp->inset;
+       if (it2 != insetlist.end()) {
+               pos = (*it2).pos;
+               return (*it2).inset;
        }
        return 0;
-               
 #else
        InsetTable * tmpi = insettable;
        InsetTable * tmpi2 = 0;
@@ -2219,70 +2228,57 @@ Inset * LyXParagraph::ReturnNextInsetPointer(LyXParagraph::size_type & pos)
 }
 
 
-/* returns -1 if inset not found */
+// returns -1 if inset not found
 int LyXParagraph::GetPositionOfInset(Inset * inset) const
 {
-#ifdef NEW_TABLE
-       for (InsetList::iterator it = insetlist.begin();
-            it != insetlist.end(); ++it) {
-               if ((*it).inset == inset) {
-                       return (*it).pos;
+       // Find the entry.
+#ifdef NEW_INSETTABLE
+       for (InsetList::const_iterator cit = insetlist.begin();
+            cit != insetlist.end(); ++cit) {
+               if ((*cit).inset == inset) {
+                       return (*cit).pos;
                }
        }
-       // Think about footnotes
-       if (footnoteflag == LyXParagraph::NO_FOOTNOTE
-           && next && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) {
-               int further = NextAfterFootnote()->GetPositionOfInset(inset);
-               if (further != -1)
-                       return size() + 1 + further;
-       }
-       return -1;
 #else
-       /* find the entry */ 
        InsetTable * tmpi = insettable;
        while (tmpi && tmpi->inset != inset) {
                tmpi = tmpi->next;
        }
        if (tmpi && tmpi->inset)
                return tmpi->pos;
-       else{
-               /* think about footnotes */
-               if (footnoteflag == LyXParagraph::NO_FOOTNOTE 
-                   && next && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) {
-                       int further = 
-                               NextAfterFootnote()->GetPositionOfInset(inset);
-                       if (further != -1)
-                               return text.size() + 1 + further;
-               }
-               return -1;
-       }
 #endif
+       // Think about footnotes.
+       if (footnoteflag == LyXParagraph::NO_FOOTNOTE 
+           && next && next->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE) {
+               int further = 
+                       NextAfterFootnote()->GetPositionOfInset(inset);
+               if (further != -1)
+                       return text.size() + 1 + further;
+       }
+       return -1;
 }
 
 
-void LyXParagraph::readSimpleWholeFile(FILE * myfile)
+void LyXParagraph::readSimpleWholeFile(istream & is)
 {
-       rewind(myfile);
-   
-       if (!feof(myfile)) {
-               char c = 0;
-               do {
-                       c = fgetc(myfile);
-                       InsertChar(text.size(), c);
-               } while (!feof(myfile));
-      
-       }
+       is.seekg(0);
+       char c = 0;
+       while(!is.eof()) {
+               is.get(c);
+               InsertChar(text.size(), c);
+       };
 }
 
 
 LyXParagraph * LyXParagraph::TeXOnePar(string & file, TexRow & texrow,
-                                     string & foot, TexRow & foot_texrow,
-                                     int & foot_count)
+                                      string & foot, TexRow & foot_texrow,
+                                      int & foot_count)
 {
        lyxerr[Debug::LATEX] << "TeXOnePar...     " << this << endl;
        LyXParagraph * par = next;
-       LyXLayout const & style = textclasslist.Style(GetCurrentTextClass(),
-                                                     layout);
+       LyXLayout const & style =
+               textclasslist.Style(current_view->buffer()->params.textclass,
+                                   layout);
 
        bool further_blank_line = false;
        if (IsDummy())
@@ -2361,8 +2357,8 @@ LyXParagraph * LyXParagraph::TeXOnePar(string & file, TexRow & texrow,
                        file += ' ';
                }
                file += "\\par}";
-       } else if (textclasslist.Style(GetCurrentTextClass(),
-                                 GetLayout()).isCommand()){
+       } else if (textclasslist.Style(current_view->buffer()->params.textclass,
+                                      GetLayout()).isCommand()){
                if (style.resfont.size() != font.size()) {
                        file += '\\';
                        file += font.latexSize();
@@ -2450,10 +2446,10 @@ bool LyXParagraph::SimpleTeXOnePar(string & file, TexRow & texrow)
        
        bool return_value = false;
 
-       LyXLayout const & style = textclasslist.Style(GetCurrentTextClass(), GetLayout());
+       LyXLayout const & style = textclasslist.Style(current_view->buffer()->params.textclass, GetLayout());
        LyXFont basefont;
 
-       /* maybe we have to create a optional argument */ 
+       // Maybe we have to create a optional argument.
        if (style.labeltype != LABEL_MANUAL)
                main_body = 0;
        else
@@ -2473,10 +2469,10 @@ bool LyXParagraph::SimpleTeXOnePar(string & file, TexRow & texrow)
            && !IsDummy()) {
                if (style.isCommand()) {
                        file += '{';
-                       column++;
+                       ++column;
                } else if (align != LYX_ALIGN_LAYOUT) {
                        file += '{';
-                       column++;
+                       ++column;
                        return_value = true;
                }
        }
@@ -2489,7 +2485,7 @@ bool LyXParagraph::SimpleTeXOnePar(string & file, TexRow & texrow)
        texrow.start(this, 0);
 
        for (size_type i = 0; i < size(); ++i) {
-               column++;
+               ++column;
                // First char in paragraph or after label?
                if (i == main_body && !IsDummy()) {
                        if (main_body > 0) {
@@ -2500,11 +2496,11 @@ bool LyXParagraph::SimpleTeXOnePar(string & file, TexRow & texrow)
                                basefont = getFont(-1); // Now use the layout font
                                running_font = basefont;
                                file += ']';
-                               column++;
+                               ++column;
                        }
                        if (style.isCommand()) {
                                file += '{';
-                               column++;
+                               ++column;
                        } else if (align != LYX_ALIGN_LAYOUT) {
                                file += "{\\par";
                                column += 4;
@@ -2546,7 +2542,7 @@ bool LyXParagraph::SimpleTeXOnePar(string & file, TexRow & texrow)
                if (open_font && c == ' ' && i <= size() - 2 
                    && !getFont(i+1).equalExceptLatex(running_font) 
                    && !getFont(i+1).equalExceptLatex(font)) {
-                       font = getFont(i+1);
+                       font = getFont(i + 1);
                }
                // We end font definition before blanks
                if (!font.equalExceptLatex(running_font) && open_font) {
@@ -2593,7 +2589,7 @@ bool LyXParagraph::SimpleTeXOnePar(string & file, TexRow & texrow)
                                file += "\\\\\n";
                        }
                        texrow.newline();
-                       texrow.start(this, i+1);
+                       texrow.start(this, i + 1);
                        column = 0;
                } else {
                        SimpleTeXSpecialChars(file, texrow,
@@ -2607,7 +2603,7 @@ bool LyXParagraph::SimpleTeXOnePar(string & file, TexRow & texrow)
                running_font.latexWriteEndChanges(file, basefont);
        }
 
-       /* needed if there is an optional argument but no contents */ 
+       // Needed if there is an optional argument but no contents.
        if (main_body > 0 && main_body == size()) {
                file += "]~";
                return_value = false;
@@ -2622,28 +2618,21 @@ bool LyXParagraph::SimpleTeXOnePar(string & file, TexRow & texrow)
 bool LyXParagraph::SimpleTeXOneTablePar(string & file, TexRow & texrow)
 {
        lyxerr[Debug::LATEX] << "SimpleTeXOneTablePar...     " << this << endl;
-       char c;
-       int tmp;
    
        bool return_value = false;
-       int current_cell_number = -1;
 
        LyXLayout const & style = 
-               textclasslist.Style(GetCurrentTextClass(), GetLayout());
-       LyXFont basefont = getFont(-1); // Get layout font
-       // Which font is currently active?
-       LyXFont running_font = basefont;
-       // Do we have an open font change?
-       bool open_font = false;
+               textclasslist.Style(current_view->buffer()->params.textclass,
+                                   GetLayout());
  
        int column = 0;
        if (!IsDummy()) { // it is dummy if it is in a float!!!
                if (style.isCommand()) {
                        file += '{';
-                       column++;
+                       ++column;
                } else if (align != LYX_ALIGN_LAYOUT) {
                        file += '{';
-                       column++;
+                       ++column;
                        return_value = true;
                }
                if (noindent) {
@@ -2669,27 +2658,35 @@ bool LyXParagraph::SimpleTeXOneTablePar(string & file, TexRow & texrow)
                        break;
                }
        }
-       current_cell_number = -1;
-       tmp = table->TexEndOfCell(file, current_cell_number);
-       for (; tmp >0 ; --tmp)
+
+       LyXFont basefont = getFont(-1); // Get layout font
+       // Which font is currently active?
+       LyXFont running_font = basefont;
+       // Do we have an open font change?
+       bool open_font = false;
+       int current_cell_number = -1;
+       int tmp = table->TexEndOfCell(file, current_cell_number);
+       for (; tmp > 0 ; --tmp)
                texrow.newline();
        
        texrow.start(this, 0);
 
        for (size_type i = 0; i < size(); ++i) {
-               c = GetChar(i);
-               if (table->IsContRow(current_cell_number+1)) {
+               char c = GetChar(i);
+               if (table->IsContRow(current_cell_number + 1)) {
                        if (c == LyXParagraph::META_NEWLINE)
-                               current_cell_number++;
+                               ++current_cell_number;
                        continue;
                }
-               column++;
+               ++column;
                
                // Fully instantiated font
                LyXFont font = getFont(i);
 
-               // Spaces at end of font change are simulated to be outside font change.
-               // i.e. we write "\textXX{text} " rather than "\textXX{text }". (Asger)
+               // Spaces at end of font change are simulated to be
+               // outside font change.
+               // i.e. we write "\textXX{text} " rather than
+               // "\textXX{text }". (Asger)
                if (open_font && c == ' ' && i <= size() - 2
                    && getFont(i+1) != running_font && getFont(i+1) != font) {
                        font = getFont(i+1);
@@ -2722,16 +2719,18 @@ bool LyXParagraph::SimpleTeXOneTablePar(string & file, TexRow & texrow)
                }
                if (c == LyXParagraph::META_NEWLINE) {
                        // special case for inside a table
-                       // different from default case in SimpleTeXSpecialChars()
+                       // different from default case in
+                       // SimpleTeXSpecialChars()
                        if (open_font) {
-                               column += running_font.latexWriteEndChanges(file, basefont);
+                               column += running_font
+                                       .latexWriteEndChanges(file, basefont);
                                open_font = false;
                        }
                        basefont = getFont(-1);
                        running_font = basefont;
-                       current_cell_number++;
+                       ++current_cell_number;
                        if (table->CellHasContRow(current_cell_number) >= 0) {
-                               TeXContTableRows(file, i+1,
+                               TeXContTableRows(file, i + 1,
                                                 current_cell_number,
                                                 column, texrow);
                        }
@@ -2739,20 +2738,20 @@ bool LyXParagraph::SimpleTeXOneTablePar(string & file, TexRow & texrow)
                        // put the EndOfCell because it is put after the
                        // for(...)
                        if (table->ShouldBeVeryLastCell(current_cell_number)) {
-                            current_cell_number--;
-                            break;
+                               --current_cell_number;
+                               break;
                        }
                        int tmp = table->TexEndOfCell(file,
                                                      current_cell_number);
-                       if (tmp>0) {
+                       if (tmp > 0) {
                                column = 0;
                        } else if (tmp < 0) {
                                tmp = -tmp;
                        }
-                       for (;tmp--;) {
+                       for (; tmp--;) {
                                texrow.newline();
                        }
-                       texrow.start(this, i+1);
+                       texrow.start(this, i + 1);
                } else {
                        SimpleTeXSpecialChars(file, texrow,
                                              font, running_font, basefont,
@@ -2764,7 +2763,7 @@ bool LyXParagraph::SimpleTeXOneTablePar(string & file, TexRow & texrow)
        if (open_font) {
                running_font.latexWriteEndChanges(file, basefont);
        }
-       current_cell_number++;
+       ++current_cell_number;
        tmp = table->TexEndOfCell(file, current_cell_number);
        for (; tmp > 0; --tmp)
                texrow.newline();
@@ -2786,11 +2785,10 @@ bool LyXParagraph::TeXContTableRows(string & file,
        char c;
    
        bool return_value = false;
-       LyXLayout const & style = textclasslist.Style(GetCurrentTextClass(),
-                                                     GetLayout());
-       LyXFont basefont;
-
-       basefont = getFont(-1); // Get layout font
+       LyXLayout const & style =
+               textclasslist.Style(current_view->buffer()->params.textclass,
+                                   GetLayout());
+       LyXFont basefont = getFont(-1); // Get layout font
        // Which font is currently active?
        LyXFont running_font = basefont;
        // Do we have an open font change?
@@ -2798,14 +2796,14 @@ bool LyXParagraph::TeXContTableRows(string & file,
 
        size_type lastpos = i;
        int cell = table->CellHasContRow(current_cell_number);
-       current_cell_number++;
+       ++current_cell_number;
        while(cell >= 0) {
                // first find the right position
                i = lastpos;
                for (; (i < size()) && (current_cell_number<cell); ++i) {
                        c = GetChar(i);
                        if (c == LyXParagraph::META_NEWLINE)
-                               current_cell_number++;
+                               ++current_cell_number;
                }
                lastpos = i;
                c = GetChar(i);
@@ -2817,15 +2815,18 @@ bool LyXParagraph::TeXContTableRows(string & file,
                        file += ' ';
                }
 
-               for (; i < size() && (c = GetChar(i)) != LyXParagraph::META_NEWLINE;
+               for (; i < size()
+                            && (c = GetChar(i)) != LyXParagraph::META_NEWLINE;
                     ++i) {
                        ++column;
 
                        // Fully instantiated font
                        LyXFont font = getFont(i);
 
-                       // Spaces at end of font change are simulated to be outside font change.
-                       // i.e. we write "\textXX{text} " rather than "\textXX{text }". (Asger)
+                       // Spaces at end of font change are simulated to
+                       // be outside font change. i.e. we write
+                       // "\textXX{text} " rather than "\textXX{text }".
+                       // (Asger)
                        if (open_font && c == ' ' && i <= size() - 2 
                            && getFont(i + 1) != running_font
                            && getFont(i + 1) != font) {
@@ -2934,7 +2935,7 @@ bool LyXParagraph::linuxDocConvertChar(char c, string & sgml_string)
                retval = true;
                sgml_string = ' ';
                break;
-       case '\0': /* Ignore :-) */
+       case '\0': // Ignore :-)
                sgml_string.clear();
                break;
        default:
@@ -2944,58 +2945,58 @@ bool LyXParagraph::linuxDocConvertChar(char c, string & sgml_string)
        return retval;
 }
 
+
 void LyXParagraph::SimpleDocBookOneTablePar(string & file, string & extra,
                                            int & desc_on, int depth) 
 {
-       if (!table)
-               return;
+       if (!table) return;
        lyxerr[Debug::LATEX] << "SimpleDocbookOneTablePar... " << this << endl;
-       int column, tmp;
-       int current_cell_number = -1;
+       int column = 0;
        LyXFont font1, font2;
        char c;
-       Inset *inset;
+       Inset * inset;
        size_type main_body;
        string emph = "emphasis";
-       bool emph_flag= false;
-       int char_line_count= 0;
+       bool emph_flag = false;
        
-       LyXLayout const & style = textclasslist.Style(GetCurrentTextClass(), GetLayout());
+       LyXLayout const & style =
+               textclasslist.Style(current_view->buffer()->params.textclass,
+                                   GetLayout());
        
        if (style.labeltype != LABEL_MANUAL)
                main_body = 0;
        else
                main_body = BeginningOfMainBody();
        
-       /* gets paragraph main font */
+       // Gets paragraph main font.
        if (main_body > 0)
                font1 = style.labelfont;
        else
                font1 = style.font;
        
-       char_line_count = depth;
+       int char_line_count = depth;
        addNewlineAndDepth(file, depth);
        if (footnoteflag == LyXParagraph::NO_FOOTNOTE) {
                file += "<INFORMALTABLE>";
                addNewlineAndDepth(file, ++depth);
        }
-       current_cell_number = -1;
-       tmp = table->DocBookEndOfCell(file, current_cell_number, depth);
+       int current_cell_number = -1;
+       int tmp = table->DocBookEndOfCell(file, current_cell_number, depth);
        
-       /* parsing main loop */
+       // Parsing main loop.
        for (size_type i = 0; i < size(); ++i) {
                c = GetChar(i);
                if (table->IsContRow(current_cell_number+1)) {
                        if (c == LyXParagraph::META_NEWLINE)
-                               current_cell_number++;
+                               ++current_cell_number;
                        continue;
                }
-               column++;
+               ++column;
                
                // Fully instantiated font
                font2 = getFont(i);
                
-               /* handle <emphasis> tag */
+               // Handle <emphasis> tag.
                if (font1.emph() != font2.emph() && i) {
                        if (font2.emph() == LyXFont::ON) {
                                file += "<emphasis>";
@@ -3006,13 +3007,13 @@ void LyXParagraph::SimpleDocBookOneTablePar(string & file, string & extra,
                        }
                }
                if (c == LyXParagraph::META_NEWLINE) {
-                       // we have only to control for emphasis open here!
+                       // We have only to control for emphasis open here!
                        if (emph_flag) {
                                file += "</emphasis>";
                                emph_flag= false;
                        }
                        font1 = font2 = getFont(-1);
-                       current_cell_number++;
+                       ++current_cell_number;
                        if (table->CellHasContRow(current_cell_number) >= 0) {
                                DocBookContTableRows(file, extra, desc_on, i+1,
                                                     current_cell_number,
@@ -3022,11 +3023,12 @@ void LyXParagraph::SimpleDocBookOneTablePar(string & file, string & extra,
                        // put the EndOfCell because it is put after the
                        // for(...)
                        if (table->ShouldBeVeryLastCell(current_cell_number)) {
-                               current_cell_number--;
+                               --current_cell_number;
                                break;
                        }
-                       tmp= table->DocBookEndOfCell(file, current_cell_number,
-                                                   depth);
+                       tmp = table->DocBookEndOfCell(file,
+                                                     current_cell_number,
+                                                     depth);
                        
                        if (tmp > 0)
                                column = 0;
@@ -3037,19 +3039,24 @@ void LyXParagraph::SimpleDocBookOneTablePar(string & file, string & extra,
                        //
                        // This code needs some explanation:
                        // Two insets are treated specially
-                       //   label if it is the first element in a command paragraph
+                       //   label if it is the first element in a
+                       //   command paragraph
                        //         desc_on == 3
-                       //   graphics inside tables or figure floats can't go on
-                               //   title (the equivalente in latex for this case is caption
+                       //   graphics inside tables or figure floats
+                       //   can't go on
+                       //   title (the equivalente in latex for this
+                       //   case is caption
                        //   and title should come first
                        //         desc_on == 4
                        //
                        if(desc_on != 3 || i != 0) {
                                if(tmp_out[0] == '@') {
                                        if(desc_on == 4)
-                                               extra += frontStrip(tmp_out, '@');
+                                               extra += frontStrip(tmp_out,
+                                                                   '@');
                                        else
-                                               file += frontStrip(tmp_out, '@');
+                                               file += frontStrip(tmp_out,
+                                                                  '@');
                                } else
                                        file += tmp_out;
                        }
@@ -3057,7 +3064,7 @@ void LyXParagraph::SimpleDocBookOneTablePar(string & file, string & extra,
                        // "TeX"-Mode on == > SGML-Mode on.
                        if (c != '\0')
                                file += c;
-                       char_line_count++;
+                       ++char_line_count;
                } else {
                        string sgml_string;
                        if (linuxDocConvertChar(c, sgml_string) 
@@ -3066,7 +3073,7 @@ void LyXParagraph::SimpleDocBookOneTablePar(string & file, string & extra,
                                // non-breaking characters
                                // char is ' '
                                if (desc_on == 1) {
-                                       char_line_count++;
+                                       ++char_line_count;
                                        file += '\n';
                                        file += "</term><listitem><para>";
                                        desc_on = 2;
@@ -3080,7 +3087,7 @@ void LyXParagraph::SimpleDocBookOneTablePar(string & file, string & extra,
                font1 = font2;
        }
        
-       /* needed if there is an optional argument but no contents */
+       // Needed if there is an optional argument but no contents.
        if (main_body > 0 && main_body == size()) {
                font1 = style.font;
        }
@@ -3089,12 +3096,12 @@ void LyXParagraph::SimpleDocBookOneTablePar(string & file, string & extra,
                file += "</emphasis>";
        }
        
-       current_cell_number++;
+       ++current_cell_number;
        tmp = table->DocBookEndOfCell(file, current_cell_number, depth);
-       /* resets description flag correctly */
+       // Resets description flag correctly.
        switch(desc_on){
        case 1:
-               /* <term> not closed... */
+               // <term> not closed...
                file += "</term>";
                break;
        }
@@ -3107,49 +3114,49 @@ void LyXParagraph::SimpleDocBookOneTablePar(string & file, string & extra,
 
 
 void LyXParagraph::DocBookContTableRows(string & file, string & extra,
-                                        int & desc_on, LyXParagraph::size_type i,
+                                        int & desc_on,
+                                       LyXParagraph::size_type i,
                                         int current_cell_number, int &column) 
 
 {
-       if (!table)
-               return;
+       if (!table) return;
        
        lyxerr[Debug::LATEX] << "DocBookContTableRows... " << this << endl;
 
-       int cell;
-       LyXFont font1, font2;
+       LyXFont font2;
        char c;
        Inset * inset;
-       size_type main_body;
-       size_type lastpos;
        string emph= "emphasis";
        bool emph_flag= false;
        int char_line_count= 0;
        
-       LyXLayout const & style = textclasslist.Style(GetCurrentTextClass(),
-                                                     GetLayout());
+       LyXLayout const & style =
+               textclasslist.Style(current_view->buffer()->params.textclass,
+                                   GetLayout());
        
+       size_type main_body;
        if (style.labeltype != LABEL_MANUAL)
                main_body = 0;
        else
                main_body = BeginningOfMainBody();
        
-       /* gets paragraph main font */
+       // Gets paragraph main font.
+       LyXFont font1;
        if (main_body > 0)
                font1 = style.labelfont;
        else
                font1 = style.font;
        
-       lastpos = i;
-       cell = table->CellHasContRow(current_cell_number);
-       current_cell_number++;
+       size_type lastpos = i;
+       int cell = table->CellHasContRow(current_cell_number);
+       ++current_cell_number;
        while(cell >= 0) {
                // first find the right position
                i = lastpos;
                for (; i < size() && current_cell_number < cell; ++i) {
                        c = GetChar(i);
                        if (c == LyXParagraph::META_NEWLINE)
-                               current_cell_number++;
+                               ++current_cell_number;
                }
                lastpos = i;
                c = GetChar(i);
@@ -3171,7 +3178,7 @@ void LyXParagraph::DocBookContTableRows(string & file, string & extra,
                        // Fully instantiated font
                        font2 = getFont(i);
                        
-                       /* handle <emphasis> tag */
+                       // Handle <emphasis> tag.
                        if (font1.emph() != font2.emph() && i) {
                                if (font2.emph() == LyXFont::ON) {
                                        file += "<emphasis>";
@@ -3188,11 +3195,13 @@ void LyXParagraph::DocBookContTableRows(string & file, string & extra,
                                //
                                // This code needs some explanation:
                                // Two insets are treated specially
-                               //   label if it is the first element in a command paragraph
+                               //   label if it is the first element in a
+                               //   command paragraph
                                //       desc_on == 3
-                               //   graphics inside tables or figure floats can't go on
-                               //   title (the equivalente in latex for this case is caption
-                               //   and title should come first
+                               //   graphics inside tables or figure floats
+                               //   can't go on title (the equivalente in
+                               //   latex for this case is caption and title
+                               //   should come first
                                //       desc_on == 4
                                //
                                if(desc_on != 3 || i != 0) {
@@ -3208,16 +3217,16 @@ void LyXParagraph::DocBookContTableRows(string & file, string & extra,
                                // "TeX"-Mode on == > SGML-Mode on.
                                if (c!= '\0')
                                        file += c;
-                               char_line_count++;
+                               ++char_line_count;
                        } else {
                                string sgml_string;
                                if (linuxDocConvertChar(c, sgml_string) 
                                    && !style.free_spacing) {
-                               // in freespacing mode, spaces are
-                               // non-breaking characters
-                               // char is ' '
+                                       // in freespacing mode, spaces are
+                                       // non-breaking characters
+                                       // char is ' '
                                        if (desc_on == 1) {
-                                               char_line_count++;
+                                               ++char_line_count;
                                                file += '\n';
                                                file += "</term><listitem><para>";
                                                desc_on = 2;
@@ -3240,6 +3249,7 @@ void LyXParagraph::DocBookContTableRows(string & file, string & extra,
        lyxerr[Debug::LATEX] << "DocBookContTableRows...done " << this << endl;
 }
 
+
 void LyXParagraph::SimpleTeXBlanks(string & file, TexRow & texrow,
                                   LyXParagraph::size_type const i,
                                   int & column, LyXFont const & font,
@@ -3306,7 +3316,7 @@ void LyXParagraph::SimpleTeXSpecialChars(string & file, TexRow & texrow,
                        } else {
                                column += file.length() - len;
                        }
-                       for (;tmp--;) {
+                       for (; tmp--;) {
                                texrow.newline();
                        }
                }
@@ -3464,7 +3474,7 @@ void LyXParagraph::SimpleTeXSpecialChars(string & file, TexRow & texrow,
                                break;
 
                        case ' ':
-                               /* blanks are printed before font switching */
+                               // Blanks are printed before font switching.
                                // Sure? I am not! (try nice-latex)
                                // I am sure it's correct. LyX might be smarter
                                // in the future, but for now, nothing wrong is
@@ -3473,7 +3483,7 @@ void LyXParagraph::SimpleTeXSpecialChars(string & file, TexRow & texrow,
 
                        default:
                                /* idea for labels --- begin*/
-                               /* check for LyX */
+                               // Check for "LyX"
                                if (c ==  'L'
                                    && i <= size() - 3
                                    && font.family() != LyXFont::TYPEWRITER_FAMILY
@@ -3483,7 +3493,7 @@ void LyXParagraph::SimpleTeXSpecialChars(string & file, TexRow & texrow,
                                        i += 2;
                                        column += 5;
                                }
-                               /* check for TeX */ 
+                               // Check for "TeX"
                                else if (c == 'T'
                                         && i <= size() - 3
                                         && font.family() != LyXFont::TYPEWRITER_FAMILY
@@ -3493,7 +3503,7 @@ void LyXParagraph::SimpleTeXSpecialChars(string & file, TexRow & texrow,
                                        i += 2;
                                        column += 5;
                                }
-                               /* check for LaTeX2e */ 
+                               // Check for "LaTeX2e"
                                else if (c == 'L'
                                         && i <= size() - 7
                                         && font.family() != LyXFont::TYPEWRITER_FAMILY
@@ -3507,7 +3517,7 @@ void LyXParagraph::SimpleTeXSpecialChars(string & file, TexRow & texrow,
                                        i += 6;
                                        column += 8;
                                }
-                               /* check for LaTeX */ 
+                               // Check for "LaTeX"
                                else if (c == 'L'
                                         && i <= size() - 5
                                         && font.family() != LyXFont::TYPEWRITER_FAMILY
@@ -3529,7 +3539,7 @@ void LyXParagraph::SimpleTeXSpecialChars(string & file, TexRow & texrow,
 }
 
 
-bool LyXParagraph::RoffContTableRows(FILE * fp,
+bool LyXParagraph::RoffContTableRows(ostream & os,
                                     LyXParagraph::size_type i,
                                     int actcell)
 {
@@ -3540,24 +3550,23 @@ bool LyXParagraph::RoffContTableRows(FILE * fp,
        LyXFont font2;
        Inset * inset;
        char c;
-       FILE * fp2;
 
        string fname2 = TmpFileName(string(), "RAT2");
        int lastpos = i;
        int cell = table->CellHasContRow(actcell);
-       actcell++;
+       ++actcell;
        while(cell >= 0) {
                // first find the right position
                i = lastpos;
                for (; i < size() && actcell < cell; ++i) {
                        c = GetChar(i);
                        if (c == LyXParagraph::META_NEWLINE)
-                               actcell++;
+                               ++actcell;
                }
                lastpos = i;
                c = GetChar(i);
                if ((c != ' ') && (c != LyXParagraph::META_NEWLINE))
-                       fprintf(fp, " ");
+                       os << " ";
                for (; i < size()
                             && (c = GetChar(i)) != LyXParagraph::META_NEWLINE;
                     ++i) {
@@ -3570,23 +3579,25 @@ bool LyXParagraph::RoffContTableRows(FILE * fp,
                        switch (c) {
                        case LyXParagraph::META_INSET:
                                if ((inset = GetInset(i))) {
-                                       if (!(fp2= fopen(fname2.c_str(), "w+"))) {
+                                       fstream fs(fname2.c_str(),
+                                                  ios::in|ios::out);
+                                       if (!fs) {
                                                WriteAlert(_("LYX_ERROR:"),
                                                           _("Cannot open temporary file:"),
                                                           fname2);
                                                return false;
                                        }
-                                       inset->Latex(fp2,-1);
-                                       rewind(fp2);
-                                       c = fgetc(fp2);
-                                       while(!feof(fp2)) {
+                                       inset->Latex(fs, -1);
+                                       fs.seekp(0);
+                                       fs.get(c);
+                                       while (!fs) {
                                                if (c == '\\')
-                                                       fprintf(fp, "\\\\");
+                                                       os << "\\\\";
                                                else
-                                                       fputc(c, fp);
-                                               c = fgetc(fp2);
+                                                       os << c;
+                                               fs.get(c);
                                        }
-                                       fclose(fp2);
+                                       fs.close();
                                }
                                break;
                        case LyXParagraph::META_NEWLINE:
@@ -3596,13 +3607,15 @@ bool LyXParagraph::RoffContTableRows(FILE * fp,
                        case LyXParagraph::META_PROTECTED_SEPARATOR:
                                break;
                        case '\\': 
-                               fprintf(fp, "\\\\");
+                               os << "\\\\";
                                break;
                        default:
                                if (c != '\0')
-                                       fprintf(fp, "%c", c);
+                                       os << c;
                                else
-                                       lyxerr.debug() << "RoffAsciiTable: NULL char in structure." << endl;
+                                       lyxerr.debug() << "RoffAsciiTable: "
+                                               "NULL char in structure."
+                                                      << endl;
                                break;
                        }
                }
@@ -3622,8 +3635,8 @@ LyXParagraph * LyXParagraph::TeXDeeper(string & file, TexRow & texrow,
        while (par && par->depth == depth) {
                if (par->IsDummy())
                        lyxerr << "ERROR (LyXParagraph::TeXDeeper)" << endl;
-               if (textclasslist.Style(GetCurrentTextClass()
-                                  par->layout).isEnvironment()
+               if (textclasslist.Style(current_view->buffer()->params.textclass
+                                       par->layout).isEnvironment()
                    || par->pextra_type != PEXTRA_NONE) 
                        {
                                par = par->TeXEnvironment(file, texrow,
@@ -3642,12 +3655,13 @@ LyXParagraph * LyXParagraph::TeXDeeper(string & file, TexRow & texrow,
 
 
 LyXParagraph * LyXParagraph::TeXEnvironment(string & file, TexRow & texrow,
-                                          string & foot, TexRow & foot_texrow,
-                                          int & foot_count)
+                                           string & foot,
+                                           TexRow & foot_texrow,
+                                           int & foot_count)
 {
        bool eindent_open = false;
        bool foot_this_level = false;
-               // flags when footnotetext should be appended to file.
+       // flags when footnotetext should be appended to file.
         static bool minipage_open = false;
         static int minipage_open_depth = 0;
        char par_sep = current_view->buffer()->params.paragraph_separation;
@@ -3656,8 +3670,9 @@ LyXParagraph * LyXParagraph::TeXEnvironment(string & file, TexRow & texrow,
        if (IsDummy())
                lyxerr << "ERROR (LyXParagraph::TeXEnvironment)" << endl;
 
-       LyXLayout const & style = textclasslist.Style(GetCurrentTextClass(),
-                                                     layout);
+       LyXLayout const & style =
+               textclasslist.Style(current_view->buffer()->params.textclass,
+                                   layout);
        
        if (pextra_type == PEXTRA_INDENT) {
                if (!pextra_width.empty()) {
@@ -3785,24 +3800,25 @@ LyXParagraph * LyXParagraph::TeXEnvironment(string & file, TexRow & texrow,
                 if (minipage_open && par && !style.isEnvironment() &&
                     (par->pextra_type == PEXTRA_MINIPAGE) &&
                     par->pextra_start_minipage) {
-                    file += "\\end{minipage}\n";
-                    texrow.newline();
-                    if (par_sep == BufferParams::PARSEP_INDENT) {
-                        file += "}\n";
+                       file += "\\end{minipage}\n";
                        texrow.newline();
-                    }
-                    minipage_open = false;
+                       if (par_sep == BufferParams::PARSEP_INDENT) {
+                               file += "}\n";
+                               texrow.newline();
+                       }
+                       minipage_open = false;
                 }
                if (par && par->depth > depth) {
-                       if (textclasslist.Style(GetCurrentTextClass(),
-                                          par->layout).isParagraph()
+                       if (textclasslist.Style(current_view->buffer()->params.textclass,
+                                               par->layout).isParagraph()
                            && !par->table
                            && !suffixIs(file, "\n\n")) {
-                           // There should be at least one '\n' already
-                           // but we need there to be two for Standard 
-                           // paragraphs that are depth-increment'ed to be
-                           // output correctly.  However, tables can also be
-                           // paragraphs so don't adjust them.  ARRae
+                               // There should be at least one '\n' already
+                               // but we need there to be two for Standard 
+                               // paragraphs that are depth-increment'ed to be
+                               // output correctly.  However, tables can
+                               // also be paragraphs so don't adjust them.
+                               // ARRae
                                file += '\n';
                                texrow.newline();
                        }
@@ -3927,8 +3943,8 @@ LyXParagraph * LyXParagraph::TeXFootnote(string & file, TexRow & texrow,
                        "No footnote!" << endl;
 
        LyXParagraph * par = this;
-       LyXLayout const & style = textclasslist.Style(GetCurrentTextClass()
-                                          previous->GetLayout());
+       LyXLayout const & style = textclasslist.Style(current_view->buffer()->params.textclass
+                                                     previous->GetLayout());
        
        if (style.needprotect && footnotekind != LyXParagraph::FOOTNOTE){
                lyxerr << "ERROR (LyXParagraph::TeXFootnote): "
@@ -3988,7 +4004,8 @@ LyXParagraph * LyXParagraph::TeXFootnote(string & file, TexRow & texrow,
                                sprintf(bufr, "\\begin{floatingfigure}{%s}\n",
                                        pextra_width.c_str());
                        else
-                               sprintf(bufr, "\\begin{floatingfigure}{%f\\textwidth}\n",
+                               sprintf(bufr,
+                                       "\\begin{floatingfigure}{%f\\textwidth}\n",
                                        atoi(pextra_widthp.c_str())/100.0);
                        file += bufr;
                } else {
@@ -4042,16 +4059,18 @@ LyXParagraph * LyXParagraph::TeXFootnote(string & file, TexRow & texrow,
            || !footer_in_body) {
                // Process text for all floats except footnotes in body
                do {
-                       LyXLayout const & style = textclasslist.Style(GetCurrentTextClass(),
-                                                         par->layout);
+                       LyXLayout const & style =
+                               textclasslist.Style(current_view->buffer()->params.textclass,
+                                                   par->layout);
                        if (par->IsDummy())
                                lyxerr << "ERROR (LyXParagraph::TeXFootnote)"
                                       << endl;
                        if (style.isEnvironment()
                            || par->pextra_type == PEXTRA_MINIPAGE) { /* && !minipage_open ?? */
-                               // Allows the use of minipages within float environments.
-                               // Shouldn't be circular because we don't support
-                               // footnotes inside floats (yet). ARRae
+                               // Allows the use of minipages within float
+                               // environments. Shouldn't be circular because
+                               // we don't support footnotes inside
+                               // floats (yet). ARRae
                                par = par->TeXEnvironment(file, texrow,
                                                          foot, foot_texrow,
                                                          foot_count);
@@ -4075,16 +4094,18 @@ LyXParagraph * LyXParagraph::TeXFootnote(string & file, TexRow & texrow,
                TexRow dummy_texrow;
                int dummy_count = 0;
                do {
-                       LyXLayout const & style = textclasslist.Style(GetCurrentTextClass(),
-                                                         par->layout);
+                       LyXLayout const & style =
+                               textclasslist.Style(current_view->buffer()->params.textclass,
+                                                   par->layout);
                        if (par->IsDummy())
                                lyxerr << "ERROR (LyXParagraph::TeXFootnote)"
                                       << endl;
                        if (style.isEnvironment()
                            || par->pextra_type == PEXTRA_MINIPAGE) { /* && !minipage_open ?? */
-                               // Allows the use of minipages within float environments.
-                               // Shouldn't be circular because we don't support
-                               // footnotes inside floats (yet). ARRae
+                               // Allows the use of minipages within float
+                               // environments. Shouldn't be circular because
+                               // we don't support footnotes inside
+                               // floats (yet). ARRae
                                par = par->TeXEnvironment(foot, foot_texrow,
                                                          dummy, dummy_texrow,
                                                          dummy_count);
@@ -4099,7 +4120,8 @@ LyXParagraph * LyXParagraph::TeXFootnote(string & file, TexRow & texrow,
                                                     dummy, dummy_texrow,
                                                     dummy_count);
                        }
-               } while (par && par->footnoteflag != LyXParagraph::NO_FOOTNOTE);
+               } while (par
+                        && par->footnoteflag != LyXParagraph::NO_FOOTNOTE);
                if (dummy_count) {
                        lyxerr << "ERROR (LyXParagraph::TeXFootnote): "
                                "Footnote in a Footnote -- not supported"
@@ -4160,80 +4182,84 @@ LyXParagraph * LyXParagraph::TeXFootnote(string & file, TexRow & texrow,
 void LyXParagraph::SetPExtraType(int type, char const * width,
                                 char const * widthp)
 {
-    pextra_type = type;
-    pextra_width = width;
-    pextra_widthp = widthp;
-
-    if (textclasslist.Style(GetCurrentTextClass(), 
-                       layout).isEnvironment()) {
-        LyXParagraph
-            * par = this,
-            * ppar = par;
-
-        while (par && (par->layout == layout) && (par->depth == depth)) {
-            ppar = par;
-            par = par->Previous();
-            if (par)
-                par = par->FirstPhysicalPar();
-            while (par && par->depth > depth) {
-                par = par->Previous();
-                if (par)
-                    par = par->FirstPhysicalPar();
-            }
-        }
-        par = ppar;
-        while (par && (par->layout == layout) && (par->depth == depth)) {
-            par->pextra_type = type;
-            par->pextra_width = width;
-            par->pextra_widthp = widthp;
-            par = par->NextAfterFootnote();
-            if (par && (par->depth > depth))
-                par->SetPExtraType(type, width, widthp);
-            while (par && ((par->depth > depth) || par->IsDummy()))
-                par = par->NextAfterFootnote();
-        }
-    }
+       pextra_type = type;
+       pextra_width = width;
+       pextra_widthp = widthp;
+
+       if (textclasslist.Style(current_view->buffer()->params.textclass, 
+                               layout).isEnvironment()) {
+               LyXParagraph
+                       * par = this,
+                       * ppar = par;
+
+               while (par && (par->layout == layout)
+                      && (par->depth == depth)) {
+                       ppar = par;
+                       par = par->Previous();
+                       if (par)
+                               par = par->FirstPhysicalPar();
+                       while (par && par->depth > depth) {
+                               par = par->Previous();
+                               if (par)
+                                       par = par->FirstPhysicalPar();
+                       }
+               }
+               par = ppar;
+               while (par && (par->layout == layout)
+                      && (par->depth == depth)) {
+                       par->pextra_type = type;
+                       par->pextra_width = width;
+                       par->pextra_widthp = widthp;
+                       par = par->NextAfterFootnote();
+                       if (par && (par->depth > depth))
+                               par->SetPExtraType(type, width, widthp);
+                       while (par && ((par->depth > depth) || par->IsDummy()))
+                               par = par->NextAfterFootnote();
+               }
+       }
 }
 
 
 void LyXParagraph::UnsetPExtraType()
 {
-    if (pextra_type == PEXTRA_NONE)
-        return;
+       if (pextra_type == PEXTRA_NONE)
+               return;
     
-    pextra_type = PEXTRA_NONE;
-    pextra_width.clear();
-    pextra_widthp.clear();
-
-    if (textclasslist.Style(GetCurrentTextClass(), 
-                       layout).isEnvironment()) {
-        LyXParagraph
-            * par = this,
-            * ppar = par;
-
-        while (par && (par->layout == layout) && (par->depth == depth)) {
-            ppar = par;
-            par = par->Previous();
-            if (par)
-                par = par->FirstPhysicalPar();
-            while (par && par->depth > depth) {
-                par = par->Previous();
-                if (par)
-                    par = par->FirstPhysicalPar();
-            }
-        }
-        par = ppar;
-        while (par && (par->layout == layout) && (par->depth == depth)) {
-            par->pextra_type = PEXTRA_NONE;
-            par->pextra_width.clear();
-            par->pextra_widthp.clear();
-            par = par->NextAfterFootnote();
-            if (par && (par->depth > depth))
-                par->UnsetPExtraType();
-            while (par && ((par->depth > depth) || par->IsDummy()))
-                par = par->NextAfterFootnote();
-        }
-    }
+       pextra_type = PEXTRA_NONE;
+       pextra_width.clear();
+       pextra_widthp.clear();
+
+       if (textclasslist.Style(current_view->buffer()->params.textclass, 
+                               layout).isEnvironment()) {
+               LyXParagraph
+                       * par = this,
+                       * ppar = par;
+
+               while (par && (par->layout == layout)
+                      && (par->depth == depth)) {
+                       ppar = par;
+                       par = par->Previous();
+                       if (par)
+                               par = par->FirstPhysicalPar();
+                       while (par && par->depth > depth) {
+                               par = par->Previous();
+                               if (par)
+                                       par = par->FirstPhysicalPar();
+                       }
+               }
+               par = ppar;
+               while (par && (par->layout == layout)
+                      && (par->depth == depth)) {
+                       par->pextra_type = PEXTRA_NONE;
+                       par->pextra_width.clear();
+                       par->pextra_widthp.clear();
+                       par = par->NextAfterFootnote();
+                       if (par && (par->depth > depth))
+                               par->UnsetPExtraType();
+                       while (par && ((par->depth > depth) || par->IsDummy()))
+                               par = par->NextAfterFootnote();
+               }
+       }
 }
 
 
@@ -4302,5 +4328,5 @@ bool LyXParagraph::IsLetter(LyXParagraph::size_type pos) const
  
 bool LyXParagraph::IsWord(size_type pos ) const
 {
-       return IsWordChar( GetChar(pos) ) ;
+       return IsWordChar(GetChar(pos)) ;
 }