frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

hw/xwin: Changes in error logging to make libwinclipboard more useful as a library 

Consistently use ErrorF() rather than winErrorFVerb()

Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Reviewed-by: Colin Harrison <colin.harrison@virgin.net>
This commit is contained in:
Jon TURNEY 2013-06-17 20:03:04 +01:00
parent e1cf2b47be
commit 5ae674ae6d
1 changed files with 25 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
hw/xwin/winclipboard/wndproc.c
View File

@ -189,7 +189,7 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
s_hwndNextViewer = (HWND) lParam; s_hwndNextViewer = (HWND) lParam;
if (s_hwndNextViewer == hwnd) { if (s_hwndNextViewer == hwnd) {
s_hwndNextViewer = NULL; s_hwndNextViewer = NULL;
winErrorFVerb(1, "winClipboardWindowProc - WM_CHANGECBCHAIN: " ErrorF("winClipboardWindowProc - WM_CHANGECBCHAIN: "
"attempted to set next window to ourselves."); "attempted to set next window to ourselves.");
} }
} }
@ -262,7 +262,7 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
s_fCBCInitialized = FALSE; s_fCBCInitialized = FALSE;
ChangeClipboardChain(hwnd, s_hwndNextViewer); ChangeClipboardChain(hwnd, s_hwndNextViewer);
winFixClipboardChain(); winFixClipboardChain();
winErrorFVerb(1, "winClipboardWindowProc - WM_DRAWCLIPBOARD - " ErrorF("winClipboardWindowProc - WM_DRAWCLIPBOARD - "
"Nested calls detected. Re-initing.\n"); "Nested calls detected. Re-initing.\n");
winDebug("winClipboardWindowProc - WM_DRAWCLIPBOARD: Exit\n"); winDebug("winClipboardWindowProc - WM_DRAWCLIPBOARD: Exit\n");
s_fProcessingDrawClipboard = FALSE; s_fProcessingDrawClipboard = FALSE;
@ -325,8 +325,8 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
XSetSelectionOwner(pDisplay, XA_PRIMARY, None, CurrentTime); XSetSelectionOwner(pDisplay, XA_PRIMARY, None, CurrentTime);
} }
else if (BadWindow == iReturn || BadAtom == iReturn) else if (BadWindow == iReturn || BadAtom == iReturn)
winErrorFVerb(1, "winClipboardWindowProc - WM_DRAWCLIPBOARD - " ErrorF("winClipboardWindowProc - WM_DRAWCLIPBOARD - "
"XGetSelection failed for PRIMARY: %d\n", "XGetSelectionOwner failed for PRIMARY: %d\n",
iReturn); iReturn);
/* Release CLIPBOARD selection if owned */ /* Release CLIPBOARD selection if owned */
@ -337,8 +337,8 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
XSetSelectionOwner(pDisplay, atoms->atomClipboard, None, CurrentTime); XSetSelectionOwner(pDisplay, atoms->atomClipboard, None, CurrentTime);
} }
else if (BadWindow == iReturn || BadAtom == iReturn) else if (BadWindow == iReturn || BadAtom == iReturn)
winErrorFVerb(1, "winClipboardWindowProc - WM_DRAWCLIPBOARD - " ErrorF("winClipboardWindowProc - WM_DRAWCLIPBOARD - "
"XGetSelection failed for CLIPBOARD: %d\n", "XGetSelectionOwner failed for CLIPBOARD: %d\n",
iReturn); iReturn);
winDebug("winClipboardWindowProc - WM_DRAWCLIPBOARD: Exit\n"); winDebug("winClipboardWindowProc - WM_DRAWCLIPBOARD: Exit\n");
@ -353,7 +353,7 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
XA_PRIMARY, iWindow, CurrentTime); XA_PRIMARY, iWindow, CurrentTime);
if (iReturn == BadAtom || iReturn == BadWindow || if (iReturn == BadAtom || iReturn == BadWindow ||
XGetSelectionOwner(pDisplay, XA_PRIMARY) != iWindow) { XGetSelectionOwner(pDisplay, XA_PRIMARY) != iWindow) {
winErrorFVerb(1, "winClipboardWindowProc - WM_DRAWCLIPBOARD - " ErrorF("winClipboardWindowProc - WM_DRAWCLIPBOARD - "
"Could not reassert ownership of PRIMARY\n"); "Could not reassert ownership of PRIMARY\n");
} }
else { else {
@ -367,7 +367,7 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
if (iReturn == BadAtom || iReturn == BadWindow || if (iReturn == BadAtom || iReturn == BadWindow ||
XGetSelectionOwner(pDisplay, atoms->atomClipboard) != iWindow) { XGetSelectionOwner(pDisplay, atoms->atomClipboard) != iWindow) {
winErrorFVerb(1, "winClipboardWindowProc - WM_DRAWCLIPBOARD - " ErrorF("winClipboardWindowProc - WM_DRAWCLIPBOARD - "
"Could not reassert ownership of CLIPBOARD\n"); "Could not reassert ownership of CLIPBOARD\n");
} }
else { else {
@ -421,7 +421,7 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
atoms->atomLocalProperty, atoms->atomLocalProperty,
iWindow, CurrentTime); iWindow, CurrentTime);
if (iReturn == BadAtom || iReturn == BadWindow) { if (iReturn == BadAtom || iReturn == BadWindow) {
winErrorFVerb(1, "winClipboardWindowProc - WM_RENDER*FORMAT - " ErrorF("winClipboardWindowProc - WM_RENDER*FORMAT - "
"XConvertSelection () failed\n"); "XConvertSelection () failed\n");
break; break;
} }
@ -436,14 +436,14 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
} }
if (!OpenClipboard(hwnd)) { if (!OpenClipboard(hwnd)) {
winErrorFVerb(1, "winClipboardWindowProc - WM_RENDER*FORMATS - " ErrorF("winClipboardWindowProc - WM_RENDER*FORMATS - "
"OpenClipboard () failed: %08x\n", "OpenClipboard () failed: %08x\n",
GetLastError()); GetLastError());
break; break;
} }
if (!EmptyClipboard()) { if (!EmptyClipboard()) {
winErrorFVerb(1, "winClipboardWindowProc - WM_RENDER*FORMATS - " ErrorF("winClipboardWindowProc - WM_RENDER*FORMATS - "
"EmptyClipboard () failed: %08x\n", "EmptyClipboard () failed: %08x\n",
GetLastError()); GetLastError());
break; break;
@ -479,8 +479,7 @@ winClipboardWindowProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
/* We must close the clipboard */ /* We must close the clipboard */
if (!CloseClipboard()) { if (!CloseClipboard()) {
winErrorFVerb(1, ErrorF("winClipboardWindowProc - WM_RENDERALLFORMATS - "
"winClipboardWindowProc - WM_RENDERALLFORMATS - "
"CloseClipboard () failed: %08x\n", "CloseClipboard () failed: %08x\n",
GetLastError()); GetLastError());
break; break;