]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insettext.C
prevent crash when inserting minipage in table cell,
[lyx.git] / src / insets / insettext.C
index 9e499a3745d0b767b95f112c90d74fc2bb85ccc1..78bb89071e290ff205c68048b367bec53a8d7550 100644 (file)
@@ -70,6 +70,7 @@ using std::make_pair;
 using std::vector;
 using std::pair;
 using std::for_each;
+using std::auto_ptr;
 
 using namespace lyx::support;
 using namespace lyx::graphics;
@@ -204,9 +205,9 @@ void InsetText::clear(bool just_mark_erased)
 }
 
 
-InsetBase * InsetText::clone() const
+auto_ptr<InsetBase> InsetText::clone() const
 {
-       return new InsetText(*this);
+       return auto_ptr<InsetBase>(new InsetText(*this));
 }
 
 
@@ -251,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;
                }
 
@@ -275,16 +276,15 @@ void InsetText::read(Buffer const * buf, LyXLex & lex)
 
 void InsetText::metrics(MetricsInfo & mi, Dimension & dim) const
 {
-       //lyxerr << "InsetText::metrics: " << getInsetName()
-       //      << " width: " << mi.base.textwidth << "\n";
-       if (mi.base.textwidth)
-               textwidth_ = mi.base.textwidth;
+       //lyxerr << "InsetText::metrics: width: " << mi.base.textwidth << "\n";
+
+       textwidth_ = mi.base.textwidth;
        BufferView * bv = mi.base.bv;
        setViewCache(bv);
-       text_.rebuild(mi.base.textwidth);
-       dim.asc = text_.rows().begin()->ascent_of_text() + TEXT_TO_INSET_OFFSET;
-       dim.des = text_.height - dim.asc + TEXT_TO_INSET_OFFSET;
-       dim.wid = max(textwidth_, int(text_.width)) + 2 * TEXT_TO_INSET_OFFSET;
+       text_.metrics(mi, dim);
+       dim.asc += TEXT_TO_INSET_OFFSET;
+       dim.des += TEXT_TO_INSET_OFFSET;
+       dim.wid += 2 * TEXT_TO_INSET_OFFSET;
        dim.wid = max(dim.wid, 10);
        dim_ = dim;
 }
@@ -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;
@@ -607,7 +598,7 @@ bool InsetText::unlockInsetInInset(BufferView * bv, UpdatableInset * inset,
 }
 
 
-bool InsetText::updateInsetInInset(BufferView * bv, Inset * inset)
+bool InsetText::updateInsetInInset(BufferView * bv, InsetOld * inset)
 {
        if (!autoBreakRows && paragraphs.size() > 1)
                collapseParagraphs(bv);
@@ -629,11 +620,10 @@ bool InsetText::updateInsetInInset(BufferView * bv, Inset * inset)
                                return false;
                        found = tl_inset->updateInsetInInset(bv, inset);
                        ustat = FULL;
-               }
-               if (found)
+               } else {
                        text_.updateInset(tl_inset);
-               if (found)
                        setUpdateStatus(ustat);
+               }
                return found;
        }
        bool found = text_.updateInset(inset);
@@ -642,7 +632,7 @@ bool InsetText::updateInsetInInset(BufferView * bv, Inset * 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;
                }
        }
@@ -667,7 +657,7 @@ void InsetText::lfunMousePress(FuncRequest const & cmd)
 
        int tmp_x = cmd.x - drawTextXOffset;
        int tmp_y = cmd.y + dim_.asc - getLyXText(bv)->top_y();
-       Inset * inset = getLyXText(bv)->checkInsetHit(tmp_x, tmp_y);
+       InsetOld * inset = getLyXText(bv)->checkInsetHit(tmp_x, tmp_y);
 
        if (the_locking_inset) {
                if (the_locking_inset == inset) {
@@ -751,7 +741,7 @@ bool InsetText::lfunMouseRelease(FuncRequest const & cmd)
 
        int tmp_x = cmd.x - drawTextXOffset;
        int tmp_y = cmd.y + dim_.asc - getLyXText(bv)->top_y();
-       Inset * inset = getLyXText(bv)->checkInsetHit(tmp_x, tmp_y);
+       InsetOld * inset = getLyXText(bv)->checkInsetHit(tmp_x, tmp_y);
        bool ret = false;
        if (inset) {
 // This code should probably be removed now. Simple insets
@@ -801,15 +791,11 @@ 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);
 }
 
 
-Inset::RESULT InsetText::localDispatch(FuncRequest const & cmd)
+InsetOld::RESULT InsetText::localDispatch(FuncRequest const & cmd)
 {
        BufferView * bv = cmd.view();
        setViewCache(bv);
@@ -825,7 +811,9 @@ Inset::RESULT InsetText::localDispatch(FuncRequest const & cmd)
 
                locked = true;
                the_locking_inset = 0;
-               inset_pos = inset_x = inset_y = 0;
+               inset_pos = 0;
+               inset_x = 0;
+               inset_y = 0;
                inset_boundary = false;
                inset_par = paragraphs.end();
                old_par = paragraphs.end();
@@ -835,11 +823,7 @@ Inset::RESULT InsetText::localDispatch(FuncRequest const & cmd)
                        if (cmd.argument == "left")
                                text_.setCursorIntern(paragraphs.begin(), 0);
                        else {
-                               ParagraphList::iterator it = paragraphs.begin();
-                               ParagraphList::iterator end = paragraphs.end();
-                               while (boost::next(it) != end)
-                                       ++it;
-               //              int const pos = (p->size() ? p->size()-1 : p->size());
+                               ParagraphList::iterator it = boost::prior(paragraphs.end());
                                text_.setCursor(it, it->size());
                        }
                } else {
@@ -967,7 +951,7 @@ Inset::RESULT InsetText::localDispatch(FuncRequest const & cmd)
                         * true (on). */
 #if 0
                        // This should not be needed here and is also WRONG!
-                       setUndo(bv, Undo::INSERT, text_.cursor.par());
+                       recordUndo(bv, Undo::INSERT, text_.cursor.par());
 #endif
                        bv->switchKeyMap();
                        if (lyxrc.auto_region_delete) {
@@ -1313,6 +1297,27 @@ int InsetText::ascii(Buffer const * buf, ostream & os, int linelen) const
 }
 
 
+int InsetText::linuxdoc(Buffer const * buf, ostream & os) const
+{
+       ParagraphList::iterator pit = const_cast<ParagraphList&>(paragraphs).begin();
+       ParagraphList::iterator pend = const_cast<ParagraphList&>(paragraphs).end();
+
+       // There is a confusion between the empty paragraph and the default paragraph
+       // The default paragraph is <p></p>, the empty paragraph is *empty*
+       // Since none of the floats of linuxdoc accepts standard paragraphs
+       // I disable them. I don't expect problems. (jamatos 2003/07/27)
+       for (; pit != pend; ++pit) {
+               const string name = pit->layout()->latexname();
+               if (name != "p")
+                       sgml::openTag(os, 1, 0, name);
+               buf->simpleLinuxDocOnePar(os, pit, 0);
+               if (name != "p")
+                       sgml::closeTag(os, 1, 0, name);
+       }
+       return 0;
+}
+
+
 int InsetText::docbook(Buffer const * buf, ostream & os, bool mixcont) const
 {
        unsigned int lines = 0;
@@ -1480,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();
 }
 
@@ -1491,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;
 }
 
@@ -1501,7 +1506,7 @@ int InsetText::insetInInsetY() const
        if (!the_locking_inset)
                return 0;
 
-       return (inset_y + the_locking_inset->insetInInsetY());
+       return inset_y + the_locking_inset->insetInInsetY();
 }
 
 
@@ -1511,17 +1516,18 @@ void InsetText::fitInsetCursor(BufferView * bv) const
                the_locking_inset->fitInsetCursor(bv);
                return;
        }
-       LyXFont const font = text_.getFont(bv->buffer(), cpar(), cpos());
+
+       LyXFont const font = text_.getFont(cpar(), cpos());
 
        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;
 }
 
 
-Inset::RESULT
+InsetOld::RESULT
 InsetText::moveRight(BufferView * bv, bool activate_inset, bool selecting)
 {
        if (text_.cursor.par()->isRightToLeftPar(bv->buffer()->params))
@@ -1531,7 +1537,7 @@ InsetText::moveRight(BufferView * bv, bool activate_inset, bool selecting)
 }
 
 
-Inset::RESULT
+InsetOld::RESULT
 InsetText::moveLeft(BufferView * bv, bool activate_inset, bool selecting)
 {
        if (text_.cursor.par()->isRightToLeftPar(bv->buffer()->params))
@@ -1541,7 +1547,7 @@ InsetText::moveLeft(BufferView * bv, bool activate_inset, bool selecting)
 }
 
 
-Inset::RESULT
+InsetOld::RESULT
 InsetText::moveRightIntern(BufferView * bv, bool front,
                           bool activate_inset, bool selecting)
 {
@@ -1558,7 +1564,7 @@ InsetText::moveRightIntern(BufferView * bv, bool front,
 }
 
 
-Inset::RESULT
+InsetOld::RESULT
 InsetText::moveLeftIntern(BufferView * bv, bool front,
                          bool activate_inset, bool selecting)
 {
@@ -1573,7 +1579,7 @@ InsetText::moveLeftIntern(BufferView * bv, bool front,
 }
 
 
-Inset::RESULT InsetText::moveUp(BufferView * bv)
+InsetOld::RESULT InsetText::moveUp(BufferView * bv)
 {
        if (crow() == text_.rows().begin())
                return FINISHED_UP;
@@ -1583,7 +1589,7 @@ Inset::RESULT InsetText::moveUp(BufferView * bv)
 }
 
 
-Inset::RESULT InsetText::moveDown(BufferView * bv)
+InsetOld::RESULT InsetText::moveDown(BufferView * bv)
 {
        if (boost::next(crow()) == text_.rows().end())
                return FINISHED_DOWN;
@@ -1593,7 +1599,7 @@ Inset::RESULT InsetText::moveDown(BufferView * bv)
 }
 
 
-bool InsetText::insertInset(BufferView * bv, Inset * inset)
+bool InsetText::insertInset(BufferView * bv, InsetOld * inset)
 {
        if (the_locking_inset) {
                if (the_locking_inset->insetAllowed(inset))
@@ -1608,7 +1614,7 @@ bool InsetText::insertInset(BufferView * bv, Inset * inset)
 }
 
 
-bool InsetText::insetAllowed(Inset::Code code) const
+bool InsetText::insetAllowed(InsetOld::Code code) const
 {
        // in_insetAllowed is a really gross hack,
        // to allow us to call the owner's insetAllowed
@@ -1634,7 +1640,7 @@ UpdatableInset * InsetText::getLockingInset() const
 }
 
 
-UpdatableInset * InsetText::getFirstLockingInsetOfType(Inset::Code c)
+UpdatableInset * InsetText::getFirstLockingInsetOfType(InsetOld::Code c)
 {
        if (c == lyxCode())
                return this;
@@ -1681,7 +1687,7 @@ void InsetText::setFont(BufferView * bv, LyXFont const & font, bool toggleall,
 
 
        if (text_.selection.set())
-               setUndo(bv, Undo::EDIT, text_.cursor.par());
+               recordUndo(bv, Undo::ATOMIC, text_.cursor.par());
 
        if (selectall) {
                text_.cursorTop();
@@ -1708,19 +1714,15 @@ void InsetText::setFont(BufferView * bv, LyXFont const & font, bool toggleall,
 
 bool InsetText::checkAndActivateInset(BufferView * bv, bool front)
 {
-       if (cpar()->isInset(cpos())) {
-               Inset * inset =
-                       static_cast<UpdatableInset*>(cpar()->getInset(cpos()));
-               if (!isHighlyEditableInset(inset))
-                       return false;
-               FuncRequest cmd(bv, LFUN_INSET_EDIT, front ? "left" : "right");
-               inset->localDispatch(cmd);
-               if (!the_locking_inset)
-                       return false;
-               updateLocal(bv, CURSOR, false);
-               return true;
-       }
-       return false;
+       InsetOld * inset = cpar()->getInset(cpos());
+       if (!isHighlyEditableInset(inset))
+               return false;
+       FuncRequest cmd(bv, LFUN_INSET_EDIT, front ? "left" : "right");
+       inset->localDispatch(cmd);
+       if (!the_locking_inset)
+               return false;
+       updateLocal(bv, CURSOR, false);
+       return true;
 }
 
 
@@ -1730,7 +1732,7 @@ bool InsetText::checkAndActivateInset(BufferView * bv, int x, int y,
        x -= drawTextXOffset;
        int dummyx = x;
        int dummyy = y + dim_.asc;
-       Inset * inset = getLyXText(bv)->checkInsetHit(dummyx, dummyy);
+       InsetOld * inset = getLyXText(bv)->checkInsetHit(dummyx, dummyy);
        // we only do the edit() call if the inset was hit by the mouse
        // or if it is a highly editable inset. So we should call this
        // function from our own edit with button < 0.
@@ -1739,21 +1741,20 @@ bool InsetText::checkAndActivateInset(BufferView * bv, int x, int y,
        if (button == mouse_button::none && !isHighlyEditableInset(inset))
                return false;
 
-       if (inset) {
-               if (x < 0)
-                       x = dim_.wid;
-               if (y < 0)
-                       y = dim_.des;
-               inset_x = cix(bv) - top_x + drawTextXOffset;
-               inset_y = ciy() + drawTextYOffset;
-               FuncRequest cmd(bv, LFUN_INSET_EDIT, x - inset_x, y - inset_y, button);
-               inset->localDispatch(cmd);
-               if (!the_locking_inset)
-                       return false;
-               updateLocal(bv, CURSOR, false);
-               return true;
-       }
-       return false;
+       if (!inset)
+               return false;
+       if (x < 0)
+               x = dim_.wid;
+       if (y < 0)
+               y = dim_.des;
+       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);
+       if (!the_locking_inset)
+               return false;
+       updateLocal(bv, CURSOR, false);
+       return true;
 }
 
 
@@ -1771,8 +1772,7 @@ void InsetText::setParagraphData(ParagraphList const & plist)
        ParagraphList::const_iterator end = plist.end();
        for (; it != end; ++it) {
                paragraphs.push_back(*it);
-               Paragraph & tmp = paragraphs.back();
-               tmp.setInsetOwner(this);
+               paragraphs.back().setInsetOwner(this);
        }
 
        reinitLyXText();
@@ -1835,12 +1835,11 @@ void InsetText::setFrameColor(BufferView * bv, LColor::color col)
 }
 
 
-int InsetText::cx(BufferView * bv) const
+int InsetText::cx() const
 {
        int x = text_.cursor.x() + top_x + TEXT_TO_INSET_OFFSET;
        if (the_locking_inset) {
-               LyXFont font = text_.getFont(bv->buffer(), text_.cursor.par(),
-                                           text_.cursor.pos());
+               LyXFont font = text_.getFont(text_.cursor.par(), text_.cursor.pos());
                if (font.isVisibleRightToLeft())
                        x -= the_locking_inset->width();
        }
@@ -1848,12 +1847,11 @@ int InsetText::cx(BufferView * bv) const
 }
 
 
-int InsetText::cix(BufferView * bv) const
+int InsetText::cix() const
 {
        int x = text_.cursor.ix() + top_x + TEXT_TO_INSET_OFFSET;
        if (the_locking_inset) {
-               LyXFont font = text_.getFont(bv->buffer(), text_.cursor.par(),
-                                           text_.cursor.pos());
+               LyXFont font = text_.getFont(text_.cursor.par(), text_.cursor.pos());
                if (font.isVisibleRightToLeft())
                        x -= the_locking_inset->width();
        }
@@ -1925,7 +1923,7 @@ void InsetText::deleteLyXText(BufferView * bv, bool recursive) const
 }
 
 
-void InsetText::resizeLyXText(BufferView * bv, bool force) const
+void InsetText::resizeLyXText(BufferView * bv, bool /*force*/) const
 {
        if (paragraphs.size() == 1 && paragraphs.begin()->empty()) {
                // no data, resize not neccessary!
@@ -1951,8 +1949,10 @@ void InsetText::resizeLyXText(BufferView * bv, bool force) const
        text_.init(bv);
        restoreLyXTextState();
 
+       // 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;
        }
 
@@ -1964,6 +1964,7 @@ void InsetText::resizeLyXText(BufferView * bv, bool force) const
        } else {
                need_update |= FULL;
        }
+#endif
 }
 
 
@@ -1983,12 +1984,11 @@ 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_));
        if (!owner()) {
-               const_cast<InsetText*>(this)->updateLocal(bv, FULL, false);
                // this will scroll the screen such that the cursor becomes visible
                bv->updateScrollbar();
        } else {
@@ -2075,7 +2075,7 @@ LyXCursor const & InsetText::cursor(BufferView * bv) const
 }
 
 
-Inset * InsetText::getInsetFromID(int id_arg) const
+InsetOld * InsetText::getInsetFromID(int id_arg) const
 {
        if (id_arg == id())
                return const_cast<InsetText *>(this);
@@ -2088,7 +2088,7 @@ Inset * InsetText::getInsetFromID(int id_arg) const
                for (; it != end; ++it) {
                        if (it->inset->id() == id_arg)
                                return it->inset;
-                       Inset * in = it->inset->getInsetFromID(id_arg);
+                       InsetOld * in = it->inset->getInsetFromID(id_arg);
                        if (in)
                                return in;
                }
@@ -2132,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;
@@ -2162,10 +2161,10 @@ bool InsetText::nextChange(BufferView * bv, lyx::pos_type & length)
                        return true;
                text_.cursorRight(true);
        }
-       lyxfind::SearchResult result =
-               lyxfind::findNextChange(bv, &text_, length);
+       lyx::find::SearchResult result =
+               lyx::find::findNextChange(bv, &text_, length);
 
-       if (result == lyxfind::SR_FOUND) {
+       if (result == lyx::find::SR_FOUND) {
                LyXCursor cur = text_.cursor;
                bv->unlockInset(bv->theLockingInset());
                if (bv->lockInset(this))
@@ -2174,7 +2173,7 @@ bool InsetText::nextChange(BufferView * bv, lyx::pos_type & length)
                text_.setSelectionRange(length);
                updateLocal(bv, SELECTION, false);
        }
-       return result != lyxfind::SR_NOT_FOUND;
+       return result != lyx::find::SR_NOT_FOUND;
 }
 
 
@@ -2186,10 +2185,10 @@ bool InsetText::searchForward(BufferView * bv, string const & str,
                        return true;
                text_.cursorRight(true);
        }
-       lyxfind::SearchResult result =
-               lyxfind::LyXFind(bv, &text_, str, true, cs, mw);
+       lyx::find::SearchResult result =
+               lyx::find::find(bv, &text_, str, true, cs, mw);
 
-       if (result == lyxfind::SR_FOUND) {
+       if (result == lyx::find::SR_FOUND) {
                LyXCursor cur = text_.cursor;
                bv->unlockInset(bv->theLockingInset());
                if (bv->lockInset(this))
@@ -2198,9 +2197,10 @@ bool InsetText::searchForward(BufferView * bv, string const & str,
                text_.setSelectionRange(str.length());
                updateLocal(bv, SELECTION, false);
        }
-       return (result != lyxfind::SR_NOT_FOUND);
+       return result != lyx::find::SR_NOT_FOUND;
 }
 
+
 bool InsetText::searchBackward(BufferView * bv, string const & str,
                               bool cs, bool mw)
 {
@@ -2217,10 +2217,10 @@ bool InsetText::searchBackward(BufferView * bv, string const & str,
 
                text_.setCursor(pit, pit->size());
        }
-       lyxfind::SearchResult result =
-               lyxfind::LyXFind(bv, &text_, str, false, cs, mw);
+       lyx::find::SearchResult result =
+               lyx::find::find(bv, &text_, str, false, cs, mw);
 
-       if (result == lyxfind::SR_FOUND) {
+       if (result == lyx::find::SR_FOUND) {
                LyXCursor cur = text_.cursor;
                bv->unlockInset(bv->theLockingInset());
                if (bv->lockInset(this))
@@ -2229,7 +2229,7 @@ bool InsetText::searchBackward(BufferView * bv, string const & str,
                text_.setSelectionRange(str.length());
                updateLocal(bv, SELECTION, false);
        }
-       return (result != lyxfind::SR_NOT_FOUND);
+       return result != lyx::find::SR_NOT_FOUND;
 }