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

Stefano Sabatini stefano.sabatini-lala
Tue Mar 2 00:17:49 CET 2010


On date Thursday 2010-02-25 20:39:14 +0100, Stefano Sabatini encoded:
> 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.

Applied.
-- 
FFmpeg = Fanciful & Faithful Magic Prodigious Explosive Generator



More information about the ffmpeg-devel mailing list