[FFmpeg-devel] [PATCH 1/2] mpegaudiodec: move imdct36() to MPADSPContext

[Michael Niedermayer]

On Wed, Aug 24, 2011 at 08:55:03PM +0200, Michael Niedermayer wrote:
> On Sun, Aug 21, 2011 at 09:55:03PM +0200, Vitor Sessak wrote:
> > On Sun, Aug 21, 2011 at 9:19 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > > Hi Vitor
> > >
> > > On Sun, Aug 21, 2011 at 04:51:48PM +0200, Vitor Sessak wrote:
> > >> $subj, should allow SIMD optimizations.
> > > [...]
> > >>          /* NOTE: we do frequency inversion adter the MDCT by changing
> > >>             the sign of the right window coefs */
> > >>          for(j=0;j<4;j++) {
> > >> -            for(i=0;i<36;i+=2) {
> > >
> > >> +            mdct_win[j    ][18] = mdct_win[j    ][19] = 0;
> > >> +            mdct_win[j + 4][18] = mdct_win[j + 4][19] = 0;
> > >
> > > these seem unneeded, or maybe i miss something ?
> > 
> > Indeed, I just forgot static buffer are already initialized with zeros.
> > 
> > > [...]
> > >> @@ -1403,15 +1294,15 @@ static void compute_imdct(MPADecodeContext *s,
> > >>          else
> > >>              win1 = mdct_win[g->block_type];
> > >>          /* select frequency inversion */
> > >> -        win = win1 + ((4 * 36) & -(j & 1));
> > >> -        imdct36(out_ptr, buf, ptr, win);
> > >> +        win = win1 + ((4 * 40) & -(j & 1));
> > >> +     s->mpadsp.RENAME(imdct36)(out_ptr, buf, ptr, win);
> > >
> > > tabs
> > >
> > >
> > > [...]
> > >> +/* using Lee like decomposition followed by hand coded 9 points DCT */
> > >> +void RENAME(ff_imdct36)(INTFLOAT *out, INTFLOAT *buf, INTFLOAT *in,
> > >
> > > trailing whitespace
> > 
> > Fixed both locally (and together with it my .emacs).
> 
> patch LGTM then

ping
i cant apply your local patches ;)

[...]


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- 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/20111002/2360a472/attachment.asc>