]> git.lyx.org Git - lyx.git/blobdiff - src/BufferView.C
"Inter-word Space"
[lyx.git] / src / BufferView.C
index fc1ac3a97ea110087fc9dbf59f67bd016a17aa29..6e1ef133efa0d6cc3ee6f8d63d2d95992ee3f5e9 100644 (file)
@@ -331,142 +331,6 @@ bool BufferView::insertLyXFile(string const & filen)
 }
 
 
-bool BufferView::removeAutoInsets()
-{
-       // keep track of which pos and par the cursor was on
-       Paragraph * cursor_par = &*text->cursor.par();
-       Paragraph * cursor_par_prev = cursor_par ? cursor_par->previous() : 0;
-       Paragraph * cursor_par_next = cursor_par ? cursor_par->next() : 0;
-       pos_type cursor_pos = text->cursor.pos();
-
-       bool found = false;
-
-       // Trap the deletion of the paragraph the cursor is in.
-       // Iterate until we find a paragraph that won't be immediately deleted.
-       // In reality this should mean we only execute the body of the while
-       // loop once at most.  However for safety we iterate rather than just
-       // make this an if () conditional.
-       while ((cursor_par_prev || cursor_par_next)
-              && text->setCursor(
-                                 cursor_par_prev ? cursor_par_prev : cursor_par_next,
-                                 0)) {
-               // We just removed cursor_par so have to fix the "cursor"
-               if (cursor_par_prev) {
-                       // '.' = cursor_par
-                       //  a -> a.
-                       // .
-                       cursor_par = cursor_par_prev;
-                       cursor_pos = cursor_par->size();
-               } else {
-                       // .  -> .a
-                       //  a
-                       cursor_par = cursor_par_next;
-                       cursor_pos = 0;
-               }
-               cursor_par_prev = cursor_par->previous();
-               cursor_par_next = cursor_par->next();
-       }
-
-       // Iterate through the paragraphs removing autoDelete insets as we go.
-       // If the paragraph ends up empty after all the autoDelete insets are
-       // removed that paragraph will be removed by the next setCursor() call.
-       ParIterator it = buffer()->par_iterator_begin();
-       ParIterator end = buffer()->par_iterator_end();
-       for (; it != end; ++it) {
-               Paragraph * par = &*(*it);
-               Paragraph * par_prev = par ? par->previous() : 0;
-               bool removed = false;
-
-               if (text->setCursor(par, 0)
-                   && cursor_par == par_prev) {
-                       // The previous setCursor line was deleted and that
-                       // was the cursor_par line.  This can only happen if an
-                       // error box was the sole item on cursor_par.
-                       // It is possible for cursor_par_prev to be stray if
-                       // the line it pointed to only had a error box on it
-                       // so we have to set it to a known correct value.
-                       // This is often the same value it already had.
-                       cursor_par_prev = par->previous();
-                       if (cursor_par_prev) {
-                               // '|' = par, '.' = cursor_par, 'E' = error box
-                               // First step below may occur before while{}
-                               //  a    |a      a     a     a.
-                               //  E -> .E -> |.E -> .  -> |b
-                               // .      b      b    |b
-                               //  b
-                               cursor_par = cursor_par_prev;
-                               cursor_pos = cursor_par_prev->size();
-                               cursor_par_prev = cursor_par->previous();
-                               // cursor_par_next remains the same
-                       } else if (cursor_par_next) {
-                               // First step below may occur before while{}
-                               // .
-                               //  E -> |.E -> |.  -> . -> .|a
-                               //  a      a      a    |a
-                               cursor_par = cursor_par_next;
-                               cursor_pos = 0;
-                               // cursor_par_prev remains unset
-                               cursor_par_next = cursor_par->next();
-                       } else {
-                               // I can't find a way to trigger this
-                               // so it should be unreachable code
-                               // unless the buffer is corrupted.
-                               lyxerr << "BufferView::removeAutoInsets() is bad\n";
-                       }
-               }
-
-               InsetList::iterator pit = par->insetlist.begin();
-               InsetList::iterator pend = par->insetlist.end();
-
-               while (pit != pend) {
-                       if (pit.getInset()->autoDelete()) {
-                               removed = true;
-                               pos_type const pos = pit.getPos();
-
-                               par->erase(pos);
-                               // We just invalidated par's inset iterators so
-                               // we get the next valid iterator position
-                               pit = par->insetlist.insetIterator(pos);
-                               // and ensure we have a valid end iterator.
-                               pend = par->insetlist.end();
-
-                               if (cursor_par == par) {
-                                       // update the saved cursor position
-                                       if (cursor_pos > pos)
-                                               --cursor_pos;
-                               }
-                       } else {
-                               ++pit;
-                       }
-               }
-               if (removed) {
-                       found = true;
-                       text->redoParagraph();
-               }
-       }
-
-       // It is possible that the last line is empty if it was cursor_par
-       // and/or only had an error inset on it.  So we set the cursor to the
-       // start of the doc to force its removal and ensure a valid saved cursor
-       if (text->setCursor(&*text->ownerParagraphs().begin(), 0)
-           && 0 == cursor_par_next) {
-               cursor_par = cursor_par_prev;
-               cursor_pos = cursor_par->size();
-       } else if (cursor_pos > cursor_par->size()) {
-               // Some C-Enter lines were removed by the setCursor call which
-               // then invalidated cursor_pos. It could still be "wrong" because
-               // the cursor may appear to have jumped but since we collapsed
-               // some C-Enter lines this should be a reasonable compromise.
-               cursor_pos = cursor_par->size();
-       }
-
-       // restore the original cursor in its corrected location.
-       text->setCursorIntern(cursor_par, cursor_pos);
-
-       return found;
-}
-
-
 void BufferView::resetErrorList()
 {
        pimpl_->errorlist_.clear();
@@ -671,15 +535,15 @@ bool BufferView::lockInset(UpdatableInset * inset)
                        InsetList::iterator it = pit->insetlist.begin();
                        InsetList::iterator end = pit->insetlist.end();
                        for (; it != end; ++it) {
-                               if (it.getInset() == inset) {
-                                       text->setCursorIntern(pit, it.getPos());
+                               if (it->inset == inset) {
+                                       text->setCursorIntern(pit, it->pos);
                                        theLockingInset(inset);
                                        return true;
                                }
-                               if (it.getInset()->getInsetFromID(id)) {
-                                       text->setCursorIntern(pit, it.getPos());
+                               if (it->inset->getInsetFromID(id)) {
+                                       text->setCursorIntern(pit, it->pos);
                                        FuncRequest cmd(this, LFUN_INSET_EDIT, "left");
-                                       it.getInset()->localDispatch(cmd);
+                                       it->inset->localDispatch(cmd);
                                        return theLockingInset()->lockInsetInInset(this, inset);
                                }
                        }
@@ -740,9 +604,7 @@ void BufferView::lockedInsetStoreUndo(Undo::undo_kind kind)
                return; // shouldn't happen
        if (kind == Undo::EDIT) // in this case insets would not be stored!
                kind = Undo::FINISH;
-       setUndo(this, kind,
-               text->cursor.par(),
-               boost::next(text->cursor.par()));
+       setUndo(this, kind, text->cursor.par());
 }
 
 
@@ -764,12 +626,11 @@ bool BufferView::ChangeInsets(Inset::Code code,
        ParIterator end = buffer()->par_iterator_end();
        for (ParIterator it = buffer()->par_iterator_begin();
             it != end; ++it) {
-               Paragraph * par = &*(*it);
                bool changed_inset = false;
-               for (InsetList::iterator it2 = par->insetlist.begin();
-                    it2 != par->insetlist.end(); ++it2) {
-                       if (it2.getInset()->lyxCode() == code) {
-                               InsetCommand * inset = static_cast<InsetCommand *>(it2.getInset());
+               for (InsetList::iterator it2 = it->insetlist.begin();
+                    it2 != it->insetlist.end(); ++it2) {
+                       if (it2->inset->lyxCode() == code) {
+                               InsetCommand * inset = static_cast<InsetCommand *>(it2->inset);
                                if (inset->getContents() == from) {
                                        inset->setContents(to);
                                        changed_inset = true;
@@ -784,7 +645,7 @@ bool BufferView::ChangeInsets(Inset::Code code,
                        // The test it.size()==1 was needed to prevent crashes.
                        // How to set the cursor corretly when it.size()>1 ??
                        if (it.size() == 1) {
-                               text->setCursorIntern(par, 0);
+                               text->setCursorIntern(*it, 0);
                                text->redoParagraphs(text->cursor,
                                                     boost::next(text->cursor.par()));
                                text->fullRebreak();