changelogs

[lyx.git] / src / BufferView_pimpl.C

diff --git a/src/BufferView_pimpl.C b/src/BufferView_pimpl.C
index 1fdc75a779953f43b533583a1e87b2f6d3787d7b..62c17dbb500b0778d062713acdf99a9b2b192880 100644 (file)
--- a/src/BufferView_pimpl.C
+++ b/src/BufferView_pimpl.C
@@ -23,6 +23,7 @@
 #include "buffer_funcs.h"
 #include "bufferlist.h"
 #include "bufferparams.h"
+#include "coordcache.h"
 #include "cursor.h"
 #include "debug.h"
 #include "dispatchresult.h"
@@ -71,6 +72,8 @@
 
 #include <boost/bind.hpp>
 
+#include <functional>
+
 using lyx::pos_type;
 
 using lyx::support::AddPath;
@@ -80,6 +83,7 @@ using lyx::support::FileSearch;
 using lyx::support::ForkedcallsController;
 using lyx::support::IsDirWriteable;
 using lyx::support::MakeDisplayPath;
+using lyx::support::MakeAbsPath;
 using lyx::support::strToUnsignedInt;
 using lyx::support::system_lyxdir;
 
@@ -88,6 +92,7 @@ using std::istringstream;
 using std::make_pair;
 using std::min;
 using std::string;
+using std::mem_fun_ref;
 
 
 extern BufferList bufferlist;
@@ -325,8 +330,17 @@ void BufferView::Pimpl::setBuffer(Buffer * b)
        if (buffer_)
                disconnectBuffer();
 
-       // set current buffer
-       buffer_ = b;
+       // if we are closing current buffer, switch to the first in
+       // buffer list.
+       if (!b) {
+               lyxerr[Debug::INFO] << "  No Buffer!" << endl;
+               // we are closing the buffer, use the first buffer as current
+               buffer_ = bufferlist.first();
+               owner_->getDialogs().hideBufferDependent();
+       } else {
+               // set current buffer
+               buffer_ = b;
+       }
 
        // reset old cursor
        top_y_ = 0;
@@ -353,11 +367,6 @@ void BufferView::Pimpl::setBuffer(Buffer * b)
                // hidden. This should go here because some dialogs (eg ToC)
                // require bv_->text.
                owner_->getDialogs().updateBufferDependent(true);
-       } else {
-               lyxerr[Debug::INFO] << "  No Buffer!" << endl;
-               // we are closing the buffer, use the first buffer as current
-               buffer_ = bufferlist.first();
-               owner_->getDialogs().hideBufferDependent();
        }
 
        update();
@@ -380,7 +389,7 @@ bool BufferView::Pimpl::fitCursor()
 {
        // to get the correct y cursor info
        lyxerr[Debug::DEBUG] << "BufferView::fitCursor" << std::endl;
-       lyx::par_type const pit = bv_->cursor().bottom().par();
+       lyx::pit_type const pit = bv_->cursor().bottom().pit();
        bv_->text()->redoParagraph(pit);
        refreshPar(*bv_, *bv_->text(), pit);
 
@@ -593,7 +602,7 @@ void BufferView::Pimpl::update()
                buffer_->buildMacros();
 
                // update all 'visible' paragraphs
-               lyx::par_type beg, end;
+               lyx::pit_type beg, end;
                getParsInRange(buffer_->paragraphs(),
                               top_y(), top_y() + workarea().workHeight(),
                               beg, end);
@@ -602,7 +611,13 @@ void BufferView::Pimpl::update()
                // and the scrollbar
                updateScrollbar();
        }
+
+       // remove old position cache
+       theCoords.clear();
+
+       // The real, big redraw.
        screen().redraw(*bv_);
+
        bv_->owner()->view_state_changed();
 }
 
@@ -642,7 +657,7 @@ Change const BufferView::Pimpl::getCurrentChange()
        if (!cur.selection())
                return Change(Change::UNCHANGED);
 
-       return text->getPar(cur.selBegin().par()).
+       return text->getPar(cur.selBegin().pit()).
                        lookupChangeFull(cur.selBegin().pos());
 }
 
@@ -789,12 +804,19 @@ void BufferView::Pimpl::MenuInsertLyXFile(string const & filenm)
 
        string const disp_fn = MakeDisplayPath(filename);
        owner_->message(bformat(_("Inserting document %1$s..."), disp_fn));
-       if (bv_->insertLyXFile(filename))
-               owner_->message(bformat(_("Document %1$s inserted."),
-                                       disp_fn));
-       else
-               owner_->message(bformat(_("Could not insert document %1$s"),
-                                       disp_fn));
+
+       bv_->cursor().clearSelection();
+       bv_->text()->breakParagraph(bv_->cursor());
+
+       BOOST_ASSERT(bv_->cursor().inTexted());
+
+       string const fname = MakeAbsPath(filename);
+       bool const res = bv_->buffer()->readFile(fname, bv_->cursor().pit());
+       bv_->resize();
+
+       string s = res ? _("Document %1$s inserted.")
+                      : _("Could not insert document %1$s");
+       owner_->message(bformat(s, disp_fn));
 }
 
 
@@ -804,9 +826,9 @@ void BufferView::Pimpl::trackChanges()
        bool const tracking = buf->params().tracking_changes;
 
        if (!tracking) {
-               ParIterator const end = buf->par_iterator_end();
-               for (ParIterator it = buf->par_iterator_begin(); it != end; ++it)
-                       it->trackChanges();
+               for_each(buf->par_iterator_begin(),
+                        buf->par_iterator_end(),
+                        bind(&Paragraph::trackChanges, _1, Change::UNCHANGED));
                buf->params().tracking_changes = true;
 
                // we cannot allow undos beyond the freeze point
@@ -823,9 +845,10 @@ void BufferView::Pimpl::trackChanges()
                        return;
                }
 
-               ParIterator const end = buf->par_iterator_end();
-               for (ParIterator it = buf->par_iterator_begin(); it != end; ++it)
-                       it->untrackChanges();
+               for_each(buf->par_iterator_begin(),
+                        buf->par_iterator_end(),
+                        mem_fun_ref(&Paragraph::untrackChanges));
+
                buf->params().tracking_changes = false;
        }
 
@@ -835,8 +858,8 @@ void BufferView::Pimpl::trackChanges()
 
 bool BufferView::Pimpl::workAreaDispatch(FuncRequest const & cmd0)
 {
-       //lyxerr << "BufferView::Pimpl::workAreaDispatch: request: "
-       //  << cmd << std::endl;
+       lyxerr << "BufferView::Pimpl::workAreaDispatch: request: "
+         << cmd0 << std::endl;
        // this is only called for mouse related events including
        // LFUN_FILE_OPEN generated by drag-and-drop.
        FuncRequest cmd = cmd0;
@@ -869,8 +892,9 @@ bool BufferView::Pimpl::workAreaDispatch(FuncRequest const & cmd0)
 
        // Build temporary cursor.
        InsetBase * inset = bv_->text()->editXY(cur, cmd.x, cmd.y);
-       lyxerr << "hit inset at tip: " << inset << endl;
-       lyxerr << "created temp cursor:\n" << cur << endl;
+       lyxerr << " * created temp cursor: " << inset << endl;
+       lyxerr << " * hit inset at tip: " << inset << endl;
+       lyxerr << " * created temp cursor:" << cur << endl;
 
        // Put anchor at the same position.
        cur.resetAnchor();
@@ -878,7 +902,7 @@ bool BufferView::Pimpl::workAreaDispatch(FuncRequest const & cmd0)
        // Try to dispatch to an non-editable inset near this position
        // via the temp cursor. If the inset wishes to change the real
        // cursor it has to do so explicitly by using
-       //  cur.bv().cursor() = cur;  (or similar)'
+       //  cur.bv().cursor() = cur;  (or similar)
        if (inset)
                inset->dispatch(cur, cmd);