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

Steinar H. Gunderson steinar+ffmpeg at gunderson.no
Mon Jan 9 21:21:25 EET 2017


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.

/* Steinar */
-- 
Homepage: https://www.sesse.net/


More information about the ffmpeg-devel mailing list