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

Derek Buitenhuis git at videolan.org
Tue Aug 28 16:33:22 CEST 2012


ffmpeg | branch: master | Derek Buitenhuis <derek.buitenhuis at gmail.com> | Sun Aug 26 20:57:08 2012 -0400| [5395d882b4c06825f43239d2e8869c82275bb3b6] | committer: Derek Buitenhuis

vp56: Don't use DECLARE_ALIGN on a typedef name

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>

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

 libavcodec/vp56.h |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavcodec/vp56.h b/libavcodec/vp56.h
index d03b733..000dfe0 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);



More information about the ffmpeg-cvslog mailing list