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

Already on GitHub? Sign in to your account

Jpeg encoder #909

merged 6 commits into from May 7, 2012


None yet
1 participant

theuni commented Apr 30, 2012

A bit late to the May party, but we'll see...

This gives us a native Jpeg encoder. One of the bigger saves actually comes from not using cximage's create* methods, which use their slow open.

The code is pretty ugly, but i wanted to re-use as much as possible. So the encode functions are essentially a waterfall. They can't be made static because there is too much to save in between the decode and the encode, and I didn't want to introduce a struct just to work with that data.

Passes all the test-files that accumulated for me after writing the decoder.

@theuni theuni added a commit that referenced this pull request May 7, 2012

@theuni theuni Merge pull request #909 from theuni/jpeg-encoder
jpeg: Add a native Jpeg encoder bypassing cximage

Gives ~1.5x speedup for standard images.

Gives ~15x speedup and big size decrease on thumbs due to the fact that we were (wrongly) encoding them to pngs before.

@theuni theuni merged commit 1d99e8b into xbmc:master May 7, 2012

@tru tru added a commit to plexinc/plex-home-theater-public that referenced this pull request Dec 4, 2013

@tru tru Merge pull request #909 from ziggimon/build-fix
Fixing linux compile.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment