[FFmpeg-devel] [PATCH] bink: correctly align local arrays

Måns Rullgård mans
Wed Jun 2 17:23:02 CEST 2010


Kostya <kostya.shishkov at gmail.com> writes:

> On Wed, Jun 02, 2010 at 03:29:03PM +0100, Mans Rullgard wrote:
>> ---
>>  libavcodec/bink.c |    4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/libavcodec/bink.c b/libavcodec/bink.c
>> index a988a34..b5921f9 100644
>> --- a/libavcodec/bink.c
>> +++ b/libavcodec/bink.c
>> @@ -681,8 +681,8 @@ static int bink_decode_plane(BinkContext *c, GetBitContext *gb, int plane_idx,
>>      int v, col[2];
>>      const uint8_t *scan;
>>      int xoff, yoff;
>> -    DECLARE_ALIGNED(16, DCTELEM, block[64]);
>> -    DECLARE_ALIGNED(16, uint8_t, ublock[64]);
>> +    LOCAL_ALIGNED_16(DCTELEM, block, [64]);
>> +    LOCAL_ALIGNED_16(uint8_t, ublock, [64]);
>>      int coordmap[64];
>>  
>>      const int stride = c->pic.linesize[plane_idx];
>
> Just apply that without sending patch, you're the guard of
> proper alignment here.

Done.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list