[FFmpeg-devel] [PATCH] Move DECLARE_ALIGNED_{8, 16} macros to mem.h

[Måns Rullgård]

Pavel Pavlov <pavel at summit-tech.ca> writes:

>> > 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
>
> 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

Ugh, I've missed those.  Patches welcome.

-- 
M?ns Rullg?rd
mans at mansr.com