[FFmpeg-devel] [PATCH] Implement a cmdutils functions for listing available elements

Michael Niedermayer michaelni
Sun Nov 15 01:38:28 CET 2009


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
> 
> 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

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

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- 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/20091115/0db9f5b0/attachment.pgp>



More information about the ffmpeg-devel mailing list