[FFmpeg-devel] [PATCH] Implement -sample_fmts
Thu Dec 24 18:41:00 CET 2009
On Thu, Dec 24, 2009 at 11:49:25AM +0100, Stefano Sabatini wrote:
> On date Tuesday 2009-12-22 14:53:24 +0100, Michael Niedermayer encoded:
> > On Tue, Dec 22, 2009 at 12:38:35AM +0100, Stefano Sabatini wrote:
> > > On date Monday 2009-12-21 23:43:05 +0100, Michael Niedermayer encoded:
> > > > On Mon, Dec 21, 2009 at 11:11:30PM +0100, Stefano Sabatini wrote:
> > > > > Hi,
> > > > > similar to -pix_fmts.
> > > > > --
> > > > > FFmpeg = Faboulous and Friendly Mournful Puritan Eager Governor
> > > >
> > > > > cmdutils.c | 6 ++++++
> > > > > cmdutils.h | 6 ++++++
> > > > > cmdutils_common_opts.h | 1 +
> > > > > doc/fftools-common-opts.texi | 3 +++
> > > > > 4 files changed, 16 insertions(+)
> > > > > bd7d7f8721f2cc7a910d52e1d40493246eae3881 implement-sample-fmts.patch
> > > >
> > > > we already have -sample_fmt list, so this is not needed
> > >
> > > 1) the same should be applied to -pix_fmts, on the other hand for
> > > example ffplay has a -pix_fmt option but in that case -pix_fmt list is
> > > not implemented.
> > yes, -pix_fmt list should be implemented
> > >
> > > 2) there are already many options showing elements, -codecs, -formats,
> > > -pix_fmts etc. so the user may expect to find a similar thing for
> > > sample formats.
> > you could remove them
> Are you suggesting to remove all the listing options (-formats,
> -codecs, ...), then have something like -f list, -codec list?
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
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel