[Ffmpeg-devel] [PATCH] Use ff_eval for AVOption parsing

[Panagiotis Issaris]

Hi,

On di, 2006-09-26 at 18:53 +0200, Michael Niedermayer wrote:
> [...] 
> > Something like the attached patch? Followed by a patch using ff_eval2()
> > ofcourse...
> > 
> >  eval.c |   56 +++++++++++++++++++++++++++++++++++---------------------
> >  1 file changed, 35 insertions(+), 21 deletions(-)
> > 
> > Regression tests succeeded.
> 
> isnt it simpler to put error in the struct instead of passing it as arg
> into every functon?

Ahum... yes :) Updated patch attached. Regression tests succeed.

 eval.c |   26 +++++++++++++++++++++-----
 1 files changed, 21 insertions(+), 5 deletions(-)

With friendly regards,
Takis
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pi-20060926T223519-ffmpeg-ff_eval_logging.diff
Type: text/x-patch
Size: 2549 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060926/e02aec0a/attachment.bin>