[MPlayer-dev-eng] [PATCH] Crash in vf_spp.c and other things

[Gianluigi Tiesi]

On Mon, Aug 28, 2006 at 12:00:09AM +0200, Diego Biurrun wrote:
> On Sun, Aug 27, 2006 at 10:33:03AM +0200, Michael Niedermayer wrote:
> > 
> > On Sun, Aug 27, 2006 at 06:22:58AM +0200, Gianluigi Tiesi wrote:
> > > Attached patch removes an if-def that rip out some code
> > > if CONFIG_ENCODERS is not defined.
> > > 
> > > vf_spp.c at line 412:
> > > 
> > > p->dsp.fdct(block);
> > > 
> > > but dsp.fdct is never assigned if CONFIG_ENCODERS
> > > is not defined in libavcodec/i386/dsputil_mmx.c
> > 
> > rejected, dct belongs to the encoding stuff and should be under that ifdef
> > add a CONFIG_DCT if you want instead
> 
> The crash is fixed now, I've reenabled CONFIG_ENCODERS for MPlayer as
> well.
> 
nevermind gcc cheats and rips of a lot of unused code

Bye

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