[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