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

Reimar Döffinger Reimar.Doeffinger at gmx.de
Tue Nov 11 08:53:38 CET 2014



On 07.11.2014, at 13:33, Matt Oliver <protogonoi at gmail.com> 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 forgot to say: looks good to me.
I'll try to find time to commit, unless someone beats me to it.


More information about the MPlayer-dev-eng mailing list