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

[Diego Biurrun]

On Fri, Aug 01, 2008 at 05:01:48PM +0200, Michael Niedermayer wrote:
> On Fri, Aug 01, 2008 at 04:37:46PM +0200, Michael Niedermayer wrote:
> > On Fri, Aug 01, 2008 at 02:12:21PM +0100, M?ns Rullg?rd wrote:
> [...]
> > > Now please stop trolling, and get back to coding, which you do very well.
> > > Quite frankly though, I doubt you have the experience necessary to for
> > > an informed opinion on organisational matters like these.  That a certain
> > 
> > maybe. I do not claim to be an expert in header inclusion rules and do not
> > claim to have reviewed dozends of projects what they do and what problems
> > it causes. I just see how well it "works" here, and how little the claims
> > match reality.
> 
> Also, its interresting to note that the "all files must directly include their
> dependancies" rule is a rather recent thing in relation to ffmpeg. ffmpeg has
> existed MUCH longer than it, and it has never been a problem for _us_ so one
> has to ask what exactly did this rule fix that caused problems before?

I can give you one very concrete example that just bit me: I added ffaac
support to my local MPlayer tree by importing the SoC stuff into it.
Compilation of aac.c failed because it was missing math.h and string.h
headers.  But the first error message was about ENOMEM which puzzled me.

This did indeed take some time to fix, which could have been avoided if
Robert had #included all necessary headers from the start.

Diego