[FFmpeg-cvslog] r18381 - in trunk/libavcodec: mpeg12.c mpegvideo.c

[Baptiste Coudurier]

Michael Niedermayer wrote:
> On Wed, May 20, 2009 at 10:58:49PM -0700, Baptiste Coudurier wrote:
>> Baptiste Coudurier wrote:
>>> On 4/8/2009 1:48 PM, michael wrote:
>>>> Author: michael
>>>> Date: Wed Apr  8 22:48:44 2009
>>>> New Revision: 18381
>>>>
>>>> Log:
>>>> Fix mb_height for interlaced mpeg2.
>>>> This correctly fixes issue961, thanks to steven warren for spoting my
>>>> mistake.
>>> I believe this commit broke interlaced encoding in 320x240.
>>> I'm investigating.
>>>
>>> ffmpeg -i <file> -flags +ildct test.m2v
>>>
>>> ffplay test.m2v:
>>>
>>> [mpeg2video @ 0x9bb320]Warning MVs not available
>>> [mpeg2video @ 0x9bb320]concealing 20 DC, 20 AC, 20 MV errors
>>> [mpeg2video @ 0x9bb320]Warning MVs not available
>>> [mpeg2video @ 0x9bb320]concealing 20 DC, 20 AC, 20 MV errors
>>> [mpeg2video @ 0x9bb320]Warning MVs not available
>>> [mpeg2video @ 0x9bb320]concealing 20 DC, 20 AC, 20 MV errors
>>>
>> And here is the patch.
> [...]
>>  mpegvideo_enc.c |    6 ++++--
>>  1 file changed, 4 insertions(+), 2 deletions(-)
>> 3e24f4fec2798ee5c2246bed821de13401d82179  interlaced_encoding_not_mul32.patch
> 
> ok
> 

Thanks, applied.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org