[FFmpeg-devel] [PATCH] Remove some unnecessary alignment specifiers

Måns Rullgård mans
Wed Aug 26 14:13:17 CEST 2009


Baptiste Coudurier <baptiste.coudurier at gmail.com> writes:

> Hi Mans,
>
> On 08/25/2009 04:32 PM, Mans Rullgard wrote:
>> None of these arrays are used in ways requiring extra alignment.
>>
>> [...]
>>
>> diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c
>> index ed349d2..25ccd13 100644
>> --- a/libavcodec/dnxhddec.c
>> +++ b/libavcodec/dnxhddec.c
>> @@ -40,7 +40,7 @@ typedef struct {
>>       int last_dc[3];
>>       DSPContext dsp;
>>       DECLARE_ALIGNED_16(DCTELEM, blocks[8][64]);
>> -    DECLARE_ALIGNED_8(ScanTable, scantable);
>> +    ScanTable scantable;
>>       const CIDEntry *cid_table;
>>   } DNXHDContext;
>
> Ok.
>
>> diff --git a/libavcodec/dv.c b/libavcodec/dv.c
>> index 96d8c39..95b2b79 100644
>> --- a/libavcodec/dv.c
>> +++ b/libavcodec/dv.c
>> @@ -527,8 +527,8 @@ static int dv_decode_video_segment(AVCodecContext *avctx, void *arg)
>>       GetBitContext gb;
>>       BlockInfo mb_data[5 * DV_MAX_BPM], *mb, *mb1;
>>       DECLARE_ALIGNED_16(DCTELEM, sblock[5*DV_MAX_BPM][64]);
>> -    DECLARE_ALIGNED_8(uint8_t, mb_bit_buffer[80 + 4]); /* allow some slack */
>> -    DECLARE_ALIGNED_8(uint8_t, vs_bit_buffer[5 * 80 + 4]); /* allow some slack */
>> +    uint8_t mb_bit_buffer[80 + 4]; /* allow some slack */
>> +    uint8_t vs_bit_buffer[5 * 80 + 4]; /* allow some slack */
>>       const int log2_blocksize = 3-s->avctx->lowres;
>>       int is_field_mode[5];
>
> Ok.

Applied.

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



More information about the ffmpeg-devel mailing list