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

Baptiste Coudurier baptiste.coudurier
Fri Apr 13 22:44:43 CEST 2007


Hi

Michael Niedermayer wrote:
> Hi
> 
> On Wed, Apr 11, 2007 at 03:39:28PM +0200, Baptiste Coudurier wrote:
>> Michael Niedermayer wrote:
>>> Hi
>>>
>>> On Wed, Apr 11, 2007 at 02:17:14PM +0200, Baptiste Coudurier wrote:
>>>> Hi
>>>>
>>>> 3 patches:
>>>>
>>>> - remove useless MpegEncContext.
>>> ok if it works
>>>
>>>
>>>> - 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.
>>> as org_height is only set during coedec init i dont see how doing
>>> height= org_height per field could change the height per field
>> It changes avctx height, not height per field ? Feel free to fix it
>> in another way if you prefer.
> 
> fine ill accept the 2nd patch too

Nice, what about the third ?

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312




More information about the ffmpeg-devel mailing list