[FFmpeg-devel] [PATCH] Move the video size and rate abbreviations system from libavformat to libavcodec

[Michael Niedermayer]

Hi

On Sat, Jun 09, 2007 at 11:46:09AM +0200, Stefano Sabatini wrote:
[...]
> +int av_parse_video_frame_rate(int *frame_rate, int *frame_rate_base, const char *arg)
> +{

instead of 2 int *, a AVRational * could be used, its the correct data
type for the frame rate

[...]
> Index: libavcodec/avcodec.h
> ===================================================================
> --- libavcodec/avcodec.h	(revision 9235)
> +++ libavcodec/avcodec.h	(working copy)
> @@ -3143,6 +3143,32 @@
>  
>  extern unsigned int av_xiphlacing(unsigned char *s, unsigned int v);
>  
> +/**
> + * Parses \p str and put in \p width_ptr and \p height_ptr the detected values.
> + *
> + * @return 0 in case of a successfull parsing, -1 otherwise

<0 for fail not just -1 (same for all other functions



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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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/20070610/3573fc48/attachment.pgp>