[FFmpeg-devel] [PATCH] Fix av_set_string()

[Stefano Sabatini]

On date Monday 2008-05-12 22:44:35 +0200, Michael Niedermayer encoded:
> On Mon, May 12, 2008 at 10:22:32PM +0200, Stefano Sabatini wrote:
> > Hi all,
> > as in subject.
> > 
> > Best regards.
> > -- 
> > FFmpeg = Free and Foolish MultiPurpose EnGine
> 
> > Index: libavcodec/opt.c
> > ===================================================================
> > --- libavcodec/opt.c	(revision 13138)
> > +++ libavcodec/opt.c	(working copy)
> > @@ -172,8 +172,8 @@
> >                  else if(!strcmp(buf, "none"   )) d= 0;
> >                  else if(!strcmp(buf, "all"    )) d= ~0;
> >                  else {
> > -                    if (!error)
> > +                    if (error)
> 
> ok
> 
> 
> > -                        av_log(NULL, AV_LOG_ERROR, "Unable to parse option value \"%s\": %s\n", val, error);
> > +                        av_log(NULL, AV_LOG_ERROR, "Unable to parse option value \"%s\": %s\n", buf, error);
> 
> not ok
[...]

I agree they are two different issues, so I'm splitting the patch.

Regards.
-- 
FFmpeg = Foolish and Freak MultiPurpose EntanGlement
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix-av-set-string-01.patch
Type: text/x-diff
Size: 551 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080513/c1c356ea/attachment.patch>