]> git.lyx.org Git - lyx.git/blobdiff - src/BufferView2.C
bug 183
[lyx.git] / src / BufferView2.C
index 9641a26017f0d7ef7483e8300f3f0e064a8fe64e..6072b4548345bb74ee85ee1708d6b4a1bb9f27ec 100644 (file)
@@ -8,34 +8,40 @@
  *
  * ====================================================== */
 
-#include <fstream>
-#include <algorithm>
-
 #include <config.h>
 
 #include "BufferView.h"
 #include "buffer.h"
 #include "lyxcursor.h"
 #include "lyxtext.h"
-#include "insets/inseterror.h"
 #include "LyXView.h"
 #include "bufferlist.h"
-#include "support/FileInfo.h"
 #include "lyxscreen.h"
-#include "support/filetools.h"
-#include "frontends/Alert.h"
 #include "LaTeX.h"
 #include "BufferView_pimpl.h"
-#include "insets/insetcommand.h" //ChangeRefs
-#include "support/lyxfunctional.h" //equal_1st_in_pair
 #include "language.h"
 #include "gettext.h"
 #include "undo_funcs.h"
 #include "debug.h"
 #include "iterators.h"
 
+#include "frontends/Alert.h"
+
+#include "insets/insetcommand.h" //ChangeRefs
+#include "insets/inseterror.h"
+
+#include "support/FileInfo.h"
+#include "support/filetools.h"
+#include "support/lyxfunctional.h" //equal_1st_in_pair
+#include "support/types.h"
+
+#include <fstream>
+#include <algorithm>
+
 extern BufferList bufferlist;
 
+using lyx::pos_type;
+
 using std::pair;
 using std::endl;
 using std::ifstream;
@@ -102,32 +108,105 @@ bool BufferView::insertLyXFile(string const & filen)
 
 bool BufferView::removeAutoInsets()
 {
-       Paragraph * par = buffer()->paragraph;
-
        LyXCursor tmpcursor = text->cursor;
-       LyXCursor cursor;
-
-       bool a = false;
-
-       while (par) {
-               // this has to be done before the delete
-               text->setCursor(this, cursor, par, 0);
-               if (par->autoDeleteInsets()){
-                       a = true;
-                       text->redoParagraphs(this, cursor,
-                                            cursor.par()->next());
-                       text->fullRebreak(this);
+       Paragraph * cur_par = tmpcursor.par();
+       Paragraph * cur_par_prev = cur_par ? cur_par->previous() : 0;
+       Paragraph * cur_par_next = cur_par ? cur_par->next() : 0;
+       pos_type cur_pos = tmpcursor.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 ((cur_par_prev || cur_par_next)
+              && text->setCursor(this,
+                                 cur_par_prev ? cur_par_prev : cur_par_next,
+                                 0)) {
+               // We just removed cur_par so have to fix the "cursor"
+               if (cur_par_prev) {
+                       // '.' = cur_par
+                       //  a -> a.
+                       // .
+                       cur_par = cur_par_prev;
+                       cur_pos = cur_par->size();
+               } else {
+                       // .  -> .a
+                       //  a
+                       cur_par = cur_par_next;
+                       cur_pos = 0;
                }
-               par = par->next();
+               cur_par_prev = cur_par->previous();
+               cur_par_next = cur_par->next();
        }
 
-       // avoid forbidden cursor positions caused by error removing
-       if (tmpcursor.pos() > tmpcursor.par()->size())
-               tmpcursor.pos(tmpcursor.par()->size());
+       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(this, par, 0)
+                   && cur_par == par_prev) {
+                       // The previous setCursor line was deleted and that
+                       // was the cur_par line.  This can only happen if an
+                       // error box was the sole item on cur_par.
+                       if (cur_par_prev) {
+                               // '|' = par, '.' = cur_par, 'E' = error box
+                               // First step below may occur before while{}
+                               //  a    |a      a     a     a.
+                               //  E -> .E -> |.E -> .  -> |b
+                               // .      b      b    |b
+                               //  b
+                               cur_par = cur_par_prev;
+                               cur_pos = cur_par_prev->size();
+                               cur_par_prev = cur_par->previous();
+                               // cur_par_next remains the same 
+                       } else if (cur_par_next) {
+                               // First step below may occur before while{}
+                               // .
+                               //  E -> |.E -> |.  -> . -> .|a
+                               //  a      a      a    |a
+                               cur_par = cur_par_next;
+                               cur_pos = 0;
+                               // cur_par_prev remains unset
+                               cur_par_next = cur_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";
+                       }
+               }
+
+               Paragraph::inset_iterator pit = par->inset_iterator_begin();
+               Paragraph::inset_iterator pend = par->inset_iterator_end();
+               while (pit != pend) {
+                       if (pit->autoDelete()) {
+                               removed = true;
+                               pos_type const pos = pit.getPos();
+                               
+                               par->erase(pos);
+                               if (cur_par == par) {
+                                       if (cur_pos > pos)
+                                               --cur_pos;
+                               }
+                       } else {
+                               ++pit;
+                       }
+               }
+               if (removed) {
+                       found = true;
+                       text->redoParagraph(this);
+               }
+       }
 
-       text->setCursorIntern(this, tmpcursor.par(), tmpcursor.pos());
+       text->setCursorIntern(this, cur_par, cur_pos);
 
-       return a;
+       return found;
 }
 
 
@@ -136,9 +215,9 @@ void BufferView::insertErrors(TeXErrors & terr)
        // Save the cursor position
        LyXCursor cursor = text->cursor;
 
-       for (TeXErrors::Errors::const_iterator cit = terr.begin();
-            cit != terr.end();
-            ++cit) {
+       TeXErrors::Errors::const_iterator cit = terr.begin();
+       TeXErrors::Errors::const_iterator end = terr.end();
+       for (; cit != end; ++cit) {
                string const desctext(cit->error_desc);
                string const errortext(cit->error_text);
                string const msgtxt = desctext + '\n' + errortext;
@@ -155,7 +234,7 @@ void BufferView::insertErrors(TeXErrors & terr)
                Paragraph * texrowpar = 0;
 
                if (tmpid == -1) {
-                       texrowpar = text->firstParagraph();
+                       texrowpar = text->ownerParagraph();
                        tmppos = 0;
                } else {
                        texrowpar = buffer()->getParFromID(tmpid);
@@ -184,7 +263,7 @@ void BufferView::setCursorFromRow(int row)
        Paragraph * texrowpar;
 
        if (tmpid == -1) {
-               texrowpar = text->firstParagraph();
+               texrowpar = text->ownerParagraph();
                tmppos = 0;
        } else {
                texrowpar = buffer()->getParFromID(tmpid);
@@ -241,10 +320,12 @@ void BufferView::menuUndo()
 
 void BufferView::menuRedo()
 {
+#if 0 // this should not be here (Jug 20011206)
        if (theLockingInset()) {
                owner()->message(_("Redo not yet supported in math mode"));
                return;
-       }    
+       }
+#endif
    
        if (available()) {
                owner()->message(_("Redo"));
@@ -264,10 +345,6 @@ void BufferView::copyEnvironment()
 {
        if (available()) {
                text->copyEnvironmentType();
-               // clear the selection, even if mark_set
-               toggleSelection();
-               text->clearSelection();
-               update(text, BufferView::SELECT|BufferView::FITCUR);
                owner()->message(_("Paragraph environment type copied"));
        }
 }
@@ -287,12 +364,6 @@ void BufferView::copy()
 {
        if (available()) {
                text->copySelection(this);
-#if 0
-               // clear the selection, even if mark_set
-               toggleSelection();
-               text->clearSelection();
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-#endif
                owner()->message(_("Copy"));
        }
 }
@@ -400,19 +471,53 @@ void BufferView::replaceWord(string const & replacestring)
 
 bool BufferView::lockInset(UpdatableInset * inset)
 {
-       if (!theLockingInset() && inset) {
-               theLockingInset(inset);
+       if (!inset)
+               return false;
+       // don't relock if we're already locked
+       if (theLockingInset() == inset)
                return true;
-       } else if (inset) {
-           return theLockingInset()->lockInsetInInset(this, inset);
+       if (!theLockingInset()) {
+               // first check if it's the inset under the cursor we want lock
+               // should be most of the time
+               char const c = text->cursor.par()->getChar(text->cursor.pos());
+               if (c == Paragraph::META_INSET) {
+                       Inset * in = text->cursor.par()->getInset(text->cursor.pos());
+                       if (inset == in) {
+                               theLockingInset(inset);
+                               return true;
+                       }
+               }
+               // Then do a deep look of the inset and lock the right one
+               Paragraph * par = buffer()->paragraph;
+               int const id = inset->id();
+               while(par) {
+                       Paragraph::inset_iterator it =
+                               par->inset_iterator_begin();
+                       Paragraph::inset_iterator const end =
+                               par->inset_iterator_end();
+                       for (; it != end; ++it) {
+                               if ((*it) == inset) {
+                                       text->setCursorIntern(this, par, it.getPos());
+                                       theLockingInset(inset);
+                                       return true;
+                               }
+                               if ((*it)->getInsetFromID(id)) {
+                                       text->setCursorIntern(this, par, it.getPos());
+                                       theLockingInset(static_cast<UpdatableInset *>(*it));
+                                       return theLockingInset()->lockInsetInInset(this, inset);
+                               }
+                       }
+                       par = par->next();
+               }
+               return false;
        }
-       return false;
+       return theLockingInset()->lockInsetInInset(this, inset);
 }
 
 
 void BufferView::showLockedInsetCursor(int x, int y, int asc, int desc)
 {
-       if (available() && theLockingInset()) {
+       if (available() && theLockingInset() && !theLockingInset()->nodraw()) {
                LyXCursor cursor = text->cursor;
                Inset * locking_inset = theLockingInset()->getLockingInset();
 
@@ -460,13 +565,18 @@ void BufferView::fitLockedInsetCursor(int x, int y, int asc, int desc)
 
 int BufferView::unlockInset(UpdatableInset * inset)
 {
+       if (!inset)
+               return 0;
        if (inset && theLockingInset() == inset) {
                inset->insetUnlock(this);
                theLockingInset(0);
+               // make sure we update the combo !
+               owner()->setLayout(getLyXText()->cursor.par()->getLayout());
                finishUndo();
                return 0;
        } else if (inset && theLockingInset() &&
                   theLockingInset()->unlockInsetInInset(this, inset)) {
+               // owner inset has updated the layout combo 
                finishUndo();
                return 0;
        }
@@ -495,7 +605,7 @@ void BufferView::updateInset(Inset * inset, bool mark_dirty)
 bool BufferView::ChangeInsets(Inset::Code code,
                              string const & from, string const & to)
 {
-       bool flag = false;
+       bool need_update = false;
        LyXCursor cursor = text->cursor;
        LyXCursor tmpcursor = cursor;
        cursor.par(tmpcursor.par());
@@ -505,20 +615,22 @@ bool BufferView::ChangeInsets(Inset::Code code,
        for (ParIterator it = buffer()->par_iterator_begin();
             it != end; ++it) {
                Paragraph * par = *it;
-               bool flag2 = false;
+               bool changed_inset = false;
                for (Paragraph::inset_iterator it2 = par->inset_iterator_begin();
                     it2 != par->inset_iterator_end(); ++it2) {
                        if ((*it2)->lyxCode() == code) {
                                InsetCommand * inset = static_cast<InsetCommand *>(*it2);
                                if (inset->getContents() == from) {
                                        inset->setContents(to);
-                                       flag2 = true;
+                                       changed_inset = true;
                                }
                        }
                }
-               if (flag2) {
-                       flag = true;
-#warning Fix me
+               if (changed_inset) {
+                       need_update = true;
+#ifdef WITH_WARNINGS
+#warning FIXME
+#endif
                        // The test it.size()==1 was needed to prevent crashes.
                        // How to set the cursor corretly when it.size()>1 ??
                        if (it.size() == 1) {
@@ -530,7 +642,7 @@ bool BufferView::ChangeInsets(Inset::Code code,
                }
        }
        text->setCursorIntern(this, cursor.par(), cursor.pos());
-       return flag;
+       return need_update;
 }
 
 
@@ -538,7 +650,10 @@ bool BufferView::ChangeRefsIfUnique(string const & from, string const & to)
 {
        // Check if the label 'from' appears more than once
        vector<string> labels = buffer()->getLabelList();
-       if (count(labels.begin(), labels.end(), from) > 1)
+       // count is broken on some systems, so use the HP version
+       int res;
+       count(labels.begin(), labels.end(), from, res);
+       if (res > 1)
                return false;
 
        return ChangeInsets(Inset::REF_CODE, from, to);