Re: [Qemu-devel] [PATCH v2 1/2] coreaudio: Fix OSStatus format specifier

2011-06-28 Thread Alexandre Raymond
Sorry for the delay. Commit 744d3644181ddb16ef5944a0f9217e46961c8c84 works fine on OSX 10.6. Alexandre On Thu, Jun 23, 2011 at 10:58 AM, malc wrote: > On Thu, 23 Jun 2011, Andreas F?rber wrote: > >> OSStatus type is defined as SInt32. That's signed int on __LP64__ and >> signed long otherwise.

Re: [Qemu-devel] [PATCH v2 1/2] coreaudio: Fix OSStatus format specifier

2011-06-23 Thread malc
On Thu, 23 Jun 2011, Andreas F?rber wrote: > OSStatus type is defined as SInt32. That's signed int on __LP64__ and > signed long otherwise. > Since it is an explicit 32-bit-width type, cast to corresponsing POSIX type > and use PRId32 format specifier. This avoids a warning on ppc64. > [..snip..

[Qemu-devel] [PATCH v2 1/2] coreaudio: Fix OSStatus format specifier

2011-06-23 Thread Andreas Färber
OSStatus type is defined as SInt32. That's signed int on __LP64__ and signed long otherwise. Since it is an explicit 32-bit-width type, cast to corresponsing POSIX type and use PRId32 format specifier. This avoids a warning on ppc64. Cc: malc Signed-off-by: Andreas Faerber --- audio/coreaudio.c