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

Stefano Sabatini stefano.sabatini-lala
Fri Nov 20 09:21:31 CET 2009


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.



More information about the ffmpeg-cvslog mailing list