Use symbols file to lookup entities for delimiters. Fixes bug #8280.

Based upon work by Josh Hieronymus.

diff --git a/src/mathed/InsetMathBig.cpp b/src/mathed/InsetMathBig.cpp
index 969dd472c72fd5e512df6675cd30cda043952037..293ff8b80db6cb3bab84342537a5f01ed7794225 100644 (file)
--- a/src/mathed/InsetMathBig.cpp
+++ b/src/mathed/InsetMathBig.cpp
@@ -107,48 +107,9 @@ void InsetMathBig::normalize(NormalStream & os) const
 
 void InsetMathBig::mathmlize(MathStream & os) const
 {
-       os << "<mo form='prefix' fence='true' stretchy='true' symmetric='true'>";
-       if (delim_ == "(" || delim_ == ")"
-                       || delim_ == "[" || delim_ == "]"
-                       || delim_ == "|" || delim_ == "/")
-               os << delim_;
-       else if (delim_ == "\\{" || delim_ == "\\lbrace")
-               os << "{";
-       else if (delim_ == "\\}" || delim_ == "\\rbrace")
-               os << "}";
-       else if (delim_ == "\\slash")
-               os << "/";
-       else if (delim_ == "\\|" || delim_ == "\\vert")
-               os << "|";
-       else if (delim_ == "\\Vert")
-               os << "&par;";
-       else if (delim_ == "\\\\" || delim_ == "\\backslash")
-               os <<" \\";
-       else if (delim_ == "\\langle")
-               os << "&lt;";
-       else if (delim_ == "\\rangle")
-               os << "&gt;";
-       else if (delim_ == "\\lceil")
-               os << "&lceil;";
-       else if (delim_ == "\\rceil")
-               os << "&rceil;";
-       else if (delim_ == "\\lfloor")
-               os << "&lfloor;";
-       else if (delim_ == "\\rfloor")
-               os << "&rfloor;";
-       else if (delim_ == "\\downarrow")
-               os << "&darr;";
-       else if (delim_ == "\\uparrow")
-               os << "&uarr;";
-       else if (delim_ == "\\Downarrow")
-               os << "&dArr;";
-       else if (delim_ == "\\Uparrow")
-               os << "&uArr;";
-       else if (delim_ == "\\updownarrow")
-               os << "&varr;";
-       else if (delim_ == "\\Updownarrow")
-               os << "&vArr;";
-       os << "</mo>";
+       os << "<mo form='prefix' fence='true' stretchy='true' symmetric='true'>"
+          << convertDelimToXMLEscape(delim_)
+          << "</mo>";
 }
 
 
@@ -161,48 +122,9 @@ void InsetMathBig::htmlize(HtmlStream & os) const
        case 4: case 5: name = "biggg"; break;
        default: name  = "big"; break;
        }
-       os << MTag("span", "class='" + name + "symbol'");
-       if (delim_ == "(" || delim_ == ")"
-                       || delim_ == "[" || delim_ == "]"
-                       || delim_ == "|" || delim_ == "/")
-               os << delim_;
-       else if (delim_ == "\\{" || delim_ == "\\lbrace")
-               os << "{";
-       else if (delim_ == "\\}" || delim_ == "\\rbrace")
-               os << "}";
-       else if (delim_ == "\\slash")
-               os << "/";
-       else if (delim_ == "\\|" || delim_ == "\\vert")
-               os << "|";
-       else if (delim_ == "\\Vert")
-               os << "&par;";
-       else if (delim_ == "\\\\" || delim_ == "\\backslash")
-               os <<" \\";
-       else if (delim_ == "\\langle")
-               os << "&lt;";
-       else if (delim_ == "\\rangle")
-               os << "&gt;";
-       else if (delim_ == "\\lceil")
-               os << "&lceil;";
-       else if (delim_ == "\\rceil")
-               os << "&rceil;";
-       else if (delim_ == "\\lfloor")
-               os << "&lfloor;";
-       else if (delim_ == "\\rfloor")
-               os << "&rfloor;";
-       else if (delim_ == "\\downarrow")
-               os << "&darr;";
-       else if (delim_ == "\\uparrow")
-               os << "&uarr;";
-       else if (delim_ == "\\Downarrow")
-               os << "&dArr;";
-       else if (delim_ == "\\Uparrow")
-               os << "&uArr;";
-       else if (delim_ == "\\updownarrow")
-               os << "&varr;";
-       else if (delim_ == "\\Updownarrow")
-               os << "&vArr;";
-       os << ETag("span");
+       os << MTag("span", "class='" + name + "symbol'")
+          << convertDelimToXMLEscape(delim_)
+          << ETag("span");
 }
 
 

diff --git a/src/mathed/InsetMathDelim.cpp b/src/mathed/InsetMathDelim.cpp
index b10d37ad698b81e0048dd5c2961fd5d5c3aadb15..68e4430514775eb8e8f832e4d3a04d92e635e8ad 100644 (file)
--- a/src/mathed/InsetMathDelim.cpp
+++ b/src/mathed/InsetMathDelim.cpp
@@ -193,14 +193,21 @@ void InsetMathDelim::mathematica(MathematicaStream & os) const
 
 void InsetMathDelim::mathmlize(MathStream & os) const
 {
-       os << "<mo form='prefix' fence='true' stretchy='true' symmetric='true'>" << left_ << "</mo>"
-               << cell(0) << "<mo form='postfix' fence='true' stretchy='true' symmetric='true'>" << right_ << "</mo>";
+       os << "<mo form='prefix' fence='true' stretchy='true' symmetric='true'>"
+          << convertDelimToXMLEscape(left_) 
+          << "</mo>\n"
+          << cell(0) 
+          << "\n<mo form='postfix' fence='true' stretchy='true' symmetric='true'>" 
+          << convertDelimToXMLEscape(right_) 
+          << "</mo>\n";
 }
 
 
 void InsetMathDelim::htmlize(HtmlStream & os) const
 {
-       os << left_ << cell(0) << right_;
+       os << convertDelimToXMLEscape(left_) 
+          << cell(0) 
+          << convertDelimToXMLEscape(right_);
 }
 
 

diff --git a/src/mathed/MathStream.cpp b/src/mathed/MathStream.cpp
index 78838ccd0ab25bb151e9441c4dbb42b87efb9faf..c1c0830f5a9be2a52803d7b30976894948f82d1c 100644 (file)
--- a/src/mathed/MathStream.cpp
+++ b/src/mathed/MathStream.cpp
@@ -12,6 +12,7 @@
 
 #include "MathStream.h"
 
+#include "MathFactory.h"
 #include "MathData.h"
 #include "MathExtern.h"
 
@@ -694,4 +695,25 @@ OctaveStream & operator<<(OctaveStream & os, string const & s)
 }
 
 
+docstring convertDelimToXMLEscape(docstring const & name)
+{
+       if (name.size() == 1) {
+               char_type const c = name[0];
+               if (c == '<')
+                       return from_ascii("&lt;");
+               else if (c == '>')
+                       return from_ascii("&gt;");
+               else
+                       return name;
+       }
+       MathWordList const & words = mathedWordList();
+       MathWordList::const_iterator it = words.find(name);
+       if (it != words.end()) {
+               docstring const escape = it->second.xmlname;
+               return escape;
+       }
+       LYXERR0("Unable to find `" << name <<"' in the mathWordList.");
+       return name;
+}
+
 } // namespace lyx

diff --git a/src/mathed/MathStream.h b/src/mathed/MathStream.h
index 479e9d459e8e931cad9d749aab1b047ddbc2a566..a9465cc4945ec10ce838fb1ebdaffbaffdc1d509 100644 (file)
--- a/src/mathed/MathStream.h
+++ b/src/mathed/MathStream.h
@@ -584,6 +584,9 @@ OctaveStream & operator<<(OctaveStream &, char);
 ///
 OctaveStream & operator<<(OctaveStream &, int);
 
+
+docstring convertDelimToXMLEscape(docstring const & name);
+
 } // namespace lyx
 
 #endif