[MPlayer-dev-eng] [PATCH] Fix AC3 passthrough on Intel Macs

compn tempn at twmi.rr.com
Mon Aug 16 06:06:58 CEST 2010


On Sun, 15 Aug 2010 14:54:19 -0400, Sean McGovern wrote:
>
>As I mentioned previously I have been looking into the AC3 passthrough
>problem on Intel Macs and have made a patch to fix the issue. Also included
>the changes from the previous patch that squelches some compiler warnings.
>
>Comments, suggestions, constructive criticisms welcome.
>
>Thanks in advance,
>Sean McGovern
>
>
>[ao_coreaudio-mactel.diff  application/octet-stream (5.4KB)]
>Index: libao2/ao_coreaudio.c
>-static OSStatus theRenderProc(void *inRefCon,
>-                              AudioUnitRenderActionFlags *inActionFlags,
>-                              const AudioTimeStamp *inTimeStamp,
>-                              UInt32 inBusNumber, UInt32 inNumFrames,
>+static OSStatus theRenderProc(__unused void *inRefCon,
>+                              __unused AudioUnitRenderActionFlags *inActionFlags,
>+                              __unused const AudioTimeStamp *inTimeStamp,
>+                              __unused UInt32 inBusNumber, UInt32 inNumFrames,

if the patch without the compiler warnings is smaller, you should undo
that patch. this __unused stuff looks terrible and makes the patch
harder to read.

-compn


More information about the MPlayer-dev-eng mailing list