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

Stefano Sabatini stefano.sabatini-lala
Sun May 16 02:42:00 CEST 2010


On date Friday 2010-05-14 00:22:56 +0200, Michael Niedermayer encoded:
> On Thu, May 13, 2010 at 12:39:05AM +0200, Stefano Sabatini wrote:
[...]
> > > From 0096d85e5deec5deb7114dc5165d59e17d9e8391 Mon Sep 17 00:00:00 2001
> > > From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> > > Date: Tue, 6 Apr 2010 00:13:04 +0200
> > > Subject: [PATCH 2/3] Change ff_parse_expr() and ff_parse_and_eval_expr() interface.
> > > 
> > > Make them print error messages using a log_ctx rather than set a
> > > constant error string in the Parser. A log_level_offset is used to
> > > change the log_level of the log_ctx, for example to silence eventual
> > > errors issued when evaluating the expression.
> > > 
> > > Allow the error message to be more expressive, as it is not anymore a
> > > generic const char * string.
> > [...]
> > 
> > Ping.
> 
> what the patch does is not in line with the recent AVClass changes

Updated.

Two questions:
Is it necessary to bump minor when adding an option to AVCodecContext?

Is OK to bump micro when changing the ff_parse_expr() and
ff_parse_and_eval_expr()? (see the thread "[RFC] Updating internl API
policy").

Regards.
-- 
FFmpeg = Fantastic Fantastic Miracolous Porno Evanescent Game



More information about the ffmpeg-devel mailing list