Skip to content

Conversation

@kornelski
Copy link
Member

This is implementation that adds support for reading PNG in cjpeg. All PNG variants are supported with caveats: alpha channel is stripped, 48bit is converted to 24bit, color profiles are ignored.

However, I'm not touching autotools with a 10-foot pole, so I haven't done anything to link with libpng and define PNG_SUPPORTED.

Can somebody else add the libpng detection part?

@kornelski kornelski mentioned this pull request Aug 4, 2014
@bdaehlie bdaehlie added this to the v3.0 milestone Aug 6, 2014
@dwbuiten
Copy link
Contributor

@pornel For the autotools part, you want to use the standard pkg-config macros: https://www.flameeyes.eu/autotools-mythbuster/pkgconfig/pkg_check_modules.html

For cmake... whatever it uses for pkg-config.

@kornelski
Copy link
Member Author

Thanks @dwbuiten! It works with autotools now.

bdaehlie pushed a commit that referenced this pull request Aug 15, 2014
PNG reading support
@bdaehlie bdaehlie merged commit df9b78a into mozilla:master Aug 15, 2014
@kornelski kornelski deleted the png branch August 15, 2014 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants