patch for image.el

patch for image.el

[chrisl]

Hi,

I'm using GNU emacs 21.3.1 under Linux.  I discovered that image.el
didn't recognize some of my jpeg files (it reported them as not being
image files).  By using file(1) and looking at the magic(5) file, I
saw that there is a different jpeg format in addition to JFIF: Exif.
I also found that image-jpeg-p can cause substring to run past the end
of the data array, which causes an error.  I made the following change
that fixes both problems:

67,69c67
< 		     (string-match "JFIF\\|Exif" 
< 				   (substring data i 
< 					      (min (+ i nbytes) len)))))
---
> 		     (string-match "JFIF" (substring data i (+ i nbytes)))))

Thanks for maintaining The One True Editor. :)

- Chris

Re: patch for image.el

[Richard Stallman]

      By using file(1) and looking at the magic(5) file, I
    saw that there is a different jpeg format in addition to JFIF: Exif.

We have fixed this in the version in CVS on Savannah.gnu.org, but
thanks anyway.

    I'm using GNU emacs 21.3.1 under Linux...
    Thanks for maintaining The One True Editor. :)

This editor is one of the programs we developed for the GNU operating
system.  It looks like you're using the entire GNU system.

Most users of GNU don't realize it is GNU, because they think the
whole system is "Linux".  You could help inform them, simply by saying
you are running GNU Emacs 21.3.1 under GNU/Linux.  That is the form
of thanks that will really help.

See http://www.gnu.org/gnu/gnu-linux-faq.html for more
explanation of the situation.