On Wed, 19 Feb 2014, Diego Biurrun wrote:

On Wed, Feb 19, 2014 at 04:02:31PM +0200, Martin Storsjö wrote:
On Wed, 19 Feb 2014, Diego Biurrun wrote:
--- a/libavutil/atomic.c
+++ b/libavutil/atomic.c
@@ -97,8 +97,10 @@ void *avpriv_atomic_ptr_cas(void * volatile *ptr, void 
*oldval, void *newval)

-#else
+#else /* HAVE_THREADS */

+/* This should never trigger, unless a new threading implementation
+ * without corresponding atomics implementation is added. */

"atomics implementation or dependency on atomics_native within
configure" or something similar?

I think I'll go for

 This should never trigger, unless a new threading implementation without
 correct atomics dependencies in configure or a corresponding atomics
 implementation is added.

Sounds ok to me.

// Martin
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to