* Licence details can be found in the file COPYING.
*
* \author Baruch Even
- * \author Herbert Voß
+ * \author Herbert Voß
* \author Angus Leeming
*
* Full author contact details are available in file CREDITS.
#include <config.h>
#include "GraphicsCacheItem.h"
+
+#include "GraphicsCache.h"
#include "GraphicsConverter.h"
#include "GraphicsImage.h"
#include "ConverterCache.h"
-#include "debug.h"
#include "Format.h"
+#include "support/debug.h"
+#include "support/FileName.h"
#include "support/filetools.h"
#include "support/FileMonitor.h"
-#include "support/lyxlib.h"
#include <boost/bind.hpp>
+using namespace std;
+using namespace lyx::support;
namespace lyx {
-using support::FileMonitor;
-using support::FileName;
-using support::makeDisplayPath;
-using support::onlyFilename;
-using support::tempName;
-using support::unlink;
-using support::unzipFile;
-
-using std::endl;
-using std::string;
-
-
namespace graphics {
class CacheItem::Impl : public boost::signals::trackable {
///
Impl(FileName const & file);
+ /**
+ * If no file conversion is needed, then tryDisplayFormat() calls
+ * loadImage() directly.
+ * \return true if a conversion is necessary and no error occurred.
+ */
+ bool tryDisplayFormat(FileName & filename, string & from);
+
/** Start the image conversion process, checking first that it is
* necessary. If it is necessary, then a conversion task is started.
* CacheItem asumes that the conversion is asynchronous and so
* is finished, this Signal is emitted, returning the converted
* file to this->imageConverted.
*
- * If no file conversion is needed, then convertToDisplayFormat() calls
- * loadImage() directly.
- *
* convertToDisplayFormat() will set the loading status flag as
* approriate through calls to setStatus().
*/
/** Load the image into memory. This is called either from
* convertToDisplayFormat() direct or from imageConverted().
*/
- void loadImage();
+ bool loadImage();
/** Get a notification when the image conversion is done.
* Connected to a signal on_finish_ which is passed to
*/
void imageConverted(bool);
- /** Get a notification when the image loading is done.
- * Connected to a signal on_finish_ which is passed to
- * lyx::graphics::Image::loadImage.
- */
- void imageLoaded(bool);
-
/** Sets the status of the loading process. Also notifies
* listeners that the status has changed.
*/
/// This signal is emitted when the image loading status changes.
boost::signal<void()> statusChanged;
- /// The connection to the signal Image::finishedLoading
- boost::signals::connection cl_;
-
/// The connection of the signal ConvProcess::finishedConversion,
boost::signals::connection cc_;
CacheItem::~CacheItem()
-{}
+{
+ delete pimpl_;
+}
FileName const & CacheItem::filename() const
}
+bool CacheItem::tryDisplayFormat() const
+{
+ if (pimpl_->status_ != WaitingToLoad)
+ pimpl_->reset();
+ FileName filename;
+ string from;
+ bool const conversion_needed = pimpl_->tryDisplayFormat(filename, from);
+ bool const success = status() == Loaded && !conversion_needed;
+ if (!success)
+ pimpl_->reset();
+ return success;
+}
+
+
void CacheItem::startLoading() const
{
pimpl_->startLoading();
{
zipped_ = false;
if (!unzipped_filename_.empty())
- unlink(unzipped_filename_);
+ unzipped_filename_.removeFile();
unzipped_filename_.erase();
if (remove_loaded_file_ && !file_to_load_.empty())
- unlink(file_to_load_);
+ file_to_load_.removeFile();
remove_loaded_file_ = false;
file_to_load_.erase();
to_.erase();
status_ = WaitingToLoad;
- if (cl_.connected())
- cl_.disconnect();
-
if (cc_.connected())
cc_.disconnect();
converter_.reset();
cc_.disconnect();
- success = !file_to_load_.empty() && file_to_load_.isFileReadable();
+ success = !file_to_load_.empty() && file_to_load_.isReadableFile();
if (!success) {
LYXERR(Debug::GRAPHICS, "Unable to find converted file!");
setStatus(ErrorConverting);
if (zipped_)
- unlink(unzipped_filename_);
+ unzipped_filename_.removeFile();
return;
}
// Add the converted file to the file cache
ConverterCache::get().add(filename_, to_, file_to_load_);
- loadImage();
+ setStatus(loadImage() ? Loaded : ErrorLoading);
}
// This function gets called from the callback after the image has been
// converted successfully.
-void CacheItem::Impl::loadImage()
+bool CacheItem::Impl::loadImage()
{
- setStatus(Loading);
LYXERR(Debug::GRAPHICS, "Loading image.");
- image_ = Image::newImage();
-
- cl_.disconnect();
- cl_ = image_->finishedLoading.connect(
- boost::bind(&Impl::imageLoaded, this, _1));
- image_->load(file_to_load_);
-}
-
+ image_.reset(newImage());
-void CacheItem::Impl::imageLoaded(bool success)
-{
+ bool success = image_->load(file_to_load_);
string const text = success ? "succeeded" : "failed";
LYXERR(Debug::GRAPHICS, "Image loading " << text << '.');
// Clean up after loading.
if (zipped_)
- unlink(unzipped_filename_);
+ unzipped_filename_.removeFile();
if (remove_loaded_file_ && unzipped_filename_ != file_to_load_)
- unlink(file_to_load_);
+ file_to_load_.removeFile();
- cl_.disconnect();
-
- if (!success) {
- setStatus(ErrorLoading);
- return;
- }
-
- // Inform the outside world.
- setStatus(Loaded);
+ return success;
}
static string const findTargetFormat(string const & from)
{
- typedef lyx::graphics::Image::FormatList FormatList;
- FormatList const formats = lyx::graphics::Image::loadableFormats();
+ typedef vector<string> FormatList;
+ FormatList const & formats = Cache::get().loadableFormats();
// There must be a format to load from.
- BOOST_ASSERT(!formats.empty());
+ LASSERT(!formats.empty(), /**/);
// Use the standard converter if we don't know the format to load
// from.
}
-void CacheItem::Impl::convertToDisplayFormat()
+bool CacheItem::Impl::tryDisplayFormat(FileName & filename, string & from)
{
- setStatus(Converting);
-
// First, check that the file exists!
- if (!filename_.isFileReadable()) {
+ if (!filename_.isReadableFile()) {
if (status_ != ErrorNoFile) {
- setStatus(ErrorNoFile);
+ status_ = ErrorNoFile;
LYXERR(Debug::GRAPHICS, "\tThe file is not readable");
}
- return;
+ return false;
}
- // Make a local copy in case we unzip it
- FileName filename;
zipped_ = filename_.isZippedFile();
if (zipped_) {
- unzipped_filename_ = tempName(FileName(), filename_.toFilesystemEncoding());
+ unzipped_filename_ = FileName::tempName(
+ filename_.toFilesystemEncoding());
if (unzipped_filename_.empty()) {
- setStatus(ErrorConverting);
+ status_ = ErrorConverting;
LYXERR(Debug::GRAPHICS, "\tCould not create temporary file.");
- return;
+ return false;
}
filename = unzipFile(filename_, unzipped_filename_.toFilesystemEncoding());
} else {
<< "\tAttempting to convert image file: " << filename
<< "\n\twith displayed filename: " << to_utf8(displayed_filename));
- string const from = formats.getFormatFromFile(filename);
+ from = formats.getFormatFromFile(filename);
if (from.empty()) {
- setStatus(ErrorConverting);
+ status_ = ErrorConverting;
LYXERR(Debug::GRAPHICS, "\tCould not determine file format.");
}
LYXERR(Debug::GRAPHICS, "\n\tThe file contains " << from << " format data.");
// No conversion needed!
LYXERR(Debug::GRAPHICS, "\tNo conversion needed (from == to)!");
file_to_load_ = filename;
- loadImage();
- return;
+ status_ = loadImage() ? Loaded : ErrorLoading;
+ return false;
}
if (ConverterCache::get().inCache(filename, to_)) {
LYXERR(Debug::GRAPHICS, "\tNo conversion needed (file in file cache)!");
file_to_load_ = ConverterCache::get().cacheName(filename, to_);
- loadImage();
- return;
+ status_ = loadImage() ? Loaded : ErrorLoading;
+ return false;
}
+ return true;
+}
+
+void CacheItem::Impl::convertToDisplayFormat()
+{
LYXERR(Debug::GRAPHICS, "\tConverting it to " << to_ << " format.");
+ // Make a local copy in case we unzip it
+ FileName filename;
+ string from;
+ if (!tryDisplayFormat(filename, from)) {
+ // The image status has changed, tell it to the outside world.
+ statusChanged();
+ return;
+ }
+
+ // We will need a conversion, tell it to the outside world.
+ setStatus(Converting);
+
// Add some stuff to create a uniquely named temporary file.
// This file is deleted in loadImage after it is loaded into memory.
- FileName const to_file_base(tempName(FileName(), "CacheItem"));
+ FileName const to_file_base = FileName::tempName("CacheItem");
remove_loaded_file_ = true;
- // Remove the temp file, we only want the name...
- // FIXME: This is unsafe!
- unlink(to_file_base);
-
// Connect a signal to this->imageConverted and pass this signal to
// the graphics converter so that we can load the modified file
// on completion of the conversion process.