]> git.lyx.org Git - lyx.git/blobdiff - src/graphics/GraphicsCacheItem.C
If the graphics loader has a copy c-tor it should have copy assignment that does...
[lyx.git] / src / graphics / GraphicsCacheItem.C
index 44dd44c867ac7fb11635f40ab728658faa2d4558..fe38fd16af8856726d452e06de754127e30b3d6c 100644 (file)
@@ -1,23 +1,23 @@
-/*
+/**
  * \file GraphicsCacheItem.C
- * Copyright 2002 the LyX Team
- * Read the file COPYING
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- * \author Baruch Even <baruch.even@writeme.com>
- * \author Herbert Voss <voss@lyx.org>
- * \author Angus Leeming <leeming@lyx.org>
+ * \author Baruch Even
+ * \author Herbert Voss
+ * \author Angus Leeming
+ *
+ * Full author contact details are available in file CREDITS
  */
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
 #include "GraphicsCacheItem.h"
 #include "GraphicsImage.h"
 #include "GraphicsConverter.h"
 
+#include "support/FileMonitor.h"
+
 #include "debug.h"
 
 #include "support/LAssert.h"
@@ -73,8 +73,22 @@ struct CacheItem::Impl : public boost::signals::trackable {
         */
        void setStatus(ImageStatus new_status);
 
+       /** Can be invoked directly by the user, but is also connected to the
+        *  FileMonitor and so is invoked when the file is changed
+        *  (if monitoring is taking place).
+        */
+       void startLoading();
+
+       /** If we are asked to load the file for a second or further time,
+        *  (because the file has changed), then we'll have to first reset
+        *  many of the variables below.
+        */
+       void reset();
+
        /// The filename we refer too.
        string const filename_;
+       ///
+       FileMonitor const monitor_;
 
        /// Is the file compressed?
        bool zipped_;
@@ -123,10 +137,26 @@ string const & CacheItem::filename() const
 
 void CacheItem::startLoading() const
 {
-       if (pimpl_->status_ != WaitingToLoad)
-               return;
+       pimpl_->startLoading();
+}
 
-       pimpl_->convertToDisplayFormat();
+
+void CacheItem::startMonitoring() const
+{
+       if (!pimpl_->monitor_.monitoring())
+               pimpl_->monitor_.start();
+}
+
+
+bool CacheItem::monitoring() const
+{
+       return pimpl_->monitor_.monitoring();
+}
+
+
+unsigned long CacheItem::checksum() const
+{
+       return pimpl_->monitor_.checksum();
 }
 
 
@@ -154,9 +184,51 @@ boost::signals::connection CacheItem::connect(slot_type const & slot) const
 
 
 CacheItem::Impl::Impl(string const & file)
-       : filename_(file), zipped_(false),
-         remove_loaded_file_(false), status_(WaitingToLoad)
-{}
+       : filename_(file),
+         monitor_(file, 2000),
+         zipped_(false),
+         remove_loaded_file_(false),
+         status_(WaitingToLoad)
+{
+       monitor_.connect(boost::bind(&Impl::startLoading, this));
+}
+
+
+void CacheItem::Impl::startLoading()
+{
+       if (status_ != WaitingToLoad)
+               reset();
+
+       convertToDisplayFormat();
+}
+
+
+void CacheItem::Impl::reset()
+{
+       zipped_ = false;
+       if (!unzipped_filename_.empty())
+               lyx::unlink(unzipped_filename_);
+       unzipped_filename_.erase();
+
+       if (remove_loaded_file_ && !file_to_load_.empty())
+               lyx::unlink(file_to_load_);
+       remove_loaded_file_ = false;
+       file_to_load_.erase();
+
+       if (image_.get())
+               image_.reset();
+
+       status_ = WaitingToLoad;
+
+       if (cl_.connected())
+               cl_.disconnect();
+
+       if (cc_.connected())
+               cc_.disconnect();
+
+       if (converter_.get())
+               converter_.reset();
+}
 
 
 void CacheItem::Impl::setStatus(ImageStatus new_status)
@@ -172,7 +244,7 @@ void CacheItem::Impl::setStatus(ImageStatus new_status)
 void CacheItem::Impl::imageConverted(bool success)
 {
        string const text = success ? "succeeded" : "failed";
-       lyxerr[Debug::GRAPHICS] << "Image conversion " << text << "." << endl;
+       lyxerr[Debug::GRAPHICS] << "Image conversion " << text << '.' << endl;
 
        file_to_load_ = converter_.get() ?
                converter_->convertedFile() : string();
@@ -180,9 +252,10 @@ void CacheItem::Impl::imageConverted(bool success)
        cc_.disconnect();
 
        success = !file_to_load_.empty() && IsFileReadable(file_to_load_);
-       lyxerr[Debug::GRAPHICS] << "Unable to find converted file!" << endl;
 
        if (!success) {
+               lyxerr[Debug::GRAPHICS] << "Unable to find converted file!"
+                                       << endl;
                setStatus(ErrorConverting);
 
                if (zipped_)
@@ -214,7 +287,7 @@ void CacheItem::Impl::loadImage()
 void CacheItem::Impl::imageLoaded(bool success)
 {
        string const text = success ? "succeeded" : "failed";
-       lyxerr[Debug::GRAPHICS] << "Image loading " << text << "." << endl;
+       lyxerr[Debug::GRAPHICS] << "Image loading " << text << '.' << endl;
 
        // Clean up after loading.
        if (zipped_)
@@ -249,23 +322,27 @@ string const findTargetFormat(string const & from)
        lyx::Assert(!formats.empty());
 
        // First ascertain if we can load directly with no conversion
-       FormatList::const_iterator it1 = formats.begin();
+       FormatList::const_iterator it  = formats.begin();
        FormatList::const_iterator end = formats.end();
-       for (; it1 != end; ++it1) {
-               if (from == *it1)
-                       return *it1;
+       for (; it != end; ++it) {
+               if (from == *it)
+                       return *it;
        }
 
        // So, we have to convert to a loadable format. Can we?
-       FormatList::const_iterator it2 = formats.begin();
-       for (; it2 != end; ++it2) {
-               if (grfx::Converter::isReachable(from, *it2))
-                       return *it2;
+       it = formats.begin();
+       for (; it != end; ++it) {
+               if (grfx::Converter::isReachable(from, *it))
+                       return *it;
+               else
+                       lyxerr[Debug::GRAPHICS]
+                               << "Unable to convert from " << from
+                               << " to " << *it << std::endl;
        }
 
-       // Failed! so we have to try to convert it to XPM format
+       // Failed! so we have to try to convert it to PPM format
        // with the standard converter
-       return string("xpm");
+       return string("ppm");
 }
 
 } // anon namespace
@@ -276,6 +353,17 @@ namespace grfx {
 void CacheItem::Impl::convertToDisplayFormat()
 {
        setStatus(Converting);
+
+       // First, check that the file exists!
+       if (!IsFileReadable(filename_)) {
+               if (status_ != ErrorNoFile) {
+                       setStatus(ErrorNoFile);
+                       lyxerr[Debug::GRAPHICS]
+                               << "\tThe file is not readable" << endl;
+               }
+               return;
+       }
+
        // Make a local copy in case we unzip it
        string const filename = zippedFile(filename_) ?
                unzipFile(filename_) : filename_;
@@ -285,13 +373,6 @@ void CacheItem::Impl::convertToDisplayFormat()
                << "\n\twith displayed filename: " << displayed_filename
                << endl;
 
-       // First, check that the file exists!
-       if (!IsFileReadable(filename)) {
-               setStatus(ErrorNoFile);
-               lyxerr[Debug::GRAPHICS] << "\tThe file is not readable" << endl;
-               return;
-       }
-
        string from = getExtFromContents(filename);
        lyxerr[Debug::GRAPHICS]
                << "\n\tThe file contains " << from << " format data." << endl;