[Ffmpeg-cvslog] CVS: ffmpeg/libavcodec/i386 idct_mmx_xvid.c, NONE, 1.1 dsputil_mmx.c, 1.99, 1.100

Michael Niedermayer michaelni
Sat Sep 10 21:16:39 CEST 2005


Hi

On Sat, Sep 10, 2005 at 08:59:52PM +0200, Guillaume POIRIER wrote:
> Hi,
> 
> On 9/10/05, Michael Niedermayer <michaelni at gmx.at> wrote:
> > Hi
> > 
> > On Sat, Sep 10, 2005 at 07:07:10PM +0200, Guillaume POIRIER wrote:
> > > Hi,
> > >
> > > On 9/10/05, Michael Niedermayer CVS <michael at mplayerhq.hu> wrote:
> > > > Update of /cvsroot/ffmpeg/ffmpeg/libavcodec/i386
> > > > In directory mail:/var2/tmp/cvs-serv23555/i386
> > > >
> > > > Modified Files:
> > > >         dsputil_mmx.c
> > > > Added Files:
> > > >         idct_mmx_xvid.c
> > > > Log Message:
> > > > xvids mmx&mmx2 idcts
> > > > needed to decode xvid without some minor artefacts
> > > > under #ifdef CONFIG_GPL of course
> > >
> > > Is that code enabled on AMD-64? I assume it's not, but one never know... :-)
> > 
> > it should be but well, hmm i havnt really considered amd64 when porting it so
> > maybe it doesnt work, bugreports welcome
> 
> Both are just as fast, but I can't see any visual difference (not that
> I saw that much of a difference between both implementations to begin

elaborate please, do you mean xvid looks better then lavc and old and new
lavc looks the same? or all 3 look the same?
and please ensure that the new idct is actually used at all (needs CONFIG_GPL
in config.h/mak) and latest lavc (with the change in h263dec.c)
maybe add a printf/avlog() into the idct to see if its really used ...

[...]
-- 
Michael





More information about the ffmpeg-cvslog mailing list