[FFmpeg-devel] [PATCH] Move DECLARE_ALIGNED_{8, 16} macros to mem.h
Pavel Pavlov
pavel
Mon Mar 8 22:32:08 CET 2010
> > On Sat, Mar 06, 2010 at 01:27:56PM +0000, M?ns Rullg?rd wrote:
> >> Michael Niedermayer <michaelni at gmx.at> writes:
> >>
> >> > On Sat, Mar 06, 2010 at 02:50:30AM +0000, Mans Rullgard wrote:
> >> >> These macros naturally belong next to the generic DECLARE_ALIGNED
> >> >> macro.
> >> >> ---
> >> >> libavcodec/dsputil.h | 3 ---
> >> >> libavutil/mem.h | 2 ++
> >> >> 2 files changed, 2 insertions(+), 3 deletions(-)
> >> >
> >> > ok if they arent redundant but they look a little redundant
> >> > relative to just using DECLARE_ALIGNED directly
> >>
> >> I always wondered what they were for. OK to do a global search and
> >> replace?
> >
> > ok
>
> Done
>
> --
> M?ns Rullg?rd
There is also ATTR_ALIGN left out in a couple of files (dsputil_vis.c, fdct_mmx.c, idct_mmx.c). Should also be replaced with DECLARE_ALIGNED
More information about the ffmpeg-devel
mailing list