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

Rostislav Pehlivanov atomnuker at gmail.com
Thu Apr 13 10:51:17 EEST 2017


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.
>
> Thank you for the explanation, Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

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.


More information about the ffmpeg-devel mailing list