[FFmpeg-devel] [PATCH]HE-AACv1 try 3 (all missing functionality added)

Alex Converse alex.converse
Tue Feb 16 23:54:34 CET 2010


On Sun, Feb 14, 2010 at 8:26 PM, Alex Converse <alex.converse at gmail.com>wrote:

> On Sat, Feb 13, 2010 at 11:31 AM, Vitor Sessak <vitor1001 at gmail.com>
> wrote:
> > Alex Converse wrote:
> >[...]
>


> >> +        if (div) {
> >> +            memset(X[0][l]+32, 0, 32*sizeof(float));
> >> +            memset(X[1][l]+32, 0, 32*sizeof(float));
> >> +        }
> >> +        ff_imdct_half(mdct, mdct_buf[0], X[0][l]);
> >> +        ff_imdct_half(mdct, mdct_buf[1], X[1][l]);
> >> +        for (n = 0; n < 64 >> div; n++) {
> >> +            v[               n] = -mdct_buf[0][64-1-(n<<div)    ] +
> >> mdct_buf[1][ n<<div     ];
> >> +            v[(128>>div)-1 - n] =  mdct_buf[0][64-1-(n<<div)-div] +
> >> mdct_buf[1][(n<<div)+div];
> >> +        }
> >
> > I still think you can avoid the temporary scratch buffer...
>
> I will fix
>
>
This made the code slower. Unfixed.



More information about the ffmpeg-devel mailing list