[MPlayer-cvslog] r31712 - trunk/libao2/ao_coreaudio.c

[Reimar Döffinger]

On Tue, Jul 13, 2010 at 02:04:31PM -0700, Alexander Strange wrote:
> On Tue, Jul 13, 2010 at 1:58 PM, Adrian Stutz <adrian at sttz.ch> wrote:
> > On Mon, Jul 12, 2010 at 19:45, Diego Biurrun <diego at biurrun.de> wrote:
> >> I still get a large number of these, maybe you can fix them:
> >
> > Hm, it seems my fix wasn't right and I'm not sure how to handle this correctly.
> >
> > UInt32 is defined as:
> >> #if __LP64__
> >> typedef unsigned int                    UInt32;
> >> #else
> >> typedef unsigned long                   UInt32;
> >> #endif
> >
> > So it should be %u in 64bit and %lu on 32bit systems (which I guess is
> > why the warnings disappeared for me but appeared for you).
> >
> > Should I just cast them to unsigned long and use %lu?
> 
> Yes, I believe this is the recommended way to print UInt32 and OSStatus.

Uhg, that would cast then to a 64 bit value. Both unsigned and uint32_t
would be reasonable values to cast to, but long isn't really.