[FFmpeg-devel] [PATCH] Split libavcodec/utils.c:{AbvEntry, frame_abvs} into VideoFrame{Size, Rate}Abbr and video_frame_{rate, size}_abbrs

[Benoit Fouet]

Michael Niedermayer wrote:
> Hi
>
> On Wed, Jun 20, 2007 at 11:56:56AM +0200, Stefano Sabatini wrote:
>   
>> Hi to all,
>>
>> this patch does: 
>>
>> * splits struct libavcodec/utils.c:AbvEntry into
>>   VideoFrame{Size,Rate}Abbr
>>
>> * splits the array libavcodec/utils.c:frame_abvs into
>>   video_frame_{rate,size}_abbrs
>>
>> * changes libavcodec/utils.c:av_parse_video_frame_{rate,size} accordingly.
>>
>> This patch fixes the bug raised by the command:
>> ffmpeg -i input -r qcif output
>>
>> (ffmpeg doesn't complain about the invalid rate abbreviation, then
>> sets the video frame rate to {0,0}).
>>
>> With the patch applied ffmpeg complains about the incorrect frame
>> rate and quits.
>>     
>
> patch looks ok
>
>   

Applied

Ben
-- 
Purple Labs S.A.
www.purplelabs.com