]> git.lyx.org Git - features.git/commitdiff
Optimize the graphics loader: don't put the image in the queue if it is directly...
authorAbdelrazak Younes <younes@lyx.org>
Fri, 6 Jun 2008 13:30:38 +0000 (13:30 +0000)
committerAbdelrazak Younes <younes@lyx.org>
Fri, 6 Jun 2008 13:30:38 +0000 (13:30 +0000)
git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@25166 a592a061-630c-0410-9148-cb99ea01b6c8

src/graphics/GraphicsCacheItem.cpp
src/graphics/GraphicsCacheItem.h
src/graphics/GraphicsLoader.cpp

index 592e51c5257cdf9c14f77754fd2b77cbbeac291b..3359682ce875e5b79c8cbc16eb391251289a46d7 100644 (file)
@@ -41,6 +41,13 @@ public:
        ///
        Impl(FileName const & file);
 
+       /**
+        *  If no file conversion is needed, then tryDisplayFormat() calls
+        *  loadImage() directly.
+        * \return true if a conversion is necessary.
+        */
+       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
@@ -48,9 +55,6 @@ public:
         *  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().
         */
@@ -135,6 +139,16 @@ FileName const & CacheItem::filename() const
 }
 
 
+bool CacheItem::tryDisplayFormat() const
+{
+       if (pimpl_->status_ != WaitingToLoad)
+               pimpl_->reset();
+       FileName filename;
+       string from;
+       return pimpl_->tryDisplayFormat(filename, from);
+}
+
+
 void CacheItem::startLoading() const
 {
        pimpl_->startLoading();
@@ -335,7 +349,7 @@ static string const findTargetFormat(string const & from)
 }
 
 
-void CacheItem::Impl::convertToDisplayFormat()
+bool CacheItem::Impl::tryDisplayFormat(FileName & filename, string & from)
 {
        setStatus(Converting);
 
@@ -345,11 +359,9 @@ void CacheItem::Impl::convertToDisplayFormat()
                        setStatus(ErrorNoFile);
                        LYXERR(Debug::GRAPHICS, "\tThe file is not readable");
                }
-               return;
+               return true;
        }
 
-       // Make a local copy in case we unzip it
-       FileName filename;
        zipped_ = filename_.isZippedFile();
        if (zipped_) {
                unzipped_filename_ = FileName::tempName(
@@ -357,7 +369,7 @@ void CacheItem::Impl::convertToDisplayFormat()
                if (unzipped_filename_.empty()) {
                        setStatus(ErrorConverting);
                        LYXERR(Debug::GRAPHICS, "\tCould not create temporary file.");
-                       return;
+                       return true;
                }
                filename = unzipFile(filename_, unzipped_filename_.toFilesystemEncoding());
        } else {
@@ -369,7 +381,7 @@ void CacheItem::Impl::convertToDisplayFormat()
                << "\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);
                LYXERR(Debug::GRAPHICS, "\tCould not determine file format.");
@@ -382,18 +394,29 @@ void CacheItem::Impl::convertToDisplayFormat()
                LYXERR(Debug::GRAPHICS, "\tNo conversion needed (from == to)!");
                file_to_load_ = filename;
                loadImage();
-               return;
+               return true;
        }
 
        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;
+               return true;
        }
+       return false;
+}
+
 
+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))
+               return;
+
        // 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");
index 3f8f7e100f75188fcbc44515dc66415c73669279..481e4b117a6a770f132daa2c4125f46644ccf547 100644 (file)
@@ -53,6 +53,9 @@ public:
        ///
        support::FileName const & filename() const;
 
+       /// Try to load a display format.
+       bool tryDisplayFormat() const;
+
        /// It's in the cache. Now start the loading process.
        void startLoading() const;
 
index 8a3d7bb5394c2a5e47a5e1350d35d47f918f1a05..eaa24493f67cd96936c792445f41da681944dba7 100644 (file)
@@ -433,6 +433,9 @@ void Loader::Impl::startLoading()
        if (status_ != WaitingToLoad)
                return;
 
+       if (cached_item_->tryDisplayFormat())
+               return;
+
        LoaderQueue::get().touch(cached_item_);
 }