Andrew Riedi : user32: Uncomment and explain a test in test_DestroyCursor() .
Alexandre Julliard
julliard at winehq.org
Mon Dec 10 08:54:24 CST 2007
Module: wine
Branch: master
Commit: 573fcad0b4beaeb8287f0b4156562ecd6baf2aef
URL: http://source.winehq.org/git/wine.git/?a=commit;h=573fcad0b4beaeb8287f0b4156562ecd6baf2aef
Author: Andrew Riedi <andrewriedi at gmail.com>
Date: Sun Dec 9 23:05:29 2007 -0800
user32: Uncomment and explain a test in test_DestroyCursor().
---
dlls/user32/tests/cursoricon.c | 11 +++++++++--
1 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/dlls/user32/tests/cursoricon.c b/dlls/user32/tests/cursoricon.c
index 9858e32..6be44c9 100644
--- a/dlls/user32/tests/cursoricon.c
+++ b/dlls/user32/tests/cursoricon.c
@@ -411,9 +411,16 @@ static void test_DestroyCursor(void)
SetCursor(NULL);
- /* Trying to destroy the cursor properly fails now for some reason with ERROR_INVALID_CURSOR_HANDLE */
+ /* Trying to destroy the cursor properly fails now with
+ * ERROR_INVALID_CURSOR_HANDLE. This happens because we called
+ * DestroyCursor() 2+ times after calling SetCursor(). The calls to
+ * GetCursor() and SetCursor(NULL) in between make no difference. */
ret = DestroyCursor(cursor);
- /* ok(ret, "DestroyCursor failed, GetLastError=%d\n", GetLastError()); */
+ todo_wine {
+ ok(!ret, "DestroyCursor succeeded.\n");
+ error = GetLastError();
+ ok(error == ERROR_INVALID_CURSOR_HANDLE, "Last error: 0x%08x\n", error);
+ }
DeleteObject(cursorInfo.hbmMask);
DeleteObject(cursorInfo.hbmColor);
More information about the wine-cvs
mailing list