Diego Biurrun <[email protected]> writes: > This makes them safe to use in non-fully braced if-blocks and similar. > --- > libavcodec/x86/dsputil_mmx.c | 16 ++++++++++++---- > 1 files changed, 12 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/x86/dsputil_mmx.c b/libavcodec/x86/dsputil_mmx.c > index 3cc0f6b..b8d7e6b 100644 > --- a/libavcodec/x86/dsputil_mmx.c > +++ b/libavcodec/x86/dsputil_mmx.c > @@ -2337,6 +2337,7 @@ extern void ff_butterflies_float_interleave_avx(float > *dst, const float *src0, > const float *src1, int len); > > #define SET_QPEL_FUNCS(PFX, IDX, SIZE, CPU, PREFIX) \ > + do { > \ > c->PFX ## _pixels_tab[IDX][ 0] = PREFIX ## PFX ## SIZE ## _mc00_ ## CPU; > \ > c->PFX ## _pixels_tab[IDX][ 1] = PREFIX ## PFX ## SIZE ## _mc10_ ## CPU; > \ > c->PFX ## _pixels_tab[IDX][ 2] = PREFIX ## PFX ## SIZE ## _mc20_ ## CPU; > \ > @@ -2352,25 +2353,32 @@ extern void ff_butterflies_float_interleave_avx(float > *dst, const float *src0, > c->PFX ## _pixels_tab[IDX][12] = PREFIX ## PFX ## SIZE ## _mc03_ ## CPU; > \ > c->PFX ## _pixels_tab[IDX][13] = PREFIX ## PFX ## SIZE ## _mc13_ ## CPU; > \ > c->PFX ## _pixels_tab[IDX][14] = PREFIX ## PFX ## SIZE ## _mc23_ ## CPU; > \ > - c->PFX ## _pixels_tab[IDX][15] = PREFIX ## PFX ## SIZE ## _mc33_ ## CPU > + c->PFX ## _pixels_tab[IDX][15] = PREFIX ## PFX ## SIZE ## _mc33_ ## CPU; > \ > + } while (0);
Wrong indentation. There are several acceptable forms for such macros. This is not one of them. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
