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

[Michael Niedermayer]

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


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070620/a667e50c/attachment.pgp>