Remove macro duplication between common.h and intmath.h
authorMåns Rullgård <mans@mansr.com>
Wed, 7 Jul 2010 17:27:43 +0000 (17:27 +0000)
committerMåns Rullgård <mans@mansr.com>
Wed, 7 Jul 2010 17:27:43 +0000 (17:27 +0000)
Originally committed as revision 24086 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavutil/common.h
libavutil/intmath.h

index afce3c8a12acb7ebe82746c2d70dc490ea0744e2..68626666175d5c88cf156c2141e4f4841aab4ca8 100644 (file)
@@ -90,12 +90,8 @@ static inline av_const int av_log2_16bit_c(unsigned int v)
 #   include "intmath.h"
 #endif
 
-#ifndef av_log2
-#   define av_log2       av_log2_c
-#endif
-#ifndef av_log2_16bit
-#   define av_log2_16bit av_log2_16bit_c
-#endif
+/* Pull in unguarded fallback defines at the end of this file. */
+#include "common.h"
 
 /**
  * Clip a signed integer value into the amin-amax range.
@@ -312,3 +308,15 @@ static inline av_const int av_ceil_log2(int x)
 #endif /* HAVE_AV_CONFIG_H */
 
 #endif /* AVUTIL_COMMON_H */
+
+/*
+ * The following definitions are outside the multiple inclusion guard
+ * to ensure they are immediately available in intmath.h.
+ */
+
+#ifndef av_log2
+#   define av_log2       av_log2_c
+#endif
+#ifndef av_log2_16bit
+#   define av_log2_16bit av_log2_16bit_c
+#endif
index 7ba3afb913e6fd23cd8d0902a66912cd5d113ebb..8b400ef05464c92bdb345d3dcccd3f15cd65b104 100644 (file)
@@ -52,22 +52,8 @@ extern const uint32_t ff_inverse[257];
 #   endif
 #endif /* FASTDIV */
 
-/*
- * Get definition of av_log2_c from common.h.  In the event we got
- * here through common.h including this file, including it again will
- * be a no-op due to multi-inclusion guards, so we must duplicate the
- * fallback defines here.
- */
-
 #include "common.h"
 
-#ifndef av_log2
-#   define av_log2       av_log2_c
-#endif
-#ifndef av_log2_16bit
-#   define av_log2_16bit av_log2_16bit_c
-#endif
-
 extern const uint8_t ff_sqrt_tab[256];
 
 static inline av_const unsigned int ff_sqrt(unsigned int a)