RE: [PATCH] print format does not match argument type

"Patrick Monnerat" <[email protected]> Wed, 11 Apr 2012 19:24:11 +0200
Newsgroups gmane.comp.debugging.insight
Message-ID <AB5E58B87EB73C46A38073D8F459F113D9F888@dataspheresrv01>
Roland Schwingel wrote:
> %llx appears to work (tested it on win32,win64 and linux 32bit).
> We can even remove the cast at all. I got no warning here.

No warning, but you may have a wrong value converted at run-time: this
is the case if the %llx descriptor uses a value larger than the one that
gets pushed and your're running on a big-endian machine or the bytes
above the pushed value are not zero...
I would really keep the (unsigned long long) cast, at least for run-time
purpose.

> So ok... Who is doing the change?

Not me: I'm not a committer ;-))

Cheers,
Patrick