if we can load direct without conversion, then do that.

[lyx.git] / src / graphics / GraphicsCacheItem.C

diff --git a/src/graphics/GraphicsCacheItem.C b/src/graphics/GraphicsCacheItem.C
index ed69a468e8973507625e3ccb9812629d211cab69..19fdc7bdccbcdc7c40978489eaf335bc77b0d1ae 100644 (file)
--- a/src/graphics/GraphicsCacheItem.C
+++ b/src/graphics/GraphicsCacheItem.C
@@ -401,19 +401,25 @@ string const findTargetFormat(string const & from)
        // There must be a format to load from. 
        lyx::Assert(!formats.empty());
 
-       grfx::GConverter const & graphics_converter = grfx::GConverter::get();
-
-       FormatList::const_iterator it  = formats.begin();
+       // First ascertain if we can load directly with no conversion
+       FormatList::const_iterator it1  = formats.begin();
        FormatList::const_iterator end = formats.end();
-       for (; it != end; ++it) {
-               if (graphics_converter.isReachable(from, *it))
-                       break;
+       for (; it1 != end; ++it1) {
+               if (from == *it1)
+                       return *it1;
        }
 
-       if (it == end)
-               return string();
+       // So, we have to convert to a loadable format. Can we?
+       grfx::GConverter const & graphics_converter = grfx::GConverter::get();
+
+       FormatList::const_iterator it2  = formats.begin();
+       for (; it2 != end; ++it2) {
+               if (graphics_converter.isReachable(from, *it2))
+                       return *it2;
+       }
 
-       return *it;
+       // Failed!
+       return string();
 }
 
 } // anon namespace