Hi Caolán, all,<div><br></div><div>Concerning your open question, the proposed patch checks if the searched string belongs to the selection by using the same mechanism as the common search command. Therefore it should have the same behaviour as the search command in both cases (common or advanced search/replace).</div>
<div><br></div><div>Best Regards,</div><div><br></div><div>--</div><div>Benoit<br><br><div class="gmail_quote">2011/7/2 Caolán McNamara <span dir="ltr">&lt;<a href="mailto:caolanm@redhat.com">caolanm@redhat.com</a>&gt;</span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On Fri, 2011-07-01 at 01:08 +0200, Benoit Dolives wrote:<br>
&gt; Hi,<br>
&gt;<br>
&gt;<br>
&gt; Please find in attachment the patch for the bug 36181.<br>
<br>
</div>initially looks good, unless someone else has a look before Monday I&#39;ll<br>
take this one for review then. Only open question I have, without<br>
reading the bug report or building sw with the patch, is if it<br>
does-the-right-thing if the search/replace is one of the regular<br>
expression &quot;advanced&quot; search/replaces ?<br>
<font color="#888888"><br>
C.<br>
<br>
<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Benoit Dolives<br>
</div>