changeset 11660:dd10b0c7d0de libavcodec

Make two functions out of #define hackery. Patch by Sebastian Vater, cdgs D basty A googlemail
author cehoyos
date Mon, 26 Apr 2010 22:00:57 +0000
parents 45512d8b6b35
children 7a5f3c94b9ad
files iff.c
diffstat 1 files changed, 31 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/iff.c	Sun Apr 25 15:05:16 2010 +0000
+++ b/iff.c	Mon Apr 26 22:00:57 2010 +0000
@@ -87,27 +87,44 @@
 }
 
 /**
- * Decode interleaved plane buffer
+ * Decode interleaved plane buffer up to 8bpp
+ * @param dst Destination buffer
+ * @param buf Source buffer
+ * @param buf_size
+ * @param bps bits_per_coded_sample (must be <= 8)
+ * @param plane plane number to decode as
+ */
+static void decodeplane8(uint8_t *dst, const uint8_t *const buf, int buf_size, int bps, int plane)
+{
+    GetBitContext gb;
+    int i, b;
+    init_get_bits(&gb, buf, buf_size * 8);
+    for(i = 0; i < (buf_size * 8 + bps - 1) / bps; i++) {
+        for (b = 0; b < bps; b++) {
+            dst[ i*bps + b ] |= get_bits1(&gb) << plane;
+        }
+    }
+}
+
+/**
+ * Decode interleaved plane buffer up to 24bpp
  * @param dst Destination buffer
  * @param buf Source buffer
  * @param buf_size
  * @param bps bits_per_coded_sample
  * @param plane plane number to decode as
  */
-#define DECLARE_DECODEPLANE(suffix, type) \
-static void decodeplane##suffix(void *dst, const uint8_t *const buf, int buf_size, int bps, int plane) \
-{ \
-    GetBitContext gb; \
-    int i, b; \
-    init_get_bits(&gb, buf, buf_size * 8); \
-    for(i = 0; i < (buf_size * 8 + bps - 1) / bps; i++) { \
-        for (b = 0; b < bps; b++) { \
-            ((type *)dst)[ i*bps + b ] |= get_bits1(&gb) << plane; \
-        } \
-    } \
+static void decodeplane32(uint32_t *dst, const uint8_t *const buf, int buf_size, int bps, int plane)
+{
+    GetBitContext gb;
+    int i, b;
+    init_get_bits(&gb, buf, buf_size * 8);
+    for(i = 0; i < (buf_size * 8 + bps - 1) / bps; i++) {
+        for (b = 0; b < bps; b++) {
+            dst[ i*bps + b ] |= get_bits1(&gb) << plane;
+        }
+    }
 }
-DECLARE_DECODEPLANE(8, uint8_t)
-DECLARE_DECODEPLANE(32, uint32_t)
 
 static int decode_frame_ilbm(AVCodecContext *avctx,
                             void *data, int *data_size,