[FFmpeg-devel] [PATCH] set avg frame rate in mkv demuxer

Baptiste Coudurier baptiste.coudurier
Mon May 17 20:13:14 CEST 2010


Hi Aurel,

On 05/16/2010 09:34 AM, Aurelien Jacobs wrote:
> On Fri, May 14, 2010 at 03:24:44PM -0700, Baptiste Coudurier wrote:
>> Hi
>>
>> $subject.
>>
>
>> Index: libavformat/matroskadec.c
>> ===================================================================
>> --- libavformat/matroskadec.c	(revision 23138)
>> +++ libavformat/matroskadec.c	(working copy)
>> @@ -1390,6 +1390,8 @@
>>                         255);
>>               if (st->codec->codec_id != CODEC_ID_H264)
>>               st->need_parsing = AVSTREAM_PARSE_HEADERS;
>> +            if (track->default_duration>  0)
>
> default_duration is unsigned, so no need to test for>0. The following
> should be enough:
>    if (track->default_duration)
>
> Except this, patch OK.
>

Thanks, changed and applied.

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



More information about the ffmpeg-devel mailing list