[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 10:10:59 EEST 2017
2017-04-13 8:44 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
> On 4/13/17, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>> 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.
> Your suggestions should always be ignored.
Would you like to elaborate or are you just trolling?
More information about the ffmpeg-devel