Iain sent me a xpm that didn't load. It used "g" instead of "c" in it's

Iain sent me a xpm that didn't load.  It used "g" instead of "c" in it's
color table.

I thought that was borked until I saw Imlib, xv, and Gimp load it up w/o
problem...

So, fixed that (less than one line fix).  HOWEVER, if this is correct,
then the gdk XPM loader in gtk+ also has the same problem.

Owen, got an opinion on this?

Mark
This commit is contained in:
Mark Crichton 2000-02-14 23:26:01 +00:00
parent 97f2c2b7bb
commit d0f73034cf
2 changed files with 6 additions and 1 deletions

View File

@ -1,3 +1,8 @@
2000-02-14 Mark Crichton <crichton@gimp.org>
* gdk-pixbuf/io-xpm.c (xpm_extract_color): Added check for "g" color type.
Imlib/libXpm handle this, but we didn't.
2000-02-14 Darin Adler <darin@eazel.com>
* gdk-pixbuf/testpixbuf.c: Remove now-unnecessary cast.

View File

@ -195,7 +195,7 @@ xpm_extract_color (const gchar *buffer)
counter = 0;
while (ptr == NULL) {
if (buffer[counter] == 'c') {
if ((buffer[counter] == 'c') || (buffer[counter] == 'g')) {
ch = buffer[counter + 1];
if (ch == 0x20 || ch == 0x09)
ptr = &buffer[counter + 1];