]> git.lyx.org Git - lyx.git/blobdiff - src/mathed/math_cursor.C
fix for the first item on Martin's list
[lyx.git] / src / mathed / math_cursor.C
index 3a2f30501302b9117cf6159ce1e85cbae3269e38..e62ac1920d6f343362eaed166eae004ff5ad6eff 100644 (file)
@@ -37,6 +37,8 @@
 #include "math_factory.h"
 #include "math_hullinset.h"
 #include "math_iterator.h"
+#include "math_macroarg.h"
+#include "math_macrotemplate.h"
 #include "math_mathmlstream.h"
 #include "math_parser.h"
 #include "math_replace.h"
@@ -55,6 +57,8 @@ using std::min;
 using std::max;
 using std::swap;
 using std::isalnum;
+using std::vector;
+using std::ostringstream;
 
 namespace {
 
@@ -75,18 +79,18 @@ struct Selection
                c1 = p->col(i1.idx_);
                c2 = p->col(i2.idx_);
                if (c1 > c2)
-                       std::swap(c1, c2);
+                       swap(c1, c2);
                r1 = p->row(i1.idx_);
                r2 = p->row(i2.idx_);
                if (r1 > r2)
-                       std::swap(r1, r2);
+                       swap(r1, r2);
        }
 
        void grab(MathCursor const & cursor)
        {
                MathCursorPos i1;
                MathCursorPos i2;
-               cursor.getSelection(i1, i2); 
+               cursor.getSelection(i1, i2);
                // shouldn'tt we assert on i1.par_ == i2.par_?
                if (i1.idx_ == i2.idx_) {
                        data_ = MathGridInset(1, 1);
@@ -94,9 +98,9 @@ struct Selection
                } else {
                        row_type r1, r2;
                        col_type c1, c2;
-                       region(i1, i2, r1, r2, c1, c2); 
+                       region(i1, i2, r1, r2, c1, c2);
                        data_ = MathGridInset(c2 - c1 + 1, r2 - r1 + 1);
-                       for (row_type row = 0; row < data_.nrows(); ++row) 
+                       for (row_type row = 0; row < data_.nrows(); ++row)
                                for (col_type col = 0; col < data_.ncols(); ++col) {
                                        idx_type i = i1.par_->index(row + r1, col + c1);
                                        data_.cell(data_.index(row, col)) = i1.par_->cell(i);
@@ -108,15 +112,15 @@ struct Selection
        {
                MathCursorPos i1;
                MathCursorPos i2;
-               cursor.getSelection(i1, i2); 
+               cursor.getSelection(i1, i2);
                if (i1.idx_ == i2.idx_)
                        i1.cell().erase(i1.pos_, i2.pos_);
                else {
                        MathInset * p = i1.par_;
                        row_type r1, r2;
                        col_type c1, c2;
-                       region(i1, i2, r1, r2, c1, c2); 
-                       for (row_type row = r1; row <= r2; ++row) 
+                       region(i1, i2, r1, r2, c1, c2);
+                       for (row_type row = r1; row <= r2; ++row)
                                for (col_type col = c1; col <= c2; ++col)
                                        p->cell(p->index(row, col)).erase();
                }
@@ -134,7 +138,7 @@ struct Selection
                        MathGridInset * p = cursor.enclosingGrid(idx);
                        col_type const numcols = min(data_.ncols(), p->ncols() - p->col(idx));
                        row_type const numrows = min(data_.nrows(), p->nrows() - p->row(idx));
-                       for (row_type row = 0; row < numrows; ++row) 
+                       for (row_type row = 0; row < numrows; ++row)
                                for (col_type col = 0; col < numcols; ++col) {
                                        idx_type i = p->index(row + p->row(idx), col + p->col(idx));
                                        p->cell(i).push_back(data_.cell(data_.index(row, col)));
@@ -199,11 +203,9 @@ void MathCursor::pushRight(MathAtom & t)
 
 bool MathCursor::popLeft()
 {
-       //cerr << "Leaving atom "; par()->write(cerr, false); cerr << " left\n";
+       //cerr << "Leaving atom to the left\n";
        if (Cursor_.size() <= 1)
                return false;
-       if (par()->asScriptInset())
-               par()->asScriptInset()->removeEmptyScripts();
        Cursor_.pop_back();
        return true;
 }
@@ -214,8 +216,6 @@ bool MathCursor::popRight()
        //cerr << "Leaving atom "; par()->write(cerr, false); cerr << " right\n";
        if (Cursor_.size() <= 1)
                return false;
-       if (par()->asScriptInset())
-               par()->asScriptInset()->removeEmptyScripts();
        Cursor_.pop_back();
        posRight();
        return true;
@@ -248,8 +248,8 @@ UpdatableInset * MathCursor::asHyperActiveInset() const
 
 bool MathCursor::isInside(MathInset const * p) const
 {
-       for (unsigned i = 0; i < Cursor_.size(); ++i) 
-               if (Cursor_[i].par_ == p) 
+       for (unsigned i = 0; i < Cursor_.size(); ++i)
+               if (Cursor_[i].par_ == p)
                        return true;
        return false;
 }
@@ -312,7 +312,7 @@ bool MathCursor::left(bool sel)
                }
                pushRight(prevAtom());
                return true;
-       } 
+       }
 
        return posLeft() || idxLeft() || popLeft() || selection_;
 }
@@ -397,7 +397,7 @@ void MathCursor::home(bool sel)
        selHandle(sel);
        macroModeClose();
        lastcode_ = LM_TC_MIN;
-       if (!par()->idxHome(idx(), pos())) 
+       if (!par()->idxHome(idx(), pos()))
                popLeft();
        dump("home 2");
 }
@@ -421,6 +421,20 @@ void MathCursor::plainErase()
 }
 
 
+void MathCursor::markInsert()
+{
+       //lyxerr << "inserting mark\n";
+       array().insert(pos(), MathAtom(new MathCharInset(0, lastcode_)));
+}
+
+
+void MathCursor::markErase()
+{
+       //lyxerr << "deleting mark\n";
+       array().erase(pos());
+}
+
+
 void MathCursor::plainInsert(MathAtom const & t)
 {
        array().insert(pos(), t);
@@ -435,6 +449,12 @@ void MathCursor::insert(char c, MathTextCodes t)
 }
 
 
+void MathCursor::insert(char c)
+{
+       insert(c, lastcode_);
+}
+
+
 void MathCursor::insert(MathAtom const & t)
 {
        macroModeClose();
@@ -450,7 +470,7 @@ void MathCursor::insert(MathAtom const & t)
 }
 
 
-void MathCursor::niceInsert(MathAtom const & t) 
+void MathCursor::niceInsert(MathAtom const & t)
 {
        selCut();
        insert(t); // inserting invalidates the pointer!
@@ -488,7 +508,7 @@ void MathCursor::backspace()
        if (pos() == 0) {
                pullArg(false);
                return;
-       }       
+       }
 
        if (selection_) {
                selDel();
@@ -498,7 +518,7 @@ void MathCursor::backspace()
        MathScriptInset * p = prevAtom()->asScriptInset();
        if (p) {
                p->removeScript(p->hasUp());
-               // Don't delete if there is anything left 
+               // Don't delete if there is anything left
                if (p->hasUp() || p->hasDown())
                        return;
        }
@@ -534,7 +554,7 @@ void MathCursor::erase()
        MathScriptInset * p = nextAtom()->asScriptInset();
        if (p) {
                p->removeScript(p->hasUp());
-               // Don't delete if there is anything left 
+               // Don't delete if there is anything left
                if (p->hasUp() || p->hasDown())
                        return;
        }
@@ -558,8 +578,8 @@ void MathCursor::delLine()
                par()->asGridInset()->delRow(hullRow());
        }
 
-       if (idx() > par()->nargs())
-               idx() = par()->nargs();
+       if (idx() >= par()->nargs())
+               idx() = par()->nargs() - 1;
 
        if (pos() > size())
                pos() = size();
@@ -617,9 +637,9 @@ void MathCursor::macroModeClose()
 }
 
 
-int MathCursor::macroNamePos() const
+MathInset::difference_type MathCursor::macroNamePos() const
 {
-       for (MathInset::difference_type i = pos() - 1; i >= 0; --i) { 
+       for (MathInset::difference_type i = pos() - 1; i >= 0; --i) {
                MathAtom & p = array().at(i);
                if (p->code() == LM_TC_TEX && p->getChar() == '\\')
                        return i;
@@ -631,8 +651,8 @@ int MathCursor::macroNamePos() const
 string MathCursor::macroName() const
 {
        string s;
-       MathInset::difference_type i = macroNamePos(); 
-       for ( ; i >= 0 && i < int(pos()); ++i) 
+       MathInset::difference_type i = macroNamePos();
+       for (; i >= 0 && i < int(pos()); ++i)
                s += array().at(i)->getChar();
        return s;
 }
@@ -739,7 +759,7 @@ void MathCursor::drawSelection(Painter & pain) const
                int y2 = c.yo() + c.descent();
                pain.fillRectangle(x1, y1, x2 - x1, y2 - y1, LColor::selection);
        } else {
-               std::vector<MathInset::idx_type> indices
+               vector<MathInset::idx_type> indices
                        = i1.par_->idxBetween(i1.idx_, i2.idx_);
                for (unsigned i = 0; i < indices.size(); ++i) {
                        MathXArray & c = i1.xcell(indices[i]);
@@ -771,13 +791,13 @@ void MathCursor::handleFont(MathTextCodes t)
        if (selection_) {
                MathCursorPos i1;
                MathCursorPos i2;
-               getSelection(i1, i2); 
+               getSelection(i1, i2);
                if (i1.idx_ == i2.idx_) {
                        MathArray & ar = i1.cell();
                        for (MathInset::pos_type pos = i1.pos_; pos != i2.pos_; ++pos)
                                ar.at(pos)->handleFont(t);
                }
-       } else 
+       } else
                lastcode_ = (lastcode_ == t) ? LM_TC_VAR : t;
 }
 
@@ -851,6 +871,12 @@ bool MathCursor::inMacroMode() const
 }
 
 
+bool MathCursor::inMacroArgMode() const
+{
+       return pos() > 0 && prevAtom()->getChar() == '#';
+}
+
+
 bool MathCursor::selection() const
 {
        return selection_;
@@ -859,7 +885,7 @@ bool MathCursor::selection() const
 
 MathGridInset * MathCursor::enclosingGrid(MathCursor::idx_type & idx) const
 {
-       for (int i = Cursor_.size() - 1; i >= 0; --i) {
+       for (MathInset::difference_type i = Cursor_.size() - 1; i >= 0; --i) {
                MathGridInset * p = Cursor_[i].par_->asGridInset();
                if (p) {
                        idx = Cursor_[i].idx_;
@@ -891,22 +917,35 @@ void MathCursor::pullArg(bool goright)
        if (popLeft()) {
                plainErase();
                array().insert(pos(), a);
-               if (goright) 
+               if (goright)
                        pos() += a.size();
+       } else {
+               formula()->mutateToText();
        }
 }
 
 
+void MathCursor::touch()
+{
+       cursor_type::const_iterator it = Cursor_.begin();
+       cursor_type::const_iterator et = Cursor_.end();
+       for ( ; it != et; ++it)
+               it->xcell().touch();
+}
+
+
 void MathCursor::normalize()
 {
+#if 0
        // rebreak
        {
                MathIterator it = ibegin(formula()->par().nucleus());
                MathIterator et = iend(formula()->par().nucleus());
-               for ( ; it != et; ++it) 
+               for (; it != et; ++it)
                        if (it.par()->asBoxInset())
                                it.par()->asBoxInset()->rebreak();
        }
+#endif
 
        if (idx() >= par()->nargs()) {
                lyxerr << "this should not really happen - 1: "
@@ -925,6 +964,19 @@ void MathCursor::normalize()
                dump("error 4");
        }
        pos() = min(pos(), size());
+
+       // remove empty scripts if possible
+       for (pos_type i = 0; i < size(); ++i) {
+               MathScriptInset * p = array().at(i)->asScriptInset();
+               if (p) {
+                       p->removeEmptyScripts();
+                       if (p->empty())
+                               array().erase(i);
+               }
+       }
+
+       // fix again position
+       pos() = min(pos(), size());
 }
 
 
@@ -1031,7 +1083,7 @@ void MathCursor::idxPrev()
 
 void MathCursor::splitCell()
 {
-       if (idx() + 1 == par()->nargs()) 
+       if (idx() + 1 == par()->nargs())
                return;
        MathArray ar = array();
        ar.erase(0, pos());
@@ -1173,22 +1225,28 @@ bool MathCursor::bruteFind(int x, int y, int xlow, int xhigh, int ylow, int yhig
 
        MathIterator it = ibegin(formula()->par().nucleus());
        MathIterator et = iend(formula()->par().nucleus());
-       for ( ; it != et; ++it) {
-               // avoid invalid nesting hen selecting
-               if (selection_ && !positionable(it.cursor(), Anchor_))
-                       continue;
-               int xo = it.position().xpos();
-               int yo = it.position().ypos();
-               if (xlow - 2 <= xo && xo <= xhigh + 2 &&
-                   ylow - 2 <= yo && yo <= yhigh + 2)
-               {
-                       double d = (x - xo) * (x - xo) + (y - yo) * (y - yo);
-                       if (d < best_dist) {
-                               best_dist   = d;
-                               best_cursor = it.cursor();
+       while (1) {
+               // avoid invalid nesting when selecting
+               if (!selection_ || positionable(it.cursor(), Anchor_)) {
+                       MathCursorPos const & top = it.position();
+                       int xo = top.xpos();
+                       int yo = top.ypos();
+                       if (xlow - 2 <= xo && xo <= xhigh + 2 &&
+                                       ylow - 2 <= yo && yo <= yhigh + 2)
+                       {
+                               double d = (x - xo) * (x - xo) + (y - yo) * (y - yo);
+                               if (d < best_dist) {
+                                       best_dist   = d;
+                                       best_cursor = it.cursor();
+                               }
                        }
                }
+
+               if (it == et)
+                       break;
+               ++it;
        }
+
        if (best_dist < 1e10)
                Cursor_ = best_cursor;
        return best_dist < 1e10;
@@ -1217,14 +1275,14 @@ bool MathCursor::interpret(string const & s)
                return interpret(s[0]);
 
        //lyxerr << "char: '" << s[0] << "'  int: " << int(s[0]) << endl;
-       //owner_->getIntl()->getTrans().TranslateAndInsert(s[0], lt);   
+       //owner_->getIntl()->getTrans().TranslateAndInsert(s[0], lt);
        //lyxerr << "trans: '" << s[0] << "'  int: " << int(s[0]) << endl;
 
        if (s.size() >= 5 && s.substr(0, 5) == "cases") {
                unsigned int n = 1;
                istringstream is(s.substr(5).c_str());
                is >> n;
-               n = std::max(1u, n);
+               n = max(1u, n);
                niceInsert(MathAtom(new MathCasesInset(n)));
                return true;
        }
@@ -1236,10 +1294,10 @@ bool MathCursor::interpret(string const & s)
                string h_align;
                istringstream is(s.substr(6).c_str());
                is >> m >> n >> v_align >> h_align;
-               m = std::max(1u, m);
-               n = std::max(1u, n);
+               m = max(1u, m);
+               n = max(1u, n);
                v_align += 'c';
-               niceInsert(MathAtom(new MathArrayInset(m, n, v_align[0], h_align)));
+               niceInsert(MathAtom(new MathArrayInset("array", m, n, v_align[0], h_align)));
                return true;
        }
 
@@ -1272,6 +1330,14 @@ bool MathCursor::interpret(string const & s)
                return true;
        }
 
+       // prevent entering of recursive macros
+       if (formula()->lyxCode() == Inset::MATHMACRO_CODE 
+               && formula()->getInsetName() == s.substr(1))
+       {
+               lyxerr << "can't enter recursive macro\n";
+               return true;
+       }
+
        niceInsert(createMathInset(s.substr(1)));
        return true;
 }
@@ -1306,30 +1372,30 @@ bool MathCursor::script(bool up)
 
 bool MathCursor::interpret(char c)
 {
-
-       // Removed super/subscript handling from here  to ::script -MV
+       if (inMacroArgMode()) {
+               --pos();
+               plainErase();
+               int n = c - '0';
+               MathMacroTemplate * p = formula()->par()->asMacroTemplate();
+               if (p && 1 <= n && n <= p->numargs())
+                       insert(MathAtom(new MathMacroArgument(c - '0')));
+               else {
+                       insert(MathAtom(new MathSpecialCharInset('#')));
+                       interpret(c); // try again
+               }
+               return true;
+       }
 
        // handle macroMode
        if (inMacroMode()) {
                string name = macroName();
 
-               if (name == "\\" && c == '#') {
-                       insert(c, LM_TC_TEX);
-                       return true;
-               }
-
                if (name == "\\" && c == '\\') {
                        backspace();
                        interpret("\\backslash");
                        return true;
                }
 
-               if (name == "\\#" && '1' <= c && c <= '9') {
-                       insert(c, LM_TC_TEX);
-                       macroModeClose();
-                       return true;
-               }
-
                if (isalpha(c)) {
                        insert(c, LM_TC_TEX);
                        return true;
@@ -1342,6 +1408,12 @@ bool MathCursor::interpret(char c)
                }
 
                macroModeClose();
+
+               if (c == '\\')
+                       insert(c, LM_TC_TEX);
+               else if (c != ' ')
+                       insert(c, lastcode_);
+
                return true;
        }
 
@@ -1373,6 +1445,11 @@ bool MathCursor::interpret(char c)
                return pos() != size();
        }
 
+       if (c == '#') {
+               insert(c, LM_TC_TEX);
+               return true;
+       }
+
 /*
        if (strchr("{}", c)) {
                insert(c, LM_TC_TEX);
@@ -1389,8 +1466,8 @@ bool MathCursor::interpret(char c)
                return true;
        }
 
-       if (strchr("#$%", c)) {
-               insert(MathAtom(new MathSpecialCharInset(c)));  
+       if (strchr("$%", c)) {
+               insert(MathAtom(new MathSpecialCharInset(c)));
                lastcode_ = LM_TC_VAR;
                return true;
        }
@@ -1409,7 +1486,7 @@ bool MathCursor::interpret(char c)
        if (c == '\\') {
                insert(c, LM_TC_TEX);
                //bv->owner()->message(_("TeX mode"));
-               return true;    
+               return true;
        }
 
        // no special circumstances, so insert the character without any fuss
@@ -1449,7 +1526,7 @@ void MathCursor::stripFromLastEqualSign()
 
        // delete everything behind this position
        ar.erase(et - ar.begin(), ar.size());
-       pos() = ar.size(); 
+       pos() = ar.size();
 }
 
 
@@ -1460,3 +1537,12 @@ void MathCursor::setSelection(cursor_type const & where, size_type n)
        Cursor_ = where;
        cursor().pos_ += n;
 }
+
+
+string MathCursor::info() const
+{
+       ostringstream os;
+       if (pos() > 0)
+               prevAtom()->infoize(os);
+       return os.str();
+}