]> git.lyx.org Git - lyx.git/blobdiff - src/mathed/math_parser.C
the convert patch
[lyx.git] / src / mathed / math_parser.C
index fbd3ad56438706f3e1c4fce8f3e2d2e865b8d8fe..1903f0432e11cfbf34596d8888a587a17216b170 100644 (file)
@@ -1,5 +1,11 @@
-/** The math parser
-    \author André Pönitz (2001)
+/**
+ * \file math_parser.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
+ *
+ * \author André Pönitz
+ *
+ * Full author contact details are available in file CREDITS.
  */
 
 /*
@@ -32,51 +38,44 @@ following hack as starting point to write some macros:
 
 #include <config.h>
 
-
 #include "math_parser.h"
-#include "math_inset.h"
 #include "math_arrayinset.h"
 #include "math_braceinset.h"
-#include "math_boxinset.h"
 #include "math_charinset.h"
 #include "math_commentinset.h"
 #include "math_deliminset.h"
 #include "math_envinset.h"
-#include "math_extern.h"
 #include "math_factory.h"
 #include "math_kerninset.h"
 #include "math_macro.h"
+#include "math_macroarg.h"
 #include "math_macrotemplate.h"
-#include "math_hullinset.h"
-#include "math_parboxinset.h"
 #include "math_parinset.h"
 #include "math_rootinset.h"
 #include "math_scriptinset.h"
-#include "math_sizeinset.h"
 #include "math_sqrtinset.h"
-#include "math_stringinset.h"
 #include "math_support.h"
 #include "math_tabularinset.h"
-#include "math_xyarrowinset.h"
 
 //#include "insets/insetref.h"
 #include "ref_inset.h"
 
 #include "lyxlex.h"
 #include "debug.h"
-#include "support/LAssert.h"
-#include "support/lstrings.h"
 
-#include <cctype>
-#include <algorithm>
+#include "support/convert.h"
+
+#include <sstream>
 
-using std::istream;
-using std::ostream;
-using std::ios;
 using std::endl;
 using std::fill;
+
+using std::string;
+using std::ios;
+using std::istream;
+using std::istringstream;
+using std::ostream;
 using std::vector;
-using std::atoi;
 
 
 //#define FILEDEBUG
@@ -86,6 +85,7 @@ namespace {
 
 MathInset::mode_type asMode(MathInset::mode_type oldmode, string const & str)
 {
+       //lyxerr << "handling mode: '" << str << "'" << endl;
        if (str == "mathmode")
                return MathInset::MATH_MODE;
        if (str == "textmode" || str == "forcetext")
@@ -101,6 +101,65 @@ bool stared(string const & s)
 }
 
 
+/*!
+ * Add the row \p cellrow to \p grid.
+ * \returns wether the row could be added. Adding a row can fail for
+ * environments like "equation" that have a fixed number of rows.
+ */
+bool addRow(MathGridInset & grid, MathGridInset::row_type & cellrow,
+            string const & vskip)
+{
+       ++cellrow;
+       if (cellrow == grid.nrows()) {
+               //lyxerr << "adding row " << cellrow << endl;
+               grid.addRow(cellrow - 1);
+               if (cellrow == grid.nrows()) {
+                       // We can't add a row to this grid, so let's
+                       // append the content of this cell to the previous
+                       // one.
+                       // This does not happen in well formed .lyx files,
+                       // but LyX versions 1.3.x and older could create
+                       // such files and tex2lyx can still do that.
+                       --cellrow;
+                       lyxerr << "ignoring extra row";
+                       if (!vskip.empty())
+                               lyxerr << " with extra space " << vskip;
+                       lyxerr << '.' << endl;
+                       return false;
+               }
+       }
+       grid.vcrskip(LyXLength(vskip), cellrow - 1);
+       return true;
+}
+
+
+/*!
+ * Add the column \p cellcol to \p grid.
+ * \returns wether the column could be added. Adding a column can fail for
+ * environments like "eqnarray" that have a fixed number of columns.
+ */
+bool addCol(MathGridInset & grid, MathGridInset::col_type & cellcol)
+{
+       ++cellcol;
+       if (cellcol == grid.ncols()) {
+               //lyxerr << "adding column " << cellcol << endl;
+               grid.addCol(cellcol - 1);
+               if (cellcol == grid.ncols()) {
+                       // We can't add a column to this grid, so let's
+                       // append the content of this cell to the previous
+                       // one.
+                       // This does not happen in well formed .lyx files,
+                       // but LyX versions 1.3.x and older could create
+                       // such files and tex2lyx can still do that.
+                       --cellcol;
+                       lyxerr << "ignoring extra column." << endl;
+                       return false;
+               }
+       }
+       return true;
+}
+
+
 // These are TeX's catcodes
 enum CatCode {
        catEscape,     // 0    backslash
@@ -131,7 +190,8 @@ inline CatCode catcode(unsigned char c)
 
 
 enum {
-       FLAG_BRACE_LAST = 1 << 1,  //  last closing brace ends the parsing
+       FLAG_ALIGN      = 1 << 0,  //  next & or \\ ends the parsing process
+       FLAG_BRACE_LAST = 1 << 1,  //  next closing brace ends the parsing
        FLAG_RIGHT      = 1 << 2,  //  next \\right ends the parsing process
        FLAG_END        = 1 << 3,  //  next \\end ends the parsing process
        FLAG_BRACK_LAST = 1 << 4,  //  next closing bracket ends the parsing
@@ -146,31 +206,6 @@ enum {
 };
 
 
-void catInit()
-{
-       fill(theCatcode, theCatcode + 256, catOther);
-       fill(theCatcode + 'a', theCatcode + 'z' + 1, catLetter);
-       fill(theCatcode + 'A', theCatcode + 'Z' + 1, catLetter);
-
-       theCatcode['\\'] = catEscape;
-       theCatcode['{']  = catBegin;
-       theCatcode['}']  = catEnd;
-       theCatcode['$']  = catMath;
-       theCatcode['&']  = catAlign;
-       theCatcode['\n'] = catNewline;
-       theCatcode['#']  = catParameter;
-       theCatcode['^']  = catSuper;
-       theCatcode['_']  = catSub;
-       theCatcode['\7f'] = catIgnore;
-       theCatcode[' ']  = catSpace;
-       theCatcode['\t'] = catSpace;
-       theCatcode['\r'] = catNewline;
-       theCatcode['~']  = catActive;
-       theCatcode['%']  = catComment;
-}
-
-
-
 //
 // Helper class for parsing
 //
@@ -206,6 +241,8 @@ ostream & operator<<(ostream & os, Token const & t)
 {
        if (t.cs().size())
                os << '\\' << t.cs();
+       else if (t.cat() == catLetter)
+               os << t.character();
        else
                os << '[' << t.character() << ',' << t.cat() << ']';
        return os;
@@ -213,7 +250,6 @@ ostream & operator<<(ostream & os, Token const & t)
 
 
 class Parser {
-
 public:
        ///
        typedef  MathInset::mode_type mode_type;
@@ -328,7 +364,7 @@ Token const & Parser::nextToken() const
 Token const & Parser::getToken()
 {
        static const Token dummy;
-       //lyxerr << "looking at token " << tokens_[pos_] << " pos: " << pos_ << '\n';
+       //lyxerr << "looking at token " << tokens_[pos_] << " pos: " << pos_ << endl;
        return good() ? tokens_[pos_++] : dummy;
 }
 
@@ -383,7 +419,7 @@ void Parser::skipSpaceTokens(istream & is, char c)
        while (catcode(c) == catSpace || catcode(c) == catNewline)
                if (!is.get(c))
                        break;
-       //lyxerr << "putting back: " << c << "\n";
+       //lyxerr << "putting back: " << c << endl;
        is.putback(c);
 }
 
@@ -412,18 +448,11 @@ void Parser::tokenize(istream & is)
 
 void Parser::tokenize(string const & buffer)
 {
-       static bool init_done = false;
-
-       if (!init_done) {
-               catInit();
-               init_done = true;
-       }
-
-       istringstream is(buffer.c_str(), ios::in | ios::binary);
+       istringstream is(buffer, ios::in | ios::binary);
 
        char c;
        while (is.get(c)) {
-               //lyxerr << "reading c: " << c << "\n";
+               //lyxerr << "reading c: " << c << endl;
 
                switch (catcode(c)) {
                        case catNewline: {
@@ -473,7 +502,7 @@ void Parser::tokenize(string const & buffer)
                        }
 
                        case catIgnore: {
-                               lyxerr << "ignoring a char: " << int(c) << "\n";
+                               lyxerr << "ignoring a char: " << int(c) << endl;
                                break;
                        }
 
@@ -496,7 +525,7 @@ void Parser::dump() const
                        lyxerr << " <#> ";
                lyxerr << tokens_[i];
        }
-       lyxerr << " pos: " << pos_ << "\n";
+       lyxerr << " pos: " << pos_ << endl;
 }
 
 
@@ -529,6 +558,7 @@ bool Parser::parse(MathAtom & at)
 
 string Parser::parse_verbatim_option()
 {
+       skipSpaces();
        string res;
        if (nextToken().character() == '[') {
                Token t = getToken();
@@ -546,6 +576,7 @@ string Parser::parse_verbatim_option()
 
 string Parser::parse_verbatim_item()
 {
+       skipSpaces();
        string res;
        if (nextToken().cat() == catBegin) {
                Token t = getToken();
@@ -578,15 +609,15 @@ void Parser::parse(MathArray & array, unsigned flags, mode_type mode)
 }
 
 
-void Parser::parse2(MathAtom & at, unsigned flags, mode_type mode,
-       bool numbered)
+void Parser::parse2(MathAtom & at, const unsigned flags, const mode_type mode,
+       const bool numbered)
 {
        parse1(*(at.nucleus()->asGridInset()), flags, mode, numbered);
 }
 
 
 void Parser::parse1(MathGridInset & grid, unsigned flags,
-       mode_type mode, bool numbered)
+       const mode_type mode, const bool numbered)
 {
        int limits = 0;
        MathGridInset::row_type cellrow = 0;
@@ -597,31 +628,31 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                grid.asHullInset()->numbered(cellrow, numbered);
 
        //dump();
+       //lyxerr << " flags: " << flags << endl;
+       //lyxerr << " mode: " << mode  << endl;
        //lyxerr << "grid: " << grid << endl;
 
        while (good()) {
                Token const & t = getToken();
 
 #ifdef FILEDEBUG
-               lyxerr << "t: " << t << " flags: " << flags << "\n";
+               lyxerr << "t: " << t << " flags: " << flags << endl;
+               lyxerr << "mode: " << mode  << endl;
                cell->dump();
-               lyxerr << "\n";
+               lyxerr << endl;
 #endif
 
                if (flags & FLAG_ITEM) {
-                       if (t.cat() == catSpace)
-                               continue;
 
-                       flags &= ~FLAG_ITEM;
                        if (t.cat() == catBegin) {
                                // skip the brace and collect everything to the next matching
                                // closing brace
-                               flags |= FLAG_BRACE_LAST;
-                               continue;
+                               parse1(grid, FLAG_BRACE_LAST, mode, numbered);
+                               return;
                        }
 
                        // handle only this single token, leave the loop if done
-                       flags |= FLAG_LEAVE;
+                       flags = FLAG_LEAVE;
                }
 
 
@@ -678,7 +709,7 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        }
 
                        else {
-                               error("something strange in the parser\n");
+                               error("something strange in the parser");
                                break;
                        }
                }
@@ -691,8 +722,10 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                                cell->push_back(MathAtom(new MathCharInset(t.character())));
                }
 
-               else if (t.cat() == catNewline && mode != MathInset::MATH_MODE)
-                       cell->push_back(MathAtom(new MathCharInset(t.character())));
+               else if (t.cat() == catNewline && mode != MathInset::MATH_MODE) {
+                       if (cell->empty() || cell->back()->getChar() != ' ')
+                               cell->push_back(MathAtom(new MathCharInset(' ')));
+               }
 
                else if (t.cat() == catParameter) {
                        Token const & n = getToken();
@@ -717,18 +750,17 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        if (flags & FLAG_BRACE_LAST)
                                return;
                        error("found '}' unexpectedly");
-                       //lyx::Assert(0);
+                       //BOOST_ASSERT(false);
                        //add(cell, '}', LM_TC_TEX);
                }
 
                else if (t.cat() == catAlign) {
-                       ++cellcol;
-                       //lyxerr << " column now " << cellcol << " max: " << grid.ncols() << "\n";
-                       if (cellcol == grid.ncols()) {
-                               //lyxerr << "adding column " << cellcol << "\n";
-                               grid.addCol(cellcol - 1);
-                       }
-                       cell = &grid.cell(grid.index(cellrow, cellcol));
+                       //lyxerr << " column now " << (cellcol + 1)
+                       //       << " max: " << grid.ncols() << endl;
+                       if (flags & FLAG_ALIGN)
+                               return;
+                       if (addCol(grid, cellcol))
+                               cell = &grid.cell(grid.index(cellrow, cellcol));
                }
 
                else if (t.cat() == catSuper || t.cat() == catSub) {
@@ -751,7 +783,7 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        //if (p->nuc().size() == 1 && p->nuc().back()->asNestInset() &&
                        //    p->nuc().back()->extraBraces())
                        //      p->nuc() = p->nuc().back()->asNestInset()->cell(0);
-                       parse(p->cell(up), FLAG_ITEM, mode);
+                       parse(p->cell(p->idxOfScript(up)), FLAG_ITEM, mode);
                        if (limits) {
                                p->limits(limits);
                                limits = 0;
@@ -759,7 +791,7 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                }
 
                else if (t.character() == ']' && (flags & FLAG_BRACK_LAST)) {
-                       //lyxerr << "finished reading option\n";
+                       //lyxerr << "finished reading option" << endl;
                        return;
                }
 
@@ -787,7 +819,11 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                                cell->back().nucleus()->lock(true);
                }
 
-               else if (t.cs() == "def" || t.cs() == "newcommand") {
+               else if (t.cs() == "def" ||
+                       t.cs() == "newcommand" ||
+                       t.cs() == "renewcommand")
+               {
+                       string const type = t.cs();
                        string name;
                        int nargs = 0;
                        if (t.cs() == "def") {
@@ -801,25 +837,25 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                                        ++nargs;
                                }
                                nargs /= 2;
-                               //lyxerr << "read \\def parameter list '" << pars << "'\n";
+                               //lyxerr << "read \\def parameter list '" << pars << "'" << endl;
 
-                       } else { // t.cs() == "newcommand"
+                       } else { // t.cs() == "newcommand" || t.cs() == "renewcommand"
 
                                if (getToken().cat() != catBegin) {
-                                       error("'{' in \\newcommand expected (1) \n");
+                                       error("'{' in \\newcommand expected (1) ");
                                        return;
                                }
 
                                name = getToken().cs();
 
                                if (getToken().cat() != catEnd) {
-                                       error("'}' in \\newcommand expected\n");
+                                       error("'}' in \\newcommand expected");
                                        return;
                                }
 
-                               string arg  = getArg('[', ']');
+                               string const arg  = getArg('[', ']');
                                if (!arg.empty())
-                                       nargs = atoi(arg.c_str());
+                                       nargs = convert<int>(arg);
 
                        }
 
@@ -830,18 +866,18 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        //MathArray test;
                        //test.push_back(createMathInset(name));
                        //if (ar1.contains(test)) {
-                       //      error("we cannot handle recursive macros at all.\n");
+                       //      error("we cannot handle recursive macros at all.");
                        //      return;
                        //}
 
                        // is a version for display attached?
                        skipSpaces();
                        MathArray ar2;
-                       if (nextToken().cat() == catBegin) {
+                       if (nextToken().cat() == catBegin)
                                parse(ar2, FLAG_ITEM, MathInset::MATH_MODE);
-                       }
 
-                       cell->push_back(MathAtom(new MathMacroTemplate(name, nargs, ar1, ar2)));
+                       cell->push_back(MathAtom(new MathMacroTemplate(name, nargs, type,
+                               ar1, ar2)));
                }
 
                else if (t.cs() == "(") {
@@ -882,14 +918,16 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                }
 
                else if (t.cs() == "\\") {
-                       grid.vcrskip(LyXLength(getArg('[', ']')), cellrow);
-                       ++cellrow;
-                       cellcol = 0;
-                       if (cellrow == grid.nrows())
-                               grid.addRow(cellrow - 1);
-                       if (grid.asHullInset())
-                               grid.asHullInset()->numbered(cellrow, numbered);
-                       cell = &grid.cell(grid.index(cellrow, cellcol));
+                       if (flags & FLAG_ALIGN)
+                               return;
+                       if (addRow(grid, cellrow, getArg('[', ']'))) {
+                               cellcol = 0;
+                               if (grid.asHullInset())
+                                       grid.asHullInset()->numbered(
+                                                       cellrow, numbered);
+                               cell = &grid.cell(grid.index(cellrow,
+                                                            cellcol));
+                       }
                }
 
 #if 0
@@ -899,20 +937,19 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        parse(count, FLAG_ITEM, mode);
                        int cols = 1;
                        if (!extractNumber(count, cols)) {
-                               lyxerr << " can't extract number of cells from " << count << "\n";
+                               lyxerr << " can't extract number of cells from " << count << endl;
                        }
                        // resize the table if necessary
                        for (int i = 0; i < cols; ++i) {
-                               ++cellcol;
-                               if (cellcol == grid.ncols()) {
-                                       //lyxerr << "adding column " << cellcol << "\n";
-                                       grid.addCol(cellcol - 1);
+                               if (addCol(grid, cellcol)) {
+                                       cell = &grid.cell(grid.index(
+                                                       cellrow, cellcol));
+                                       // mark this as dummy
+                                       grid.cellinfo(grid.index(
+                                               cellrow, cellcol)).dummy_ = true;
                                }
-                               cell = &grid.cell(grid.index(cellrow, cellcol));
-                               // mark this as dummy
-                               grid.cellinfo(grid.index(cellrow, cellcol)).dummy_ = true;
                        }
-                       // the last cell is the real thng, not a dummy
+                       // the last cell is the real thing, not a dummy
                        grid.cellinfo(grid.index(cellrow, cellcol)).dummy_ = false;
 
                        // read special alignment
@@ -964,7 +1001,7 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        parse(cell->back().nucleus()->cell(0), FLAG_ITEM, mode);
                }
 
-               else if (t.cs() == "ref" || t.cs() == "prettyref" || 
+               else if (t.cs() == "ref" || t.cs() == "prettyref" ||
                                t.cs() == "pageref" || t.cs() == "vpageref" || t.cs() == "vref") {
                        cell->push_back(MathAtom(new RefInset(t.cs())));
                        parse(cell->back().nucleus()->cell(1), FLAG_OPTION, mode);
@@ -972,9 +1009,11 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                }
 
                else if (t.cs() == "left") {
+                       skipSpaces();
                        string l = getToken().asString();
                        MathArray ar;
                        parse(ar, FLAG_RIGHT, mode);
+                       skipSpaces();
                        string r = getToken().asString();
                        cell->push_back(MathAtom(new MathDelimInset(l, r, ar)));
                }
@@ -982,14 +1021,13 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                else if (t.cs() == "right") {
                        if (flags & FLAG_RIGHT)
                                return;
-                       //lyxerr << "got so far: '" << cell << "'\n";
+                       //lyxerr << "got so far: '" << cell << "'" << endl;
                        error("Unmatched right delimiter");
                        return;
                }
 
                else if (t.cs() == "begin") {
                        string const name = getArg('{', '}');
-                       skipSpaces();
 
                        if (name == "array" || name == "subarray") {
                                string const valign = parse_verbatim_option() + 'c';
@@ -998,7 +1036,7 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                                parse2(cell->back(), FLAG_END, mode, false);
                        }
 
-                       if (name == "tabular") {
+                       else if (name == "tabular") {
                                string const valign = parse_verbatim_option() + 'c';
                                string const halign = parse_verbatim_item();
                                cell->push_back(MathAtom(new MathTabularInset(name, valign[0], halign)));
@@ -1006,7 +1044,8 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        }
 
                        else if (name == "split" || name == "cases" ||
-                                        name == "gathered" || name == "aligned") {
+                                        name == "gathered" || name == "aligned" ||
+                                  name == "alignedat") {
                                cell->push_back(createMathInset(name));
                                parse2(cell->back(), FLAG_END, mode, false);
                        }
@@ -1076,10 +1115,11 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        }
 
                        else {
-                               // lyxerr << "unknow math inset begin '" << name << "'\n";
+                               dump();
+                               lyxerr << "found unknown math environment '" << name << "'" << endl;
                                // create generic environment inset
                                cell->push_back(MathAtom(new MathEnvInset(name)));
-                               parse(cell->back().nucleus()->cell(0), FLAG_END, mode);
+                               parse(cell->back().nucleus()->cell(0), FLAG_ITEM, mode);
                        }
                }
 
@@ -1103,11 +1143,13 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
 
                else if (t.cs() == "label") {
                        string label = parse_verbatim_item();
+                       MathArray ar;
+                       asArray(label, ar);
                        if (grid.asHullInset()) {
                                grid.asHullInset()->label(cellrow, label);
                        } else {
                                cell->push_back(createMathInset(t.cs()));
-                               cell->push_back(MathAtom(new MathBraceInset(asArray(label))));
+                               cell->push_back(MathAtom(new MathBraceInset(ar)));
                        }
                }
 
@@ -1120,17 +1162,20 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        return;
                }
 
-               else if (t.cs() == "substack") {
-                       cell->push_back(createMathInset(t.cs()));
-                       parse2(cell->back(), FLAG_ITEM, mode, false);
+               else if (t.cs() == "color") {
+                       MathAtom at = createMathInset(t.cs());
+                       parse(at.nucleus()->cell(0), FLAG_ITEM, MathInset::TEXT_MODE);
+                       parse(at.nucleus()->cell(1), flags, mode);
+                       cell->push_back(at);
+                       return;
                }
 
-               else if (t.cs() == "xymatrix") {
+               else if (t.cs() == "substack") {
                        cell->push_back(createMathInset(t.cs()));
                        parse2(cell->back(), FLAG_ITEM, mode, false);
                }
 
-               else if (t.cs() == "framebox") {
+               else if (t.cs() == "framebox" || t.cs() == "makebox") {
                        cell->push_back(createMathInset(t.cs()));
                        parse(cell->back().nucleus()->cell(0), FLAG_OPTION, MathInset::TEXT_MODE);
                        parse(cell->back().nucleus()->cell(1), FLAG_OPTION, MathInset::TEXT_MODE);
@@ -1147,7 +1192,7 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
 
                // Disabled
                else if (1 && t.cs() == "ar") {
-                       MathXYArrowInset * p = new MathXYArrowInset;
+                       auto_ptr<MathXYArrowInset> p(new MathXYArrowInset);
                        // try to read target
                        parse(p->cell(0), FLAG_OTPTION, mode);
                        // try to read label
@@ -1155,11 +1200,11 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                                p->up_ = nextToken().cat() == catSuper;
                                getToken();
                                parse(p->cell(1), FLAG_ITEM, mode);
-                               //lyxerr << "read label: " << p->cell(1) << "\n";
+                               //lyxerr << "read label: " << p->cell(1) << endl;
                        }
 
-                       cell->push_back(MathAtom(p));
-                       //lyxerr << "read cell: " << cell << "\n";
+                       cell->push_back(MathAtom(p.release()));
+                       //lyxerr << "read cell: " << cell << endl;
                }
 #endif
 
@@ -1168,35 +1213,35 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        if (l) {
                                if (l->inset == "font") {
                                        cell->push_back(createMathInset(t.cs()));
-                                       parse(cell->back().nucleus()->cell(0), FLAG_ITEM, asMode(mode, l->extra));
+                                       parse(cell->back().nucleus()->cell(0),
+                                               FLAG_ITEM, asMode(mode, l->extra));
                                }
 
                                else if (l->inset == "oldfont") {
                                        cell->push_back(createMathInset(t.cs()));
-                                       parse(cell->back().nucleus()->cell(0), flags, asMode(mode, l->extra));
-                                       return;
+                                       parse(cell->back().nucleus()->cell(0),
+                                               flags | FLAG_ALIGN, asMode(mode, l->extra));
+                                       if (prevToken().cat() != catAlign &&
+                                           prevToken().cs() != "\\")
+                                               return;
+                                       putback();
                                }
 
                                else if (l->inset == "style") {
                                        cell->push_back(createMathInset(t.cs()));
-                                       parse(cell->back().nucleus()->cell(0), flags, mode);
-                                       return;
-                               }
-
-                               else if (l->inset == "parbox") {
-                                       // read optional positioning and width
-                                       string pos   = parse_verbatim_option();
-                                       string width = parse_verbatim_item();
-                                       cell->push_back(createMathInset(t.cs()));
-                                       parse(cell->back().nucleus()->cell(0), FLAG_ITEM, MathInset::TEXT_MODE);
-                                       cell->back().nucleus()->asParboxInset()->setPosition(pos);
-                                       cell->back().nucleus()->asParboxInset()->setWidth(width);
+                                       parse(cell->back().nucleus()->cell(0),
+                                               flags | FLAG_ALIGN, mode);
+                                       if (prevToken().cat() != catAlign &&
+                                           prevToken().cs() != "\\")
+                                               return;
+                                       putback();
                                }
 
                                else {
                                        MathAtom at = createMathInset(t.cs());
                                        for (MathInset::idx_type i = 0; i < at->nargs(); ++i)
-                                               parse(at.nucleus()->cell(i), FLAG_ITEM, asMode(mode, l->extra));
+                                               parse(at.nucleus()->cell(i),
+                                                       FLAG_ITEM, asMode(mode, l->extra));
                                        cell->push_back(at);
                                }
                        }
@@ -1204,8 +1249,11 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                        else {
                                MathAtom at = createMathInset(t.cs());
                                MathInset::mode_type m = mode;
-                               if (m == MathInset::UNDECIDED_MODE)
+                               //if (m == MathInset::UNDECIDED_MODE)
+                               //lyxerr << "default creation: m1: " << m << endl;
+                               if (at->currentMode() != MathInset::UNDECIDED_MODE)
                                        m = at->currentMode();
+                               //lyxerr << "default creation: m2: " << m << endl;
                                MathInset::idx_type start = 0;
                                // this fails on \bigg[...\bigg]
                                //MathArray opt;
@@ -1214,8 +1262,10 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
                                //      start = 1;
                                //      at.nucleus()->cell(0) = opt;
                                //}
-                               for (MathInset::idx_type i = start; i < at->nargs(); ++i)
+                               for (MathInset::idx_type i = start; i < at->nargs(); ++i) {
                                        parse(at.nucleus()->cell(i), FLAG_ITEM, m);
+                                       skipSpaces();
+                               }
                                cell->push_back(at);
                        }
                }
@@ -1235,7 +1285,7 @@ void Parser::parse1(MathGridInset & grid, unsigned flags,
 
 void mathed_parse_cell(MathArray & ar, string const & str)
 {
-       istringstream is(str.c_str());
+       istringstream is(str);
        mathed_parse_cell(ar, is);
 }
 
@@ -1248,7 +1298,7 @@ void mathed_parse_cell(MathArray & ar, istream & is)
 
 bool mathed_parse_normal(MathAtom & t, string const & str)
 {
-       istringstream is(str.c_str());
+       istringstream is(str);
        return Parser(is).parse(t);
 }
 
@@ -1267,6 +1317,30 @@ bool mathed_parse_normal(MathAtom & t, LyXLex & lex)
 
 void mathed_parse_normal(MathGridInset & grid, string const & str)
 {
-       istringstream is(str.c_str());
+       istringstream is(str);
        Parser(is).parse1(grid, 0, MathInset::MATH_MODE, false);
 }
+
+
+void initParser()
+{
+       fill(theCatcode, theCatcode + 256, catOther);
+       fill(theCatcode + 'a', theCatcode + 'z' + 1, catLetter);
+       fill(theCatcode + 'A', theCatcode + 'Z' + 1, catLetter);
+
+       theCatcode[int('\\')] = catEscape;
+       theCatcode[int('{')]  = catBegin;
+       theCatcode[int('}')]  = catEnd;
+       theCatcode[int('$')]  = catMath;
+       theCatcode[int('&')]  = catAlign;
+       theCatcode[int('\n')] = catNewline;
+       theCatcode[int('#')]  = catParameter;
+       theCatcode[int('^')]  = catSuper;
+       theCatcode[int('_')]  = catSub;
+       theCatcode[int(0x7f)] = catIgnore;
+       theCatcode[int(' ')]  = catSpace;
+       theCatcode[int('\t')] = catSpace;
+       theCatcode[int('\r')] = catNewline;
+       theCatcode[int('~')]  = catActive;
+       theCatcode[int('%')]  = catComment;
+}