[FFmpeg-devel] [RFC] Export eval.h?

Stefano Sabatini stefano.sabatini-lala
Sat Jun 5 23:27:00 CEST 2010


On date Thursday 2008-08-21 00:04:54 +0200, Stefano Sabatini encoded:
> On date Wednesday 2008-08-20 23:17:30 +0200, Michael Niedermayer encoded:
> > On Wed, Aug 20, 2008 at 06:49:01PM +0200, Stefano Sabatini wrote:
> > > On date Wednesday 2008-08-20 03:05:24 +0200, Stefano Sabatini encoded:
> > > > On date Wednesday 2008-08-20 01:43:17 +0200, Michael Niedermayer encoded:
> > > > > On Mon, Aug 11, 2008 at 11:43:53PM +0200, Stefano Sabatini wrote:
> > > > [...]
> > > > 
> > > > Michael:
> > > > > i wonder if it would make sense to just use the whole expression evaluator
> > > > > instead of av_strtod().
> > > > 
> > > > I think this could be a very good idea, anyway maybe is still a good
> > > > idea to export av_strtod(), skip review of the following patch
> > > > otherwise.
> > > [...]
> > > 
> > > And finally I discovered that eval.h is not a public header, when we
> > > can still use it in cmdutils.c with:
> > > #include "libavcodec/eval.h"
> > > 
> > > I think ff* tools shouldn't use internal functions as they are the
> > > implementation reference for libav*.
> > > 
> > > On the other hand I find eval features quite useful per-se, so I
> > > think maybe it's arrived the time to export it, I volunteer to do all
> > > the required work if Michael and other maintainers agree.
> > 
> > i agree in principle but it might take longer than normal to review
> > because theres quite a bit i should review and should work on ...
> > 
> > > The plan could sound as follows:
> > > 1) move eval.[ch] to libavutil
> > > 2) s/ff_/av_/ in some function names
> > > 3) export eval.h interface and bump minor libavutil version
> > 
> > yes, but i have to review eval API/ABI and think about it before it can be
> > exported, i wont do this soon ...
> [...]
> 
> I see, I'll stay tuned meaningwhile...

I stayed tuned all the time, and now here it is an updated patch :-)).

Regards.
-- 
FFmpeg = Fascinating and Fast Mastering Puritan Elastic Gadget



More information about the ffmpeg-devel mailing list