[MPlayer-cvslog] r28151 - trunk/mangle.h

Diego Biurrun diego at biurrun.de
Mon Dec 15 17:38:25 CET 2008


On Mon, Dec 15, 2008 at 05:15:35PM +0100, Reimar Döffinger wrote:
> On Mon, Dec 15, 2008 at 05:52:21PM +0200, Uoti Urpala wrote:
> > On Mon, 2008-12-15 at 14:53 +0100, reimar wrote:
> > > Modified: trunk/mangle.h
> > 
> > > +#define MANGLE(a) EXTERN_PREFIX #a
> > 
> > EXTERN_PREFIX is defined in config.h but mangle.h does not #include it.
> > At least mp3lib has files using mangle.h which do not already include
> > config.h and so will fail to compile.
> 
> Hm, only two files had that issue, so I decided to fix it by including
> config.h in those files.

Please do it the other way around, otherwise you will break 'make
checkheaders' and also you have to make sure that config.h is #included
with every use of mangle.h.

Diego



More information about the MPlayer-cvslog mailing list