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

[Diego Biurrun]

On Mon, Aug 28, 2006 at 03:12:40AM +0200, Gianluigi Tiesi wrote:
> 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.
> > 
> So I cannot use vf=ssp if I use --disable-mencoder?

You could not before, now you can.

Diego