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

Carl Eugen Hoyos ceffmpeg at gmail.com
Fri Apr 14 07:50:55 EEST 2017


2017-04-13 9:51 GMT+02:00 Rostislav Pehlivanov <atomnuker at gmail.com>:
> On 13 April 2017 at 07:13, 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.

> There's nothing to do:
>> * Copyright (c) 2012 Andrew D'Addesio
>> * Copyright (c) 2013-2014 Mozilla Corporation
>> * Copyright (c) 2017 Rostislav Pehlivanov <atomnuker at gmail.com>
>
> Mozilla relicensed the decoder to LGPL when it was ported.

Sorry, I don't understand:
You write above that FFmpeg's opus decoder is based on libopus,
this also explains the similar code I saw (above). libopus is - afaict -
copyright Xiph.Org Foundation.
I suggested to add this copyright to our codec and your answer
is that there is nothing to do but I don't see Xiph mentioned: What
am I missing?
You then continue that the code was relicensed to LGPL: How is
that related to the missing copyright notice?

Thank you, Carl Eugen


More information about the ffmpeg-devel mailing list