]> git.lyx.org Git - lyx.git/blobdiff - src/lyxfind.cpp
* add PreBabelPreamble to Language definition (fixes #4786).
[lyx.git] / src / lyxfind.cpp
index 4e3dd9c17caf4df41be730b142ecbb769afe0ed6..c5adc57dd030b2d431b450b6685756f969b22455 100644 (file)
 #include "lyxfind.h"
 
 #include "Buffer.h"
-#include "BufferList.h"
 #include "buffer_funcs.h"
+#include "BufferList.h"
 #include "BufferParams.h"
 #include "BufferView.h"
 #include "Changes.h"
 #include "Cursor.h"
 #include "CutAndPaste.h"
 #include "FuncRequest.h"
-#include "OutputParams.h"
+#include "LyX.h"
 #include "output_latex.h"
+#include "OutputParams.h"
 #include "Paragraph.h"
 #include "ParIterator.h"
 #include "TexRow.h"
 #include "Text.h"
-#include "FuncRequest.h"
-#include "LyXFunc.h"
+
+#include "frontends/alert.h"
 
 #include "mathed/InsetMath.h"
 #include "mathed/InsetMathGrid.h"
 #include "mathed/InsetMathHull.h"
 #include "mathed/MathStream.h"
 
-#include "frontends/alert.h"
-
 #include "support/convert.h"
 #include "support/debug.h"
 #include "support/docstream.h"
 #include "support/gettext.h"
-#include "support/lstrings.h"
 #include "support/lassert.h"
+#include "support/lstrings.h"
 
-#include "frontends/Application.h"
-#include "frontends/LyXView.h"
-
-#include <boost/regex.hpp>
+#include "support/regex.h"
 #include <boost/next_prior.hpp>
 
 using namespace std;
@@ -75,23 +71,23 @@ class MatchString : public binary_function<Paragraph, pos_type, bool>
 {
 public:
        MatchString(docstring const & str, bool cs, bool mw)
-               : str(str), cs(cs), mw(mw)
+               : str(str), case_sens(cs), whole_words(mw)
        {}
 
        // returns true if the specified string is at the specified position
        // del specifies whether deleted strings in ct mode will be considered
        bool operator()(Paragraph const & par, pos_type pos, bool del = true) const
        {
-               return par.find(str, cs, mw, pos, del);
+               return par.find(str, case_sens, whole_words, pos, del);
        }
 
 private:
        // search string
        docstring str;
        // case sensitive
-       bool cs;
+       bool case_sens;
        // match whole words only
-       bool mw;
+       bool whole_words;
 };
 
 
@@ -136,7 +132,7 @@ bool findChange(DocIterator & cur, bool next)
 }
 
 
-bool searchAllowed(BufferView * /*bv*/, docstring const & str)
+bool searchAllowed(docstring const & str)
 {
        if (str.empty()) {
                frontend::Alert::error(_("Search error"), _("Search string is empty"));
@@ -146,21 +142,21 @@ bool searchAllowed(BufferView * /*bv*/, docstring const & str)
 }
 
 
-bool find(BufferView * bv, docstring const & searchstr,
-       bool cs, bool mw, bool fw, bool find_del = true)
+bool findOne(BufferView * bv, docstring const & searchstr,
+       bool case_sens, bool whole, bool forward, bool find_del = true)
 {
-       if (!searchAllowed(bv, searchstr))
+       if (!searchAllowed(searchstr))
                return false;
 
        DocIterator cur = bv->cursor();
 
-       MatchString const match(searchstr, cs, mw);
+       MatchString const match(searchstr, case_sens, whole);
 
-       bool found = fw ? findForward(cur, match, find_del) :
+       bool found = forward ? findForward(cur, match, find_del) :
                          findBackwards(cur, match, find_del);
 
        if (found)
-               bv->putSelectionAt(cur, searchstr.length(), !fw);
+               bv->putSelectionAt(cur, searchstr.length(), !forward);
 
        return found;
 }
@@ -168,14 +164,16 @@ bool find(BufferView * bv, docstring const & searchstr,
 
 int replaceAll(BufferView * bv,
               docstring const & searchstr, docstring const & replacestr,
-              bool cs, bool mw)
+              bool case_sens, bool whole)
 {
        Buffer & buf = bv->buffer();
 
-       if (!searchAllowed(bv, searchstr) || buf.isReadonly())
+       if (!searchAllowed(searchstr) || buf.isReadonly())
                return 0;
 
-       MatchString const match(searchstr, cs, mw);
+       DocIterator cur_orig(bv->cursor());
+
+       MatchString const match(searchstr, case_sens, whole);
        int num = 0;
 
        int const rsize = replacestr.size();
@@ -198,55 +196,80 @@ int replaceAll(BufferView * bv,
                ++num;
        }
 
-       buf.updateLabels();
        bv->putSelectionAt(doc_iterator_begin(&buf), 0, false);
        if (num)
                buf.markDirty();
+
+       cur_orig.fixIfBroken();
+       bv->setCursor(cur_orig);
+
        return num;
 }
 
 
-bool stringSelected(BufferView * bv, docstring & searchstr,
-                   bool cs, bool mw, bool fw)
+// the idea here is that we are going to replace the string that
+// is selected IF it is the search string. 
+// if there is a selection, but it is not the search string, then
+// we basically ignore it. (FIXME We ought to replace only within
+// the selection.)
+// if there is no selection, then:
+//  (i) if some search string has been provided, then we find it.
+//      (think of how the dialog works when you hit "replace" the
+//      first time.) 
+// (ii) if no search string has been provided, then we treat the
+//      word the cursor is in as the search string. (why? i have no
+//      idea.) but this only works in text?
+//
+// returns the number of replacements made (one, if any) and 
+// whether anything at all was done.
+pair<bool, int> replaceOne(BufferView * bv, docstring searchstr,
+           docstring const & replacestr, bool case_sens, 
+                       bool whole, bool forward)
 {
-       // if nothing selected and searched string is empty, this
-       // means that we want to search current word at cursor position.
-       if (!bv->cursor().selection() && searchstr.empty()) {
-               bv->cursor().innerText()->selectWord(bv->cursor(), WHOLE_WORD);
-               searchstr = bv->cursor().selectionAsString(false);
-               return true;
+       Cursor & cur = bv->cursor();
+       if (!cur.selection()) {
+               // no selection, non-empty search string: find it
+               if (!searchstr.empty()) {
+                       findOne(bv, searchstr, case_sens, whole, forward);
+                       return pair<bool, int>(true, 0);
+               }
+               // empty search string
+               if (!cur.inTexted())
+                       // bail in math
+                       return pair<int, bool>(0, false);
+               // select current word and treat it as the search string
+               cur.innerText()->selectWord(cur, WHOLE_WORD);
+               searchstr = cur.selectionAsString(false);
        }
-
-       // if nothing selected or selection does not equal search
-       // string search and select next occurance and return
-       docstring const & str1 = searchstr;
-       docstring const str2 = bv->cursor().selectionAsString(false);
-       if ((cs && str1 != str2) || compare_no_case(str1, str2) != 0) {
-               find(bv, searchstr, cs, mw, fw);
-               return false;
+       
+       // if we still don't have a search string, report the error
+       // and abort.
+       if (!searchAllowed(searchstr))
+               return pair<bool, int>(false, 0);
+       
+       bool have_selection = cur.selection();
+       docstring const selected = cur.selectionAsString(false);
+       bool match = 
+               case_sens ? searchstr == selected
+                   : compare_no_case(searchstr, selected) == 0;
+
+       // no selection or current selection is not search word:
+       // just find the search word
+       if (!have_selection || !match) {
+               findOne(bv, searchstr, case_sens, whole, forward);
+               return pair<bool, int>(true, 0);
        }
 
-       return true;
-}
-
-
-int replace(BufferView * bv, docstring & searchstr,
-           docstring const & replacestr, bool cs, bool mw, bool fw)
-{
-       if (!stringSelected(bv, searchstr, cs, mw, fw))
-               return 0;
-
-       if (!searchAllowed(bv, searchstr) || bv->buffer().isReadonly())
-               return 0;
+       // we're now actually ready to replace. if the buffer is
+       // read-only, we can't, though.
+       if (bv->buffer().isReadonly())
+               return pair<bool, int>(false, 0);
 
-       Cursor & cur = bv->cursor();
-       cap::replaceSelectionWithString(cur, replacestr, fw);
+       cap::replaceSelectionWithString(cur, replacestr, forward);
        bv->buffer().markDirty();
-       find(bv, searchstr, cs, mw, fw, false);
-       bv->buffer().updateMacros();
-       bv->processUpdateFlags(Update::Force | Update::FitCursor);
+       findOne(bv, searchstr, case_sens, whole, forward, false);
 
-       return 1;
+       return pair<bool, int>(true, 1);
 }
 
 } // namespace anon
@@ -279,9 +302,9 @@ docstring const replace2string(docstring const & replace,
 }
 
 
-bool find(BufferView * bv, FuncRequest const & ev)
+bool lyxfind(BufferView * bv, FuncRequest const & ev)
 {
-       if (!bv || ev.action != LFUN_WORD_FIND)
+       if (!bv || ev.action() != LFUN_WORD_FIND)
                return false;
 
        //lyxerr << "find called, cmd: " << ev << endl;
@@ -296,14 +319,15 @@ bool find(BufferView * bv, FuncRequest const & ev)
        bool matchword     = parse_bool(howto);
        bool forward       = parse_bool(howto);
 
-       return find(bv, search, casesensitive, matchword, forward);
+       return findOne(bv, search, casesensitive, matchword, forward);
 }
 
 
-void replace(BufferView * bv, FuncRequest const & ev, bool has_deleted)
+bool lyxreplace(BufferView * bv, 
+               FuncRequest const & ev, bool has_deleted)
 {
-       if (!bv || ev.action != LFUN_WORD_REPLACE)
-               return;
+       if (!bv || ev.action() != LFUN_WORD_REPLACE)
+               return false;
 
        // data is of the form
        // "<search>
@@ -319,34 +343,44 @@ void replace(BufferView * bv, FuncRequest const & ev, bool has_deleted)
        bool all           = parse_bool(howto);
        bool forward       = parse_bool(howto);
 
+       int replace_count = 0;
+       bool update = false;
+
        if (!has_deleted) {
-               int const replace_count = all
-                       ? replaceAll(bv, search, rplc, casesensitive, matchword)
-                       : replace(bv, search, rplc, casesensitive, matchword, forward);
+               if (all) {
+                       replace_count = replaceAll(bv, search, rplc, casesensitive, matchword);
+                       update = replace_count > 0;
+               } else {
+                       pair<bool, int> rv = 
+                               replaceOne(bv, search, rplc, casesensitive, matchword, forward);
+                       update = rv.first;
+                       replace_count = rv.second;
+               }
 
-               Buffer & buf = bv->buffer();
-               if (replace_count == 0) {
+               Buffer const & buf = bv->buffer();
+               if (!update) {
                        // emit message signal.
                        buf.message(_("String not found!"));
                } else {
-                       if (replace_count == 1) {
-                               // emit message signal.
+                       if (replace_count == 0) {
+                               buf.message(_("String found."));
+                       } else if (replace_count == 1) {
                                buf.message(_("String has been replaced."));
                        } else {
-                               docstring str = convert<docstring>(replace_count);
-                               str += _(" strings have been replaced.");
-                               // emit message signal.
+                               docstring const str = 
+                                       bformat(_("%1$d strings have been replaced."), replace_count);
                                buf.message(str);
                        }
                }
        } else {
                // if we have deleted characters, we do not replace at all, but
                // rather search for the next occurence
-               if (find(bv, search, casesensitive, matchword, forward))
-                       bv->showCursor();
+               if (findOne(bv, search, casesensitive, matchword, forward))
+                       update = true;
                else
                        bv->message(_("String not found!"));
        }
+       return update;
 }
 
 
@@ -367,7 +401,7 @@ bool findChange(BufferView * bv, bool next)
        if (bv->cursor().selection()) {
                // set the cursor at the beginning or at the end of the selection
                // before searching. Otherwise, the current change will be found.
-               if (next != (bv->cursor().top() > bv->cursor().anchor()))
+               if (next != (bv->cursor().top() > bv->cursor().normalAnchor()))
                        bv->cursor().setCursorToAnchor();
        }
 
@@ -377,11 +411,18 @@ bool findChange(BufferView * bv, bool next)
        // clear the selection and search the other way around (see the end
        // of this function). This will avoid changes to be selected half.
        bool search_both_sides = false;
-       if (cur.pos() > 1) {
-               Change change_next_pos
-                       = cur.paragraph().lookupChange(cur.pos());
+       DocIterator tmpcur = cur;
+       // Leave math first
+       while (tmpcur.inMathed())
+               tmpcur.pop_back();
+       Change change_next_pos
+               = tmpcur.paragraph().lookupChange(tmpcur.pos());
+       if (change_next_pos.changed() && cur.inMathed()) {
+               cur = tmpcur;
+               search_both_sides = true;
+       } else if (tmpcur.pos() > 0 && tmpcur.inTexted()) {
                Change change_prev_pos
-                       = cur.paragraph().lookupChange(cur.pos() - 1);
+                       = tmpcur.paragraph().lookupChange(tmpcur.pos() - 1);
                if (change_next_pos.isSimilarTo(change_prev_pos))
                        search_both_sides = true;
        }
@@ -402,14 +443,14 @@ bool findChange(BufferView * bv, bool next)
        if (next) {
                for (; !tip.at_end(); tip.forwardPos()) {
                        Change change = tip.paragraph().lookupChange(tip.pos());
-                       if (change != orig_change)
+                       if (!change.isSimilarTo(orig_change))
                                break;
                }
        } else {
                for (; !tip.at_begin();) {
                        tip.backwardPos();
                        Change change = tip.paragraph().lookupChange(tip.pos());
-                       if (change != orig_change) {
+                       if (!change.isSimilarTo(orig_change)) {
                                // take a step forward to correctly set the selection
                                tip.forwardPos();
                                break;
@@ -516,45 +557,46 @@ string escape_for_regex(string s)
 {
        size_t pos = 0;
        while (pos < s.size()) {
-                       size_t new_pos = s.find("\\regexp{", pos);
-                       if (new_pos == string::npos)
-                                       new_pos = s.size();
-                       LYXERR(Debug::FIND, "new_pos: " << new_pos);
-                       string t = apply_escapes(s.substr(pos, new_pos - pos), get_lyx_unescapes());
-                       LYXERR(Debug::FIND, "t      : " << t);
-                       t = apply_escapes(t, get_regexp_escapes());
-                       s.replace(pos, new_pos - pos, t);
-                       new_pos = pos + t.size();
-                       LYXERR(Debug::FIND, "Regexp after escaping: " << s);
-                       LYXERR(Debug::FIND, "new_pos: " << new_pos);
-                       if (new_pos == s.size())
-                                       break;
-                       size_t end_pos = find_matching_brace(s, new_pos + 7);
-                       LYXERR(Debug::FIND, "end_pos: " << end_pos);
-                       t = apply_escapes(s.substr(new_pos + 8, end_pos - (new_pos + 8)), get_lyx_unescapes());
-                       LYXERR(Debug::FIND, "t      : " << t);
-                       if (end_pos == s.size()) {
-                                       s.replace(new_pos, end_pos - new_pos, t);
-                                       pos = s.size();
-                                       LYXERR(Debug::FIND, "Regexp after \\regexp{} removal: " << s);
-                                       break;
-                       }
-                       s.replace(new_pos, end_pos + 1 - new_pos, t);
+               size_t new_pos = s.find("\\regexp{{{", pos);
+               if (new_pos == string::npos)
+                       new_pos = s.size();
+               LYXERR(Debug::FIND, "new_pos: " << new_pos);
+               string t = apply_escapes(s.substr(pos, new_pos - pos), get_lyx_unescapes());
+               LYXERR(Debug::FIND, "t      : " << t);
+               t = apply_escapes(t, get_regexp_escapes());
+               LYXERR(Debug::FIND, "t      : " << t);
+               s.replace(pos, new_pos - pos, t);
+               new_pos = pos + t.size();
+               LYXERR(Debug::FIND, "Regexp after escaping: " << s);
+               LYXERR(Debug::FIND, "new_pos: " << new_pos);
+               if (new_pos == s.size())
+                       break;
+               size_t end_pos = s.find("}}}", new_pos + 10); // find_matching_brace(s, new_pos + 7);
+               LYXERR(Debug::FIND, "end_pos: " << end_pos);
+               t = apply_escapes(s.substr(new_pos + 10, end_pos - (new_pos + 10)), get_lyx_unescapes());
+               LYXERR(Debug::FIND, "t      : " << t);
+               if (end_pos == s.size()) {
+                       s.replace(new_pos, end_pos - new_pos, t);
+                       pos = s.size();
                        LYXERR(Debug::FIND, "Regexp after \\regexp{} removal: " << s);
-                       pos = new_pos + t.size();
-                       LYXERR(Debug::FIND, "pos: " << pos);
+                       break;
+               }
+               s.replace(new_pos, end_pos + 3 - new_pos, t);
+               LYXERR(Debug::FIND, "Regexp after \\regexp{} removal: " << s);
+               pos = new_pos + t.size();
+               LYXERR(Debug::FIND, "pos: " << pos);
        }
        return s;
 }
 
-/// Wrapper for boost::regex_replace with simpler interface
+/// Wrapper for lyx::regex_replace with simpler interface
 bool regex_replace(string const & s, string & t, string const & searchstr,
        string const & replacestr)
 {
-       boost::regex e(searchstr);
+       lyx::regex e(searchstr);
        ostringstream oss;
        ostream_iterator<char, char> it(oss);
-       boost::regex_replace(it, s.begin(), s.end(), e, replacestr);
+       lyx::regex_replace(it, s.begin(), s.end(), e, replacestr);
        // tolerate t and s be references to the same variable
        bool rv = (s != oss.str());
        t = oss.str();
@@ -570,7 +612,8 @@ bool regex_replace(string const & s, string & t, string const & searchstr,
  ** Number of open braces that must remain open at the end for the verification to succeed.
  **/
 bool braces_match(string::const_iterator const & beg,
-       string::const_iterator const & end, int unmatched = 0)
+                 string::const_iterator const & end,
+                 int unmatched = 0)
 {
        int open_pars = 0;
        string::const_iterator it = beg;
@@ -628,6 +671,9 @@ public:
        FindAndReplaceOptions const & opt;
 
 private:
+       /// Auxiliary find method (does not account for opt.matchword)
+       int findAux(DocIterator const & cur, int len = -1, bool at_begin = true) const;
+
        /** Normalize a stringified or latexified LyX paragraph.
         **
         ** Normalize means:
@@ -645,9 +691,9 @@ private:
        // normalized string to search
        string par_as_string;
        // regular expression to use for searching
-       boost::regex regexp;
+       lyx::regex regexp;
        // same as regexp, but prefixed with a ".*"
-       boost::regex regexp2;
+       lyx::regex regexp2;
        // unmatched open braces in the search string/regexp
        int open_braces;
        // number of (.*?) subexpressions added at end of search regexp for closing
@@ -683,7 +729,6 @@ MatchStringAdv::MatchStringAdv(lyx::Buffer & buf, FindAndReplaceOptions const &
                        break;
                } while (true);
                LYXERR(Debug::FIND, "Open braces: " << open_braces);
-               LASSERT(braces_match(par_as_string.begin(), par_as_string.end(), open_braces), /* */);
                LYXERR(Debug::FIND, "Built MatchStringAdv object: par_as_string = '" << par_as_string << "'");
        } else {
                par_as_string = escape_for_regex(par_as_string);
@@ -705,17 +750,16 @@ MatchStringAdv::MatchStringAdv(lyx::Buffer & buf, FindAndReplaceOptions const &
                LYXERR(Debug::FIND, "par_as_string now is '" << par_as_string << "'");
                LYXERR(Debug::FIND, "Open braces: " << open_braces);
                LYXERR(Debug::FIND, "Close .*?  : " << close_wildcards);
-               LASSERT(braces_match(par_as_string.begin(), par_as_string.end(), open_braces), /* */);
                LYXERR(Debug::FIND, "Replaced text (to be used as regex): " << par_as_string);
                // If entered regexp must match at begin of searched string buffer
-               regexp = boost::regex(string("\\`") + par_as_string);
+               regexp = lyx::regex(string("\\`") + par_as_string);
                // If entered regexp may match wherever in searched string buffer
-               regexp2 = boost::regex(string("\\`.*") + par_as_string);
+               regexp2 = lyx::regex(string("\\`.*") + par_as_string);
        }
 }
 
 
-int MatchStringAdv::operator()(DocIterator const & cur, int len, bool at_begin) const
+int MatchStringAdv::findAux(DocIterator const & cur, int len, bool at_begin) const
 {
        docstring docstr = stringifyFromForSearch(opt, cur, len);
        LYXERR(Debug::FIND, "Matching against     '" << lyx::to_utf8(docstr) << "'");
@@ -733,11 +777,11 @@ int MatchStringAdv::operator()(DocIterator const & cur, int len, bool at_begin)
        } else {
                // Try all possible regexp matches, 
                //until one that verifies the braces match test is found
-               boost::regex const *p_regexp = at_begin ? &regexp : &regexp2;
-               boost::sregex_iterator re_it(str.begin(), str.end(), *p_regexp);
-               boost::sregex_iterator re_it_end;
+               regex const *p_regexp = at_begin ? &regexp : &regexp2;
+               sregex_iterator re_it(str.begin(), str.end(), *p_regexp);
+               sregex_iterator re_it_end;
                for (; re_it != re_it_end; ++re_it) {
-                       boost::match_results<string::const_iterator> const & m = *re_it;
+                       match_results<string::const_iterator> const & m = *re_it;
                        // Check braces on the segment that matched the entire regexp expression,
                        // plus the last subexpression, if a (.*?) was inserted in the constructor.
                        if (! braces_match(m[0].first, m[0].second, open_braces))
@@ -758,6 +802,26 @@ int MatchStringAdv::operator()(DocIterator const & cur, int len, bool at_begin)
 }
 
 
+int MatchStringAdv::operator()(DocIterator const & cur, int len, bool at_begin) const
+{
+       int res = findAux(cur, len, at_begin);
+       if (res == 0 || !at_begin || !opt.matchword || !cur.inTexted())
+               return res;
+       Paragraph const & par = cur.paragraph();
+       bool ws_left = cur.pos() > 0 ?
+               par.isWordSeparator(cur.pos() - 1) : true;
+       bool ws_right = cur.pos() + res < par.size() ?
+               par.isWordSeparator(cur.pos() + res) : true;
+       LYXERR(Debug::FIND,
+              "cur.pos()=" << cur.pos() << ", res=" << res
+              << ", separ: " << ws_left << ", " << ws_right
+              << endl);
+       if (ws_left && ws_right)
+               return res;
+       return 0;
+}
+
+
 string MatchStringAdv::normalize(docstring const & s) const
 {
        string t;
@@ -844,9 +908,6 @@ docstring latexifyFromCursor(DocIterator const & cur, int len)
                        ParagraphList::const_iterator pit = cur.innerText()->paragraphs().begin();
                        for (int i = 0; i < cur.pit(); ++i)
                                        ++pit;
-//             ParagraphList::const_iterator pit_end = pit;
-//             ++pit_end;
-//             lyx::latexParagraphs(buf, cur.innerText()->paragraphs(), ods, texrow, runparams, string(), pit, pit_end);
                pos_type const endpos = (len == -1 || cur.pos() + len > int(pit->size()))
                        ? pit->size() : cur.pos() + len;
                TeXOnePar(buf, *cur.innerText(), pit, ods, texrow, runparams, string(),
@@ -870,13 +931,6 @@ docstring latexifyFromCursor(DocIterator const & cur, int len)
                for (MathData::const_iterator it = md.begin() + cs.pos(); it != it_end; ++it)
                                ods << *it;
 
-               // MathData md = cur.cell();
-               // MathData::const_iterator it_end = ( ( len == -1 || cur.pos() + len > int(md.size()) ) ? md.end() : md.begin() + cur.pos() + len );
-               // for (MathData::const_iterator it = md.begin() + cur.pos(); it != it_end; ++it) {
-               //      MathAtom const & ma = *it;
-               //      ma.nucleus()->latex(buf, ods, runparams);
-               // }
-
                // Retrieve the math environment type, and add '$' or '$]'
                // or others (\end{equation}) accordingly
                for (int s = cur.depth() - 1; s >= 0; --s) {
@@ -918,6 +972,8 @@ int findAdvFinalize(DocIterator & cur, MatchStringAdv const & match)
 
        // Compute the match length
        int len = 1;
+       if (cur.pos() + len > cur.lastpos())
+               return 0;
        LYXERR(Debug::FIND, "verifying unmatch with len = " << len);
        while (cur.pos() + len <= cur.lastpos() && match(cur, len) == 0) {
                ++len;
@@ -936,257 +992,95 @@ int findAdvFinalize(DocIterator & cur, MatchStringAdv const & match)
 }
 
 
-/** Switch p_buf to point to next document buffer.
- **
- ** Return true if restarted from master-document buffer.
- **
- ** @note
- ** Not using p_buf->allRelatives() here, because I'm not sure
- ** whether or not the returned order is independent of p_buf.
- **/
-bool next_document_buffer(Buffer * & p_buf) {
-       Buffer *p_master = p_buf;
-       Buffer *p_old;
-       do {
-               p_old = p_master;
-               p_master = const_cast<Buffer *>(p_master->masterBuffer());
-               LYXERR(Debug::FIND, "p_old=" << p_old << ", p_master=" << p_master);
-       } while (p_master != p_old);
-       LASSERT(p_master != NULL, /**/);
-       vector<Buffer *> v_children;
-       /* Root master added as first buffer in the vector */
-       v_children.push_back(p_master);
-       p_master->getChildren(v_children, true);
-       LYXERR(Debug::FIND, "v_children.size()=" << v_children.size());
-       vector<Buffer *>::const_iterator it = find(v_children.begin(), v_children.end(), p_buf);
-       LASSERT(it != v_children.end(), /**/)
-       ++it;
-       if (it == v_children.end()) {
-               p_buf = *v_children.begin();
-               return true;
-       }
-       p_buf = *it;
-       return false;
-}
-
-
-/** Switch p_buf to point to previous document buffer.
- **
- ** Return true if restarted from last child buffer.
- **
- ** @note
- ** Not using p_buf->allRelatives() here, because I'm not sure
- ** whether or not the returned order is independent of p_buf.
- **/
-bool prev_document_buffer(Buffer * & p_buf) {
-       Buffer *p_master = p_buf;
-       Buffer *p_old;
-       do {
-               p_old = p_master;
-               p_master = const_cast<Buffer *>(p_master->masterBuffer());
-               LYXERR(Debug::FIND, "p_old=" << p_old << ", p_master=" << p_master);
-       } while (p_master != p_old);
-       LASSERT(p_master != NULL, /**/);
-       vector<Buffer *> v_children;
-       /* Root master added as first buffer in the vector */
-       v_children.push_back(p_master);
-       p_master->getChildren(v_children, true);
-       LYXERR(Debug::FIND, "v_children.size()=" << v_children.size());
-       vector<Buffer *>::const_iterator it = find(v_children.begin(), v_children.end(), p_buf);
-       LASSERT(it != v_children.end(), /**/)
-       if (it == v_children.begin()) {
-               it = v_children.end();
-               --it;
-               p_buf = *it;
-               return true;
-       }
-       --it;
-       p_buf = *it;
-       return false;
-}
-
-
-/** Switch p_buf to point to next open buffer.
- **
- ** Return true if restarted from first open buffer.
- **/
-bool next_open_buffer(Buffer * & p_buf) {
-       p_buf = theBufferList().next(p_buf);
-       return p_buf == *theBufferList().begin();
-}
-
-
-/** Switch p_buf to point to previous open buffer.
- **
- ** Return true if restarted from last open buffer.
- **/
-bool prev_open_buffer(Buffer * & p_buf) {
-       p_buf = theBufferList().previous(p_buf);
-       return p_buf == *(theBufferList().end() - 1);
-}
-
-
 /// Finds forward
 int findForwardAdv(DocIterator & cur, MatchStringAdv & match)
 {
        if (!cur)
                return 0;
-       int wrap_answer = -1;
-       do {
-               while (cur && !match(cur, -1, false)) {
-                       if (cur.pit() < cur.lastpit())
-                               cur.forwardPar();
-                       else {
-                               cur.forwardPos();
-                       }
-               }
-               for (; cur; cur.forwardPos()) {
-                       if (match(cur))
-                               return findAdvFinalize(cur, match);
+       while (cur && !match(cur, -1, false)) {
+               if (cur.pit() < cur.lastpit())
+                       cur.forwardPar();
+               else {
+                       cur.forwardPos();
                }
-               // No match has been found in current buffer
-               bool prompt = false;
-               switch (match.opt.scope) {
-               case FindAndReplaceOptions::S_BUFFER:
-                       prompt = true;
-                       break;
-               case FindAndReplaceOptions::S_DOCUMENT:
-                       prompt = next_document_buffer(match.p_buf);
-                       break;
-               case FindAndReplaceOptions::S_OPEN_BUFFERS:
-                       prompt = next_open_buffer(match.p_buf);
-                       break;
-               }
-               if (prompt) {
-                       if (wrap_answer != -1)
-                               break;
-                       wrap_answer = frontend::Alert::prompt(
-                               _("Wrap search?"),
-                               _("End of document/scope reached while searching forward.\n"
-                                       "\n"
-                                       "Continue searching from beginning?"),
-                               0, 1, _("&Yes"), _("&No"));
-               }
-               cur.clear();
-               cur.push_back(CursorSlice(match.p_buf->inset()));
-       } while (wrap_answer != 1);
+       }
+       for (; cur; cur.forwardPos()) {
+               if (match(cur))
+                       return findAdvFinalize(cur, match);
+       }
        return 0;
 }
 
 
 /// Find the most backward consecutive match within same paragraph while searching backwards.
-void findMostBackwards(DocIterator & cur, MatchStringAdv const & match, int & len) {
+int findMostBackwards(DocIterator & cur, MatchStringAdv const & match)
+{
        DocIterator cur_begin = doc_iterator_begin(cur.buffer());
-       len = findAdvFinalize(cur, match);
-       if (cur != cur_begin) {
-               Inset & inset = cur.inset();
-               int old_len;
-               DocIterator old_cur;
-               DocIterator dit2;
-               do {
-                       old_cur = cur;
-                       old_len = len;
-                       cur.backwardPos();
-                       LYXERR(Debug::FIND, "findMostBackwards(): old_cur=" 
-                               << old_cur << ", old_len=" << len << ", cur=" << cur);
-                       dit2 = cur;
-               } while (cur != cur_begin && &cur.inset() == &inset && match(cur)
-                        && (len = findAdvFinalize(dit2, match)) > old_len);
-               cur = old_cur;
-               len = old_len;
+       DocIterator tmp_cur = cur;
+       int len = findAdvFinalize(tmp_cur, match);
+       Inset & inset = cur.inset();
+       for (; cur != cur_begin; cur.backwardPos()) {
+               LYXERR(Debug::FIND, "findMostBackwards(): cur=" << cur);
+               DocIterator new_cur = cur;
+               new_cur.backwardPos();
+               if (new_cur == cur || &new_cur.inset() != &inset || !match(new_cur))
+                       break;
+               int new_len = findAdvFinalize(new_cur, match);
+               if (new_len == len)
+                       break;
+               len = new_len;
        }
-       LYXERR(Debug::FIND, "findMostBackwards(): cur=" << cur);
+       LYXERR(Debug::FIND, "findMostBackwards(): exiting with cur=" << cur);
+       return len;
 }
 
+
 /// Finds backwards
 int findBackwardsAdv(DocIterator & cur, MatchStringAdv & match) {
        if (! cur)
                return 0;
-       // Backup of original position (for restoring it in case match not found)
-       DocIterator cur_orig(cur);
-       // Position beyond which match is not considered
-       // (set to end of document after wrap-around question)
-       DocIterator cur_orig2(cur);
+       // Backup of original position
        DocIterator cur_begin = doc_iterator_begin(cur.buffer());
-/*     if (match(cur_orig)) */
-/*             findAdvFinalize(cur_orig, match); */
-       int wrap_answer = 0;
+       if (cur == cur_begin)
+               return 0;
+       cur.backwardPos();
+       DocIterator cur_orig(cur);
        bool found_match;
+       bool pit_changed = false;
+       found_match = false;
        do {
-               bool pit_changed = false;
-               found_match = false;
-               // Search in current par occurs from start to end, 
-               // but in next loop match is discarded if pos > original pos
                cur.pos() = 0;
                found_match = match(cur, -1, false);
-               LYXERR(Debug::FIND, "findBackAdv0: found_match=" << found_match << ", cur: " << cur);
-               while (cur != cur_begin) {
-                       if (found_match)
-                               break;
-                       if (cur.pit() > 0)
-                               --cur.pit();
-                       else
-                               cur.backwardPos();
-                       pit_changed = true;
-                       // Search in previous pars occurs from start to end
-                       cur.pos() = 0;
-                       found_match = match(cur, -1, false);
-                       LYXERR(Debug::FIND, "findBackAdv1: found_match=" 
-                               << found_match << ", cur: " << cur);
-               }
-               if (pit_changed)
-                       cur.pos() = cur.lastpos();
-               else
-                       cur.pos() = cur_orig2.pos();
-               LYXERR(Debug::FIND, "findBackAdv2: cur: " << cur);
-               DocIterator cur_prev_iter;
+
                if (found_match) {
-                       while (true) {
-                               found_match=match(cur);
+                       if (pit_changed)
+                               cur.pos() = cur.lastpos();
+                       else
+                               cur.pos() = cur_orig.pos();
+                       LYXERR(Debug::FIND, "findBackAdv2: cur: " << cur);
+                       DocIterator cur_prev_iter;
+                       do {
+                               found_match = match(cur);
                                LYXERR(Debug::FIND, "findBackAdv3: found_match=" 
-                                       << found_match << ", cur: " << cur);
-                               if (found_match) {
-                                       int len;
-                                       findMostBackwards(cur, match, len);
-                                       if (&cur.inset() != &cur_orig2.inset()
-                                           || !(cur.pit() == cur_orig2.pit())
-                                           || cur.pos() < cur_orig2.pos())
-                                               return len;
-                               }
-                               // Prevent infinite loop at begin of document
-                               if (cur == cur_begin || cur == cur_prev_iter)
+                                      << found_match << ", cur: " << cur);
+                               if (found_match)
+                                       return findMostBackwards(cur, match);
+
+                               // Stop if begin of document reached
+                               if (cur == cur_begin)
                                        break;
                                cur_prev_iter = cur;
                                cur.backwardPos();
-                       };
+                       } while (true);
                }
-               // No match has been found in current buffer
-               bool prompt = false;
-               switch (match.opt.scope) {
-               case FindAndReplaceOptions::S_BUFFER:
-                       prompt = true;
-                       break;
-               case FindAndReplaceOptions::S_DOCUMENT:
-                       prompt = prev_document_buffer(match.p_buf);
-                       break;
-               case FindAndReplaceOptions::S_OPEN_BUFFERS:
-                       prompt = prev_open_buffer(match.p_buf);
+               if (cur == cur_begin)
                        break;
-               }
-               if (prompt) {
-                       wrap_answer = frontend::Alert::prompt(
-                               _("Wrap search?"),
-                               _("Beginning of document/scope reached while searching backwards\n"
-                                 "\n"
-                                 "Continue searching from end?"),
-                               0, 1, _("&Yes"), _("&No"));
-               }
-               cur = doc_iterator_end(match.p_buf);
-               cur.backwardPos();
-               LYXERR(Debug::FIND, "findBackAdv5: cur: " << cur);
-               cur_orig2 = cur;
-       } while (wrap_answer == 0);
-       cur = cur_orig;
+               if (cur.pit() > 0)
+                       --cur.pit();
+               else
+                       cur.backwardPos();
+               pit_changed = true;
+       } while (true);
        return 0;
 }
 
@@ -1215,16 +1109,7 @@ FindAndReplaceOptions::FindAndReplaceOptions(docstring const & search, bool case
 }
 
 
-/** Checks if the supplied character is lower-case */
-static bool isLowerCase(char_type ch) {
-       return lowercase(ch) == ch;
-}
-
-
-/** Checks if the supplied character is upper-case */
-static bool isUpperCase(char_type ch) {
-       return uppercase(ch) == ch;
-}
+namespace {
 
 
 /** Check if 'len' letters following cursor are all non-lowercase */
@@ -1266,31 +1151,99 @@ static void changeFirstCase(Buffer & buffer, TextCase first_case, TextCase other
        right = pit->size() + 1;
        pit->changeCase(buffer.params(), right, right, others_case);
 }
+} // anon namespace
+
+///
+static void findAdvReplace(BufferView * bv, FindAndReplaceOptions const & opt, MatchStringAdv & matchAdv)
+{
+       Cursor & cur = bv->cursor();
+       if (opt.replace == docstring(from_utf8(LYX_FR_NULL_STRING)))
+               return;
+       DocIterator sel_beg = cur.selectionBegin();
+       DocIterator sel_end = cur.selectionEnd();
+       if (&sel_beg.inset() != &sel_end.inset()
+           || sel_beg.pit() != sel_end.pit())
+               return;
+       int sel_len = sel_end.pos() - sel_beg.pos();
+       LYXERR(Debug::FIND, "sel_beg: " << sel_beg << ", sel_end: " << sel_end
+              << ", sel_len: " << sel_len << endl);
+       if (sel_len == 0)
+               return;
+       LASSERT(sel_len > 0, /**/);
+
+       if (!matchAdv(sel_beg, sel_len))
+               return;
+
+       string lyx = to_utf8(opt.replace);
+       // FIXME: Seems so stupid to me to rebuild a buffer here,
+       // when we already have one (replace_work_area_.buffer())
+       Buffer repl_buffer("", false);
+       repl_buffer.setUnnamed(true);
+       LASSERT(repl_buffer.readString(lyx), /**/);
+       repl_buffer.changeLanguage(
+               repl_buffer.language(),
+               cur.getFont().language());
+       if (opt.keep_case && sel_len >= 2) {
+               if (cur.inTexted()) {
+                       if (firstUppercase(cur))
+                               changeFirstCase(repl_buffer, text_uppercase, text_lowercase);
+                       else if (allNonLowercase(cur, sel_len))
+                               changeFirstCase(repl_buffer, text_uppercase, text_uppercase);
+               }
+       }
+       cap::cutSelection(cur, false, false);
+       if (!cur.inMathed()) {
+               LYXERR(Debug::FIND, "Replacing by pasteParagraphList()ing repl_buffer");
+               cap::pasteParagraphList(cur, repl_buffer.paragraphs(),
+                                       repl_buffer.params().documentClassPtr(),
+                                       bv->buffer().errorList("Paste"));
+       } else {
+               odocstringstream ods;
+               OutputParams runparams(&repl_buffer.params().encoding());
+               runparams.nice = false;
+               runparams.flavor = OutputParams::LATEX;
+               runparams.linelen = 8000; //lyxrc.plaintext_linelen;
+               runparams.dryrun = true;
+               TexRow texrow;
+               TeXOnePar(repl_buffer, repl_buffer.text(), 
+                         repl_buffer.paragraphs().begin(), ods, texrow, runparams);
+               //repl_buffer.getSourceCode(ods, 0, repl_buffer.paragraphs().size(), false);
+               docstring repl_latex = ods.str();
+               LYXERR(Debug::FIND, "Latexified replace_buffer: '" << repl_latex << "'");
+               string s;
+               regex_replace(to_utf8(repl_latex), s, "\\$(.*)\\$", "$1");
+               regex_replace(s, s, "\\\\\\[(.*)\\\\\\]", "$1");
+               repl_latex = from_utf8(s);
+               LYXERR(Debug::FIND, "Replacing by niceInsert()ing latex: '" << repl_latex << "'");
+               cur.niceInsert(repl_latex);
+       }
+       bv->buffer().markDirty();
+       cur.pos() -= repl_buffer.paragraphs().begin()->size();
+       bv->putSelectionAt(DocIterator(cur), repl_buffer.paragraphs().begin()->size(), !opt.forward);
+}
 
 
 /// Perform a FindAdv operation.
 bool findAdv(BufferView * bv, FindAndReplaceOptions const & opt)
 {
-       DocIterator cur = bv->cursor();
-       int match_len = 0;
+       DocIterator cur;
+       int match_len;
 
        if (opt.search.empty()) {
-                       bv->message(_("Search text is empty!"));
-                       return false;
+               bv->message(_("Search text is empty!"));
+               return false;
        }
-//     if (! bv->buffer()) {
-//             bv->message(_("No open document !"));
-//             return false;
-//     }
 
-       MatchStringAdv matchAdv(bv->buffer(), opt);
        try {
+               MatchStringAdv matchAdv(bv->buffer(), opt);
+               findAdvReplace(bv, opt, matchAdv);
+               cur = bv->cursor();
                if (opt.forward)
                                match_len = findForwardAdv(cur, matchAdv);
                else
                                match_len = findBackwardsAdv(cur, matchAdv);
        } catch (...) {
-               // This may only be raised by boost::regex()
+               // This may only be raised by lyx::regex()
                bv->message(_("Invalid regular expression!"));
                return false;
        }
@@ -1300,81 +1253,16 @@ bool findAdv(BufferView * bv, FindAndReplaceOptions const & opt)
                return false;
        }
 
-       LYXERR(Debug::FIND, "Putting selection at buf=" << matchAdv.p_buf
-               << "cur=" << cur << " with len: " << match_len);
+       bv->message(_("Match found!"));
 
-       lyx::dispatch(FuncRequest(LFUN_BUFFER_SWITCH,
-                                 matchAdv.p_buf->absFileName()));
-       bv = theApp()->currentWindow()->documentBufferView();
-
-       bv->putSelectionAt(cur, match_len, ! opt.forward);
-       if (opt.replace == docstring(from_utf8(LYX_FR_NULL_STRING))) {
-               bv->message(_("Match found !"));
-       } else {
-               string lyx = to_utf8(opt.replace);
-               // FIXME: Seems so stupid to me to rebuild a buffer here,
-               // when we already have one (replace_work_area_.buffer())
-               Buffer repl_buffer("", false);
-               repl_buffer.setUnnamed(true);
-               if (repl_buffer.readString(lyx)) {
-                       if (opt.keep_case && match_len >= 2) {
-                               if (cur.inTexted()) {
-                                       if (firstUppercase(cur))
-                                               changeFirstCase(repl_buffer, text_uppercase, text_lowercase);
-                                       else if (allNonLowercase(cur, match_len))
-                                               changeFirstCase(repl_buffer, text_uppercase, text_uppercase);
-                               }
-                       }
-                       cap::cutSelection(bv->cursor(), false, false);
-                       if (! cur.inMathed()) {
-                               LYXERR(Debug::FIND, "Replacing by pasteParagraphList()ing repl_buffer");
-                               cap::pasteParagraphList(bv->cursor(), repl_buffer.paragraphs(),
-                                                       repl_buffer.params().documentClassPtr(),
-                                                       bv->buffer().errorList("Paste"));
-                       } else {
-                               odocstringstream ods;
-                               OutputParams runparams(&repl_buffer.params().encoding());
-                               runparams.nice = false;
-                               runparams.flavor = OutputParams::LATEX;
-                               runparams.linelen = 8000; //lyxrc.plaintext_linelen;
-                               runparams.dryrun = true;
-                               TexRow texrow;
-                               TeXOnePar(repl_buffer, repl_buffer.text(), 
-                                         repl_buffer.paragraphs().begin(), ods, texrow, runparams);
-                               //repl_buffer.getSourceCode(ods, 0, repl_buffer.paragraphs().size(), false);
-                               docstring repl_latex = ods.str();
-                               LYXERR(Debug::FIND, "Latexified replace_buffer: '" << repl_latex << "'");
-                               string s;
-                               regex_replace(to_utf8(repl_latex), s, "\\$(.*)\\$", "$1");
-                               regex_replace(s, s, "\\\\\\[(.*)\\\\\\]", "$1");
-                               repl_latex = from_utf8(s);
-                               LYXERR(Debug::FIND, "Replacing by niceInsert()ing latex: '" << repl_latex << "'");
-                               bv->cursor().niceInsert(repl_latex);
-                       }
-                       bv->putSelectionAt(cur, repl_buffer.paragraphs().begin()->size(), ! opt.forward);
-                       bv->message(_("Match found and replaced !"));
-               } else
-                       LASSERT(false, /**/);
-               // dispatch(FuncRequest(LFUN_SELF_INSERT, opt.replace));
-       }
+       LYXERR(Debug::FIND, "Putting selection at cur=" << cur << " with len: " << match_len);
+       bv->putSelectionAt(cur, match_len, !opt.forward);
 
        return true;
 }
 
 
-void findAdv(BufferView * bv, FuncRequest const & ev)
-{
-       if (!bv || ev.action != LFUN_WORD_FINDADV)
-               return;
-
-       FindAndReplaceOptions opt;
-       istringstream iss(to_utf8(ev.argument()));
-       iss >> opt;
-       findAdv(bv, opt);
-}
-
-
-ostringstream & operator<<(ostringstream & os, lyx::FindAndReplaceOptions const & opt)
+ostringstream & operator<<(ostringstream & os, FindAndReplaceOptions const & opt)
 {
        os << to_utf8(opt.search) << "\nEOSS\n"
           << opt.casesensitive << ' '
@@ -1392,7 +1280,7 @@ ostringstream & operator<<(ostringstream & os, lyx::FindAndReplaceOptions const
        return os;
 }
 
-istringstream & operator>>(istringstream & is, lyx::FindAndReplaceOptions & opt)
+istringstream & operator>>(istringstream & is, FindAndReplaceOptions & opt)
 {
        LYXERR(Debug::FIND, "parsing");
        string s;