Move "block" variable in rtjpeg decoder to context to ensure sufficient alignment
authorReimar Döffinger <Reimar.Doeffinger@gmx.de>
Sun, 24 May 2009 08:36:52 +0000 (08:36 +0000)
committerReimar Döffinger <Reimar.Doeffinger@gmx.de>
Sun, 24 May 2009 08:36:52 +0000 (08:36 +0000)
for idct_put on architectures/compilers that have problems aligning on the stack.

Originally committed as revision 18924 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavcodec/rtjpeg.c
libavcodec/rtjpeg.h

index d9ca035..2736807 100644 (file)
@@ -98,7 +98,6 @@ static inline int get_block(GetBitContext *gb, DCTELEM *block, const uint8_t *sc
  */
 int rtjpeg_decode_frame_yuv420(RTJpegContext *c, AVFrame *f,
                                const uint8_t *buf, int buf_size) {
-    DECLARE_ALIGNED_16(DCTELEM, block[64]);
     GetBitContext gb;
     int w = c->w / 16, h = c->h / 16;
     int x, y;
@@ -107,6 +106,7 @@ int rtjpeg_decode_frame_yuv420(RTJpegContext *c, AVFrame *f,
     init_get_bits(&gb, buf, buf_size * 8);
     for (y = 0; y < h; y++) {
         for (x = 0; x < w; x++) {
+            DCTELEM *block = c->block;
             if (get_block(&gb, block, c->scan, c->lquant))
                 c->dsp->idct_put(y1, f->linesize[0], block);
             y1 += 8;
index 02f2058..d2745f8 100644 (file)
@@ -31,6 +31,7 @@ typedef struct {
     uint8_t scan[64];
     uint32_t lquant[64];
     uint32_t cquant[64];
+    DECLARE_ALIGNED_16(DCTELEM, block[64]);
 } RTJpegContext;
 
 void rtjpeg_decode_init(RTJpegContext *c, DSPContext *dsp,