Janne Grunau <janne-li...@jannau.net> writes: > On 2012-12-07 23:03:07 +0000, Måns Rullgård wrote: >> This changes the LOCAL_ALIGNED definition on systems where >> DECLARE_ALIGNED is used such it matches the manual alignment >> case, ensuring invalid use will not compile on x86 only to >> fail on everything else. >> >> Signed-off-by: Mans Rullgard <m...@mansr.com> >> --- >> libavcodec/dsputil.h | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h >> index 5640f3a..c1bd401 100644 >> --- a/libavcodec/dsputil.h >> +++ b/libavcodec/dsputil.h >> @@ -610,7 +610,9 @@ void ff_dsputil_init_dwt(DSPContext *c); >> uint8_t la_##v[sizeof(t s o) + (a)]; \ >> t (*v) o = (void *)FFALIGN((uintptr_t)la_##v, a) >> >> -#define LOCAL_ALIGNED_D(a, t, v, s, o, ...) DECLARE_ALIGNED(a, t, v) s o >> +#define LOCAL_ALIGNED_D(a, t, v, s, o, ...) \ >> + DECLARE_ALIGNED(a, t, la_##v) s o; \ >> + t (*v) o = la_##v > > Not really happy about changing arrays to pointers but since we can't > use the array properties (like sizeof array) anyway since that would > break too it's not a great loss. I wouldn't be surprised there some > memsets or memcpys arround using sizeof(array) of a LOCAL_ALIGNED > variable. > > Ok for me but wait a couple of days
It's been a few days of silence... I've fixed the typo in the subject so no need to point that out. -- Måns Rullgård m...@mansr.com _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
