[MPlayer-dev-eng] [PATCH] fix printf format string warnings
Diego Biurrun
diego at biurrun.de
Thu Nov 22 16:13:58 CET 2007
On Thu, Nov 22, 2007 at 10:12:40AM -0500, Rich Felker wrote:
> On Thu, Nov 22, 2007 at 01:34:38AM +0100, Diego Biurrun wrote:
> > On Wed, Nov 21, 2007 at 01:22:03PM -0500, Rich Felker wrote:
> > > On Wed, Nov 21, 2007 at 11:49:10AM +0100, Diego Biurrun wrote:
> > > > This patch fixes a couple of "format '%d' expects type 'int', but
> > > > argument 2 has type 'long int'" warnings. OK to apply?
> > >
> > > What type are those struct fields? If they're something like uint32_t
> > > then either use constants from inttypes.h or cast to long before
> > > printing... Just assuming they're the same as long would be very wrong
> > > and break 64bit archs!
> >
> > They are type long, see loader/wine/vfw.h.
>
> Nonetheless I'm pretty sure that's a mistake and the correct type
> would be int32_t..
Isn't that orthogonal to my patch? And Windows DLLs only work on 32 bit
systems anyway, so ve_vfw.c couldn't care less. Am I mistaken?
Diego
More information about the MPlayer-dev-eng
mailing list