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

[Diego Biurrun]

On Fri, Aug 01, 2008 at 11:59:41PM +0200, Michael Niedermayer wrote:
> On Fri, Aug 01, 2008 at 07:19:54PM +0300, Uoti Urpala wrote:
> > On Fri, 2008-08-01 at 14:42 +0200, Michael Niedermayer wrote:
> > > On Fri, Aug 01, 2008 at 05:20:33AM +0300, Uoti Urpala wrote:
> > > > it's not realistic to keep
> > > > them in sync with indirect dependencies, at least not without automated
> > > > tools that aren't used in practice.
> > > 
> > > It worked in mplayer under arpi without a single issue i can remember.
> > > 
> > > So all the pretty arguments stand in stark contrast to actual practice.
> > 
> > MPlayer files included lots of unnecessary headers after dependencies
> > changed.
> 
> Including all dependancies directly also causes many things to be included
> several times redundantly.

No problem, we have multiple inclusion guards everywhere.

Diego