[FFmpeg-devel] [patch] mjpeg restart marker parsing

tom harper d.thomas.harper
Wed Mar 31 00:20:48 CEST 2010


I updated the patch per patch check - can we start by getting the fix in for
not checking the padding bytes and go from there?

re: ff_er_add_slice() and ff_er_frame_end() - it seems like the ff_er api
could be used but it would require a bit more study/testing, as I would have
to simulate losing sets of blocks between markers- i don't have an example
of this off hand.

Also the loop would need to be advanced by the estimated number of blocks
lost, which would be wrong if we had cycled past more than mod 8 *
restart_interval blocks, as far as I understand.  If you didn't lose a set
of blocks (i.e. bit error, huffman coding error) I think the code would
basically exit.  So the safest (and easiest) thing to do would be to just
bail and call ff_er_frame_end if the code exits before processing all of the
blocks.  I don't know how useful it would be to recover slices w/o any
examples.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg_mjpeg_restart_marker_patch.diff
Type: application/octet-stream
Size: 2002 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100330/0efae5e1/attachment.obj>



More information about the ffmpeg-devel mailing list