[FFmpeg-cvslog] r12311 - in trunk/libavcodec: dsputil.c dsputil.h mpegvideo.c mpegvideo.h

[Aurelien Jacobs]

Luca Barbato wrote:

> Luca Barbato wrote:
> > Mike Melanson wrote:
> >> aurel wrote:
> >>> Author: aurel
> >>> Date: Tue Mar  4 11:26:56 2008
> >>> New Revision: 12311
> >>>
> >>> Log:
> >>> move ff_init_scantable() into dsputil
> >>>
> >>> Modified:
> >>>    trunk/libavcodec/dsputil.c
> >>>    trunk/libavcodec/dsputil.h
> >>>    trunk/libavcodec/mpegvideo.c
> >>>    trunk/libavcodec/mpegvideo.h
> >> Boom. This change broke all of the PowerPC builds. Check FATE:
> >>
> >>   http://fate.multimedia.cx/
> >>
> >> Click on the history link for a particular configuration to see
> >> which change broke compilation. The error boils down to:
> >>
> >> /tmp/fate/source/libavcodec/dsputil.h:165: error: expected
> >> specifier-qualifier-list before 'DECLARE_ALIGNED_8'
> >>
> >> repeated over and over for many files.
> >>
> > 
> > s/DECLARE_ALIGNED_8(/DECLARE_ALIGNED(8,
> > 
> err make it 16, I'll fix it tonight...

Obviously yes. Fixed. Thanks.

When moving this code, I've seen this #ifdef ARCH_POWERPC and
thought it was ugly. But I didn't even thought it could fail to
build :-(
Anyway, thanks to FATE !

Aurel