Re: [FFmpeg-devel] [PATCH]Silence a warning when decoding hap

2015-07-06 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes:

   hapdec.c |3 ++-
   1 file changed, 2 insertions(+), 1 deletion(-)
  126826ea8c86d05606122d00f4f8d633829f94f9  patchhap.diff
 
 probably the call can be removed entirely but this 
 is strictly taken safer
 LGTM

The patch was merged.

Thank you, Carl Eugen

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


[FFmpeg-devel] [PATCH]Silence a warning when decoding hap

2015-06-29 Thread Carl Eugen Hoyos
Hi!

Attached patch silences many lines of Multiple ff_thread_finish_setup() 
calls when decoding hap.

Please comment, Carl Eugen
diff --git a/libavcodec/hapdec.c b/libavcodec/hapdec.c
index 6e3c138..e6b7d61 100644
--- a/libavcodec/hapdec.c
+++ b/libavcodec/hapdec.c
@@ -177,7 +177,8 @@ static int hap_decode(AVCodecContext *avctx, void *data,
 ret = ff_thread_get_buffer(avctx, tframe, 0);
 if (ret  0)
 return ret;
-ff_thread_finish_setup(avctx);
+if (avctx-codec-update_thread_context)
+ff_thread_finish_setup(avctx);
 
 /* Use the decompress function on the texture, one block per thread */
 avctx-execute2(avctx, decompress_texture_thread, tframe.f, NULL, blocks);
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH]Silence a warning when decoding hap

2015-06-29 Thread Michael Niedermayer
On Mon, Jun 29, 2015 at 11:28:04PM +0200, Carl Eugen Hoyos wrote:
 Hi!
 
 Attached patch silences many lines of Multiple ff_thread_finish_setup() 
 calls when decoding hap.
 
 Please comment, Carl Eugen

  hapdec.c |3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)
 126826ea8c86d05606122d00f4f8d633829f94f9  patchhap.diff

probably the call can be removed entirely but this is strictly
taken safer
LGTM

thx

-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

DNS cache poisoning attacks, popular search engine, Google internet authority
dont be evil, please


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel