[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