Created attachment 18963 [details] [review] libjpeg based jpeg decoder So this new revision have a couple of changes: - fix a brown paper bag, i was reading the compressed data 1 (!!) byte at time, because the silly of me not only was using sizeof() on a malloc'ed area but i was even dereferencing it. Should now be a lot faster. - improve the jpeg fscked soi workaround, now it doesn't try to patch the wrong soi, if there is a sane soi after that it justs skips data. Don't think the old was correct because libjpeg refused to decode the picture. Feel free to use this bug to track regressions with dsjpeg.
(In reply to comment #0) > - improve the jpeg fscked soi workaround, now it doesn't try to patch the wrong > soi, if there is a sane soi after that it justs skips data. Don't think the old > was correct because libjpeg refused to decode the picture. Just noticed it fixes some testcase like the orisins game but there are stil cases like pitch hitter 2 that are not working. So not ready for inclusion.
Another site with jpeg image corruption: http://thefiresign.com/firesign_121706.swf Note to self: I've checked gnash and it works fine for both pinch hitter and the one above, there should be something dumb i'm doing.
Created attachment 21505 [details] [review] ijg libjepg based jpeg decoding - v9 This is an updated version against origin/master with these changes: - better bogus jpeg header workaround, now same as gnash - handling of cmyk and grayscale color space - every file that previously failed now works \o/ The patch is till not ready for inclusion because: - it needs review :) - swfdec_jpeg_cmyk_to_argb loop does not compile - more testing to sort out if i can remove ifdeffed out code in swfdec_jpeg_decode_argb, think so
Created attachment 21518 [details] ijg libjepg based jpeg decoding - v10 Changelog from previous version: - reworked swfdec_jpeg_x_to_argb functions a bit: cinfo->output_componenents is used to loop over the input image and a check of its expected size is added. Side effect is that swfdec_jpeg_cmyk_to_argb now compiles. - removed code under #if 0
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.