[FFmpeg-devel] [PATCH] Move eval to lavu

[Michael Niedermayer]

On Tue, Oct 20, 2009 at 09:38:54AM +0100, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > On Mon, Oct 19, 2009 at 08:58:58PM +0200, Stefano Sabatini wrote:
> >> Hi,
> >> 
> >> as in subject, this doesn't comport any external interface change
> >> since eval.h is not public, and libavutil seems the right place for
> >> the eval code.
> >
> > if eval.h is not public then eval can not be used from outside the lib
> > in which eval is. That makes it impossible for libavcodec to use it.
> > Your patch is clearly broken in this respect as libavcodec then uses
> > non public API from libavutil
> 
> How about making it public?

thats possible, but it requires the API to be carefully reviewed first
and that review might find things that require changes ...
also it seems alot of work without an advantage being stated first,
does some application want to use the eval code?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
-------------- 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/20091020/d9cd9f4d/attachment.pgp>