prevent crash when inserting minipage in table cell,

[lyx.git] / src / insets / insettext.C

diff --git a/src/insets/insettext.C b/src/insets/insettext.C
index 83adc9605b3372116231834072956f6ab4da2f3a..78bb89071e290ff205c68048b367bec53a8d7550 100644 (file)
--- a/src/insets/insettext.C
+++ b/src/insets/insettext.C
@@ -252,8 +252,8 @@ void InsetText::read(Buffer const * buf, LyXLex & lex)
                        break;
                }
 
-               if (token == "\\the_end") {
-                       lex.printError("\\the_end read in inset! Error in document!");
+               if (token == "\\end_document") {
+                       lex.printError("\\end_document read in inset! Error in document!");
                        return;
                }
 
@@ -321,6 +321,8 @@ void InsetText::draw(PainterInfo & pi, int x, int baseline) const
                return;
        }
 
+       bv->hideCursor();
+
        if (!owner())
                x += scroll();
 
@@ -333,23 +335,23 @@ void InsetText::draw(PainterInfo & pi, int x, int baseline) const
        }
 
        if (the_locking_inset && cpar() == inset_par && cpos() == inset_pos) {
-               inset_x = cix(bv) - x + drawTextXOffset;
+               inset_x = cix() - x + drawTextXOffset;
                inset_y = ciy() + drawTextYOffset;
        }
 
        x += TEXT_TO_INSET_OFFSET;
 
-       RowList::iterator rowit = text_.rows().begin();
+       RowList::iterator rit = text_.rows().begin();
        RowList::iterator end = text_.rows().end();
 
-       int y_offset = baseline - rowit->ascent_of_text();
-       int ph = pain.paperHeight();
+       int y_offset = baseline - rit->ascent_of_text();
+       int y2 = pain.paperHeight();
        int first = 0;
        int y = y_offset;
-       while (rowit != end && y + rowit->height() <= 0) {
-               y += rowit->height();
-               first += rowit->height();
-               ++rowit;
+       while (rit != end && y + rit->height() <= 0) {
+               y += rit->height();
+               first += rit->height();
+               ++rit;
        }
        if (y_offset < 0) {
                text_.top_y(-y_offset);
@@ -360,28 +362,17 @@ void InsetText::draw(PainterInfo & pi, int x, int baseline) const
                first = 0;
        }
 
-       int yf = y_offset + first;
-       y = 0;
+       int yo = y_offset + first;
 
-       bv->hideCursor();
-
-       while (rowit != end && yf < ph) {
-               paintRows(*bv, text_, rowit,
-                       y + y_offset + first, int(x), y + text_.top_y());
-               y += rowit->height();
-               yf += rowit->height();
-               ++rowit;
-       }
+       paintRows(*bv, text_, rit, x, 0, yo, y2, yo);
 
        text_.clearPaint();
 
-       if (drawFrame_ == ALWAYS || (drawFrame_ == LOCKED && locked)) {
+       if (drawFrame_ == ALWAYS || (drawFrame_ == LOCKED && locked))
                drawFrame(pain, int(start_x));
-       }
 
-       if (need_update != INIT) {
+       if (need_update != INIT)
                need_update = NONE;
-       }
 }
 
 
@@ -515,10 +506,10 @@ void InsetText::lockInset(BufferView * bv)
 }
 
 
-void InsetText::lockInset(BufferView * bv, UpdatableInset * inset)
+void InsetText::lockInset(BufferView * /*bv*/, UpdatableInset * inset)
 {
        the_locking_inset = inset;
-       inset_x = cix(bv) - top_x + drawTextXOffset;
+       inset_x = cix() - top_x + drawTextXOffset;
        inset_y = ciy() + drawTextYOffset;
        inset_pos = cpos();
        inset_par = cpar();
@@ -571,7 +562,7 @@ bool InsetText::lockInsetInInset(BufferView * bv, UpdatableInset * inset)
        if (the_locking_inset && the_locking_inset == inset) {
                if (cpar() == inset_par && cpos() == inset_pos) {
                        lyxerr[Debug::INSETS] << "OK" << endl;
-                       inset_x = cix(bv) - top_x + drawTextXOffset;
+                       inset_x = cix() - top_x + drawTextXOffset;
                        inset_y = ciy() + drawTextYOffset;
                } else {
                        lyxerr[Debug::INSETS] << "cursor.pos != inset_pos" << endl;
@@ -629,7 +620,7 @@ bool InsetText::updateInsetInInset(BufferView * bv, InsetOld * inset)
                                return false;
                        found = tl_inset->updateInsetInInset(bv, inset);
                        ustat = FULL;
-               } else { 
+               } else {
                        text_.updateInset(tl_inset);
                        setUpdateStatus(ustat);
                }
@@ -641,7 +632,7 @@ bool InsetText::updateInsetInInset(BufferView * bv, InsetOld * inset)
                if (the_locking_inset &&
                    cpar() == inset_par && cpos() == inset_pos)
                {
-                       inset_x = cix(bv) - top_x + drawTextXOffset;
+                       inset_x = cix() - top_x + drawTextXOffset;
                        inset_y = ciy() + drawTextYOffset;
                }
        }
@@ -800,11 +791,7 @@ void InsetText::lfunMouseMotion(FuncRequest const & cmd)
        if (cur == text_.cursor)
                return;
        text_.setSelection();
-       bool flag = (text_.toggle_cursor.par() != text_.toggle_end_cursor.par() ||
-                                text_.toggle_cursor.pos() != text_.toggle_end_cursor.pos());
-       if (flag) {
-               updateLocal(bv, SELECTION, false);
-       }
+       updateLocal(bv, SELECTION, false);
 }
 
 
@@ -1498,7 +1485,7 @@ void InsetText::getCursor(BufferView & bv, int & x, int & y) const
                the_locking_inset->getCursor(bv, x, y);
                return;
        }
-       x = cx(&bv);
+       x = cx();
        y = cy() + InsetText::y();
 }
 
@@ -1509,7 +1496,7 @@ void InsetText::getCursorPos(BufferView * bv, int & x, int & y) const
                the_locking_inset->getCursorPos(bv, x, y);
                return;
        }
-       x = cx(bv) - top_x - TEXT_TO_INSET_OFFSET;
+       x = cx() - top_x - TEXT_TO_INSET_OFFSET;
        y = cy() - TEXT_TO_INSET_OFFSET;
 }
 
@@ -1535,7 +1522,7 @@ void InsetText::fitInsetCursor(BufferView * bv) const
        int const asc = font_metrics::maxAscent(font);
        int const desc = font_metrics::maxDescent(font);
 
-       if (bv->fitLockedInsetCursor(cx(bv), cy(), asc, desc))
+       if (bv->fitLockedInsetCursor(cx(), cy(), asc, desc))
                need_update |= FULL;
 }
 
@@ -1760,7 +1747,7 @@ bool InsetText::checkAndActivateInset(BufferView * bv, int x, int y,
                x = dim_.wid;
        if (y < 0)
                y = dim_.des;
-       inset_x = cix(bv) - top_x + drawTextXOffset;
+       inset_x = cix() - top_x + drawTextXOffset;
        inset_y = ciy() + drawTextYOffset;
        FuncRequest cmd(bv, LFUN_INSET_EDIT, x - inset_x, y - inset_y, button);
        inset->localDispatch(cmd);
@@ -1848,7 +1835,7 @@ void InsetText::setFrameColor(BufferView * bv, LColor::color col)
 }
 
 
-int InsetText::cx(BufferView *) const
+int InsetText::cx() const
 {
        int x = text_.cursor.x() + top_x + TEXT_TO_INSET_OFFSET;
        if (the_locking_inset) {
@@ -1860,7 +1847,7 @@ int InsetText::cx(BufferView *) const
 }
 
 
-int InsetText::cix(BufferView *) const
+int InsetText::cix() const
 {
        int x = text_.cursor.ix() + top_x + TEXT_TO_INSET_OFFSET;
        if (the_locking_inset) {
@@ -1965,7 +1952,7 @@ void InsetText::resizeLyXText(BufferView * bv, bool /*force*/) const
        // seems to be unneeded
 #if 1
        if (the_locking_inset) {
-               inset_x = cix(bv) - top_x + drawTextXOffset;
+               inset_x = cix() - top_x + drawTextXOffset;
                inset_y = ciy() + drawTextYOffset;
        }
 
@@ -1997,7 +1984,7 @@ void InsetText::reinitLyXText() const
        text_.init(bv);
        restoreLyXTextState();
        if (the_locking_inset) {
-               inset_x = cix(bv) - top_x + drawTextXOffset;
+               inset_x = cix() - top_x + drawTextXOffset;
                inset_y = ciy() + drawTextYOffset;
        }
        text_.top_y(bv->screen().topCursorVisible(&text_));
@@ -2145,19 +2132,18 @@ void InsetText::selectSelectedWord(BufferView * bv)
 
 void InsetText::toggleSelection(BufferView * bv, bool kill_selection)
 {
-       if (the_locking_inset) {
+       if (the_locking_inset)
                the_locking_inset->toggleSelection(bv, kill_selection);
-       }
 
        int x = top_x + TEXT_TO_INSET_OFFSET;
 
-       RowList::iterator rowit = text_.rows().begin();
+       RowList::iterator rit = text_.rows().begin();
        RowList::iterator end = text_.rows().end();
-       int y_offset = top_baseline - rowit->ascent_of_text();
+       int y_offset = top_baseline - rit->ascent_of_text();
        int y = y_offset;
-       while ((rowit != end) && ((y + rowit->height()) <= 0)) {
-               y += rowit->height();
-               ++rowit;
+       while (rit != end && y + rit->height() <= 0) {
+               y += rit->height();
+               ++rit;
        }
        if (y_offset < 0)
                y_offset = y;
@@ -2211,9 +2197,10 @@ bool InsetText::searchForward(BufferView * bv, string const & str,
                text_.setSelectionRange(str.length());
                updateLocal(bv, SELECTION, false);
        }
-       return (result != lyx::find::SR_NOT_FOUND);
+       return result != lyx::find::SR_NOT_FOUND;
 }
 
+
 bool InsetText::searchBackward(BufferView * bv, string const & str,
                               bool cs, bool mw)
 {
@@ -2242,7 +2229,7 @@ bool InsetText::searchBackward(BufferView * bv, string const & str,
                text_.setSelectionRange(str.length());
                updateLocal(bv, SELECTION, false);
        }
-       return (result != lyx::find::SR_NOT_FOUND);
+       return result != lyx::find::SR_NOT_FOUND;
 }