[FFmpeg-cvslog] r14484 - in trunk/libavcodec: audioconvert.c audioconvert.h

[Diego Biurrun]

On Sat, Aug 02, 2008 at 06:45:21PM +0200, Michael Niedermayer wrote:
> On Sat, Aug 02, 2008 at 06:20:03PM +0200, Diego Biurrun wrote:
> > On Sat, Aug 02, 2008 at 05:33:50PM +0200, Michael Niedermayer wrote:
> > > 
> > > no, audioconvert.h already incldues avcodec.h.
> > > the #include "avcodec.h" does nothing its just because the diego-mans
> > > rule says it should be there.
> > 
> > So far we have Mans, Baptiste and Diego in favor, with only you
> > against..
> 
> ehh?
> 
> mans clearly said he considers common.h and internal.h as exceptions
> baptiste said he still has to think about if the #include of
> avcodec.h in audioconvert.c should be there or if audioconvert.h is sufficient
> And the wanderer clearly said he is against the include every header
> in every c and h file directly and redundantly.

So the Wanderer suddenly turned into an FFmpeg developer?

My impression is that he just enjoys arguing for the sake of it.  Which
is bad, as it causes extra traffic and lengthens discussions
considerably.

Diego