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

[Stefano Sabatini]

On date Sunday 2007-06-10 12:20:24 +0200, Michael Niedermayer encoded:
> 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

Fixed, now it is:
int av_parse_video_frame_rate(AVRational *frame_rate, const char *arg)

> 
> [...]
> > 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

Fixed.
 
Cheers 
-- 
Stefano Sabatini
Linux user number 337176 (see http://counter.li.org)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: move-video-abbrs-support-to-libavcodec-05.patch
Type: text/x-diff
Size: 10715 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070611/2355cea7/attachment.patch>