[FFmpeg-devel] [PATCH] swscale.c

Pavel Pavlov pavel
Thu Jun 4 18:29:56 CEST 2009


 

> -----Original Message-----
> > I simply prefer to use one macro everywhere, instead of ten others. 
> > DECLARE_ASM_CONST just makes it less clear; with 
> DECLARE_ALIGNED it's 
> > obvious that alignment is required and that variable is 
> static or not.
> 
> If you don't use DECLARE_ASM_CONST for "static const" stuff 
> that is used via MANGLE in asm code you will break ICC compilation.


In this regard my path is correct. It doesn't modify anything except it 
specifies alignment via DECLARE_ALIGNED macro instead of gcc specific
way.
Static/non-static modifiers are not touched at all.
Check all the diffs I sent. They are small and you can inspect all of
them 
in 5 minutes.



More information about the ffmpeg-devel mailing list