[FFmpeg-devel] [PATCH 1/5] ra144: use scalarproduct_int16

[Michael Niedermayer]

On Fri, May 10, 2013 at 11:39:04AM +0200, Christophe Gisquet wrote:
> 2013/5/10 Christophe Gisquet <christophe.gisquet at gmail.com>:
> > @@ -374,7 +376,7 @@ static void ra144_encode_subblock(RA144Context *ractx,
> >          memcpy(cba, work + LPC_ORDER, sizeof(cba));
> >
> >          ff_copy_and_dup(cba_vect, ractx->adapt_cb, cba_idx + BLOCKSIZE / 2 - 1);
> > -        m[0] = (ff_irms(cba_vect) * rms) >> 12;
> > +        m[0] = (ff_irms(&ractx->dsp, cba_vect) * rms) >> 12;
> >      }
> >      fixed_cb_search(work + LPC_ORDER, coefs, data, cba_idx, &cb1_idx, &cb2_idx);
> >      for (i = 0; i < BLOCKSIZE; i++) {
> 
> I think this hunk is wrong. What's the name of the fate test for encoding ra144?

fate-acodec-ra144 should do that

also you can find the names of tests by greping the output of
make fate-list

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130510/78746622/attachment.asc>