[FFmpeg-devel] [PATCH] Implement a cmdutils functions for listing available elements
Michael Niedermayer
michaelni
Wed Nov 18 00:43:40 CET 2009
On Sun, Nov 15, 2009 at 12:40:21PM +0100, Stefano Sabatini wrote:
> On date Sunday 2009-11-15 01:38:28 +0100, Michael Niedermayer encoded:
> > On Sun, Nov 15, 2009 at 12:51:56AM +0100, Stefano Sabatini wrote:
> > > On date Sunday 2009-11-15 00:10:22 +0100, Michael Niedermayer encoded:
> > > > On Sat, Nov 14, 2009 at 05:35:28PM +0100, Stefano Sabatini wrote:
> > > > > On date Saturday 2009-11-14 11:53:30 +0100, Michael Niedermayer encoded:
> > [...]
> > > > > +
> > > > > + return 0;
> > > > > +}
> > > > > +
> > > > > int opt_loglevel(const char *opt, const char *arg)
> > > > > {
> > > > > const struct { const char *name; int level; } log_levels[] = {
> > > > > Index: ffmpeg/cmdutils.h
> > > > > ===================================================================
> > > > > --- ffmpeg.orig/cmdutils.h 2009-11-14 17:09:02.000000000 +0100
> > > > > +++ ffmpeg/cmdutils.h 2009-11-14 17:09:25.000000000 +0100
> > > > > @@ -56,6 +56,15 @@
> > > > > int opt_loglevel(const char *opt, const char *arg);
> > > > >
> > > > > /**
> > > > > + * Lists available elements. Exits from the application if the
> > > > > + * value in arg is not supported.
> > > > > + *
> > > > > + * @param opt the name of the option associated to the function
> > > > > + * @param arg the name of the elements to show
> > > > > + */
> > > > > +int opt_list(const char *opt, const char *arg);
> > > >
> > > > This function does not do what you describe
>
> To me it looks correct,
> now it just implements the second part of the
> description " Exits from the application if the value in arg is not
> supported.".
thats like saying printf() is correctly implemented if it just returns
>
> > > Furthermore this trial-and-error process has proved frustrating and
> > > time-consumming, and you still didn't make clear if you are going to
> > > accept the overall change (having a -list option replacing the
> > > -formats one, rationale in the first mail), if we don't agree on that
> > > then there is no point for me into continuing to push patches.
> >
> > iam ok with the idea of formats -> several lists
several lists like -list-codecs -list-...
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I hate to see young programmers poisoned by the kind of thinking
Ulrich Drepper puts forward since it is simply too narrow -- Roman Shaposhnik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091118/11eae0a6/attachment.pgp>
More information about the ffmpeg-devel
mailing list