Jean-Marc Lasgouttes wrote:
> I like it. You might want to rename gotoNextInset to findNextInset (or
> find a better name).

findNextInset looks good to me. I have renamed that.

> Did you test that the searching works even in corner cases (inset is
> the first thing in a document; cursor is at beginning of document;
> wrap-around search)? The original code worked, but I may have broken
> it.

I have already tested those cases, and they still worked. I'll test again over 
the weekend and apply the stuff if there are no problems

Thanks,
Jürgen

Reply via email to