[FFmpeg-devel] [PATCH 1/5] Change eval internal functions, ff_parse_expr() and ff_parse_and_eval_expr() interface.

[Michael Niedermayer]

On Tue, May 18, 2010 at 12:56:53AM +0200, Stefano Sabatini wrote:
[...]
>  opt.c |    2 ++
>  1 file changed, 2 insertions(+)
> 11b07e9a34d65b8e88f47b5fd2d9af456258620b  0002-Silence-logging-when-evaluating-the-expression-for-b.patch
> >From c18ac9ee11ca03ab36e5acf2b72ecfc537caaced Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> Date: Tue, 18 May 2010 00:31:01 +0200
> Subject: [PATCH 2/4] Silence logging when evaluating the expression for buf in
>  av_set_string3().
> 
> ---
>  libavcodec/opt.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/libavcodec/opt.c b/libavcodec/opt.c
> index 5b56bc8..844c0ec 100644
> --- a/libavcodec/opt.c
> +++ b/libavcodec/opt.c
> @@ -155,7 +155,9 @@ int av_set_string3(void *obj, const char *name, const char *val, int alloc, cons
>                  buf[i]= val[i];
>              buf[i]=0;
>  
> +            av_set_int(obj, "log_level_offset", AV_LOG_DEBUG - AV_LOG_QUIET);
>              d = ff_parse_and_eval_expr(buf, const_names, const_values, NULL, NULL, NULL, NULL, NULL, obj);
> +            av_set_int(obj, "log_level_offset", 0);

not ok, obj is not owned by this code. Other code might via a seperate
thread print via obj at the same time.


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- 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/20100518/455c0f10/attachment.pgp>