[FFmpeg-devel] PATCH: add missing header inclusion to intreadwrite.h

Måns Rullgård mans
Mon Mar 15 16:48:24 CET 2010


Benoit Fouet <benoit.fouet at free.fr> writes:

> Hi,
>
> intreadwrite.h needs the definition of av_alias.
> (found when trying to build all tools in tools/)
> Don't know why make checkheaders didn't find it though.
>
> Index: libavutil/intreadwrite.h
> ===================================================================
> --- libavutil/intreadwrite.h	(revision 22544)
> +++ libavutil/intreadwrite.h	(working copy)
> @@ -23,6 +23,7 @@
>  #include "config.h"
>  #include "bswap.h"
>  #include "common.h"
> +#include "internal.h"

common.h pulls in internal.h when HAVE_AV_CONFIG_H is defined.  The
error comes from this not being defined when building the tools.  Do
we want the tools to stick to public API, or should we simply define
this for them as well?

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list