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

Baptiste Coudurier baptiste.coudurier
Wed Aug 26 03:32:27 CEST 2009


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.

[...]

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org



More information about the ffmpeg-devel mailing list