diff mbox series

[FFmpeg-devel,04/23] avcodec/zmbv: Mark decoder as init-threadsafe

Message ID VI1PR0301MB2159888EF7F863E6DEAED62D8F589@VI1PR0301MB2159.eurprd03.prod.outlook.com
State Accepted
Commit e51fad097b81da868a1bc8ce9757dbd9a4e3c374
Headers show
Series [FFmpeg-devel,01/23] avcodec/ljpegenc: Mark encoder as init-threadsafe
Related show

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Andreas Rheinhardt May 6, 2021, 5:11 a.m. UTC
Initializing zlib in the way we do here is threadsafe, see
https://www.zlib.net/zlib_faq.html#faq21

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavcodec/zmbv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tomas Härdin May 7, 2021, 7:08 a.m. UTC | #1
tor 2021-05-06 klockan 07:11 +0200 skrev Andreas Rheinhardt:
> Initializing zlib in the way we do here is threadsafe, see
> https://www.zlib.net/zlib_faq.html#faq21
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  libavcodec/zmbv.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/zmbv.c b/libavcodec/zmbv.c
> index 4cc0476f4d..0a5bb40ad5 100644
> --- a/libavcodec/zmbv.c
> +++ b/libavcodec/zmbv.c
> @@ -664,5 +664,5 @@ const AVCodec ff_zmbv_decoder = {
>      .close          = decode_end,
>      .decode         = decode_frame,
>      .capabilities   = AV_CODEC_CAP_DR1,
> -    .caps_internal  = FF_CODEC_CAP_INIT_CLEANUP,
> +    .caps_internal  = FF_CODEC_CAP_INIT_THREADSAFE | FF_CODEC_CAP_INIT_CLEANUP,
>  };

Looks good to me. Same goes for the encoder version of this.

/Tomas
diff mbox series

Patch

diff --git a/libavcodec/zmbv.c b/libavcodec/zmbv.c
index 4cc0476f4d..0a5bb40ad5 100644
--- a/libavcodec/zmbv.c
+++ b/libavcodec/zmbv.c
@@ -664,5 +664,5 @@  const AVCodec ff_zmbv_decoder = {
     .close          = decode_end,
     .decode         = decode_frame,
     .capabilities   = AV_CODEC_CAP_DR1,
-    .caps_internal  = FF_CODEC_CAP_INIT_CLEANUP,
+    .caps_internal  = FF_CODEC_CAP_INIT_THREADSAFE | FF_CODEC_CAP_INIT_CLEANUP,
 };