[FFmpeg-devel] [PATCH] VC-1 MMX DSP functions
Sat Jul 7 15:55:16 CEST 2007
Zuxy Meng a ?crit :
>>> /tmp/ccaopxZe.s:25763: Error: `0(%rdx,%ecx)' is not a valid base/index
>> int where long int should be used. This probably was in the first patch,
>> but I forgot to reinsert a cast when refactoring vc1dsp_mmx.c.
> long only works for LP64; intptr_t is safer.
Then there are some modifications to be down throughout libavcodec. I
even wonder if any occurrence of something like "r"(stride) (or
line_width, or ...) should have that cast for safety.
At least for an automated search in i386 folder (my code included):
grep -rn '[SDrmRg]" *((long)' libavcodec/i386 | grep -v svn | wc -l
Could be fixed with:
sed -i 's|\([SDrmRg]" *((\)long)|\1intptr_t)|g' libavcodec/i386/*.[ch]
More information about the ffmpeg-devel