[MPlayer-dev-eng] [PATCH] build: fix lavfi dependencies (swscale and swresample).

Nicolas George nicolas.george at normalesup.org
Tue Feb 7 16:06:21 CET 2012


Le nonidi 19 pluviôse, an CCXX, Reinhard Tartler a écrit :
> I see. Well, adding the #include "avutil/mathematics.h" is anyway the
> right thing to do, imo. You should always include headers that you
> require and don't rely on headers that do that indirectly in general
> no?

That is what I did: I included avutil.h to import the general services
proposed by libavutil, including rational computations.

I do not object to your addition, but I really do not see the point of
requiring from every application programmer to remember which header provide
which service for the core part of lavu. It has been a long time since
parsing ~150 lines of headers had a significant impact on compilation time.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20120207/0956b34e/attachment.asc>


More information about the MPlayer-dev-eng mailing list