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

[Michael Niedermayer]

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

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080512/86bc167c/attachment.pgp>