[FFmpeg-cvslog] r20553 - in trunk: cmdutils.c cmdutils.h ffmpeg.c ffplay.c ffserver.c

[Michael Niedermayer]

On Fri, Nov 20, 2009 at 09:21:31AM +0100, Stefano Sabatini wrote:
> On date Friday 2009-11-20 08:58:11 +0100, Michael Niedermayer wrote:
> > On Fri, Nov 20, 2009 at 12:08:56AM +0100, Stefano Sabatini wrote:
> [...]
> > > > Modified: trunk/cmdutils.c
> > > > ==============================================================================
> > > > --- trunk/cmdutils.c	Thu Nov 19 12:49:03 2009	(r20552)
> > > > +++ trunk/cmdutils.c	Thu Nov 19 17:35:55 2009	(r20553)
> > > [...]
> > > > +void show_protocols(void)
> > > > +{
> > > > +    URLProtocol *up=NULL;
> > > >  
> > > >      printf("Supported file protocols:\n");
> > > >      while((up = av_protocol_next(up)))
> > > > @@ -541,13 +558,6 @@ void show_formats(void)
> > > >      printf("\n");
> > > >  
> > > >      printf("Frame size, frame rate abbreviations:\n ntsc pal qntsc qpal sntsc spal film ntsc-film sqcif qcif cif 4cif\n");
> > > 
> > > show_protocols() is listing abbreviations...
> > 
> > and before my commit, show_formats() did print them, what is it
> > that you are trying to say?
> >
> > That its still sub-optimal after the split?
> > I dont know where to put that line, if you have an idea send a
> > _clean_ patch
> 
> Maybe we could add a show_sizes()/show_framerates() function or even
> removing them may be an option, considering that we have the docs for
> that, and that that listing is badly outdated.

after writing my mail it came to my mind that probably -h that is the
main help screen is the best place for such single line lists

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- 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-cvslog/attachments/20091120/4896a8ad/attachment.pgp>