msvcp90: Fix 64-bit compilation warnings

David Laight david at l8s.co.uk
Sun Sep 5 04:36:59 CDT 2010


On Wed, Sep 01, 2010 at 11:56:40AM +0200, Alexandre Julliard wrote:
> Piotr Caban <piotr at codeweavers.com> writes:
> 
> > @@ -596,7 +596,7 @@ DEFINE_THISCALL_WRAPPER(MSVCP_basic_string_char_erase, 12)
> >  basic_string_char* __thiscall MSVCP_basic_string_char_erase(
> >          basic_string_char *this, size_t pos, size_t len)
> >  {
> > -    TRACE("%p %d %d\n", this, pos, len);
> > +    TRACE("%p %d %d\n", this, (int)pos, (int)len);
> 
> You should be casting to long, otherwise you'll potentially truncate the
> value.

In which case the format specifiers need to be %ld as well...

	David

-- 
David Laight: david at l8s.co.uk



More information about the wine-devel mailing list