[FFmpeg-devel] [PATCH] Clarify error message in ffprobe when more than input is provided

Stefano Sabatini stefano.sabatini-lala
Thu Feb 25 20:39:14 CET 2010


On date Thursday 2010-02-25 17:00:13 +0100, Stefano Sabatini encoded:
> On date Thursday 2010-02-25 01:52:01 +0100, Michael Niedermayer encoded:
> > On Thu, Feb 25, 2010 at 01:11:36AM +0100, Stefano Sabatini wrote:
> > [...]
> > >  ffplay.c |    5 +++++
> > >  1 file changed, 5 insertions(+)
> > > d6c6a292140d03b3c6c29bb9da9345a68fd00f01  ffplay-add-input-check.patch
> > > Index: ffmpeg/ffplay.c
> > > ===================================================================
> > > --- ffmpeg.orig/ffplay.c	2010-02-25 00:56:24.000000000 +0100
> > > +++ ffmpeg/ffplay.c	2010-02-25 01:10:17.000000000 +0100
> > > @@ -2670,6 +2670,11 @@
> > >  
> > >  static void opt_input_file(const char *filename)
> > >  {
> > > +    if (input_filename) {
> > > +        fprintf(stderr, "Argument '%s' provided as input filename, but '%s' was already specified.\n",
> > > +                filename, input_filename);
> > > +        exit(1);
> > > +    }
> > 
> > want playlist support ...
> > dont want to review silly error messages saying "no"
> 
> Everybody wants playlust support, while we don't have that better to
> say a clear "no" and avoid confusion.

I'll apply both patches after three days if I get no comments /
flames.

Regards.
-- 
FFmpeg = Fabulous and Faithful Minimal Practical Elastic Gargoyle



More information about the ffmpeg-devel mailing list