[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