On Fri, Sep 16, 2005 at 12:46:12PM +0200, Juergen Spitzmueller wrote:
> Martin,
> what is this fix for bug 1981 supposed to do?
>       case LFUN_CUT:
> +             cur.pos() = 0; // Prevent stale position >= size crash
>               cutSelection(cur, true, true);
>               cur.message(_("Cut"));
>               break;
> 
> It is the cause of a rather silly behaviour: when you cut a selected string 
> from a math nestinset, not only the selection, but also everything preceeding 
> it will be cut.
> 
> IMO (and I'm pretty sure) the correct fix is to use cur.normalize() instead, 
> which fixes the cursor position if it is outside the size() range. I have 
> tested that it fixes both bug 1981 and bug 1960, which seem to share the 
> cause.
> 
> Can this go in?

It's the proper fix.

Andre'

Reply via email to