(image-type): Use image-type-from-file-name (from trunk

2007-05-21  Chong Yidong  <cyd@stupidchicken.com>).

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 5f19204e0b589263edd6c9dedc9c56ae276869f8..86a9c4dc2ec442ac3d5125b3534f0f723b2b702b 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2008-01-14  Jason Rumney  <jasonr@gnu.org>
+
+       * image.el (image-type): Use image-type-from-file-name (from trunk
+       2007-05-21  Chong Yidong  <cyd@stupidchicken.com>).
+
 2008-01-12  Glenn Morris  <rgm@gnu.org>
 
        * woman.el (woman-parse-numeric-arg): Change handling of `==':

diff --git a/lisp/image.el b/lisp/image.el
index 2deaf0a5ebf5b82af1779023df5ff42548ff0b22..d111f7e82a817d2c7378ae74cbf6aabb3c77e6bf 100644 (file)
--- a/lisp/image.el
+++ b/lisp/image.el
@@ -313,22 +313,14 @@ use its file extension as image type.
 Optional DATA-P non-nil means FILE-OR-DATA is a string containing image data."
   (when (and (not data-p) (not (stringp file-or-data)))
     (error "Invalid image file name `%s'" file-or-data))
-  (cond ((null data-p)
-        ;; FILE-OR-DATA is a file name.
-        (unless (or type
-                    (setq type (image-type-from-file-header file-or-data)))
-          (let ((extension (file-name-extension file-or-data)))
-            (unless extension
-              (error "Cannot determine image type"))
-            (setq type (intern extension)))))
-       (t
-        ;; FILE-OR-DATA contains image data.
-        (unless type
-          (setq type (image-type-from-data file-or-data)))))
   (unless type
-    (error "Cannot determine image type"))
-  (unless (symbolp type)
-    (error "Invalid image type `%s'" type))
+    (setq type (if data-p 
+                   (image-type-from-data file-or-data)
+                 (or (image-type-from-file-header file-or-data)
+                     (image-type-from-file-name file-or-data))))
+    (or type(error "Cannot determine image type")))
+  (or (memq type (and (boundp 'image-types) image-types))
+      (error "Invalid image type `%s'" type))
   type)