[FFmpeg-devel] [PATCH] cleanup #includes in libavutil/common.h

Diego Biurrun diego
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.

Done.

Diego




More information about the ffmpeg-devel mailing list