[FFmpeg-devel] [PATCH] cleanup #includes in libavutil/common.h
Mon Jan 26 00:18:04 CET 2009
On Sun, Jan 25, 2009 at 11:15:19PM +0000, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> > On Sun, Jan 25, 2009 at 10:19:24PM +0100, Diego Biurrun wrote:
> >> libavutil/common.h conditionally #includes a bunch of system headers.
> >> This seems to be unnecessary. Here is a patch to get rid of this and
> >> collect the #includes together.
> > i dont know if this has sideeffects, so no approval nor rejection from
> > me
> All those headers are standard C headers, so unconditionally including
> them should be harmless. It could be argued, of course, that someone
> including an FFmpeg header does not expect random system headers to be
> pulled in as well. However, in this case I think the improved
> readability of our header outweighs any hypothetical problems one
> might imagine.
> In other words, go for it.
More information about the ffmpeg-devel