X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Fgraphics%2FPreviewImage.cpp;h=b80bf9415b5175bdabf77d1e50b2af144e733137;hb=94c65074e029937b549f5ed5af1001fadbe408be;hp=89289e282e016c34dd8a8e95cdc4e5d7de791ca9;hpb=f497296c30e6da2f97b16da8ad1c9e96feffb16b;p=lyx.git

diff --git a/src/graphics/PreviewImage.cpp b/src/graphics/PreviewImage.cpp
index 89289e282e..b80bf9415b 100644
--- a/src/graphics/PreviewImage.cpp
+++ b/src/graphics/PreviewImage.cpp
@@ -11,25 +11,23 @@
 #include <config.h>
 
 #include "PreviewImage.h"
+
+#include "Buffer.h"
+#include "Dimension.h"
 #include "GraphicsImage.h"
 #include "GraphicsLoader.h"
 #include "PreviewLoader.h"
 
-#include "support/filename.h"
-#include "support/lyxlib.h"
-
-#include <boost/bind.hpp>
+#include "support/FileName.h"
 
-using std::string;
 
+using namespace std;
+using namespace lyx::support;
 
 namespace lyx {
-
-using support::FileName;
-
 namespace graphics {
 
-class PreviewImage::Impl : public boost::signals::trackable {
+class PreviewImage::Impl {
 public:
 	///
 	Impl(PreviewImage & p, PreviewLoader & l,
@@ -63,7 +61,9 @@ PreviewImage::PreviewImage(PreviewLoader & l,
 
 
 PreviewImage::~PreviewImage()
-{}
+{
+	delete pimpl_;
+}
 
 
 string const & PreviewImage::snippet() const
@@ -72,54 +72,56 @@ string const & PreviewImage::snippet() const
 }
 
 
-int PreviewImage::ascent() const
+support::FileName const & PreviewImage::filename() const
 {
-	Image const * const image = pimpl_->iloader_.image();
-	if (!image)
-		return 0;
-
-	return int(pimpl_->ascent_frac_ * double(image->getHeight()));
+	return pimpl_->iloader_.filename();
 }
 
 
-int PreviewImage::descent() const
+Dimension PreviewImage::dim() const
 {
+	Dimension dim;
 	Image const * const image = pimpl_->iloader_.image();
 	if (!image)
-		return 0;
+		return dim;
 
-	// Avoids rounding errors.
-	return image->getHeight() - ascent();
+	dim.asc = int(pimpl_->ascent_frac_ * double(image->height()) + 0.5);
+	dim.des = image->height() - dim.asc;
+	dim.wid = image->width();
+	return dim;
 }
 
 
-int PreviewImage::width() const
+Image const * PreviewImage::image() const
 {
-	Image const * const image = pimpl_->iloader_.image();
-	return image ? image->getWidth() : 0;
+	return pimpl_->image();
 }
 
 
-Image const * PreviewImage::image() const
+PreviewLoader & PreviewImage::previewLoader() const
 {
-	return pimpl_->image();
+	return pimpl_->ploader_;
 }
 
 
-PreviewImage::Impl::Impl(PreviewImage & p, PreviewLoader & l,
-			 string const & s,
-			 FileName const & bf,
-			 double af)
-	: parent_(p), ploader_(l), iloader_(bf),
+PreviewImage::Impl::Impl(PreviewImage & p, PreviewLoader & l, string const & s,
+                         FileName const & bf, double af)
+	: parent_(p), ploader_(l), iloader_(l.buffer().fileName(), bf),
 	  snippet_(s), ascent_frac_(af)
 {
-	iloader_.connect(boost::bind(&Impl::statusChanged, this));
+	iloader_.setDisplayPixelRatio(l.displayPixelRatio());
+	// This connection is destroyed at the same time as this.
+	iloader_.connect([this](){ statusChanged(); });
 }
 
 
 PreviewImage::Impl::~Impl()
 {
-	support::unlink(iloader_.filename());
+	// If these images are generated for a clone, then that may be
+	// because we are previewing. We therefore do not want to delete
+	// them when this Buffer is destroyed.
+	if (!ploader_.buffer().isClone())
+		iloader_.filename().removeFile();
 }
 
 
@@ -147,12 +149,15 @@ void PreviewImage::Impl::statusChanged()
 	case ErrorLoading:
 	case ErrorGeneratingPixmap:
 	case ErrorUnknown:
-		//lyx::unlink(iloader_.filename());
+		//iloader_.filename().removeFile();
 		ploader_.remove(snippet_);
-		break;
+		// FIXME: We need to return here, because PreviewLoader::remove
+		// removes the preview image from the cache, which deletes this
+		// object, so we should not try to do anything here.
+		return;
 
 	case Ready:
-		support::unlink(iloader_.filename());
+		iloader_.filename().removeFile();
 		break;
 	}
 	ploader_.emitSignal(parent_);