]> git.lyx.org Git - lyx.git/blobdiff - src/mathed/InsetMathNest.cpp
make Pavel happy
[lyx.git] / src / mathed / InsetMathNest.cpp
index 88e163deb4edfd9da4997b966a056d541b5f5e36..837b6234e1364d191fe4170469639a02786b64d1 100644 (file)
@@ -3,13 +3,15 @@
  * This file is part of LyX, the document processor.
  * Licence details can be found in the file COPYING.
  *
- * \author André Pönitz
+ * \author André Pönitz
  *
  * Full author contact details are available in file CREDITS.
  */
 
 #include <config.h>
 
+//#define AUTOCORRECT
+
 #include "InsetMathNest.h"
 
 #include "InsetMathArray.h"
@@ -25,6 +27,9 @@
 #include "InsetMathSpace.h"
 #include "InsetMathSymbol.h"
 #include "InsetMathUnknown.h"
+#ifdef AUTOCORRECT
+#include "MathAutoCorrect.h"
+#endif
 #include "MathCompletionList.h"
 #include "MathData.h"
 #include "MathFactory.h"
@@ -336,7 +341,9 @@ MathData InsetMathNest::glue() const
 
 void InsetMathNest::write(WriteStream & os) const
 {
-       os << '\\' << name().c_str();
+       ModeSpecifier specifier(os, currentMode());
+       docstring const latex_name = name();
+       os << '\\' << latex_name;
        for (size_t i = 0; i < nargs(); ++i)
                os << '{' << cell(i) << '}';
        if (nargs() == 0)
@@ -350,7 +357,7 @@ void InsetMathNest::write(WriteStream & os) const
 
 void InsetMathNest::normalize(NormalStream & os) const
 {
-       os << '[' << name().c_str();
+       os << '[' << name();
        for (size_t i = 0; i < nargs(); ++i)
                os << ' ' << cell(i);
        os << ']';
@@ -359,7 +366,8 @@ void InsetMathNest::normalize(NormalStream & os) const
 
 int InsetMathNest::latex(odocstream & os, OutputParams const & runparams) const
 {
-       WriteStream wi(os, runparams.moving_arg, true, runparams.dryrun);
+       WriteStream wi(os, runparams.moving_arg, true, runparams.dryrun,
+                       runparams.encoding);
        write(wi);
        return wi.line();
 }
@@ -443,7 +451,7 @@ void InsetMathNest::handleNest(Cursor & cur, MathAtom const & nest,
                cur.insert(arg);
                return;
        }
-       
+
        // multiple selected cells in a simple non-grid inset
        if (i1.asInsetMath()->nrows() == 0 || i1.asInsetMath()->ncols() == 0) {
                for (idx_type i = i1.idx(); i <= i2.idx(); ++i) {
@@ -453,11 +461,11 @@ void InsetMathNest::handleNest(Cursor & cur, MathAtom const & nest,
                        cur.resetAnchor();
                        cur.pos() = cur.lastpos();
                        cur.setSelection();
-                       
+
                        // change font of cell
                        cur.handleNest(nest);
                        cur.insert(arg);
-                       
+
                        // cur is in the font inset now. If the loop continues,
                        // we need to get outside again for the next cell
                        if (i + 1 <= i2.idx())
@@ -465,7 +473,7 @@ void InsetMathNest::handleNest(Cursor & cur, MathAtom const & nest,
                }
                return;
        }
-       
+
        // the complicated case with multiple selected cells in a grid
        row_type r1, r2;
        col_type c1, c2;
@@ -478,11 +486,11 @@ void InsetMathNest::handleNest(Cursor & cur, MathAtom const & nest,
                        cur.resetAnchor();
                        cur.pos() = cur.lastpos();
                        cur.setSelection();
-                       
-                       // 
+
+                       //
                        cur.handleNest(nest);
                        cur.insert(arg);
-               
+
                        // cur is in the font inset now. If the loop continues,
                        // we need to get outside again for the next cell
                        if (col + 1 <= c2 || row + 1 <= r2)
@@ -501,7 +509,7 @@ void InsetMathNest::handleFont2(Cursor & cur, docstring const & arg)
        if (font.fontInfo().color() != Color_inherit &&
            font.fontInfo().color() != Color_ignore)
                handleNest(cur, MathAtom(new InsetMathColor(true, font.fontInfo().color())));
-       
+
        // FIXME: support other font changes here as well?
 }
 
@@ -510,9 +518,16 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
 {
        //lyxerr << "InsetMathNest: request: " << cmd << endl;
 
+       Parse::flags parseflg = Parse::QUIET;
+
        switch (cmd.action) {
 
+       case LFUN_CLIPBOARD_PASTE:
+               parseflg |= Parse::VERBATIM;
+               // fall through
        case LFUN_PASTE: {
+               if (cur.currentMode() == TEXT_MODE)
+                       parseflg |= Parse::TEXTMODE;
                cur.recordUndoSelection();
                cur.message(_("Paste"));
                replaceSelection(cur);
@@ -525,7 +540,7 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                        is >> n;
                        topaste = cap::selection(n);
                }
-               cur.niceInsert(topaste);
+               cur.niceInsert(topaste, parseflg);
                cur.clearSelection(); // bug 393
                cur.finishUndo();
                break;
@@ -578,22 +593,22 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_CHAR_BACKWARD_SELECT:
        case LFUN_CHAR_FORWARD_SELECT: {
                // are we in a selection?
-               bool select = (cmd.action == LFUN_CHAR_RIGHT_SELECT 
+               bool select = (cmd.action == LFUN_CHAR_RIGHT_SELECT
                                           || cmd.action == LFUN_CHAR_LEFT_SELECT
                                           || cmd.action == LFUN_CHAR_BACKWARD_SELECT
                                           || cmd.action == LFUN_CHAR_FORWARD_SELECT);
-               // are we moving forward or backwards? 
+               // are we moving forward or backwards?
                // If the command was RIGHT or LEFT, then whether we're moving forward
                // or backwards depends on the cursor movement mode (logical or visual):
-               //  * in visual mode, since math is always LTR, right -> forward, 
+               //  * in visual mode, since math is always LTR, right -> forward,
                //    left -> backwards
                //  * in logical mode, the mapping is determined by the
                //    reverseDirectionNeeded() function
-               
+
                bool forward;
                FuncCode finish_lfun;
 
-               if (cmd.action == LFUN_CHAR_FORWARD 
+               if (cmd.action == LFUN_CHAR_FORWARD
                                || cmd.action == LFUN_CHAR_FORWARD_SELECT) {
                        forward = true;
                        finish_lfun = LFUN_FINISHED_FORWARD;
@@ -608,7 +623,7 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                                                  || cmd.action == LFUN_CHAR_RIGHT);
                        if (lyxrc.visual_cursor || !reverseDirectionNeeded(cur))
                                forward = right;
-                       else 
+                       else
                                forward = !right;
 
                        if (right)
@@ -641,18 +656,18 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                        cur.macroModeClose();
                        break;
                }
-               
+
                // stop/start the selection
                bool select = cmd.action == LFUN_DOWN_SELECT ||
                        cmd.action == LFUN_UP_SELECT;
                cur.selHandle(select);
-               
+
                // go up/down
                bool up = cmd.action == LFUN_UP || cmd.action == LFUN_UP_SELECT;
                bool successful = cur.upDownInMath(up);
                if (successful)
                        break;
-               
+
                if (cur.fixIfBroken())
                        // FIXME: Something bad happened. We pass the corrected Cursor
                        // instead of letting things go worse.
@@ -669,7 +684,7 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                cur.pos() = 0;
                cur.idx() = 0;
                cur.resetAnchor();
-               cur.selection() = true;
+               cur.setSelection(true);
                cur.pos() = cur.lastpos();
                cur.idx() = cur.lastidx();
                break;
@@ -689,7 +704,7 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_WORD_BACKWARD_SELECT:
        case LFUN_WORD_LEFT_SELECT:
                cur.selHandle(cmd.action == LFUN_WORD_BACKWARD_SELECT ||
-                               cmd.action == LFUN_WORD_LEFT_SELECT || 
+                               cmd.action == LFUN_WORD_LEFT_SELECT ||
                                cmd.action == LFUN_LINE_BEGIN_SELECT);
                cur.macroModeClose();
                if (cur.pos() != 0) {
@@ -825,7 +840,6 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                break;
 
        //case LFUN_SERVER_GET_XY:
-       //      sprintf(dispatch_buffer, "%d %d",);
        //      break;
 
        case LFUN_SERVER_SET_XY: {
@@ -866,12 +880,18 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                break;
 
        //  Math fonts
-       case LFUN_FONT_FREE_APPLY:
-       case LFUN_FONT_FREE_UPDATE:
+       case LFUN_TEXTSTYLE_APPLY:
+       case LFUN_TEXTSTYLE_UPDATE:
                handleFont2(cur, cmd.argument());
                break;
 
        case LFUN_FONT_BOLD:
+               if (currentMode() == TEXT_MODE)
+                       handleFont(cur, cmd.argument(), "textbf");
+               else
+                       handleFont(cur, cmd.argument(), "mathbf");
+               break;
+       case LFUN_FONT_BOLDSYMBOL:
                if (currentMode() == TEXT_MODE)
                        handleFont(cur, cmd.argument(), "textbf");
                else
@@ -917,11 +937,6 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                else
                        handleFont(cur, cmd.argument(), "mathbb");
                break;
-       /*
-       case LFUN_FONT_FREE_APPLY:
-               handleFont(cur, cmd.argument(), "textrm");
-               break;
-       */
        case LFUN_FONT_DEFAULT:
                handleFont(cur, cmd.argument(), "textnormal");
                break;
@@ -931,6 +946,7 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                // ignore math-mode on when already in math mode
                if (currentMode() == Inset::MATH_MODE && cmd.argument() == "on")
                        break;
+               cur.recordUndoSelection();
                cur.macroModeClose();
                docstring const save_selection = grabAndEraseSelection(cur);
                selClearOrDel(cur);
@@ -941,6 +957,7 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                cur.niceInsert(save_selection);
 #else
                if (currentMode() == Inset::TEXT_MODE) {
+                       cur.recordUndoSelection();
                        cur.niceInsert(MathAtom(new InsetMathHull("simple")));
                        cur.message(_("create new math text environment ($...$)"));
                } else {
@@ -951,12 +968,28 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                break;
        }
 
-       case LFUN_MATH_SIZE:
-#if 0
-               cur.recordUndoSelection();
-               cur.setSize(arg);
-#endif
+       case LFUN_REGEXP_MODE: {
+               InsetMathHull * i = dynamic_cast<InsetMathHull *>(cur.inset().asInsetMath());
+               if (i && i->getType() == hullRegexp) {
+                       cur.message(_("Already in regexp mode"));
+                       break;
+               }
+               cur.macroModeClose();
+               docstring const save_selection = grabAndEraseSelection(cur);
+               selClearOrDel(cur);
+               cur.plainInsert(MathAtom(new InsetMathHull(hullRegexp)));
+               cur.posBackward();
+               cur.pushBackward(*cur.nextInset());
+               cur.niceInsert(save_selection);
+               cur.message(_("Regexp editor mode"));
                break;
+       }
+
+       case LFUN_MATH_SIZE: {
+               FuncRequest fr = FuncRequest(LFUN_MATH_INSERT, cmd.argument());
+               doDispatch(cur, fr);
+               break;
+       }
 
        case LFUN_MATH_MATRIX: {
                cur.recordUndo();
@@ -1020,9 +1053,19 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
        }
 
        case LFUN_SPACE_INSERT:
+               cur.recordUndoSelection();
+               cur.insert(MathAtom(new InsetMathSpace));
+               break;
+
        case LFUN_MATH_SPACE:
                cur.recordUndoSelection();
-               cur.insert(MathAtom(new InsetMathSpace(from_ascii(","))));
+               if (cmd.argument().empty())
+                       cur.insert(MathAtom(new InsetMathSpace));
+               else {
+                       string const name = cmd.getArg(0);
+                       string const len = cmd.getArg(1);
+                       cur.insert(MathAtom(new InsetMathSpace(name, len)));
+               }
                break;
 
        case LFUN_ERT_INSERT:
@@ -1042,7 +1085,7 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                cur.recordUndoSelection();
                interpretChar(cur, '^');
                break;
-               
+
        case LFUN_MATH_MACRO_FOLD:
        case LFUN_MATH_MACRO_UNFOLD: {
                Cursor it = cur;
@@ -1090,6 +1133,9 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
                if (name == "ref") {
                        InsetMathRef tmp(name);
                        data = tmp.createDialogStr(to_utf8(name));
+               } else if (name == "mathspace") {
+                       InsetMathSpace tmp;
+                       data = tmp.createDialogStr();
                }
                cur.bv().showDialog(to_utf8(name), data);
                break;
@@ -1121,7 +1167,7 @@ void InsetMathNest::doDispatch(Cursor & cur, FuncRequest & cmd)
 bool InsetMathNest::findMacroToFoldUnfold(Cursor & it, bool fold) const {
        // look for macro to open/close, but stay in mathed
        for (; !it.empty(); it.pop_back()) {
-                       
+
                // go backward through the current cell
                Inset * inset = it.nextInset();
                while (inset && inset->asInsetMath()) {
@@ -1130,24 +1176,24 @@ bool InsetMathNest::findMacroToFoldUnfold(Cursor & it, bool fold) const {
                                // found the an macro to open/close?
                                if (macro->folded() != fold)
                                        return true;
-                               
+
                                // Wrong folding state.
                                // If this was the first we see in this slice, look further left,
                                // otherwise go up.
                                if (inset != it.nextInset())
                                        break;
                        }
-                       
+
                        // go up if this was the left most position
                        if (it.pos() == 0)
                                break;
-                       
+
                        // go left
                        it.pos()--;
                        inset = it.nextInset();
                }
        }
-       
+
        return false;
 }
 
@@ -1186,6 +1232,7 @@ bool InsetMathNest::getStatus(Cursor & cur, FuncRequest const & cmd,
        /// We have to handle them since 1.4 blocks all unhandled actions
        case LFUN_FONT_ITAL:
        case LFUN_FONT_BOLD:
+       case LFUN_FONT_BOLDSYMBOL:
        case LFUN_FONT_SANS:
        case LFUN_FONT_EMPH:
        case LFUN_FONT_TYPEWRITER:
@@ -1232,7 +1279,7 @@ bool InsetMathNest::getStatus(Cursor & cur, FuncRequest const & cmd,
                // getStatus is not called with a valid reference and the
                // dialog would not be applyable.
                string const name = cmd.getArg(0);
-               flag.setEnabled(name == "ref");
+               flag.setEnabled(name == "ref" || name == "mathspace");
                break;
        }
 
@@ -1241,7 +1288,7 @@ bool InsetMathNest::getStatus(Cursor & cur, FuncRequest const & cmd,
                // Don't do this with multi-cell selections
                flag.setEnabled(cur.selBegin().idx() == cur.selEnd().idx());
                break;
-               
+
        case LFUN_MATH_MACRO_FOLD:
        case LFUN_MATH_MACRO_UNFOLD: {
                Cursor it = cur;
@@ -1249,7 +1296,7 @@ bool InsetMathNest::getStatus(Cursor & cur, FuncRequest const & cmd,
                flag.setEnabled(found);
                break;
        }
-               
+
        case LFUN_SPECIALCHAR_INSERT:
                // FIXME: These would probably make sense in math-text mode
                flag.setEnabled(false);
@@ -1270,7 +1317,7 @@ bool InsetMathNest::getStatus(Cursor & cur, FuncRequest const & cmd,
 void InsetMathNest::edit(Cursor & cur, bool front, EntryDirection entry_from)
 {
        cur.push(*this);
-       bool enter_front = (entry_from == Inset::ENTRY_DIRECTION_RIGHT || 
+       bool enter_front = (entry_from == Inset::ENTRY_DIRECTION_RIGHT ||
                (entry_from == Inset::ENTRY_DIRECTION_IGNORE && front));
        cur.idx() = enter_front ? 0 : cur.lastidx();
        cur.pos() = enter_front ? 0 : cur.lastpos();
@@ -1323,7 +1370,7 @@ void InsetMathNest::lfunMousePress(Cursor & cur, FuncRequest & cmd)
                // Update::FitCursor: adjust the screen to the cursor
                //                    position if needed
                // cur.result().update(): don't overwrite previously set flags.
-               cur.updateFlags(Update::Decoration | Update::FitCursor 
+               cur.updateFlags(Update::Decoration | Update::FitCursor
                                | cur.result().update());
        } else if (cmd.button() == mouse_button::button2) {
                if (cap::selection()) {
@@ -1348,7 +1395,7 @@ void InsetMathNest::lfunMouseMotion(Cursor & cur, FuncRequest & cmd)
                if (bvcur.anchor_.hasPart(cur)) {
                        //lyxerr << "## lfunMouseMotion: cursor: " << cur << endl;
                        bvcur.setCursor(cur);
-                       bvcur.selection() = true;
+                       bvcur.setSelection(true);
                        //lyxerr << "MOTION " << bvcur << endl;
                } else
                        cur.undispatched();
@@ -1365,7 +1412,7 @@ void InsetMathNest::lfunMouseRelease(Cursor & cur, FuncRequest & cmd)
                        cur.noUpdate();
                else {
                        Cursor & bvcur = cur.bv().cursor();
-                       bvcur.selection() = true;
+                       bvcur.setSelection(true);
                }
                return;
        }
@@ -1374,7 +1421,7 @@ void InsetMathNest::lfunMouseRelease(Cursor & cur, FuncRequest & cmd)
 }
 
 
-bool InsetMathNest::interpretChar(Cursor & cur, char_type c)
+bool InsetMathNest::interpretChar(Cursor & cur, char_type const c)
 {
        //lyxerr << "interpret 2: '" << c << "'" << endl;
        docstring save_selection;
@@ -1400,7 +1447,7 @@ bool InsetMathNest::interpretChar(Cursor & cur, char_type c)
                MathWordList const & mwl = mathedWordList();
                bool star_macro = c == '*'
                        && (mwl.find(name.substr(1) + "*") != mwl.end()
-                           || cur.buffer().getMacro(name.substr(1) + "*", cur, true));
+                           || cur.buffer()->getMacro(name.substr(1) + "*", cur, true));
                if (isAlphaASCII(c) || star_macro) {
                        cur.activeMacro()->setName(name + docstring(1, c));
                        return true;
@@ -1415,6 +1462,9 @@ bool InsetMathNest::interpretChar(Cursor & cur, char_type c)
                                        cur.niceInsert(createInsetMath("textbackslash"));
                                else
                                        cur.niceInsert(createInsetMath("backslash"));
+                       } else if (c == '^' && currentMode() == InsetMath::MATH_MODE) {
+                               cur.backspace();
+                               cur.niceInsert(createInsetMath("mathcircumflex"));
                        } else if (c == '{') {
                                cur.backspace();
                                cur.niceInsert(MathAtom(new InsetMathBrace));
@@ -1473,17 +1523,17 @@ bool InsetMathNest::interpretChar(Cursor & cur, char_type c)
        // This is annoying as one has to press <space> far too often.
        // Disable it.
 
-#if 0
+#ifdef AUTOCORRECT
                // leave autocorrect mode if necessary
-               if (autocorrect() && c == ' ') {
-                       autocorrect() = false;
+               if (cur.autocorrect() && c == ' ') {
+                       cur.autocorrect() = false;
                        return true;
                }
 #endif
 
        // just clear selection on pressing the space bar
        if (cur.selection() && c == ' ') {
-               cur.selection() = false;
+               cur.setSelection(false);
                return true;
        }
 
@@ -1544,7 +1594,7 @@ bool InsetMathNest::interpretChar(Cursor & cur, char_type c)
                return cur.pos() != cur.lastpos();
        }
 
-       // These shouldn't work in text mode:
+       // These should be treated differently when not in text mode:
        if (currentMode() != InsetMath::TEXT_MODE) {
                if (c == '_') {
                        script(cur, false, save_selection);
@@ -1558,18 +1608,29 @@ bool InsetMathNest::interpretChar(Cursor & cur, char_type c)
                        cur.niceInsert(createInsetMath("sim"));
                        return true;
                }
+       } else {
+               if (c == '^') {
+                       cur.niceInsert(createInsetMath("textasciicircum"));
+                       return true;
+               }
+               if (c == '~') {
+                       cur.niceInsert(createInsetMath("textasciitilde"));
+                       return true;
+               }
        }
 
        if (c == '{' || c == '}' || c == '&' || c == '$' || c == '#' ||
-           c == '%' || c == '_' || c == '^') {
+           c == '%' || c == '_') {
                cur.niceInsert(createInsetMath(docstring(1, c)));
                return true;
        }
 
 
+#ifdef AUTOCORRECT
        // try auto-correction
-       //if (autocorrect() && hasPrevAtom() && math_autocorrect(prevAtom(), c))
-       //      return true;
+       if (cur.autocorrect() && cur.pos() != 0 && math_autocorrect(cur.prevAtom(), c))
+               return true;
+#endif
 
        // no special circumstances, so insert the character without any fuss
        cur.insert(c);
@@ -1690,7 +1751,7 @@ InsetMathNest::createCompletionList(Cursor const & cur) const
 {
        if (!cur.inMacroMode())
                return 0;
-       
+
        return new MathCompletionList(cur);
 }
 
@@ -1699,7 +1760,7 @@ docstring InsetMathNest::completionPrefix(Cursor const & cur) const
 {
        if (!cur.inMacroMode())
                return docstring();
-       
+
        return cur.activeMacro()->name();
 }
 
@@ -1730,7 +1791,7 @@ bool InsetMathNest::insertCompletion(Cursor & cur, docstring const & s,
 }
 
 
-void InsetMathNest::completionPosAndDim(Cursor const & cur, int & x, int & y, 
+void InsetMathNest::completionPosAndDim(Cursor const & cur, int & x, int & y,
                                        Dimension & dim) const
 {
        Inset const * inset = cur.activeMacro();
@@ -1758,7 +1819,7 @@ bool InsetMathNest::cursorMathForward(Cursor & cur)
                cur.pushBackward(*cur.nextAtom().nucleus());
                cur.inset().idxFirst(cur);
                return true;
-       } 
+       }
        if (cur.posForward() || idxForward(cur))
                return true;
        // try to pop forwards --- but don't pop out of math! leave that to
@@ -1777,10 +1838,10 @@ bool InsetMathNest::cursorMathBackward(Cursor & cur)
                cur.push(*cur.nextAtom().nucleus());
                cur.inset().idxLast(cur);
                return true;
-       } 
+       }
        if (cur.posBackward() || idxBackward(cur))
                return true;
-       // try to pop backwards --- but don't pop out of math! leave that to 
+       // try to pop backwards --- but don't pop out of math! leave that to
        // the FINISH lfuns
        int s = cur.depth() - 2;
        if (s >= 0 && cur[s].inset().asInsetMath())
@@ -1795,10 +1856,10 @@ MathCompletionList::MathCompletionList(Cursor const & cur)
 {
        // fill it with macros from the buffer
        MacroNameSet macros;
-       cur.buffer().listMacroNames(macros);
+       cur.buffer()->listMacroNames(macros);
        MacroNameSet::const_iterator it;
        for (it = macros.begin(); it != macros.end(); ++it) {
-               if (cur.buffer().getMacro(*it, cur, false))
+               if (cur.buffer()->getMacro(*it, cur, false))
                        locals.push_back("\\" + *it);
        }
        sort(locals.begin(), locals.end());
@@ -1809,12 +1870,12 @@ MathCompletionList::MathCompletionList(Cursor const & cur)
        // fill in global macros
        macros.clear();
        MacroTable::globalMacros().getMacroNames(macros);
-       lyxerr << "Globals completion macros: ";
+       //lyxerr << "Globals completion macros: ";
        for (it = macros.begin(); it != macros.end(); ++it) {
-               lyxerr << "\\" + *it << " ";
+               //lyxerr << "\\" + *it << " ";
                globals.push_back("\\" + *it);
        }
-       lyxerr << std::endl;
+       //lyxerr << std::endl;
 
        // fill in global commands
        globals.push_back(from_ascii("\\boxed"));
@@ -1830,6 +1891,7 @@ MathCompletionList::MathCompletionList(Cursor const & cur)
        globals.push_back(from_ascii("\\alignedat"));
        globals.push_back(from_ascii("\\cases"));
        globals.push_back(from_ascii("\\substack"));
+       globals.push_back(from_ascii("\\xymatrix"));
        globals.push_back(from_ascii("\\subarray"));
        globals.push_back(from_ascii("\\array"));
        globals.push_back(from_ascii("\\sqrt"));
@@ -1864,12 +1926,12 @@ MathCompletionList::MathCompletionList(Cursor const & cur)
        globals.push_back(from_ascii("\\vphantom"));
        MathWordList const & words = mathedWordList();
        MathWordList::const_iterator it2;
-       lyxerr << "Globals completion commands: ";
+       //lyxerr << "Globals completion commands: ";
        for (it2 = words.begin(); it2 != words.end(); ++it2) {
                globals.push_back("\\" + (*it2).first);
-               lyxerr << "\\" + (*it2).first << " ";
+               //lyxerr << "\\" + (*it2).first << " ";
        }
-       lyxerr << std::endl;
+       //lyxerr << std::endl;
        sort(globals.begin(), globals.end());
 }
 
@@ -1895,7 +1957,7 @@ docstring const & MathCompletionList::data(size_t idx) const
 }
 
 
-std::string MathCompletionList::icon(size_t idx) const 
+std::string MathCompletionList::icon(size_t idx) const
 {
        // get the latex command
        docstring cmd;
@@ -1904,7 +1966,7 @@ std::string MathCompletionList::icon(size_t idx) const
                cmd = globals[idx - lsize];
        else
                cmd = locals[idx];
-       
+
        // get the icon resource name by stripping the backslash
        return "images/math/" + to_utf8(cmd.substr(1)) + ".png";
 }