]> git.lyx.org Git - lyx.git/blobdiff - src/undo_funcs.C
most of textHandleUndo
[lyx.git] / src / undo_funcs.C
index dfd8d81c467de4bf8e1447a5500d793685b4d744..f820005ebcaa49fba85ae0123657943c0c8d8413 100644 (file)
 /* This file is part of
- * ====================================================== 
- * 
+ * ======================================================
+ *
  *           LyX, The Document Processor
- *        
+ *
  *           Copyright 1995-2001 The LyX Team.
  *
  * ====================================================== */
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
 #include "undo_funcs.h"
 #include "lyxtext.h"
+#include "funcrequest.h"
 #include "BufferView.h"
 #include "buffer.h"
-#include "insets/inset.h"
+#include "insets/updatableinset.h"
+#include "insets/insettext.h"
 #include "debug.h"
 #include "support/LAssert.h"
-
 #include "iterators.h"
 
-//#define DELETE_UNUSED_PARAGRAPHS 1
-#ifdef DELETE_UNUSED_PARAGRAPHS
 #include <vector>
-#endif
 
 using std::vector;
+using boost::shared_ptr;
 
 
-/// the flag used by FinishUndo();
+/// The flag used by FinishUndo().
 bool undo_finished;
-/// a flag
+/// Whether actions are not added to the undo stacks.
 bool undo_frozen;
 
+namespace {
 
-bool textUndo(BufferView * bv)
+/// Utility to return the cursor.
+LyXCursor const & undoCursor(BufferView * bv)
 {
-       // returns false if no undo possible
-       Undo * undo = bv->buffer()->undostack.top();
-       bv->buffer()->undostack.pop();
-       if (undo) {
-               finishUndo();
-               if (!undo_frozen) {
-                       Paragraph * first = bv->buffer()->getParFromID(undo->number_of_before_par);
-                       if (first && first->next())
-                               first = first->next();
-                       else if (!first)
-                               first = firstUndoParagraph(bv, undo->number_of_inset_id);
-                       if (first) {
-                               bv->buffer()->redostack.push(
-                                       createUndo(bv, undo->kind, first,
-                                                  bv->buffer()->getParFromID(undo->number_of_behind_par)));
-                       }
-               }
-       }
-       // now we can unlock the inset for saftey because the inset pointer could
-       // be changed during the undo-function. Anyway if needed we have to lock
-       // the right inset/position if this is requested.
-       freezeUndo();
-       bv->unlockInset(bv->theLockingInset());
-       bool ret = textHandleUndo(bv, undo);
-       unFreezeUndo();
-       return ret;
+       if (bv->theLockingInset())
+               return bv->theLockingInset()->cursor(bv);
+       return bv->text->cursor;
 }
 
 
-bool textRedo(BufferView * bv)
+/**
+ * Returns a pointer to the very first Paragraph depending of where
+ * we are so it will return the first paragraph of the buffer or the
+ * first paragraph of the textinset we're in.
+ */
+ParagraphList * undoParagraphs(BufferView * bv, int inset_id)
 {
-       // returns false if no redo possible
-       Undo * undo = bv->buffer()->redostack.top();
-       bv->buffer()->redostack.pop();
-       if (undo) {
-               finishUndo();
-               if (!undo_frozen) {
-                       Paragraph * first = bv->buffer()->getParFromID(undo->number_of_before_par);
-                       if (first && first->next())
-                               first = first->next();
-                       else if (!first)
-                               first = firstUndoParagraph(bv, undo->number_of_inset_id);
-                       if (first) {
-                               bv->buffer()->undostack.push(
-                                       createUndo(bv, undo->kind, first,
-                                                  bv->buffer()->getParFromID(undo->number_of_behind_par)));
-                       }
-               }
+       Inset * inset = bv->buffer()->getInsetFromID(inset_id);
+       if (inset) {
+               ParagraphList * result = inset->getParagraphs(0);
+               if (result && !result->empty())
+                       return result;
        }
-       // now we can unlock the inset for saftey because the inset pointer could
-       // be changed during the undo-function. Anyway if needed we have to lock
-       // the right inset/position if this is requested.
+       return &bv->text->ownerParagraphs();
+}
+
+
+/**
+ * Finish the undo operation in the case there was no entry
+ * on the stack to perform.
+ */
+void finishNoUndo(BufferView * bv)
+{
        freezeUndo();
        bv->unlockInset(bv->theLockingInset());
-       bool ret = textHandleUndo(bv, undo);
+       finishUndo();
+       bv->text->postPaint(0);
        unFreezeUndo();
-       return ret;
 }
 
 
-bool textHandleUndo(BufferView * bv, Undo * undo)
+// Returns false if no undo possible.
+bool textHandleUndo(BufferView * bv, Undo & undo)
 {
-       // returns false if no undo possible
-       bool result = false;
-       if (undo) {
-               Paragraph * before =
-                       bv->buffer()->getParFromID(undo->number_of_before_par); 
-               Paragraph * behind =
-                       bv->buffer()->getParFromID(undo->number_of_behind_par); 
-               Paragraph * tmppar;
-               Paragraph * tmppar2;
-               Paragraph * tmppar5;
-
-               // if there's no before take the beginning
-               // of the document for redoing
-               if (!before) {
-                       LyXText * t = bv->text;
-                       int num = undo->number_of_inset_id;
-                       if (undo->number_of_inset_id >= 0) {
-                               Inset * in = bv->buffer()->getInsetFromID(num);
-                               if (in) {
-                                       t = in->getLyXText(bv);
-                               } else {
-                                       num = -1;
-                               }
-                       }
-                       t->setCursorIntern(bv, firstUndoParagraph(bv, num), 0);
-               }
+       Buffer * b = bv->buffer();
+
+       ParIterator const before = b->getParFromID(undo.number_of_before_par);
+       ParIterator const behind = b->getParFromID(undo.number_of_behind_par);
+       ParIterator const null   = b->par_iterator_end();
+
+       int const before_id = (before == null) ? -1 : before->id();
+       int const behind_id = (behind == null) ? -1 : behind->id();
+       int const inset_id  = undo.number_of_inset_id;
+
+       ParagraphList * plist = undoParagraphs(bv, inset_id);
+
+       ParagraphList::iterator first;
+       if (before == null) {
+               // if there's no before take the beginning of parlist.
+               first = plist->begin();
+               LyXText * t = bv->text;
+               if (inset_id >= 0)
+                       if (Inset * in = bv->buffer()->getInsetFromID(inset_id))
+                               t = in->getLyXText(bv);
+               t->setCursorIntern(plist->begin(), 0);
+       } else {
+               first = *before;
+               ++first;
+       }
+       int const first_id  = first->id();
+
+       int after_id;
+       ParagraphList::iterator after;
+       if (behind == null) {
+               // if there's no behind take the end of parlist.
+               after = plist->end();
+               after_id = -1;
+       } else {
+               after = *behind;
+               after_id = after->id();
+       }
+
+       lyxerr << "\nbefore_id: " << before_id << "\n";
+       lyxerr << "first_id:  " << first_id  << "\n";
+       lyxerr << "after_id: " << after_id << "\n";
+       lyxerr << "behind_id: " << behind_id << "\n";
+       lyxerr << "inset_id: " << inset_id << "\n";
+
+       // Set the right(new) inset-owner of the paragraph if there is any.
+       if (!undo.pars.empty()) {
+               Inset * in = 0;
+               if (before != null)
+                       in = before->inInset();
+               else if (inset_id >= 0)
+                       in = bv->buffer()->getInsetFromID(inset_id);
+               for (size_t i = 0, n = undo.pars.size(); i < n; ++i)
+                       undo.pars[i]->setInsetOwner(in);
+       }
+
+
+       // quick hack to make the common case work
+       if (undo.pars.size() == 1 && boost::next(first) == after) {
+       //      first = *undo.pars[0];
+               lyxerr << "could use special case...\n";
+       }
+
+#if 0
+       // remove stuff between first and after
+       plist->erase(first, after);
+       // re-create first
+       if (before == null) {
+               // if there's no before take the beginning of parlist.
+               first = plist->begin();
+       } else {
+               first = *before;
+               ++first;
+       }
 
-               // replace the paragraphs with the undo informations
-
-               Paragraph * tmppar3 = undo->par;
-               undo->par = 0;  /* otherwise the undo destructor would
-                                  delete the paragraph */
-
-               // get last undo par and set the right(new) inset-owner of the
-               // paragraph if there is any. This is not needed if we don't have
-               // a paragraph before because then in is automatically done in the
-               // function which assigns the first paragraph to an InsetText. (Jug)
-               Paragraph * tmppar4 = tmppar3;
-               if (tmppar4) {
-                       Inset * in = 0;
-                       if (before)
-                               in = before->inInset();
-                       tmppar4->setInsetOwner(in);
-                       while (tmppar4->next()) {
-                               tmppar4 = tmppar4->next();
-                               tmppar4->setInsetOwner(in);
-                       }
-               }
-    
-               // now remove the old text if there is any
-#ifdef DELETE_UNUSED_PARAGRAPHS
-               vector<Paragraph *> vvpar;
-#endif
-               if (before != behind || (!behind && !before)) {
-                       if (before)
-                               tmppar5 = before->next();
-                       else
-                               tmppar5 = firstUndoParagraph(bv, undo->number_of_inset_id);
-                       tmppar2 = tmppar3;
-                       while (tmppar5 && tmppar5 != behind) {
-#ifdef DELETE_UNUSED_PARAGRAPHS
-                               vvpar.push_back(tmppar5);
-#endif
-                               tmppar = tmppar5;
-                               tmppar5 = tmppar5->next();
-                               // a memory optimization for edit:
-                               // Only layout information 
-                               // is stored in the undo. So restore
-                               // the text informations. 
-                               if (undo->kind == Undo::EDIT) {
-                                       tmppar2->setContentsFromPar(tmppar);
-#ifndef DELETE_UNUSED_PARAGRAPHS
-                                       tmppar->clearContents();
 #endif
-                                       tmppar2 = tmppar2->next();
-                               }
+
+       // inset saved paragraphs
+       for (size_t i = 0, n = undo.pars.size(); i < n; ++i) {
+               lyxerr << " inserting par " << undo.pars[i]->id() << "\n";
+               lyxerr << " inserting plist: " << plist << "\n";
+               //plist->insert(first, new Paragraph(*undo.pars[i], true));
+               /*
+                       // A memory optimization for edit:
+                       // Only layout information
+                       // is stored in the undo. So restore
+                       // the text informations.
+                       if (undo.kind == Undo::EDIT) {
+                               undo.pars[par]->setContentsFromPar(*deletelist.back());
+                               ++par;
                        }
-               }
-    
-               // put the new stuff in the list if there is one
-               if (tmppar3) {
-                       if (before)
-                               before->next(tmppar3);
-                       else
-                               bv->text->ownerParagraph(tmppar3->id(),
-                                                        tmppar3);
-
-                       tmppar3->previous(before);
+               */
+       }
+       
+       // Set the cursor for redoing
+       // if we have a par before the first.
+       if (before != null) {
+               Inset * it = before->inInset();
+               if (it)
+                       it->getLyXText(bv)->setCursorIntern(*before, 0);
+               else
+                       bv->text->setCursorIntern(*before, 0);
+       }
+
+       UpdatableInset * it = 0;
+       if (first != plist->begin())
+               it = static_cast<UpdatableInset*>(first->inInset());
+
+       LyXText * text = it ? it->getLyXText(bv) : bv->text;
+
+       text->redoParagraphs(text->cursor, plist->end());
+
+       ParIterator tmppar = bv->buffer()->getParFromID(inset_id);
+
+       if (tmppar != null) {
+               LyXText * t;
+               Inset * it = tmppar->inInset();
+               if (it) {
+                       FuncRequest cmd(bv, LFUN_INSET_EDIT, "left");
+                       it->localDispatch(cmd);
+                       t = it->getLyXText(bv);
                } else {
-                       // Do we really enter here ??? (Jug)
-                       if (!before && behind) {
-                               bv->text->ownerParagraph(behind);
-                               tmppar3 = behind;
-                       }
+                       t = bv->text;
                }
-               if (tmppar4) {
-                       tmppar4->next(behind);
-                       if (behind)
-                               behind->previous(tmppar4);
+               t->setCursorIntern(*tmppar, undo.cursor_pos);
+               // Clear any selection and set the selection
+               // cursor for an evt. new selection.
+               t->clearSelection();
+               t->selection.cursor = t->cursor;
+               t->updateCounters();
+       }
+
+       if (it) {
+               bv->fitCursor();
+               bv->updateInset(it);
+               bv->text->setCursorIntern(bv->text->cursor.par(),
+                                         bv->text->cursor.pos());
+       }
+
+
+       finishUndo();
+       bv->text->postPaint(0);
+
+       return true;
+}
+
+
+bool createUndo(BufferView * bv, Undo::undo_kind kind,
+       ParagraphList::iterator itfirst, ParagraphList::iterator itlast,
+       shared_ptr<Undo> & u)
+{
+       Buffer * b = bv->buffer();
+
+       ParIterator null    = b->par_iterator_end();
+       ParIterator prev    = null;
+       ParIterator before  = null;
+       ParIterator first   = null;
+       ParIterator last    = null;
+       ParIterator behind  = null;
+
+       int const first_id  = itfirst->id();
+       int const last_id   = itlast->id();
+
+       for (ParIterator it = b->par_iterator_begin(); it != null; ++it) {
+               if ((*it)->id() == first_id) {
+                       first = it;
+                       before = prev;
                }
-    
-    
-               // Set the cursor for redoing
-               if (before) {
-                       Inset * it = before->inInset();
-                       if (it)
-                               it->getLyXText(bv)->setCursorIntern(bv, before, 0);
-                       else
-                               bv->text->setCursorIntern(bv, before, 0);
+               if ((*it)->id() == last_id) {
+                       last = it;
+                       behind = last;
+                       ++behind;
                }
+               prev = it;
+       }
 
-               Paragraph * endpar = 0;
-               // calculate the endpar for redoing the paragraphs.
-               if (behind)
-                       endpar = behind->next();
+       if (last == null)
+               last = first;
+
+       int const before_id = (before == null) ? -1 : before->id();
+       int const behind_id = (behind == null) ? -1 : behind->id();
+       int inset_id        = (first->inInset()) ? first->inInset()->id() : -1;
+
+       lyxerr << "\nbefore_id: " << before_id << "\n";
+       lyxerr << "first_id:  " << first_id  << "\n";
+       lyxerr << "last_id:   " << last_id   << "\n";
+       lyxerr << "behind_id: " << behind_id << "\n";
+       lyxerr << "inset_id:  " << inset_id  << "\n";
+
+       ParagraphList * plist = 0;
+       if (first != null)
+               plist = &first.plist();
+       else if (behind != null)
+               plist = &behind.plist();
+       else if (!plist) {
+               lyxerr << "plist from buffer (should this happen?)\n";
+               plist = &b->paragraphs;
+       }
 
-               tmppar = bv->buffer()->getParFromID(undo->number_of_cursor_par);
-               UpdatableInset* it = 0;
-               if (tmppar3)
-                       it = static_cast<UpdatableInset*>(tmppar3->inInset());
-               if (it) {
-                       it->getLyXText(bv)->redoParagraphs(bv,
-                                                          it->getLyXText(bv)->cursor,
-                                                          endpar);
-                       LyXFont font;
-                       it->update(bv, font, false);
-                       // we now would have to rebreak the whole paragraph the
-                       // undo-par was in. How we do it here is not really true.
-                       // We would have to save this information in the undo-struct
-                       // and then we could do the right rebreak. Here we only
-                       // handle the case where this was in the actual paragraph,
-                       // which not always is true.
-                       bv->text->redoParagraphs(bv, bv->text->cursor,
-                                                bv->text->cursor.par());
-                       if (tmppar) {
-                               it = static_cast<UpdatableInset*>(tmppar->inInset());
-                               LyXText * t;
-                               if (it) {
-                                       it->edit(bv);
-                                       t = it->getLyXText(bv);
-                               } else {
-                                       t = bv->text;
-                               }
-                               t->setCursorIntern(bv, tmppar, undo->cursor_pos);
-                               t->updateCounters(bv, t->cursor.row());
-                       }
-                       bv->text->setCursorIntern(bv, bv->text->cursor.par(),
-                                                 bv->text->cursor.pos());
-               } else {
-                       bv->text->redoParagraphs(bv, bv->text->cursor, endpar);
-                       if (tmppar) {
-                               LyXText * t;
-                               Inset * it = tmppar->inInset();
-                               if (it) {
-                                       it->edit(bv);
-                                       t = it->getLyXText(bv);
-                               } else {
-                                       t = bv->text;
-                               }
-                               t->setCursorIntern(bv, tmppar, undo->cursor_pos);
-                               t->updateCounters(bv, t->cursor.row());
-                       }
-               }
-               result = true;
-               delete undo;
-#ifdef DELETE_UNUSED_PARAGRAPHS
-               // And here it's save enough to delete all removed paragraphs
-               vector<Paragraph *>::iterator pit = vvpar.begin();
-               if (pit != vvpar.end()) {
-                       lyxerr << "DEL: ";
-                       for(;pit != vvpar.end(); ++pit) {
-                               lyxerr << *pit << " ";
-                               delete (*pit);
-                       }
-                       lyxerr << endl << "PARS:";
-                       ParIterator end = bv->buffer()->par_iterator_end();
-                       ParIterator it = bv->buffer()->par_iterator_begin();
-                       for (; it != end; ++it)
-                               lyxerr << *it << " ";
-                       lyxerr << endl;
+       // Undo::EDIT and Undo::FINISH are
+       // always finished. (no overlapping there)
+       // overlapping only with insert and delete inside one paragraph:
+       // Nobody wants all removed  character
+       // appear one by one when undoing.
+       // EDIT is special since only layout information, not the
+       // contents of a paragaph are stored.
+       if (!undo_finished && (kind != Undo::EDIT) && (kind != Undo::FINISH)) {
+               // Check whether storing is needed.
+               if (!b->undostack.empty() &&
+                   b->undostack.top()->kind == kind &&
+                   b->undostack.top()->number_of_before_par == before_id &&
+                   b->undostack.top()->number_of_behind_par == behind_id) {
+                       // No undo needed.
+                       return false;
                }
-#endif
        }
+
+       // Create a new Undo.
+       std::vector<Paragraph *> undo_pars;
+
+       for (ParagraphList::iterator it = *first; it != *last; ++it) 
+               undo_pars.push_back(new Paragraph(*it, true));
+       undo_pars.push_back(new Paragraph(**last, true));
+
+       // A memory optimization: Just store the layout
+       // information when only edit.
+#warning Waste...
+       //if (kind == Undo::EDIT)
+       //      for (size_t i = 0, n = undo_pars.size(); i < n; ++i)
+       //              undo_pars[i]->clearContents();
+
+       int cursor_par = undoCursor(bv).par()->id();
+       int cursor_pos = undoCursor(bv).pos();
+
+       u.reset(new Undo(kind, inset_id,
+               before_id, behind_id, cursor_par, cursor_pos, undo_pars));
+
+       undo_finished = false;
+       return true;
+}
+
+
+// Returns false if no undo possible.
+bool textUndoOrRedo(BufferView * bv,
+       limited_stack<boost::shared_ptr<Undo> > & stack,
+                   limited_stack<boost::shared_ptr<Undo> > & /*otherstack*/)
+{
+       //Buffer * b = bv->buffer();
+
+       if (stack.empty()) {
+               finishNoUndo(bv);
+               return false;
+       }
+
+       shared_ptr<Undo> undo = stack.top();
+       stack.pop();
        finishUndo();
-       bv->text->status(bv, LyXText::NEED_MORE_REFRESH);
-       return result;
+
+       if (!undo_frozen) {
+/*
+               ParIterator p = b->getParFromID(undo->number_of_before_par);
+               bool ok = false;
+               ParagraphList::iterator first;
+               // default constructed?
+               ParIterator const end = b->par_iterator_end();
+               if (p != end) {
+                       first = p.par();
+                       if (first->next())
+                               first = first->next();
+               } else
+                       first = undoParagraphs(bv, undo->number_of_inset_id)->begin();
+               if (first) {
+                       shared_ptr<Undo> u;
+                       ParIterator behind = b->getParFromID(undo->number_of_behind_par);
+                       if (createUndo(bv, undo->kind, first, behind.par(), u))
+                               otherstack.push(u);
+               }
+*/
+       }
+
+       // Now we can unlock the inset for saftey because the inset
+       // pointer could be changed during the undo-function. Anyway
+       // if needed we have to lock the right inset/position if this
+       // is requested.
+       freezeUndo();
+       bv->unlockInset(bv->theLockingInset());
+       bool const ret = textHandleUndo(bv, *undo.get());
+       unFreezeUndo();
+       return ret;
 }
 
+} // namespace anon
+
 
 void finishUndo()
 {
-       // makes sure the next operation will be stored
+       // Makes sure the next operation will be stored.
        undo_finished = true;
 }
 
 
 void freezeUndo()
 {
-       // this is dangerous and for internal use only
+       // This is dangerous and for internal use only.
        undo_frozen = true;
 }
 
 
 void unFreezeUndo()
 {
-       // this is dangerous and for internal use only
+       // This is dangerous and for internal use only.
        undo_frozen = false;
 }
 
 
-void setUndo(BufferView * bv, Undo::undo_kind kind,
-             Paragraph const * first, Paragraph const * behind)
-{
-       if (!undo_frozen) {
-               bv->buffer()->undostack.push(createUndo(bv, kind, first, behind));
-               bv->buffer()->redostack.clear();
-       }
-}
-
-
-void setRedo(BufferView * bv, Undo::undo_kind kind,
-             Paragraph const * first, Paragraph const * behind)
+bool textUndo(BufferView * bv)
 {
-       bv->buffer()->redostack.push(createUndo(bv, kind, first, behind));
+       return textUndoOrRedo(bv, bv->buffer()->undostack,
+                             bv->buffer()->redostack);
 }
 
 
-Undo * createUndo(BufferView * bv, Undo::undo_kind kind,
-                  Paragraph const * first, Paragraph const * behind)
+bool textRedo(BufferView * bv)
 {
-       lyx::Assert(first);
-
-       int before_number = -1;
-       int behind_number = -1;
-       int inset_id = -1;
-
-       if (first->previous())
-               before_number = first->previous()->id();
-       if (behind)
-               behind_number = behind->id();
-       if (first->inInset())
-               inset_id = first->inInset()->id();
-
-       // Undo::EDIT  and Undo::FINISH are
-       // always finished. (no overlapping there)
-       // overlapping only with insert and delete inside one paragraph: 
-       // Nobody wants all removed  character
-       // appear one by one when undoing. 
-       // EDIT is special since only layout information, not the
-       // contents of a paragaph are stored.
-       if (!undo_finished && (kind != Undo::EDIT) && (kind != Undo::FINISH)) {
-               // check wether storing is needed
-               if (!bv->buffer()->undostack.empty() && 
-                   bv->buffer()->undostack.top()->kind == kind &&
-                   bv->buffer()->undostack.top()->number_of_before_par == before_number &&
-                   bv->buffer()->undostack.top()->number_of_behind_par == behind_number) {
-                       // no undo needed
-                       return 0;
-               }
-       }
-       // create a new Undo
-       Paragraph * undopar;
-
-       Paragraph * start = const_cast<Paragraph *>(first);
-       Paragraph * end = 0;
-
-       if (behind)
-               end = const_cast<Paragraph*>(behind->previous());
-       else {
-               end = start;
-               while (end->next())
-                       end = end->next();
-       }
-       if (start && end && (start != end->next()) &&
-           ((before_number != behind_number) ||
-                ((before_number < 0) && (behind_number < 0))))
-       {
-               Paragraph * tmppar = start;
-               Paragraph * tmppar2 = new Paragraph(*tmppar, true);
-               
-               // a memory optimization: Just store the layout information
-               // when only edit
-               if (kind == Undo::EDIT) {
-                       tmppar2->clearContents();
-               }
-
-               undopar = tmppar2;
-  
-               while (tmppar != end && tmppar->next()) {
-                       tmppar = tmppar->next();
-#if 0
-                       tmppar2->next(new Paragraph(*tmppar, true));
-#else
-                       Paragraph * ptmp = new Paragraph(*tmppar, true);
-                       tmppar2->next(ptmp);
-#endif
-                       // a memory optimization: Just store the layout
-                       // information when only edit
-                       if (kind == Undo::EDIT) {
-                               tmppar2->clearContents();
-                       }
-                       tmppar2->next()->previous(tmppar2);
-                       
-                       tmppar2 = tmppar2->next();
-               }
-               tmppar2->next(0);
-       } else
-               undopar = 0; // nothing to replace (undo of delete maybe)
-
-       int cursor_par = undoCursor(bv).par()->id();
-       int cursor_pos =  undoCursor(bv).pos();
-       
-       Undo * undo = new Undo(kind, inset_id,
-                              before_number, behind_number,  
-                              cursor_par, cursor_pos, undopar);
-  
-       undo_finished = false;
-       return undo;
+       return textUndoOrRedo(bv, bv->buffer()->redostack,
+                             bv->buffer()->undostack);
 }
 
 
-void setCursorParUndo(BufferView * bv)
+void setUndo(BufferView * bv, Undo::undo_kind kind,
+            ParagraphList::iterator first)
 {
-       setUndo(bv, Undo::FINISH, bv->text->cursor.par(),
-               bv->text->cursor.par()->next());
+       setUndo(bv, kind, first, first);
 }
 
 
-Paragraph * firstUndoParagraph(BufferView * bv, int inset_id)
+void setUndo(BufferView * bv, Undo::undo_kind kind,
+            ParagraphList::iterator first, ParagraphList::iterator last)
 {
-       Inset * inset = bv->buffer()->getInsetFromID(inset_id);
-       if (inset) {
-               Paragraph * result = inset->getFirstParagraph(0);
-               if (result)
-                       return result;
+#warning DISABLED
+       //return;
+       if (!undo_frozen) {
+               shared_ptr<Undo> u;
+               if (createUndo(bv, kind, first, last, u))
+                       bv->buffer()->undostack.push(u);
+               bv->buffer()->redostack.clear();
        }
-       return bv->text->ownerParagraph();
 }
 
 
-LyXCursor const & undoCursor(BufferView * bv)
+void setCursorParUndo(BufferView * bv)
 {
-       if (bv->theLockingInset())
-               return bv->theLockingInset()->cursor(bv);
-       return bv->text->cursor;
+       setUndo(bv, Undo::FINISH, bv->text->cursor.par());
 }