[PATCH] user32/tests: GetClipboardData() returns no error for missing formats.

[Francois Gouget]

Despite the documentation that says if GetClipboardData() returns NULL 
one can check GetLastError() to know why!

Signed-off-by: Francois Gouget 
---
 dlls/user32/clipboard.c       | 1 +
 dlls/user32/tests/clipboard.c | 5 +++++
 2 files changed, 6 insertions(+)

diff --git a/dlls/user32/clipboard.c b/dlls/user32/clipboard.c
index e8de7d9bdf6..a13085719a9 100644
--- a/dlls/user32/clipboard.c
+++ b/dlls/user32/clipboard.c
@@ -1070,6 +1070,7 @@ HANDLE WINAPI GetClipboardData( UINT format )
         GlobalFree( data );

if (status == STATUS_BUFFER_OVERFLOW) continue;  /* retry with the new size */
+        if (status == STATUS_OBJECT_NAME_NOT_FOUND) return 0; /* no such format */
         if (status)
         {
             SetLastError( RtlNtStatusToDosError( status ));
diff --git a/dlls/user32/tests/clipboard.c b/dlls/user32/tests/clipboard.c
index c8de860f8cc..4103baeef9b 100644
--- a/dlls/user32/tests/clipboard.c
+++ b/dlls/user32/tests/clipboard.c
@@ -1543,6 +1543,11 @@ static void test_handles( HWND hwnd )
     data = GetClipboardData( 0xdeadfade );
     ok( data == ptr, "wrong data %p\n", data );

+    SetLastError( 0xdeadbeef );
+    data = GetClipboardData( CF_RIFF );
+    ok( GetLastError() == 0xdeadbeef, "unexpected last error %d\n", GetLastError() );
+    ok( !data, "wrong data %p\n", data );
+
     h = SetClipboardData( CF_PRIVATEFIRST + 7, htext4 );
     ok( h == htext4, "got %p\n", h );
     ok( is_moveable( h ), "expected moveable mem %p\n", h );

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=95221

Your paranoid android.

=== debiant2 (32 bit report) ===

user32:
menu.c:2337: Test failed: test 25