glx/windows: Fix compilation with -Werror-format
authorJon Turney <jon.turney@dronecode.org.uk>
Thu, 6 Jun 2019 15:44:08 +0000 (16:44 +0100)
committerKenneth Graunke <kenneth@whitecape.org>
Fri, 7 Jun 2019 18:28:48 +0000 (11:28 -0700)
commit87173ded6e26ac791fc7d71184c560ed9b747817
tree0afa73f73f0d00ef00eaece839c80b16b33f7943
parentcd796120c9c9306219efea9e002819ef4c9ee245
glx/windows: Fix compilation with -Werror-format

Fix compilation where the DWORD type is used with a format, after
-Werror-format added by c9c1e261.

Some Win32 API types are different fundamental types in the 32-bit and
64-bit versions. This problem is then further compounded by the fact
that whilst both 32-bit Cygwin and 32-bit MinGW use the ILP32 data
model, 64-bit MinGW uses the LLP64 data model, but 64-bit Cygwin uses
the LP64 data model. This makes it near impossible to write printf
format specifiers which are correct for all those targets.

In the Win32 API, DWORD is an unsigned, 32-bit type.  So, it is defined
in terms of an unsigned long, except in the LP64 data model used by
64-bit Cygwin, where it is an unsigned int.

It should always be safe to cast it to unsigned int and use %u or %x.

Reviewed-by: Eric Anholt <eric@anholt.net>
src/glx/windows/windows_drawable.c
src/glx/windows/windowsgl.c