[MPlayer-dev-eng] [PATCH] fix printf format string warnings
Rich Felker
dalias at aerifal.cx
Thu Nov 22 16:12:40 CET 2007
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..
Rich
More information about the MPlayer-dev-eng
mailing list