Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

resolve incompatibility with ipp-samples (Intel Performance Primitives) JPEG decoder #48

Closed
bdaehlie opened this issue May 8, 2014 · 3 comments
Assignees
Milestone

Comments

@bdaehlie
Copy link
Contributor

bdaehlie commented May 8, 2014

Images generated with mozjpeg's command line tool with mozjpeg defaults (with multiple scans) cannot be decoded by ipp-samples JPEG decoder.

@bdaehlie bdaehlie added this to the v2.0 milestone May 8, 2014
fbossen added a commit that referenced this issue May 8, 2014
Add option to have a single DC scan wherein all components are
interleaved when using progressive mode. This may resolve compatibility
issues raised in #29 and #48.
This option is available through -onedcscan in cjpeg
@bdaehlie
Copy link
Contributor Author

bdaehlie commented May 9, 2014

Using the '-onedcscan' option resolves this. We should probably enable it by default.

@bdaehlie
Copy link
Contributor Author

bdaehlie commented May 9, 2014

Resolved with the following commit, which makes one dc scan the default:

87254c1

@bdaehlie bdaehlie closed this as completed May 9, 2014
@ace-dent
Copy link

ace-dent commented May 9, 2014

Similar to comment by @pornel re Opera compatibility?...
https://github.com/pornel/ImageOptim/blob/master/jpegtran/jpegrescan#L90
It seems a shame to limit compression based on certain decoding libraries. There should be some discussion of this re. #37 to define supported majority (and highlight any minority decoder implementations that wont be supported).

kornelski pushed a commit to ImageOptim/mozjpeg-cocoa that referenced this issue Apr 28, 2016
Partially reverts 54014d9.  When
building from a git sandbox, as opposed to from an official source
tarball, it is still necessary to run autoreconf.

Closes mozilla#48
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

No branches or pull requests

3 participants