[MPlayer-dev-eng] [PATCH] XviD profile support

[Guillaume POIRIER]

Hi,

On 9/18/05, Giacomo Comes <comes at naic.edu> wrote:
> On Sun, Sep 18, 2005 at 05:51:33PM +0200, Guillaume POIRIER wrote:
> > Hi,
> >
> > On 9/18/05, Oded Shimon <ods15 at ods15.dyndns.org> wrote:
> > > On Sat, Sep 17, 2005 at 11:25:41PM +0200, eviv bulgroz wrote:
> > > > Guillaume POIRIER wrote:
> > > >
> > > > >Applied.
> > > > >
> > > > >
> > > > >
> > > > Unfortunately this patch requires a pretty recent xvid release and this
> > > > isn't detected by configure and/or proper #ifdefing.
> > > > This what you get when you try to compile it with latest stable
> > > > xvidcore-1.0.3:
> > > >
> > > > cc -c -I../libvo -I../../libvo -I/usr/X11R6/include -fno-PIC -O4
> > > > -march=athlon-4 -mtune=athlon-4 -pipe -ffast-math -fomit-frame-pointer
> > > > -D_REENTRANT -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64  -I. -Inative
> > > > -I.. -I../libmpdemux -I../loader    -D_GNU_SOURCE -o ve_xvid4.o ve_xvid4.c
> > > > ve_xvid4.c: In function 'profileFromName':
> > > > ve_xvid4.c:223: warning: initialization discards qualifiers from pointer
> > > > target type
> > > > ve_xvid4.c: In function 'dispatch_settings':
> > > > ve_xvid4.c:806: error: 'XVID_GLOBAL_DIVX5_USERDATA' undeclared (first
> > > > use in this function)
> > > > ve_xvid4.c:806: error: (Each undeclared identifier is reported only once
> > >
> > > To anyone who cares, I second this protest. please fix, MPlayer doesn't
> > > compile...
> > >
> > > ve_xvid4.c:806: error: `XVID_GLOBAL_DIVX5_USERDATA' undeclared (first use
> > > in this function)
> > > ve_xvid4.c:806: error: (Each undeclared identifier is reported only once
> > > ve_xvid4.c:806: error: for each function it appears in.)
> > > ve_xvid4.c:844: error: structure has no member named `vbv_size'
> > > ve_xvid4.c:845: error: structure has no member named `vbv_initial'
> >
> > Would the attached patch fix the problem? Does it seems acceptable
> > (though it would required the man page entry about profiles to be
> > modified to tell that profiles are only fully supported with xvid
> > 1.1.x) ?
> >
> > If it doesn't work, please post the errors that you get with it...
> 
> I can confirm now that with the patch MPlayer compiles.

Okay, nice to hear.

> But what about the --with-xvid*dir bug I already reported?
> (http://mplayerhq.hu/pipermail/mplayer-users/2005-September/055541.html)

I had a look at the history of ve_xvid4.c here:
http://www1.mplayerhq.hu/cgi-bin/cvsweb.cgi/main/libmpcodecs/ve_xvid4.c

and it seems like your problem doesn't come from there. It most likely
a configure problem, which I know little about. Should you be able to
tell us precisely which day it broke, it may help us a lot to figure
out what broke it.

Guillaume

-- 
Reading doesn't hurt, really!
  -- Dominik 'Rathann' Mierzejewski