* 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.
/**
* If no file conversion is needed, then tryDisplayFormat() calls
* loadImage() directly.
- * \return true if a conversion is necessary.
+ * \return true if a conversion is necessary and no error occurred.
*/
bool tryDisplayFormat(FileName & filename, string & from);
/** 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
pimpl_->reset();
FileName filename;
string from;
- return pimpl_->tryDisplayFormat(filename, from);
+ bool const conversion_needed = pimpl_->tryDisplayFormat(filename, from);
+ bool const success = status() == Loaded && !conversion_needed;
+ if (!success)
+ pimpl_->reset();
+ return success;
}
// 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_.reset(Image::newImage());
+ image_.reset(newImage());
bool success = image_->load(file_to_load_);
string const text = success ? "succeeded" : "failed";
if (remove_loaded_file_ && unzipped_filename_ != file_to_load_)
file_to_load_.removeFile();
- if (!success) {
- setStatus(ErrorLoading);
- return;
- }
-
- // Inform the outside world.
- setStatus(Loaded);
+ return success;
}
bool CacheItem::Impl::tryDisplayFormat(FileName & filename, string & from)
{
- setStatus(Converting);
-
// First, check that the file exists!
if (!filename_.isReadableFile()) {
if (status_ != ErrorNoFile) {
- setStatus(ErrorNoFile);
+ status_ = ErrorNoFile;
LYXERR(Debug::GRAPHICS, "\tThe file is not readable");
}
- return true;
+ return false;
}
zipped_ = filename_.isZippedFile();
unzipped_filename_ = FileName::tempName(
filename_.toFilesystemEncoding());
if (unzipped_filename_.empty()) {
- setStatus(ErrorConverting);
+ status_ = ErrorConverting;
LYXERR(Debug::GRAPHICS, "\tCould not create temporary file.");
- return true;
+ return false;
}
filename = unzipFile(filename_, unzipped_filename_.toFilesystemEncoding());
} else {
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 true;
+ 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 true;
+ status_ = loadImage() ? Loaded : ErrorLoading;
+ return false;
}
- return false;
+ return true;
}
// Make a local copy in case we unzip it
FileName filename;
string from;
- if (tryDisplayFormat(filename, 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 = FileName::tempName("CacheItem");
remove_loaded_file_ = true;
- // Remove the temp file, we only want the name...
- // FIXME: This is unsafe!
- to_file_base.removeFile();
-
// 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.