>>>>> "Jürgen" == Jürgen Spitzmüller <[EMAIL PROTECTED]> writes:

Jürgen> A missing recordUndo call und some reordering were needed.
Jürgen> I'll commit later today if I get no objections.

+                       if (!cur.selection())
+                               recordUndo(cur);
+                       cap::replaceSelection(cur);

Why the test for selection? Does that avoid a double undo? It stinks
that we have to do that. I do not know what the proper semantics
should be (do not handle undo at all?) though. Could you add a FIXME
explainign the issue?

Jürgen> Jean-Marc, this is also something for branch.

Yes.

JMarc

Reply via email to