]> git.lyx.org Git - lyx.git/blobdiff - src/lyxfind.cpp
Integrate texrow with otexstream in order to perform automatic line
[lyx.git] / src / lyxfind.cpp
index 987166411b0cd79fc9a18890f8b81857162d9866..409a2e2cf7f2a1bf8d24fbf5725d111dd2311a0a 100644 (file)
@@ -33,6 +33,7 @@
 #include "TexRow.h"
 #include "Text.h"
 
+#include "frontends/Application.h"
 #include "frontends/alert.h"
 
 #include "mathed/InsetMath.h"
@@ -47,7 +48,7 @@
 #include "support/lassert.h"
 #include "support/lstrings.h"
 
-#include <boost/regex.hpp>
+#include "support/regex.h"
 #include <boost/next_prior.hpp>
 
 using namespace std;
@@ -67,51 +68,55 @@ bool parse_bool(docstring & howto)
 }
 
 
-class MatchString : public binary_function<Paragraph, pos_type, bool>
+class MatchString : public binary_function<Paragraph, pos_type, int>
 {
 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
+       int 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;
 };
 
 
-bool findForward(DocIterator & cur, MatchString const & match,
+int findForward(DocIterator & cur, MatchString const & match,
                 bool find_del = true)
 {
        for (; cur; cur.forwardChar())
-               if (cur.inTexted() &&
-                   match(cur.paragraph(), cur.pos(), find_del))
-                       return true;
-       return false;
+               if (cur.inTexted()) {
+                       int len = match(cur.paragraph(), cur.pos(), find_del);
+                       if (len > 0)
+                               return len;
+               }
+       return 0;
 }
 
 
-bool findBackwards(DocIterator & cur, MatchString const & match,
+int findBackwards(DocIterator & cur, MatchString const & match,
                 bool find_del = true)
 {
        while (cur) {
                cur.backwardChar();
-               if (cur.inTexted() &&
-                   match(cur.paragraph(), cur.pos(), find_del))
-                       return true;
+               if (cur.inTexted()) {
+                       int len = match(cur.paragraph(), cur.pos(), find_del);
+                       if (len > 0)
+                               return len;
+               }
        }
-       return false;
+       return 0;
 }
 
 
@@ -132,7 +137,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"));
@@ -142,38 +147,38 @@ 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) :
+       int match_len = forward ? findForward(cur, match, find_del) :
                          findBackwards(cur, match, find_del);
 
-       if (found)
-               bv->putSelectionAt(cur, searchstr.length(), !fw);
+       if (match_len > 0)
+               bv->putSelectionAt(cur, match_len, !forward);
 
-       return found;
+       return match_len > 0;
 }
 
 
 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;
 
        DocIterator cur_orig(bv->cursor());
 
-       MatchString const match(searchstr, cs, mw);
+       MatchString const match(searchstr, case_sens, whole);
        int num = 0;
 
        int const rsize = replacestr.size();
@@ -181,12 +186,13 @@ int replaceAll(BufferView * bv,
 
        Cursor cur(*bv);
        cur.setCursor(doc_iterator_begin(&buf));
-       while (findForward(cur, match, false)) {
+       int match_len = findForward(cur, match, false);
+       while (match_len > 0) {
                // Backup current cursor position and font.
                pos_type const pos = cur.pos();
                Font const font = cur.paragraph().getFontSettings(buf.params(), pos);
                cur.recordUndo();
-               int striked = ssize - cur.paragraph().eraseChars(pos, pos + ssize,
+               int striked = ssize - cur.paragraph().eraseChars(pos, pos + match_len,
                                                            buf.params().trackChanges);
                cur.paragraph().insert(pos, replacestr, font,
                                       Change(buf.params().trackChanges ?
@@ -194,12 +200,10 @@ int replaceAll(BufferView * bv,
                for (int i = 0; i < rsize + striked; ++i)
                        cur.forwardChar();
                ++num;
+               match_len = findForward(cur, match, false);
        }
 
-       buf.updateBuffer();
        bv->putSelectionAt(doc_iterator_begin(&buf), 0, false);
-       if (num)
-               buf.markDirty();
 
        cur_orig.fixIfBroken();
        bv->setCursor(cur_orig);
@@ -208,49 +212,75 @@ int replaceAll(BufferView * bv,
 }
 
 
-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,
-       // but only if we are in texted() mode.
-       if (!bv->cursor().selection() && searchstr.empty()
-           && bv->cursor().inTexted()) {
-               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<bool, int>(false, 0);
+               // 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);
-       bv->buffer().markDirty();
-       find(bv, searchstr, cs, mw, fw, false);
-       bv->buffer().updateMacros();
-       bv->processUpdateFlags(Update::Force | Update::FitCursor);
+       cap::replaceSelectionWithString(cur, replacestr, forward);
+       if (forward) {
+               cur.pos() += replacestr.length();
+               LASSERT(cur.pos() <= cur.lastpos(), /* */);
+       } else {
+               cur.pos() -= replacestr.length();
+               LASSERT(cur.pos() >= 0, /* */);
+       }
+       findOne(bv, searchstr, case_sens, whole, forward, false);
 
-       return 1;
+       return pair<bool, int>(true, 1);
 }
 
 } // namespace anon
@@ -283,9 +313,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;
@@ -300,14 +330,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>
@@ -323,34 +354,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;
 }
 
 
@@ -371,7 +412,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();
        }
 
@@ -559,14 +600,14 @@ string escape_for_regex(string s)
        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();
@@ -661,25 +702,89 @@ 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
        // environments, math mode, styles, etc...
        int close_wildcards;
+       // Are we searching with regular expressions ?
+       bool use_regexp;
 };
 
 
+static docstring buffer_to_latex(Buffer & buffer) 
+{
+       OutputParams runparams(&buffer.params().encoding());
+       TexRow texrow;
+       odocstringstream ods;
+       otexstream os(ods, texrow);
+       runparams.nice = true;
+       runparams.flavor = OutputParams::LATEX;
+       runparams.linelen = 80; //lyxrc.plaintext_linelen;
+       // No side effect of file copying and image conversion
+       runparams.dryrun = true;
+       pit_type const endpit = buffer.paragraphs().size();
+       for (pit_type pit = 0; pit != endpit; ++pit) {
+               TeXOnePar(buffer, buffer.text(), pit, os, runparams);
+               LYXERR(Debug::FIND, "searchString up to here: " << ods.str());
+       }
+       return ods.str();
+}
+
+
+static docstring stringifySearchBuffer(Buffer & buffer, FindAndReplaceOptions const & opt) {
+       docstring str;
+       if (!opt.ignoreformat) {
+               str = buffer_to_latex(buffer);
+       } else {
+               ParIterator it = buffer.par_iterator_begin();
+               ParIterator end = buffer.par_iterator_end();
+               OutputParams runparams(&buffer.params().encoding());
+               odocstringstream os;
+               runparams.nice = true;
+               runparams.flavor = OutputParams::LATEX;
+               runparams.linelen = 100000; //lyxrc.plaintext_linelen;
+               runparams.dryrun = true;
+               for (; it != end; ++it) {
+                       LYXERR(Debug::FIND, "Adding to search string: '"
+                               << it->asString(false)
+                               << "'");
+                       str +=
+                               it->stringify(pos_type(0), it->size(),
+                                             AS_STR_INSETS, runparams);
+               }
+       }
+       return str;
+}
+
+
+/// Return separation pos between the leading material and the rest
+static size_t identifyLeading(string const & s)  {
+       string t = s;
+       // @TODO Support \item[text]
+       while (regex_replace(t, t, "\\\\(emph|textbf|subsubsection|subsection|section|subparagraph|paragraph|part)\\{", "")
+              || regex_replace(t, t, "^\\$", "")
+              || regex_replace(t, t, "^\\\\\\[ ", "")
+              || regex_replace(t, t, "^\\\\item ", ""))
+               LYXERR(Debug::FIND, "  after removing leading $, \\[ , \\emph{, \\textbf{, etc.: " << t);
+       return s.find(t);
+}
+
+
 MatchStringAdv::MatchStringAdv(lyx::Buffer & buf, FindAndReplaceOptions const & opt)
        : p_buf(&buf), p_first_buf(&buf), opt(opt)
 {
-       par_as_string = normalize(opt.search);
+       Buffer & find_buf = *theBufferList().getBuffer(FileName(to_utf8(opt.find_buf_name)), true);
+       par_as_string = normalize(stringifySearchBuffer(find_buf, opt));
        open_braces = 0;
        close_wildcards = 0;
 
-       if (! opt.regexp) {
+       use_regexp = !opt.ignoreformat || par_as_string.find("\\regexp") != std::string::npos;
+
+       if (!use_regexp) {
                // Remove trailing closure of math, macros and environments, so to catch parts of them.
                do {
                        LYXERR(Debug::FIND, "par_as_string now is '" << par_as_string << "'");
@@ -701,6 +806,14 @@ MatchStringAdv::MatchStringAdv(lyx::Buffer & buf, FindAndReplaceOptions const &
                LYXERR(Debug::FIND, "Open braces: " << open_braces);
                LYXERR(Debug::FIND, "Built MatchStringAdv object: par_as_string = '" << par_as_string << "'");
        } else {
+               size_t lead_size = identifyLeading(par_as_string);
+               string lead_as_regexp;
+               if (lead_size > 0) {
+                       lead_as_regexp = escape_for_regex(par_as_string.substr(0, lead_size));
+                       par_as_string = par_as_string.substr(lead_size, par_as_string.size() - lead_size);
+                       LYXERR(Debug::FIND, "lead_as_regexp is '" << lead_as_regexp << "'");
+                       LYXERR(Debug::FIND, "par_as_string now is '" << par_as_string << "'");
+               }
                par_as_string = escape_for_regex(par_as_string);
                // Insert (.*?) before trailing closure of math, macros and environments, so to catch parts of them.
                LYXERR(Debug::FIND, "par_as_string now is '" << par_as_string << "'");
@@ -722,9 +835,14 @@ MatchStringAdv::MatchStringAdv(lyx::Buffer & buf, FindAndReplaceOptions const &
                LYXERR(Debug::FIND, "Close .*?  : " << close_wildcards);
                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);
+               string regexp_str = string("\\`") + lead_as_regexp + par_as_string;
+               LYXERR(Debug::FIND, "Setting regexp to : " << regexp_str << endl);
+               regexp = lyx::regex(regexp_str);
+
                // If entered regexp may match wherever in searched string buffer
-               regexp2 = boost::regex(string("\\`.*") + par_as_string);
+               string regexp2_str = string("\\`.*") + lead_as_regexp + ".*" + par_as_string;
+               LYXERR(Debug::FIND, "Setting regexp2 to: " << regexp2_str << endl);
+               regexp2 = lyx::regex(regexp2_str);
        }
 }
 
@@ -735,30 +853,38 @@ int MatchStringAdv::findAux(DocIterator const & cur, int len, bool at_begin) con
        LYXERR(Debug::FIND, "Matching against     '" << lyx::to_utf8(docstr) << "'");
        string str = normalize(docstr);
        LYXERR(Debug::FIND, "After normalization: '" << str << "'");
-       if (! opt.regexp) {
+       if (! use_regexp) {
+               LYXERR(Debug::FIND, "Searching in normal mode: par_as_string='" << par_as_string << "', str='" << str << "'");
                if (at_begin) {
+                       LYXERR(Debug::FIND, "size=" << par_as_string.size() << ", substr='" << str.substr(0, par_as_string.size()) << "'");
                        if (str.substr(0, par_as_string.size()) == par_as_string)
                                return par_as_string.size();
                } else {
-                       size_t pos = str.find(par_as_string);
+                       string t = par_as_string;
+                       while (regex_replace(t, t, "\\\\(emph|textbf|subsubsection|subsection|section|subparagraph|paragraph|part)\\{", "")
+                              || regex_replace(t, t, "^\\$", "")
+                              || regex_replace(t, t, "^\\\\\\[ ", ""))
+                               LYXERR(Debug::FIND, "  after removing leading $, \\[ , \\emph{, \\textbf{, etc.: " << t);
+                       size_t pos = str.find(t);
                        if (pos != string::npos)
                                return par_as_string.size();
                }
        } else {
+               LYXERR(Debug::FIND, "Searching in regexp mode: at_begin=" << at_begin);
                // 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))
+                       if (!braces_match(m[0].first, m[0].second, open_braces))
                                return 0;
                        // Check braces on segments that matched all (.*?) subexpressions.
                        for (size_t i = 1; i < m.size(); ++i)
-                               if (! braces_match(m[i].first, m[i].second))
+                               if (!braces_match(m[i].first, m[i].second))
                                        return false;
                        // Exclude from the returned match length any length 
                        // due to close wildcards added at end of regexp
@@ -775,6 +901,8 @@ int MatchStringAdv::findAux(DocIterator const & cur, int len, bool at_begin) con
 int MatchStringAdv::operator()(DocIterator const & cur, int len, bool at_begin) const
 {
        int res = findAux(cur, len, at_begin);
+       LYXERR(Debug::FIND,
+              "res=" << res << ", at_begin=" << at_begin << ", matchword=" << opt.matchword << ", inTexted=" << cur.inTexted());
        if (res == 0 || !at_begin || !opt.matchword || !cur.inTexted())
                return res;
        Paragraph const & par = cur.paragraph();
@@ -811,7 +939,7 @@ string MatchStringAdv::normalize(docstring const & s) const
                t.replace(pos, 1, " ");
        // Remove stale empty \emph{}, \textbf{} and similar blocks from latexify
        LYXERR(Debug::FIND, "Removing stale empty \\emph{}, \\textbf{}, \\*section{} macros from: " << t);
-       while (regex_replace(t, t, "\\\\(emph|textbf|subsubsection|subsection|section|subparagraph|paragraph)(\\{\\})+", ""))
+       while (regex_replace(t, t, "\\\\(emph|textbf|subsubsection|subsection|section|subparagraph|paragraph|part)(\\{\\})+", ""))
                LYXERR(Debug::FIND, "  further removing stale empty \\emph{}, \\textbf{} macros from: " << t);
        return t;
 }
@@ -866,6 +994,7 @@ docstring latexifyFromCursor(DocIterator const & cur, int len)
 
        TexRow texrow;
        odocstringstream ods;
+       otexstream os(ods, texrow);
        OutputParams runparams(&buf.params().encoding());
        runparams.nice = false;
        runparams.flavor = OutputParams::LATEX;
@@ -874,14 +1003,12 @@ docstring latexifyFromCursor(DocIterator const & cur, int len)
        runparams.dryrun = true;
 
        if (cur.inTexted()) {
-                       // @TODO what about searching beyond/across paragraph breaks ?
-                       ParagraphList::const_iterator pit = cur.innerText()->paragraphs().begin();
-                       for (int i = 0; i < cur.pit(); ++i)
-                                       ++pit;
-               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(),
-                       cur.pos(), endpos);
+               // @TODO what about searching beyond/across paragraph breaks ?
+               pos_type endpos = cur.paragraph().size();
+               if (len != -1 && endpos > cur.pos() + len)
+                       endpos = cur.pos() + len;
+               TeXOnePar(buf, *cur.innerText(), cur.pit(), os, runparams,
+                       string(), cur.pos(), endpos);
                LYXERR(Debug::FIND, "Latexified text: '" << lyx::to_utf8(ods.str()) << "'");
        } else if (cur.inMathed()) {
                // Retrieve the math environment type, and add '$' or '$[' or others (\begin{equation}) accordingly
@@ -967,27 +1094,42 @@ int findForwardAdv(DocIterator & cur, MatchStringAdv & match)
 {
        if (!cur)
                return 0;
-       while (cur && !match(cur, -1, false)) {
-               if (cur.pit() < cur.lastpit())
+       while (cur) {
+               LYXERR(Debug::FIND, "findForwardAdv() cur: " << cur);
+               if (match(cur, -1, false)) {
+                       for (; cur; cur.forwardPos()) {
+                               LYXERR(Debug::FIND, "Advancing cur: " << cur);
+                               if (match(cur)) {
+                                       // Sometimes in finalize we understand it wasn't a match
+                                       // and we need to continue the outest loop
+                                       int len = findAdvFinalize(cur, match);
+                                       if (len > 0)
+                                               return len;
+                               }
+                       }
+                       if (!cur)
+                               return 0;
+               }
+               if (cur.pit() < cur.lastpit()) {
+                       LYXERR(Debug::FIND, "Advancing par: cur=" << cur);
                        cur.forwardPar();
-               else {
+               } else {
+                       // This should exit nested insets, if any, or otherwise undefine the currsor.
+                       cur.pos() = cur.lastpos();
+                       LYXERR(Debug::FIND, "Advancing pos: cur=" << cur);
                        cur.forwardPos();
                }
        }
-       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());
        DocIterator tmp_cur = cur;
-       len = findAdvFinalize(tmp_cur, match);
+       int len = findAdvFinalize(tmp_cur, match);
        Inset & inset = cur.inset();
        for (; cur != cur_begin; cur.backwardPos()) {
                LYXERR(Debug::FIND, "findMostBackwards(): cur=" << cur);
@@ -1001,6 +1143,7 @@ void findMostBackwards(DocIterator & cur, MatchStringAdv const & match, int & le
                len = new_len;
        }
        LYXERR(Debug::FIND, "findMostBackwards(): exiting with cur=" << cur);
+       return len;
 }
 
 
@@ -1032,11 +1175,9 @@ int findBackwardsAdv(DocIterator & cur, MatchStringAdv & match) {
                                found_match = match(cur);
                                LYXERR(Debug::FIND, "findBackAdv3: found_match=" 
                                       << found_match << ", cur: " << cur);
-                               if (found_match) {
-                                       int len;
-                                       findMostBackwards(cur, match, len);
-                                       return len;
-                               }
+                               if (found_match)
+                                       return findMostBackwards(cur, match);
+
                                // Stop if begin of document reached
                                if (cur == cur_begin)
                                        break;
@@ -1062,6 +1203,7 @@ int findBackwardsAdv(DocIterator & cur, MatchStringAdv & match) {
 docstring stringifyFromForSearch(FindAndReplaceOptions const & opt,
        DocIterator const & cur, int len)
 {
+       LASSERT(cur.pos() >= 0 && cur.pos() <= cur.lastpos(), /* */);
        if (!opt.ignoreformat)
                return latexifyFromCursor(cur, len);
        else
@@ -1069,27 +1211,18 @@ docstring stringifyFromForSearch(FindAndReplaceOptions const & opt,
 }
 
 
-FindAndReplaceOptions::FindAndReplaceOptions(docstring const & search, bool casesensitive,
+FindAndReplaceOptions::FindAndReplaceOptions(docstring const & find_buf_name, bool casesensitive,
        bool matchword, bool forward, bool expandmacros, bool ignoreformat,
-       bool regexp, docstring const & replace, bool keep_case,
+       docstring const & repl_buf_name, bool keep_case,
        SearchScope scope)
-       : search(search), casesensitive(casesensitive), matchword(matchword),
+       : find_buf_name(find_buf_name), casesensitive(casesensitive), matchword(matchword),
        forward(forward), expandmacros(expandmacros), ignoreformat(ignoreformat),
-       regexp(regexp), replace(replace), keep_case(keep_case), scope(scope)
+       repl_buf_name(repl_buf_name), keep_case(keep_case), scope(scope)
 {
 }
 
 
-/** 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 */
@@ -1131,14 +1264,15 @@ 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)))
+       if (opt.repl_buf_name == docstring())
                return;
+
        DocIterator sel_beg = cur.selectionBegin();
        DocIterator sel_end = cur.selectionEnd();
        if (&sel_beg.inset() != &sel_end.inset()
@@ -1154,15 +1288,14 @@ static void findAdvReplace(BufferView * bv, FindAndReplaceOptions const & opt, M
        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())
+       // Build a copy of the replace buffer, adapted to the KeepCase option
+       Buffer & repl_buffer_orig = *theBufferList().getBuffer(FileName(to_utf8(opt.repl_buf_name)), true);
+       ostringstream oss;
+       repl_buffer_orig.write(oss);
+       string lyx = oss.str();
        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))
@@ -1173,20 +1306,26 @@ static void findAdvReplace(BufferView * bv, FindAndReplaceOptions const & opt, M
        }
        cap::cutSelection(cur, false, false);
        if (!cur.inMathed()) {
+               repl_buffer.changeLanguage(
+                       repl_buffer.language(),
+                       cur.getFont().language());
                LYXERR(Debug::FIND, "Replacing by pasteParagraphList()ing repl_buffer");
+               LYXERR(Debug::FIND, "Before pasteParagraphList() cur=" << cur << endl);
                cap::pasteParagraphList(cur, repl_buffer.paragraphs(),
                                        repl_buffer.params().documentClassPtr(),
                                        bv->buffer().errorList("Paste"));
+               LYXERR(Debug::FIND, "After pasteParagraphList() cur=" << cur << endl);
+               sel_len = repl_buffer.paragraphs().begin()->size();
        } else {
+               TexRow texrow;
                odocstringstream ods;
+               otexstream os(ods, texrow);
                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);
+               TeXOnePar(repl_buffer, repl_buffer.text(), 0, os, runparams);
                //repl_buffer.getSourceCode(ods, 0, repl_buffer.paragraphs().size(), false);
                docstring repl_latex = ods.str();
                LYXERR(Debug::FIND, "Latexified replace_buffer: '" << repl_latex << "'");
@@ -1195,11 +1334,14 @@ static void findAdvReplace(BufferView * bv, FindAndReplaceOptions const & opt, M
                regex_replace(s, s, "\\\\\\[(.*)\\\\\\]", "$1");
                repl_latex = from_utf8(s);
                LYXERR(Debug::FIND, "Replacing by niceInsert()ing latex: '" << repl_latex << "'");
-               cur.niceInsert(repl_latex);
+               sel_len = 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);
+       cur.pos() -= sel_len;
+       if (cur.pos() < 0)
+               cur.pos() = 0;
+       LYXERR(Debug::FIND, "Putting selection at cur=" << cur << " with len: " << sel_len);
+       bv->putSelectionAt(DocIterator(cur), sel_len, !opt.forward);
+       bv->processUpdateFlags(Update::Force);
 }
 
 
@@ -1207,12 +1349,7 @@ static void findAdvReplace(BufferView * bv, FindAndReplaceOptions const & opt, M
 bool findAdv(BufferView * bv, FindAndReplaceOptions const & opt)
 {
        DocIterator cur;
-       int match_len;
-
-       if (opt.search.empty()) {
-               bv->message(_("Search text is empty!"));
-               return false;
-       }
+       int match_len = 0;
 
        try {
                MatchStringAdv matchAdv(bv->buffer(), opt);
@@ -1223,7 +1360,7 @@ bool findAdv(BufferView * bv, FindAndReplaceOptions const & opt)
                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;
        }
@@ -1244,14 +1381,13 @@ bool findAdv(BufferView * bv, FindAndReplaceOptions const & opt)
 
 ostringstream & operator<<(ostringstream & os, FindAndReplaceOptions const & opt)
 {
-       os << to_utf8(opt.search) << "\nEOSS\n"
+       os << to_utf8(opt.find_buf_name) << "\nEOSS\n"
           << opt.casesensitive << ' '
           << opt.matchword << ' '
           << opt.forward << ' '
           << opt.expandmacros << ' '
           << opt.ignoreformat << ' '
-          << opt.regexp << ' '
-          << to_utf8(opt.replace) << "\nEOSS\n"
+          << to_utf8(opt.repl_buf_name) << "\nEOSS\n"
           << opt.keep_case << ' '
           << int(opt.scope);
 
@@ -1260,6 +1396,7 @@ ostringstream & operator<<(ostringstream & os, FindAndReplaceOptions const & opt
        return os;
 }
 
+
 istringstream & operator>>(istringstream & is, FindAndReplaceOptions & opt)
 {
        LYXERR(Debug::FIND, "parsing");
@@ -1274,9 +1411,9 @@ istringstream & operator>>(istringstream & is, FindAndReplaceOptions & opt)
                                break;
                getline(is, line);
        }
-       LYXERR(Debug::FIND, "searching for: '" << s << "'");
-       opt.search = from_utf8(s);
-       is >> opt.casesensitive >> opt.matchword >> opt.forward >> opt.expandmacros >> opt.ignoreformat >> opt.regexp;
+       LYXERR(Debug::FIND, "file_buf_name: '" << s << "'");
+       opt.find_buf_name = from_utf8(s);
+       is >> opt.casesensitive >> opt.matchword >> opt.forward >> opt.expandmacros >> opt.ignoreformat;
        is.get();       // Waste space before replace string
        s = "";
        getline(is, line);
@@ -1288,14 +1425,14 @@ istringstream & operator>>(istringstream & is, FindAndReplaceOptions & opt)
                                break;
                getline(is, line);
        }
+       LYXERR(Debug::FIND, "repl_buf_name: '" << s << "'");
+       opt.repl_buf_name = from_utf8(s);
        is >> opt.keep_case;
        int i;
        is >> i;
        opt.scope = FindAndReplaceOptions::SearchScope(i);
        LYXERR(Debug::FIND, "parsed: " << opt.casesensitive << ' ' << opt.matchword << ' ' << opt.forward << ' '
-                  << opt.expandmacros << ' ' << opt.ignoreformat << ' ' << opt.regexp << ' ' << opt.keep_case);
-       LYXERR(Debug::FIND, "replacing with: '" << s << "'");
-       opt.replace = from_utf8(s);
+                  << opt.expandmacros << ' ' << opt.ignoreformat << ' ' << opt.keep_case);
        return is;
 }