Andreas' patch to prevent crash on click on previewd inset

[lyx.git] / src / mathed / math_factory.C

diff --git a/src/mathed/math_factory.C b/src/mathed/math_factory.C
index d9e28d85b222de7f8d178e25c78620409d761d4f..c865d52e0aad230100bff7ad86729594a0512693 100644 (file)
--- a/src/mathed/math_factory.C
+++ b/src/mathed/math_factory.C
@@ -16,11 +16,13 @@
 #include "math_amsarrayinset.h"
 #include "math_binominset.h"
 #include "math_boxinset.h"
+#include "math_boxedinset.h"
 #include "math_boldsymbolinset.h"
 #include "math_casesinset.h"
+#include "math_colorinset.h"
 #include "math_decorationinset.h"
+#include "math_dfracinset.h"
 #include "math_dotsinset.h"
-#include "math_ertinset.h"
 #include "math_fboxinset.h"
 #include "math_frameboxinset.h"
 #include "math_fontinset.h"
@@ -34,7 +36,7 @@
 #include "math_macrotemplate.h"
 #include "math_makeboxinset.h"
 #include "math_oversetinset.h"
-#include "math_parboxinset.h"
+#include "math_parser.h"
 #include "math_rootinset.h"
 #include "math_sizeinset.h"
 #include "math_spaceinset.h"
@@ -44,6 +46,7 @@
 #include "math_substackinset.h"
 #include "math_symbolinset.h"
 #include "math_tabularinset.h"
+#include "math_tfracinset.h"
 #include "math_undersetinset.h"
 #include "math_unknowninset.h"
 #include "math_xarrowinset.h"
@@ -53,16 +56,19 @@
 
 #include "debug.h"
 #include "math_support.h"
-#include "support/std_sstream.h"
+
 #include "support/filetools.h" // LibFileSearch
 #include "support/lstrings.h"
+
 #include "frontends/lyx_gui.h"
 
 #include <fstream>
+#include <sstream>
 
 using lyx::support::LibFileSearch;
 using lyx::support::split;
 
+using std::string;
 using std::endl;
 using std::istringstream;
 
@@ -118,7 +124,7 @@ void initSymbols()
 
                // special case of iffont/else/endif
                if (line.size() >= 7 && line.substr(0, 6) == "iffont") {
-                       istringstream is(STRCONV(line));
+                       istringstream is(line);
                        string tmp;
                        is >> tmp;
                        is >> tmp;
@@ -134,13 +140,12 @@ void initSymbols()
 
                // special case of pre-defined macros
                if (line.size() > 8 && line.substr(0, 5) == "\\def\\") {
-                       //lyxerr << "defining: '" << line << '\'' << endl;
-                       istringstream is(STRCONV(line));
-                       MathMacroTable::create(MathAtom(new MathMacroTemplate(is)));
+                       //lyxerr << "macro definition: '" << line << '\'' << endl;
+                       MacroTable::globalMacros().insert(line);
                        continue;
                }
 
-               istringstream is(STRCONV(line));
+               istringstream is(line);
                latexkeys tmp;
                is >> tmp.name >> tmp.inset;
                if (isFontName(tmp.inset))
@@ -158,6 +163,12 @@ void initSymbols()
                        // tmp.inset _is_ the fontname here.
                        // create fallbacks if necessary
 
+                       // store requirements as long as we can
+                       if (tmp.inset == "msa" || tmp.inset == "msb")
+                               tmp.requires = "amssymb";
+                       else if (tmp.inset == "wasy")
+                               tmp.requires = "wasysym";
+
                        // symbol font is not available sometimes
                        string symbol_font = "lyxsymbol";
 
@@ -213,6 +224,7 @@ void initMath()
 {
        static bool initialized = false;
        if (!initialized) {
+               initParser();
                initSymbols();
                initialized = true;
        }
@@ -222,21 +234,17 @@ void initMath()
 latexkeys const * in_word_set(string const & str)
 {
        WordList::iterator it = theWordList.find(str);
-       //lyxerr << "looking up '" << str << "' found: "
-       // << (it != theWordList.end()) << endl;
-       return (it != theWordList.end()) ? &(it->second) : 0;
+       return it != theWordList.end() ? &(it->second) : 0;
 }
 
 
 MathAtom createMathInset(string const & s)
 {
-       lyxerr[Debug::MATHED] << "creating inset with name: '"
-                             << s << '\'' << endl;;
+       //lyxerr << "creating inset with name: '" << s << '\'' << endl;
        latexkeys const * l = in_word_set(s);
        if (l) {
                string const & inset = l->inset;
-               lyxerr[Debug::MATHED] << " found inset: '" <<
-                       inset << '\'' << endl;
+               //lyxerr << " found inset: '" << inset << '\'' << endl;
                if (inset == "ref")
                        return MathAtom(new RefInset(l->name));
                if (inset == "overset")
@@ -250,11 +258,13 @@ MathAtom createMathInset(string const & s)
                if (inset == "dots")
                        return MathAtom(new MathDotsInset(l));
                if (inset == "mbox")
+                       // return MathAtom(new MathMBoxInset);
+                       // MathMBoxInset is proposed to replace MathBoxInset,
+                       // but is not ready yet (it needs a BufferView for
+                       // construction)
                        return MathAtom(new MathBoxInset(l->name));
-               if (inset == "parbox")
-                       return MathAtom(new MathParboxInset);
-               if (inset == "fbox")
-                       return MathAtom(new MathFboxInset(l));
+//             if (inset == "fbox")
+//                     return MathAtom(new MathFboxInset(l));
                if (inset == "style")
                        return MathAtom(new MathSizeInset(l));
                if (inset == "font")
@@ -271,6 +281,10 @@ MathAtom createMathInset(string const & s)
        if (s.size() == 3 && s[0] == '\\' && s[1] == '#'
                        && s[2] >= '1' && s[2] <= '9')
                return MathAtom(new MathMacroArgument(s[2] - '0'));
+       if (s == "boxed")
+               return MathAtom(new MathBoxedInset());
+       if (s == "fbox")
+               return MathAtom(new MathFboxInset());
        if (s == "framebox")
                return MathAtom(new MathFrameboxInset);
        if (s == "makebox")
@@ -279,7 +293,7 @@ MathAtom createMathInset(string const & s)
                return MathAtom(new MathKernInset);
        if (s == "xrightarrow" || s == "xleftarrow")
                return MathAtom(new MathXArrowInset(s));
-       if (s == "split" || s == "gathered" || s == "aligned")
+       if (s == "split" || s == "gathered" || s == "aligned" || s == "alignedat")
                return MathAtom(new MathSplitInset(s));
        if (s == "cases")
                return MathAtom(new MathCasesInset);
@@ -305,15 +319,25 @@ MathAtom createMathInset(string const & s)
                return MathAtom(new MathFracInset(true));
        if (s == "lefteqn")
                return MathAtom(new MathLefteqnInset);
-       if (s == "lyxert")
-               return MathAtom(new MathErtInset);
        if (s == "boldsymbol")
                return MathAtom(new MathBoldsymbolInset);
-
-       if (MathMacroTable::has(s))
-               return MathAtom(new MathMacro(s));
-
-       //lyxerr[Debug::MATHED] << "creating inset 2 with name: '" << s << '\'' << endl;
+       if (s == "color" || s == "normalcolor")
+               return MathAtom(new MathColorInset(true));
+       if (s == "textcolor")
+               return MathAtom(new MathColorInset(false));
+       if (s == "dfrac")
+               return MathAtom(new MathDfracInset);
+       if (s == "tfrac")
+               return MathAtom(new MathTfracInset);
+
+       if (MacroTable::globalMacros().has(s))
+               return MathAtom(new MathMacro(s,
+                       MacroTable::globalMacros().get(s).numargs()));
+       //if (MacroTable::localMacros().has(s))
+       //      return MathAtom(new MathMacro(s,
+       //              MacroTable::localMacros().get(s).numargs()));
+
+       //lyxerr << "creating unknown inset '" << s << "'" << endl;
        return MathAtom(new MathUnknownInset(s));
 }
 
@@ -340,3 +364,4 @@ bool createMathInset_fromDialogStr(string const & str, MathArray & ar)
 
        return ar[0].nucleus();
 }
+