[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