Andre Poenitz <[EMAIL PROTECTED]> writes:

| This makes undo partially work. It crashes always when used in the last
| par and after "complicated" operations, but it works if used in the middle
| of the text for simple insertions/deletions, sometimes even multi-par
| stuff. 
| 
| In any case, this is somewhat beter than what we have now.

Good... and long as it gets better we should push on.
 
| Note that I had to implement Paragraph::operator=() (Lars, you are a
| naughty boy...).

I had it implemented (with swap), but that was before the clone
fixes... so it did not really compile :-/

| Replace it with something closer to your taste if you
| want. The swap idiom is a bit more work here because the inset owners
| need to be set..

Not if a proper operator= is implemented for InsetList.
(a setOwner will still be needed though).
 
| Next change i the return type of limited_stack::top(). It's (a) used this
| way and (b) consistent with std::stack<> now.

ok.
 
| The rest is undo specific stuff, some simplification of the logic and some
| code to 'make it work'.
| 
| Ok?

Yes.

-- 
        Lgb

Reply via email to