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

Michael Niedermayer michaelni
Fri Dec 11 21:30:57 CET 2009


On Fri, Dec 11, 2009 at 08:51:44PM +0100, Stefano Sabatini wrote:
> On date Sunday 2009-12-06 23:15:24 +0100, Stefano Sabatini encoded:
> > On date Friday 2009-12-04 02:06:08 +0100, Stefano Sabatini encoded:
> > > On date Tuesday 2009-12-01 01:07:24 +0100, Stefano Sabatini encoded:
> > > > On date Sunday 2009-11-29 12:13:51 +0100, Stefano Sabatini encoded:
> > > > > On date Saturday 2009-11-28 12:18:52 +0100, Michael Niedermayer encoded:
> > > > > > On Sun, Nov 22, 2009 at 06:41:04PM +0100, Stefano Sabatini wrote:
> > > > > > 
> > > > > > [...]
> > > > > > > Index: ffmpeg.c
> > > > > > > ===================================================================
> > > > > > > --- ffmpeg.c	(revision 20577)
> > > > > > > +++ ffmpeg.c	(working copy)
> > > > > > > @@ -3829,9 +3829,7 @@
> > > > > > >      { "h", OPT_EXIT, {(void*)show_help}, "show help" },
> > > > > > >      { "version", OPT_EXIT, {(void*)show_version}, "show version" },
> > > > > > >      { "formats"  , OPT_EXIT, {(void*)show_formats  }, "show available formats" },
> > > > > > > -    { "codecs"   , OPT_EXIT, {(void*)show_codecs   }, "show available codecs" },
> > > > > > > -    { "bsfs"     , OPT_EXIT, {(void*)show_bsfs     }, "show available bit stream filters" },
> > > > > > > -    { "protocols", OPT_EXIT, {(void*)show_protocols}, "show available protocols" },
> > > > > > > +    { "list", HAS_ARG | OPT_EXIT | OPT_FUNC2, {(void*)opt_list}, "list available elements" },
> > > > > > >      { "f", HAS_ARG, {(void*)opt_format}, "force format", "fmt" },
> > > > > > >      { "i", HAS_ARG, {(void*)opt_input_file}, "input file name", "filename" },
> > > > > > >      { "y", OPT_BOOL, {(void*)&file_overwrite}, "overwrite output files" },
> > > > > > 
> > > > > > The existing options are documented in the help output, yours is not
> > > > > 
> > > > > Fixed, documentation updated accordingly.
> > > > 
> > > > Patch updated against the latest SVN.
> > > 
> > > Ping?
> > 
> > Ping / anyone else who cares?
> 
> Oh well, so my question is:
> 
> is better to have a long list of options or just one -list elts?
> 
> I want to extend support for listing, for example have
> -list pix_fmts
> -list filters
> 
> I prefer the -list idea as I repeatedly stated, but then I could also
> consider the other way (having one option for every list), please
> express your opinion, otherwise I'll commit this after a week since
> now.

My oppinon is that i prefer seperate options

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.
-------------- 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/20091211/afdb3964/attachment.pgp>



More information about the ffmpeg-devel mailing list