Alfredo Braunstein wrote:
> This patch looks fine.
committed now.
Jürgen
Jürgen Spitzmüller wrote:
> Jürgen Spitzmüller wrote:
>> On a second thought, you might be right. The attached patch does what you
>> propose and works for me.
>>
>> OK to apply?
>
> This patch also fixes bug 3160:
> http://bugzilla.lyx.org/show_bug.cgi?id=3160
This patch looks fine.
Regards,
Jürgen Spitzmüller wrote:
> On a second thought, you might be right. The attached patch does what you
> propose and works for me.
>
> OK to apply?
This patch also fixes bug 3160:
http://bugzilla.lyx.org/show_bug.cgi?id=3160
Jürgen
Michael Gerz wrote:
> Maybe I was talking nonsense. I just looked at the patch (without the
> surrounding code). It looked like you check for the first character and
> then match(...) tries to match the complete string.
On a second thought, you might be right. The attached patch does what you
pro
Jürgen Spitzmüller schrieb:
Michael Gerz wrote:
for (; cur; cur.forwardChar())
- if (cur.inTexted() && match(cur.paragraph(), cur.pos()))
+ if (cur.inTexted() &&
+ (find_del || !cur.paragraph().isDeleted(cur.pos())) &&
+ match(cur.
Michael Gerz wrote:
> > for (; cur; cur.forwardChar())
> > - if (cur.inTexted() && match(cur.paragraph(), cur.pos()))
> > + if (cur.inTexted() &&
> > + (find_del || !cur.paragraph().isDeleted(cur.pos())) &&
> > + match(cur.paragraph(), c
Jürgen,
your patch looks nice in general. I can imagine that it even fixes #3160. Below
please find some additional comments.
Michael
Index: src/lyxfind.cpp
===
--- src/lyxfind.cpp (Revision 18711)
+++ src/lyxfind.cpp (Ar