On Fri, 2005-09-16 at 12:46 +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?
> 
> Jürgen

OK, this looks better.

- Martin

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to