Introduce a return value for mathmlize(). We will need this to be able

[features.git] / src / mathed / InsetMathChar.cpp

diff --git a/src/mathed/InsetMathChar.cpp b/src/mathed/InsetMathChar.cpp
index 64e258e7798f20a661139eebb71e1f94009013a0..27093e8504ca1ed241d1602f1f5f5c32ecb94ac9 100644 (file)
--- a/src/mathed/InsetMathChar.cpp
+++ b/src/mathed/InsetMathChar.cpp
@@ -161,21 +161,25 @@ void InsetMathChar::octave(OctaveStream & os) const
 
 // We have a bit of a problem here. MathML wants to know whether the
 // character represents an "identifier" or an "operator", and we have
-// no general way of telling. So we shall guess: If it's mathalpha,
-// then we'll treat it as an identifier, otherwise as an operator.
-// Worst case: We get bad spacing.
-void InsetMathChar::mathmlize(MathStream & ms) const
+// no general way of telling. So we shall guess: If it's alpha or 
+// mathalpha, then we'll treat it as an identifier, otherwise as an 
+// operator.
+// Worst case: We get bad spacing, or bad italics.
+docstring InsetMathChar::mathmlize(MathStream & ms) const
 {
        switch (char_) {
-               case '<': ms << "<mo>&lt;</mo>"; break;
-               case '>': ms << "<mo>&gt;</mo>"; break;
-               case '&': ms << "<mo>&amp;</mo>"; break;
+               case '<': ms << "<mo>&lt;</mo>"; return docstring();
+               case '>': ms << "<mo>&gt;</mo>"; return docstring();
+               case '&': ms << "<mo>&amp;</mo>"; return docstring();
                default: break;
        }
        
-       char const * type = (isalpha(char_) || Encodings::isMathAlpha(char_)) ? "mi" : "mo";
+       char const * type = 
+               (isalpha(char_) || Encodings::isMathAlpha(char_))       
+                       ? "mi" : "mo";
        // we don't use MTag and ETag because we do not want the spacing
-       ms << "<" << type << ">" << char_ << "</" << type << ">";       
+       ms << "<" << type << ">" << char(char_) << "</" << type << ">"; 
+       return docstring();
 }