Bug 323912

Summary:	valgrind.h header isn't compatible for mingw64
Product:	[Developer tools] valgrind	Reporter:	Makoto Kato <m_kato>
Component:	general	Assignee:	Julian Seward <jseward>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bart.vanassche+kde
Priority:	NOR
Version First Reported In:	3.9.0.SVN
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	a patch to fix this.

Description Makoto Kato 2013-08-23 07:47:57 UTC

if including this header, some macros aren't compatible on mingw64 compiler

Reproducible: Always

Comment 1 Makoto Kato 2013-08-23 07:57:11 UTC

Created attachment 81868 [details]
a patch to fix this.

Since Win64 is LLP64 model, don't cast to unsigned long.

Comment 2 Bart Van Assche 2013-08-25 06:11:11 UTC

Thanks for the patch. A (slightly modified) version has been applied as r13513. Please verify whether the new version of valgrind.h works fine for you.

Comment 3 Julian Seward 2013-09-12 14:30:26 UTC

I'm assuming that r13513 is OK, and closing.