[FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

[Kieran Kunhya]

On Mon, 9 Jan 2017 at 19:21 Steinar H. Gunderson <
steinar+ffmpeg at gunderson.no> wrote:

> On Mon, Jan 09, 2017 at 08:17:18PM +0100, Michael Niedermayer wrote:
> >> +        /* Reverse the AC VLC, because INIT_VLC_LE wants it in that
> order. */
> >> +        for (i = 0; i < FF_ARRAY_ELEMS(speedhq_vlc); ++i) {
> >> +            speedhq_vlc[i][0] = reverse(speedhq_vlc[i][0],
> speedhq_vlc[i][1]);
> >> +        }
> > This makes the init thread unsafe
> > it would be better if the table was const and this wasnt done at
> > runtime
>
> Do you want all three tables, or just this one?
>
> Optionally, I can keep the original one const, and just copy into a
> temporary
> here.
>

You can also do it like this:
https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=504e3f75bf73a488d39a80a42962bae2a353dd6b

Kieran