(mh-x-image-url-cache-canonicalize): Use url-hexify-string to remove

special characters from filenames (closes SF #1396499).

lisp/mh-e/ChangeLog

+2006-01-11  Bill Wohler  <wohler@newt.com>
+
+	* mh-utils.el (mh-x-image-url-cache-canonicalize): Use
+	url-hexify-string to remove special characters from filenames
+	(closes SF #1396499). Note that this invalidates the existing
+	names in your cache so you might as well remove
+	~/Mail/.mhe-x-image-cache/* now.
+	
 2006-01-10  Bill Wohler  <wohler@newt.com>
 
 	* mh-comp.el (mh-insert-letter): If you choose a different folder,
-	the cur message is used (closes SF #1205890).
+	the cur message is used. Sync docstring with manual (closes SF
+	#1205890).
 
 	* mh-mime.el (mh-compose-forward): Use standard range argument
 	instead of messages. Use more powerful mh-read-range instead of

lisp/mh-e/mh-utils.el

@@ -1489,12 +1489,16 @@ path is generated."
 
 (defun mh-x-image-url-cache-canonicalize (url)
   "Canonicalize URL.
-Replace the ?/ character with a ?! character and append .png."
-   (format "%s/%s.png" mh-x-image-cache-directory
+Replace the ?/ character with a ?! character and append .png.
+Also replaces special characters with `url-hexify-string' since
+not all characters, such as :, are legal within Windows
+filenames. See URL `http://msdn.microsoft.com/library/default.asp?url=/library/en-us/fileio/fs/naming_a_file.asp'."
+  (format "%s/%s.png" mh-x-image-cache-directory
+          (url-hexify-string
            (with-temp-buffer
              (insert url)
              (mh-replace-string "/" "!")
-             (buffer-string))))
+             (buffer-string)))))
 
 (defun mh-x-image-set-download-state (file data)
   "Setup a symbolic link from FILE to DATA."