[FFmpeg-devel] [PATCH] cleanup #includes in libavutil/common.h
Mon Jan 26 00:15:19 CET 2009
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
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
In other words, go for it.
mans at mansr.com
More information about the ffmpeg-devel