summaryrefslogtreecommitdiff
path: root/hw/xwin/winclipboard
diff options
context:
space:
mode:
authorJon TURNEY <jon.turney@dronecode.org.uk>2015-02-06 20:22:11 +0000
committerJon TURNEY <jon.turney@dronecode.org.uk>2015-07-07 16:52:50 +0100
commite3cfeb949a9c9363beacdb00acdd9723ed54fac2 (patch)
tree36d94d9bc39ea3987f5d49877e995702f8f125a6 /hw/xwin/winclipboard
parent4f8661fac985306c56330cae69ffc19e5dd0af61 (diff)
hw/xwin: printf format fixes for WPARAM and LPARAM types
Some Win32 API types are different fundamental types in the 32-bit and 64-bit 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 impossible to write printf format specifiers which are correct for all those targets, so we use some macros to provide the correct specifier for the target. LPARAM and WPARAM are integer types which can contain a pointer LPARAM is long in ILP32 and long long in LLP64 WPARAM is unsigned int in ILP32 and unsigned long long in LLP64 Generally, these are just used to passs integer parameters, so for simplicity, cast to int and use an int-compatible format In the specific case of WM_CHANGECBCHAIN, they are used to pass HWND, so cast to that type and print using an appropriate format. Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk> Reviewed-by: Colin Harrison <colin.harrison@virgin.net>
Diffstat (limited to 'hw/xwin/winclipboard')
-rw-r--r--hw/xwin/winclipboard/wndproc.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/hw/xwin/winclipboard/wndproc.c b/hw/xwin/winclipboard/wndproc.c
index e1517f207..d17cf2e86 100644
--- a/hw/xwin/winclipboard/wndproc.c
+++ b/hw/xwin/winclipboard/wndproc.c
@@ -194,9 +194,9 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
case WM_CHANGECBCHAIN:
{
- winDebug("winClipboardWindowProc - WM_CHANGECBCHAIN: wParam(%x) "
- "lParam(%x) s_hwndNextViewer(%p)\n",
- wParam, lParam, s_hwndNextViewer);
+ winDebug("winClipboardWindowProc - WM_CHANGECBCHAIN: wParam(%p) "
+ "lParam(%p) s_hwndNextViewer(%p)\n",
+ (HWND)wParam, (HWND)lParam, s_hwndNextViewer);
if ((HWND) wParam == s_hwndNextViewer) {
s_hwndNextViewer = (HWND) lParam;
@@ -441,7 +441,7 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
int best_target = 0;
winDebug("winClipboardWindowProc - WM_RENDERFORMAT %d - Hello.\n",
- wParam);
+ (int)wParam);
/* Flag whether to convert to Unicode or not */
fConvertToUnicode = (CF_UNICODETEXT == wParam);