[FFmpeg-devel] [PATCH 5/6] Change eval internal functions and ff_parse() interface, to make them return an error code and print the error on the log rather than set the error in the Parser.

[Stefano Sabatini]

On date Sunday 2010-04-11 23:38:18 +0200, Michael Niedermayer encoded:
> On Sun, Apr 11, 2010 at 12:59:01AM +0200, Stefano Sabatini wrote:
> > On date Saturday 2010-04-10 16:33:23 +0200, Stefano Sabatini encoded:
> > > [...]
> > > Forget this patch, it needs more work/testing.
> > 
> > New patch attached.
> > -- 
> > FFmpeg = Formidable & Fantastic Mastodontic Portable EnGine
> 
> >  eval.c        |  169 +++++++++++++++++++++++++++++++++++++---------------------
> >  eval.h        |   12 ++--
> >  ratecontrol.c |   11 +--
> >  3 files changed, 122 insertions(+), 70 deletions(-)
> > 33fbe675678dbc3926a232d9006b9729d9a8d614  0005-Change-eval-internal-functions-and-ff_parse-interfac.patch
> > >From 46856e9c090f3899ebef8cda1ca9cab3064db9ca 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 05/21] Change eval internal functions and ff_parse() interface.
> 
> this mixes several unrelated interface changes, some of which i do not like
> adding a log_ctx is ok, the rest iam not sure

You're right, I'll split the patch, consider this thread/patch dead.

Regards.
-- 
FFmpeg = Funny and Fancy Magic Power Explosive Gadget