[FFmpeg-devel] [PATCH 1/2] opus: move all tables to a separate file

[Rostislav Pehlivanov]

On 8 November 2016 at 00:18, Michael Niedermayer <michael at niedermayer.cc>
wrote:

> On Mon, Nov 07, 2016 at 10:39:57PM +0000, Rostislav Pehlivanov wrote:
> > Signed-off-by: Rostislav Pehlivanov <atomnuker at gmail.com>
> > ---
> >  libavcodec/Makefile    |    2 +-
> >  libavcodec/opus_celt.c |  518 +++-------------------
> >  libavcodec/opus_silk.c |  849 +++---------------------------------
> >  libavcodec/opustab.c   | 1137 ++++++++++++++++++++++++++++++
> ++++++++++++++++++
> >  libavcodec/opustab.h   |  156 +++++++
> >  5 files changed, 1422 insertions(+), 1240 deletions(-)
> >  create mode 100644 libavcodec/opustab.c
> >  create mode 100644 libavcodec/opustab.h
>
> is there any effect on speed ?
> if theres no slowdown then this patchset should be ok
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> If you think the mosad wants you dead since a long time then you are either
> wrong or dead since a long time.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
Both patches have no effect on speed at all (CPU frequency scaling disabled
of course), tested over multiple runs on a decently long 1 hour file.
Will apply them tomorrow morning if there are no objections, thanks.