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

[Michael Niedermayer]

Hi

On Tue, Sep 26, 2006 at 10:43:35PM +0200, Panagiotis Issaris wrote:
> 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.

looks ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is