[FFmpeg-devel] [PATCH] avutil/crc: always use precalculated CRC tables for known polynomials

Derek Buitenhuis derek.buitenhuis at gmail.com
Sun Oct 22 16:15:24 EEST 2017


On 10/22/2017 2:11 PM, James Almer wrote:
> It was suggested, but nobody gave it a try (Or they did but found it
> wasn't as simple as first thought?).

[...]

> Thread sanitizer complains about this in every other run, and the tables
> are at most 1k each, so this is not a bad solution and can be replaced
> by ff_thread_once in the future.

And say that N times and you end up with Nk of tables ;).

That said, I'm not NAKing the patch, just suggesting an alternative.

- Derek


More information about the ffmpeg-devel mailing list