[patch] SDLImage

Hi,

  1. I updated Zlib to 1.2.1 and Pnglib to 1.2.5

  2. I removed reading of comments of PNG from SDLImage. There is no need for it and
    some popular programs like ACDSEE make wrong IDAT chunks, so to load such pngs correctly
    we are to omit reading IDAT chunks (anyway it is useless).

    small example:
    http://activekitten.com/files/system.png
    Photoshop and ASDCEE read it OK,
    IE and old SDL_Image can’t read it. Modified SDL_Image is able to
    read it.

This archive contains complete set of new Dll’s for SDL_image and
sources modified (img_png.c)
http://activekitten.com/files/sdlimage_new.zip (278 kb)

To compile dlls I used the latest CVS sources.

Sam, could you review it and upload to CVS?–
Best regards,
Dmitry Yakimov, ISDEF member
ActiveKitten.com

This archive contains complete set of new Dll’s for SDL_image and
sources modified (img_png.c)
http://activekitten.com/files/sdlimage_new.zip (278 kb)

To compile dlls I used the latest CVS sources.

Sam, could you review it and upload to CVS?

Done! Thanks for the patch.

See ya,
-Sam Lantinga, Software Engineer, Blizzard Entertainment

Sam Lantinga wrote:

This archive contains complete set of new Dll’s for SDL_image and
sources modified (img_png.c)
http://activekitten.com/files/sdlimage_new.zip (278 kb)

To compile dlls I used the latest CVS sources.

Sam, could you review it and upload to CVS?

Done! Thanks for the patch.

See ya,
-Sam Lantinga, Software Engineer, Blizzard Entertainment


SDL mailing list
SDL at libsdl.org
http://www.libsdl.org/mailman/listinfo/sdl

Does this use the newer libpng? Because I had to recompile SDL_image
and link it to libpng 1.0.15 to stop an eventual crash when loading PNGs
in WIndows XP (Win98 and Linux were not affected).

-TomT64

Done! Thanks for the patch.

Does this use the newer libpng? Because I had to recompile SDL_image
and link it to libpng 1.0.15 to stop an eventual crash when loading PNGs
in WIndows XP (Win98 and Linux were not affected).

Not yet. I need to update the projects by hand to preserve VC++ 5.0
compatibility, and that will take a bit of time.

See ya,
-Sam Lantinga, Software Engineer, Blizzard Entertainment