[FFmpeg-devel] [PATCH] Document av_[io]format_next() functions

Stefano Sabatini stefano.sabatini-lala
Sat Feb 14 17:13:25 CET 2009


On date Saturday 2009-02-14 16:16:47 +0100, Michael Niedermayer encoded:
> On Sat, Feb 14, 2009 at 03:15:19PM +0100, Stefano Sabatini wrote:
> > Hi,
> > as in subject.
> > 
> > Regards.
> > -- 
> > FFmpeg = Fierce and Faboulous Mastodontic Prodigious Erratic Governor
> 
> > Index: libavformat/avformat.h
> > ===================================================================
> > --- libavformat/avformat.h	(revision 17180)
> > +++ libavformat/avformat.h	(working copy)
> > @@ -687,7 +687,18 @@
> >  extern AVOutputFormat *first_oformat;
> >  #endif
> >  
> > +/**
> > + * Returns the first registered input format if \p f is NULL,
> > + * otherwise the registered input format next to \p f or NULL if \p f
> > + * is the last one.
> > + */
> >  AVInputFormat  *av_iformat_next(AVInputFormat  *f);
> > +
> > +/**
> > + * Returns the first registered output format if \p f is NULL,
> > + * otherwise the registered output format next to \p f or NULL if \p f
> > + * is the last one.
> > + */
> >  AVOutputFormat *av_oformat_next(AVOutputFormat *f);
> 
> please write this in a way that is readable, i mean
> if X then Y
> if Z then U
> ...
> not
> if X then U
> else V if T
> then Z

Maybe clearer...

Regards.
-- 
FFmpeg = Friendly and Foolish Mind-dumbing Purposeless Exploitable Guru
-------------- next part --------------
A non-text attachment was scrubbed...
Name: document-av-ioformat-next.patch
Type: text/x-diff
Size: 757 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090214/ec015216/attachment.patch>



More information about the ffmpeg-devel mailing list