[FFmpeg-devel] [PATCH] swscale.c
Pavel Pavlov
pavel
Thu Jun 4 02:19:14 CEST 2009
> -----Original Message-----
> On Wed, Jun 3, 2009 at 2:56 PM, Pavel Pavlov
> <pavel at summit-tech.ca> wrote:
> > Old:
> > ? ? ? ?static const uint8_t ?__attribute__((aligned(8)))
> > dither_2x2_4[2][8]={
> >
> > New:
> > ? ? ? ?DECLARE_ALIGNED(8, static const uint8_t,
> dither_2x2_4[2][8])={
>
> Hmm, shouldn't some of these (also in your other patches) use
> DECLARE_ASM_CONST?
>
> Also is it best to:
> DECLARE_ALIGNED(8, static const
> static const DECLARE_ALIGNED(8
> or some other combination?
>
> Some grepping shows we have a couple of different
> combinations for those.
>
I simply prefer to use one macro everywhere, instead of ten others.
DECLARE_ASM_CONST just makes it less clear; with DECLARE_ALIGNED
it's obvious that alignment is required and that variable is static or
not.
More information about the ffmpeg-devel
mailing list