[FFmpeg-devel] [PATCH 1/2] opus_pvq: add resynth support and band encoding cost function

Rostislav Pehlivanov atomnuker at gmail.com
Thu Apr 13 05:02:36 EEST 2017


On 12 April 2017 at 23:50, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:

> 2017-04-13 0:26 GMT+02:00 Rostislav Pehlivanov <atomnuker at gmail.com>:
>
> > +        /* Undo the sample reorganization going from time order to
> frequency order */
> > +        if (B0 > 1)
> > +            celt_interleave_hadamard(f->scratch, X, N_B >> recombine,
> > +                                     B0<<recombine, longblocks);
>
> Is this the same code as in opus/celt/bands.c?
> Who wrote it?
>
> Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

This exact line is also on line 1206 of the same file for PVQ decoding .
Most of the PVQ encoder is copied from our decoder since they're very
similar. This commit just copies more.
The reason why you can find the same string in libopus is because that
decoder was cut down, modified and ported as our native decoder. I have
used no libopus code in my encoder, I did everything from scratch plus
whatever code the decoder had.


More information about the ffmpeg-devel mailing list