#205 libpng 1.6.1 pngwrite.c png_image_write_main

Milestone: libpng_code

Status: closed-fixed

Owner: Glenn Randers-Pehrson

Labels: None

Priority: 5

Updated: 2013-04-26

Created: 2013-04-17

Creator: yuriy_levchenko

Private: No

pngwrite.c : 2090 (png_image_write_main)

png_set_sRGB - not present if i undefined PNG_sRGB_SUPPORTED

maybe -

ifdef PNG_sRGB_SUPPORTED

else if (!(image->flags & PNG_IMAGE_FLAG_COLORSPACE_NOT_sRGB))
png_set_sRGB(png_ptr, info_ptr, PNG_sRGB_INTENT_PERCEPTUAL);

endif

Discussion

Glenn Randers-Pehrson - 2013-04-17

status: open --> open-accepted

assigned_to: Glenn Randers-Pehrson
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Glenn Randers-Pehrson - 2013-04-17

Thanks. Patched in the libpng16 branch of the GIT repo. There are similar situations with cHRM and gAMA which I also patched.

I'm not sure how well the simplified interface will work with sRGB and gAMA not supported, though.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Glenn Randers-Pehrson - 2013-04-17

It is a mistake to enable the simplified API without enabling sRGB and gAMA as well. In libpng-1.6.3 we will attempt to prevent that condition with appropriate dependencies.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Glenn Randers-Pehrson - 2013-04-26

Fixed in libpng-1.6.2, thanks.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Glenn Randers-Pehrson - 2013-04-26

status: open-accepted --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.