[MPlayer-users] CVS works no longer with current ffmpeg onbigendian

[Gansser, Martin]

> -----Original Message-----
> From: Götz Waschk [mailto:waschk at informatik.uni-rostock.de]
> Sent: Wednesday, June 18, 2003 11:39 AM
> To: MPlayer user's list.
> Subject: Re: [MPlayer-users] CVS works no longer with current ffmpeg
> onbigendian
> 
> 
> [Automatic answer: RTFM (read DOCS, FAQ), also read 
> DOCS/bugreports.html]
> Am Mittwoch, 18. Juni 2003, 09:53:54 Uhr MET, schrieb Gansser, Martin:
> > gcc -O4   -pipe -ffast-math -fomit-frame-pointer -DHPUX 
> -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -
> > Wall -g -DHAVE_AV_CONFIG_H -I.. -D_FILE_OFFSET_BITS=64 
> -D_LARGEFILE_SOURCE -D_GNU_SOURCE -c -o mjpeg
> > .o mjpeg.c
> > mjpeg.c: In function `mjpeg_decode_frame':
> > mjpeg.c:1664: `PIX_FMT_ABGR32' undeclared (first use in 
> this function)
> > mjpeg.c:1664: (Each undeclared identifier is reported only once
> > mjpeg.c:1664: for each function it appears in.)
> > gmake[1]: *** [mjpeg.o] Error 1
> > gmake[1]: Leaving directory 
> `/users/mgansser/GNU/MPlayer-20030618/libavcodec'
> > gmake: *** [libavcodec/libavcodec.a] Error 2
> > MPlayer CVS compiles at the moment only with ffmpeg-cvs-2003-06-10
> > OS: hpux 11.00 big endian
> 
> I can confirm this problem on a Solaris machine (SPARC CPU(s), also
> big endian).
>  
> > I figured out that it works only wiht the following change 
> on my machine:
> > 
> > diff -Naur mjpeg.c.orig mjpeg.c
> > --- mjpeg.c.orig        2003-06-18 09:43:28.000000000 +0200
> > +++ mjpeg.c     2003-06-18 09:53:50.000000000 +0200
> > @@ -1661,7 +1661,7 @@
> >                          case 0x11:
> >                              if(s->rgb){
> >  #ifdef WORDS_BIGENDIAN
> > -                                avctx->pix_fmt = PIX_FMT_ABGR32;
> > +                                avctx->pix_fmt = PIX_FMT_YUV444P;
> >  #else
> >                                  avctx->pix_fmt = PIX_FMT_RGBA32;
> >  #endif
> > 
> > any suggestion ?
> 
> I don't know if there are ffmpeg developers on this list, if this
> doesn't get fixed, mail to the ffmpeg-devel list. BTW I don't think
> YUV444P is the right replacement for ABGR32, I think the real fix
> would be to define PIX_FMT_ABGR32 somewhere. 

the change was made by Michael Niedermayer see: 

http://sourceforge.net/mailarchive/message.php?msg_id=5280298

I have already send a email to him.

thanks Martin