[FFmpeg-devel] [PATCH] Improve ffpreset error messages

Stefano Sabatini stefano.sabatini-lala
Fri Nov 21 23:55:34 CET 2008


On date Friday 2008-11-21 13:35:49 +0100, Michael Niedermayer encoded:
> On Thu, Nov 20, 2008 at 08:34:30PM +0100, Stefano Sabatini wrote:
> > On date Thursday 2008-11-20 12:12:19 +0100, Michael Niedermayer encoded:
> > > On Thu, Nov 20, 2008 at 12:57:00AM +0100, Stefano Sabatini wrote:
> > > > Hi all,
> > > > 
> > > > sorry for the pickyness, maybe they should go in two separate commits.
> > > > 
> > > > The first change provides the exact line showing a syntax problem,
> > > > like here:
> > > > ./ffpresets/libx264-bugged.ffpreset: Invalid syntax: foobar baz # foo
> > > > 
> > > > the second one uses the exact line found in the file, for example the
> > > > user could specify "key=    value" while the message will tell: 
> > > > "Invalid option or argument: key=value"
> > > 
> > > could be 'Invalid option or argument: "%s", parsed as %s = %s'
> > 
> > Seconded.
> 
> looks ok

Applied, regards.
-- 
FFmpeg = Friendly and Fostering Minimalistic Programmable Elastic Guide




More information about the ffmpeg-devel mailing list