[FFmpeg-cvslog] atomics: cosmetics: Restructure ifdefs for greater clarity

Diego Biurrun git at videolan.org
Fri Dec 20 13:18:33 CET 2013


ffmpeg | branch: master | Diego Biurrun <diego at biurrun.de> | Wed Dec 18 22:28:11 2013 +0100| [e1b9de4fe15cda08cb18a2821a5729cf95e33d2a] | committer: Diego Biurrun

atomics: cosmetics: Restructure ifdefs for greater clarity

Also fix the #endif comment in a few places and #include config.h to
avoid assuming it is #included implicitly.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=e1b9de4fe15cda08cb18a2821a5729cf95e33d2a
---

 libavutil/atomic.c |    3 ++-
 libavutil/atomic.h |    7 ++++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/libavutil/atomic.c b/libavutil/atomic.c
index add489a..e4fa957 100644
--- a/libavutil/atomic.c
+++ b/libavutil/atomic.c
@@ -18,6 +18,7 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
+#include "config.h"
 #include "atomic.h"
 
 #if !HAVE_ATOMICS_NATIVE
@@ -102,7 +103,7 @@ void *avpriv_atomic_ptr_cas(void * volatile *ptr, void *oldval, void *newval)
 
 #endif /* HAVE_PTHREADS */
 
-#endif /* !HAVE_MEMORYBARRIER && !HAVE_SYNC_VAL_COMPARE_AND_SWAP && !HAVE_MACHINE_RW_BARRIER */
+#endif /* !HAVE_ATOMICS_NATIVE */
 
 #ifdef TEST
 #include <assert.h>
diff --git a/libavutil/atomic.h b/libavutil/atomic.h
index a5c5fe8..57a0c19 100644
--- a/libavutil/atomic.h
+++ b/libavutil/atomic.h
@@ -23,12 +23,16 @@
 
 #include "config.h"
 
+#if HAVE_ATOMICS_NATIVE
+
 #if HAVE_ATOMICS_GCC
 #include "atomic_gcc.h"
 #elif HAVE_ATOMICS_WIN32
 #include "atomic_win32.h"
 #elif HAVE_ATOMICS_SUNCC
 #include "atomic_suncc.h"
+#endif
+
 #else
 
 /**
@@ -70,5 +74,6 @@ int avpriv_atomic_int_add_and_fetch(volatile int *ptr, int inc);
  */
 void *avpriv_atomic_ptr_cas(void * volatile *ptr, void *oldval, void *newval);
 
-#endif /* HAVE_MEMORYBARRIER */
+#endif /* HAVE_ATOMICS_NATIVE */
+
 #endif /* AVUTIL_ATOMIC_H */



More information about the ffmpeg-cvslog mailing list