[FFmpeg-devel] [PATCH] split vc1dsp_mmx in its own compilation unit
Christophe GISQUET
christophe.gisquet
Tue Nov 27 01:05:23 CET 2007
Hello,
Aurelien Jacobs a ?crit :
> This allows disabling vc1dsp code when vc1 is disabled.
> This patch duplicates the ff_pw_3 const integer. I think it's not
+static const uint64_t ff_pw_3 attribute_used __attribute__
((aligned(8))) = 0x0003000300030003ULL;
+
Nitpicking but:
- I prefer use of macro DECLARE_ALIGNED_8/16 (though you have an
attribute_used here)
- if it's static, then it doesn't need ff_ prefix; code throughout the
file should be modified if you choose to perform that change
Otherwise, no objection, on the contrary, that's a good move.
Best regards,
--
Christophe GISQUET
More information about the ffmpeg-devel
mailing list