[FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

Michael Niedermayer michael at niedermayer.cc
Mon Oct 30 15:07:27 EET 2017


On Mon, Oct 30, 2017 at 02:14:35PM +0700, Muhammad Faiz wrote:
> On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz <mfcc64 at gmail.com> wrote:
> > Fix tsan warnings.
> >
> > Signed-off-by: Muhammad Faiz <mfcc64 at gmail.com>
> > ---
> >  libavutil/crc.c | 49 +++++++++++++++++++++++++++++--------------------
> >  1 file changed, 29 insertions(+), 20 deletions(-)
> 
> Ping.

I assume james patch is faster than both ?

If this code is never run in speed relevant loops then your solution is
better. Otherwise i think james patch is better

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171030/899a6778/attachment.sig>


More information about the ffmpeg-devel mailing list