[FFmpeg-devel] [PATCH] Implement -pix_fmts for listing all the supported pixel formats

[Michael Niedermayer]

On Tue, Feb 16, 2010 at 01:21:35AM +0100, Stefano Sabatini wrote:
> On date Saturday 2009-12-12 19:19:48 +0100, Michael Niedermayer encoded:
> > On Sat, Dec 12, 2009 at 07:07:06PM +0100, Stefano Sabatini wrote:
> > > On date Saturday 2009-12-12 18:22:33 +0100, Michael Niedermayer encoded:
> > > > On Fri, Nov 20, 2009 at 12:34:48AM +0100, Stefano Sabatini wrote:
> > > > > Hi, requires two steps, moving list_fmts to cmdutils.c and implement
> > > > > the function.
> > > > 
> > > > ok
> > > 
> > > Applied the first step, I'm waiting for the common opts texi
> > > factorization patch
> > 
> > not my area ...
> > 
> > 
> > > to be reviewed before to apply the second, and
> > > another patch to remove -pix_fmt list.
> > 
> > -pix_fmt list removial is rejected, its one of the most obviosu ways
> > one would try to get a list -pix_fmt help would be the second.
> 
> Another idea would be to have:
> -codec/-pix_fmt/-sample_fmt etc.
> 
> list just a plain list of the values which can be used for completion
> (that is no header and just one column with the name of the possible
> values), *and* keep the listing options.

This would be the most obfuscated user interface ive seen recently

its like -h -? giving different help texts this would confuse everyone


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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- 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/20100216/42b98256/attachment.pgp>