[Ffmpeg-devel] [PATCH] mjpeg cleanup and again interlaced fix

Baptiste Coudurier baptiste.coudurier
Wed Apr 11 14:17:14 CEST 2007


Hi

3 patches:

- remove useless MpegEncContext.
- fix odd field height decoding.
- fix some interlaced content.

justification:
odd_heigth.mov, this sample is 720x405, and setting height *= 2 is then
wrong, correct is to set it to container height.
Since height change every field, do that for every field, not only for
first picture.

ODML specs specify the use of AVI1 tag, and meaning of polarity (0 for
prog, 1 means encoded from first field, 2 means encoded from second
field), and only set polarity when first field is being decoded, use of
new second_field in context.

If anyone has some special/broken mjpeg, mention them I'll test against
this patch.

I'll remove useless struct fields after that.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mjpeg_idct.patch
Type: text/x-diff
Size: 2242 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070411/963aea20/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: odd_field.patch
Type: text/x-diff
Size: 792 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070411/963aea20/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: interlaced_fix.patch
Type: text/x-diff
Size: 4516 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070411/963aea20/attachment-0002.patch>



More information about the ffmpeg-devel mailing list