[FFmpeg-devel] [PATCH] vp56: Don't use DECLARE_ALIGN on a typedef name

Derek Buitenhuis derek.buitenhuis at gmail.com
Mon Aug 27 04:09:10 CEST 2012


Instead, use it on the first member, since by definition, if
any member is aligned, the whole struct must be, in order to
maintain that alignment.

Fixes compilation with some finicky compilers.

Idea for fix from Måns Rullgård.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
---
 libavcodec/vp56.h |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavcodec/vp56.h b/libavcodec/vp56.h
index 12f9380..d8f7ce6 100644
--- a/libavcodec/vp56.h
+++ b/libavcodec/vp56.h
@@ -36,9 +36,9 @@
 typedef struct vp56_context VP56Context;
 
 typedef struct {
-    int16_t x;
+    DECLARE_ALIGNED(4, int16_t, x);
     int16_t y;
-} DECLARE_ALIGNED(4, , VP56mv);
+} VP56mv;
 
 typedef void (*VP56ParseVectorAdjustment)(VP56Context *s,
                                           VP56mv *vect);
-- 
1.7.10.4



More information about the ffmpeg-devel mailing list