[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.

Michael Niedermayer michaelni
Sun Apr 11 23:38:18 CEST 2010


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

[...]
-- 
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/20100411/065147bf/attachment.pgp>



More information about the ffmpeg-devel mailing list