[MPlayer-dev-eng] [PATCH] fixes for gcc4

[Gianluigi Tiesi]

On Fri, Feb 25, 2005 at 03:31:49AM +0100, Michael Niedermayer wrote:
> Hi
> 
> On Thursday 24 February 2005 19:15, Gianluigi Tiesi wrote:
> > I've added some fixes to compile mplayer with gcc4, anyway they are even
> > nonsenses like asm memory addressing of a binary operation of two vars.
> > There are some stuff should be fixed like includes in libavcodec,
> > gcc4 seams to not like struct AVOption without a type if used not as
> > pointer (also this seams reasonable to me).
> > The patch is splitted in two part, libavcodec and mplayer.
> > I don't like much the way I've patched common.h and avcodec.h, feel free
> > to find a better way.
> >
> > I'm also in doubt with:
> > extern struct STRTAB chanlist_names[];
> > that I've changed in:
> > extern struct CHANLISTS   chanlist_names[];
> >
> > I cannot found STRTAB anyway in the source ;(
> > I've no clue to compile libmpcodecs/native/RTjpegN.c with gcc4, it
> > complains for a wrong operand in mmx (?) instructions.
> 
> [...]
> 
> > #ifdef HAVE_MMX
> >        assert(filterSize % 4 == 0 && filterSize>0);
> >+       int ptr;
> 
> mixing of statements and declarations breaks gcc 2.95
> 
You are right but it's enough to put the var at the top of the block,
I'm still searching for a better way to patch common.h and avcodec.h



-- 
Gianluigi Tiesi <sherpya at netfarm.it>
EDP Project Leader
Netfarm S.r.l. - http://www.netfarm.it/
Free Software: http://oss.netfarm.it/