[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