[FFmpeg-devel] [PATCH] remove unnecessary header files

Måns Rullgård mans
Thu May 29 10:02:10 CEST 2008

Erik Hovland <erik at hovland.org> writes:

> This patch removes any header files that unnecessary to the files that
> include them.

In what way are they unnecessary?  If anything declared in a header is
explicitly used, the header should be included, even if another header
includes it indirectly.  The only exception is libavutil/common.h (and
with it internal.h), which is included by avcodec.h and others.  If
everything declared in a header is truly unused in a source file, it
should obviously not be included.  Patches to fix such instances are

M?ns Rullg?rd
mans at mansr.com

More information about the ffmpeg-devel mailing list