[FFmpeg-cvslog] r14484 - in trunk/libavcodec: audioconvert.c audioconvert.h
Aurelien Jacobs
aurel
Thu Jul 31 19:47:32 CEST 2008
compn wrote:
> On Thu, 31 Jul 2008 16:32:20 +0200, Aurelien Jacobs wrote:
> >Not only unnecessary, but even bad.
> >The usage here is that every file must include every headers that it
> >requires directly. You should not rely on one header including another
> >(except when it's explicitely documented so, such as inttypes.h
> >including stdint.h).
> >So here you should add back the avcodec.h include.
>
> is this policy documented somewhere?
In the mailing list archives... ;-)
More seriously, that's indeed the kind of thing which would need to be
written somewhere in the doc. Do you volunteer ?
Aurel
More information about the ffmpeg-cvslog
mailing list