[MPlayer-dev-eng] [Patch] mpcodec: Use XXX_INLINE defines for inline asm

Erik Auerswald auerswal at unix-ag.uni-kl.de
Fri Nov 7 14:38:09 CET 2014


Hi,

On Fri, Nov 07, 2014 at 11:33:40PM +1100, Matt Oliver wrote:
> This patch is based on one supplied to FFmpeg devel. Currently the mpcodec
> code uses pre-processor defines such as HAVE_MMX for inline asm, given
> mplayer and ffmpeg have a HAVE_MMX_INLINE define specific for inline asm
> then the preprocessors can be changed to the _INLINE variants which were
> made for this purpose.
> 
> This allows for proper compilation on architectures where things like MMX
> are available but inline asm isnt and is required for compiling mpcodec
> with compilers that dont support inline asm (such as msvc as used by
> FFmpeg).
> 
> This patch updates all sections in mpcodecs not just those used by FFmpeg
> in order to keep everything consistent. Let me know what you think.

I think you have missed changing configure, so this patch cannot work
correctly.

Thanks,
Erik
-- 
If it ain't broke, don't fix it.


More information about the MPlayer-dev-eng mailing list