Il 26/05/2011 19:48, Richard Heck ha scritto:
On 05/26/2011 01:46 PM, Richard Heck wrote:
On 05/26/2011 01:08 PM, tomm...@lyx.org wrote:
Author: tommaso
Date: Thu May 26 19:08:48 2011
New Revision: 38846
URL: http://www.lyx.org/trac/changeset/38846

Log:
Bugfix in how the search buffer was exported for the Advanced Find&  Replace.
For example, before this footnotes were not correctly found.
Added also accompanying regression test.

Modified: lyx-devel/trunk/src/lyxfind.cpp
==============================================================================
--- lyx-devel/trunk/src/lyxfind.cpp     Thu May 26 18:54:51 2011        (r38845)
+++ lyx-devel/trunk/src/lyxfind.cpp     Thu May 26 19:08:48 2011        (r38846)
@@ -743,21 +743,19 @@
        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) {
+               for (pos_type pit = pos_type(0); pit<  
(pos_type)buffer.paragraphs().size(); ++pit) {

This change doesn't look necessary for the rest. Using the ParIterator
is a bit more elegant....

And it looks OK for branch to me.

r38850. Please, update satus.20x with these few commits I did in these days.

    T.

Reply via email to