[Ffmpeg-devel] [PATCH] remove non library include header files

Limin Wang lance.lmwang
Mon Mar 12 03:05:20 CET 2007


Hi,

I think ffmpeg.c shouldn't include the headers which aren't part of ffmpeg
library header file. If anybody need dsputil.h and framehook.h, then install
it as library header to include directory? Now without them, I can compile
ffmpeg without issue by my configure. Please review the patch.

One more question, any possible to move all library header files to every
library include directory, so that we can get which is public header files
easily? Now I had to check the install directory to sure which is public.


Thanks,
Limin
-------------- next part --------------
Index: ffmpeg.c
===================================================================
--- ffmpeg.c	(revision 8335)
+++ ffmpeg.c	(working copy)
@@ -23,8 +23,6 @@
 #include <limits.h>
 #include "avformat.h"
 #include "swscale.h"
-#include "framehook.h"
-#include "dsputil.h"
 #include "opt.h"
 #include "fifo.h"
 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 481 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070312/8e05ab5f/attachment.pgp>



More information about the ffmpeg-devel mailing list