[FFmpeg-devel] [PATCH] Split libavcodec/utils.c:{AbvEntry, frame_abvs} into VideoFrame{Size, Rate}Abbr and video_frame_{rate, size}_abbrs
Benoit Fouet
benoit.fouet
Thu Jun 21 09:47:00 CEST 2007
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
More information about the ffmpeg-devel
mailing list