* remove various xforms relicts, in particular:

[lyx.git] / src / BufferView_pimpl.C

diff --git a/src/BufferView_pimpl.C b/src/BufferView_pimpl.C
index b4ed4db808ed16887591fcaa990eb0456fb54dac..2365d64f11a17e1f5110d7c4ef7332305fc41e8f 100644 (file)
--- a/src/BufferView_pimpl.C
+++ b/src/BufferView_pimpl.C
@@ -64,7 +64,6 @@
 #include "frontends/font_metrics.h"
 #include "frontends/Gui.h"
 #include "frontends/LyXView.h"
-#include "frontends/Painter.h"
 #include "frontends/WorkArea.h"
 
 #include "graphics/Previews.h"
@@ -137,9 +136,9 @@ T * getInsetByCode(LCursor & cur, InsetBase::Code code)
 } // anon namespace
 
 
-BufferView::Pimpl::Pimpl(BufferView & bv, LyXView * owner, WorkArea * workArea)
-       : bv_(&bv), owner_(owner), buffer_(0), wh_(0), cursor_timeout(400), 
-         using_xterm_cursor(false), workArea_(workArea), cursor_(bv), 
+BufferView::Pimpl::Pimpl(BufferView & bv, LyXView * owner)
+       : bv_(&bv), owner_(owner), buffer_(0), wh_(0), cursor_timeout(400),
+         using_xterm_cursor(false), cursor_(bv),
          multiparsel_cache_(false), anchor_ref_(0), offset_ref_(0)
 {
        xsel_cache_.set = false;
@@ -149,7 +148,7 @@ BufferView::Pimpl::Pimpl(BufferView & bv, LyXView * owner, WorkArea * workArea)
                .connect(boost::bind(&BufferView::Pimpl::cursorToggle, this));
 
        cursor_timeout.start();
-       
+
        saved_positions.resize(saved_positions_num);
        // load saved bookmarks
        lyx::Session::BookmarkList & bmList = LyX::ref().session().loadBookmarks();
@@ -319,12 +318,6 @@ lyx::frontend::Gui & BufferView::Pimpl::gui() const
 }
 
 
-lyx::frontend::Painter & BufferView::Pimpl::painter() const
-{
-       return workArea_->getPainter();
-}
-
-
 int BufferView::Pimpl::width() const
 {
        return width_;
@@ -688,50 +681,55 @@ bool BufferView::Pimpl::multiParSel()
 }
 
 
+ViewMetricsInfo const & BufferView::Pimpl::viewMetricsInfo()
+{
+       return metrics_info_;
+}
+
+
 void BufferView::Pimpl::update(Update::flags flags)
 {
-       lyxerr[Debug::DEBUG]
-               << BOOST_CURRENT_FUNCTION
-               << "[fitcursor = " << (flags & Update::FitCursor)
-               << ", forceupdate = " << (flags & Update::Force)
-               << ", singlepar = " << (flags & Update::SinglePar)
-               << "]  buffer: " << buffer_ << endl;
+       // This is close to a hot-path.
+       if (lyxerr.debugging(Debug::DEBUG)) {
+               lyxerr[Debug::DEBUG]
+                       << BOOST_CURRENT_FUNCTION
+                       << "[fitcursor = " << (flags & Update::FitCursor)
+                       << ", forceupdate = " << (flags & Update::Force)
+                       << ", singlepar = " << (flags & Update::SinglePar)
+                       << "]  buffer: " << buffer_ << endl;
+       }
 
        // Check needed to survive LyX startup
        if (buffer_) {
                // Update macro store
                buffer_->buildMacros();
 
-               CoordCache backup;
-               std::swap(theCoords, backup);
-
                // This, together with doneUpdating(), verifies (using
                // asserts) that screen redraw is not called from
                // within itself.
                theCoords.startUpdating();
 
                // First drawing step
-               ViewMetricsInfo vi = metrics(flags & Update::SinglePar);
+               bool singlePar = flags & Update::SinglePar;
                bool forceupdate(flags & (Update::Force | Update::SinglePar));
 
-               if ((flags & Update::FitCursor) && fitCursor()) {
-                       forceupdate = true;
-                       vi = metrics();
-               }
-               if ((flags & Update::MultiParSel) && multiParSel()) {
+               if ((flags & (Update::FitCursor | Update::MultiParSel))
+                   && (fitCursor() || multiParSel())) {
                        forceupdate = true;
-                       vi = metrics();
+                       singlePar = false;
                }
+
                if (forceupdate) {
                        // Second drawing step
-                       workArea_->redraw(*bv_, vi);
+                       updateMetrics(singlePar);
+                       owner_->workArea()->redraw(*bv_);
                } else {
                        // Abort updating of the coord
                        // cache - just restore the old one
-                       std::swap(theCoords, backup);
+                       theCoords.doneUpdating();
                }
        } else
-               workArea_->greyOut();
+               owner_->workArea()->greyOut();
 
        owner_->view_state_changed();
 }
@@ -872,11 +870,6 @@ void BufferView::Pimpl::center()
 }
 
 
-void BufferView::Pimpl::stuffClipboard(string const & content) const
-{
-       owner_->gui().clipboard().put(content);
-}
-
 
 void BufferView::Pimpl::menuInsertLyXFile(string const & filenm)
 {
@@ -1254,7 +1247,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & cmd)
        case LFUN_OUTLINE_IN:
                lyx::toc::outline(lyx::toc::In, cursor_);
                updateLabels(*buffer_);
-               break;  
+               break;
        case LFUN_OUTLINE_OUT:
                lyx::toc::outline(lyx::toc::Out, cursor_);
                updateLabels(*buffer_);
@@ -1416,7 +1409,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & cmd)
 }
 
 
-ViewMetricsInfo BufferView::Pimpl::metrics(bool singlepar)
+void BufferView::Pimpl::updateMetrics(bool singlepar)
 {
        // Remove old position cache
        theCoords.clear();
@@ -1513,5 +1506,5 @@ ViewMetricsInfo BufferView::Pimpl::metrics(bool singlepar)
                << "size: " << size
                << endl;
 
-       return ViewMetricsInfo(pit1, pit2, y1, y2, singlepar, size);
+       metrics_info_ = ViewMetricsInfo(pit1, pit2, y1, y2, singlepar, size);
 }