[MPlayer-dev-eng] [PATCH] disable filters that depend on dsputil.h

Diego Biurrun diego at biurrun.de
Wed Mar 15 00:09:25 CET 2006


On Tue, Mar 14, 2006 at 02:09:05PM +0200, Ivan Kalvachev wrote:
> 2006/3/14, Nico Sabbi <nicola_sabbi at fastwebnet.it>:
> > Hi,
> > last months we added dsputil.h to the install target of libavcodec;
> > it was a mistake because that file is not supposed to be used outside of
> > libavcodec.
> >
> > This patch disables the inclusion of filters that depend on that header
> > when libavcodec
> > is not imported in main/; if applied I'll post a patch to ffmpeg-devel
> > to restore the previous
> > behaviour (not install dsputil.h).
> >
> > Please, review.
> 
> I'm afraid this could be considered feature removal.

No, because shared lavc has only lived in CVS for a few weeks now.

> The reason (interface purity) doesn't justify the result.

Michael has clearly spoken out against dsputil.h being considered part
of the public interface of FFmpeg...

Diego




More information about the MPlayer-dev-eng mailing list