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

Carl Eugen Hoyos ceffmpeg at gmail.com
Thu Apr 13 09:13:56 EEST 2017


2017-04-13 4:02 GMT+02:00 Rostislav Pehlivanov <atomnuker at gmail.com>:
> 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?
>
> 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.

Then I suggest you add the missing copyright notices to the decoder
and the encoder.

Thank you for the explanation, Carl Eugen


More information about the ffmpeg-devel mailing list