Austin English : kernel32/tests: Make sure to use return values (LLVM/Clang ).

Alexandre Julliard julliard at winehq.org
Thu Feb 17 13:06:35 CST 2011


Module: wine
Branch: master
Commit: 194e43f6e3d68691c502fece1c8503b6354f3b9d
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=194e43f6e3d68691c502fece1c8503b6354f3b9d

Author: Austin English <austinenglish at gmail.com>
Date:   Wed Feb 16 18:18:11 2011 -0800

kernel32/tests: Make sure to use return values (LLVM/Clang).

---

 dlls/kernel32/tests/file.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/dlls/kernel32/tests/file.c b/dlls/kernel32/tests/file.c
index 375b5f0..4e8136a 100644
--- a/dlls/kernel32/tests/file.c
+++ b/dlls/kernel32/tests/file.c
@@ -178,11 +178,11 @@ static void test__hwrite( void )
     ok( 0 != memory_object, "LocalAlloc fails. (Could be out of memory.)\n" );
 
     contents = LocalLock( memory_object );
+    ok( NULL != contents, "LocalLock whines\n" );
 
     filehandle = _lopen( filename, OF_READ );
 
     contents = LocalLock( memory_object );
-
     ok( NULL != contents, "LocalLock whines\n" );
 
     ok( bytes_written == _hread( filehandle, contents, bytes_written), "read length differ from write length\n" );
@@ -541,11 +541,11 @@ static void test__lwrite( void )
     ok( 0 != memory_object, "LocalAlloc fails, could be out of memory\n" );
 
     contents = LocalLock( memory_object );
+    ok( NULL != contents, "LocalLock whines\n" );
 
     filehandle = _lopen( filename, OF_READ );
 
     contents = LocalLock( memory_object );
-
     ok( NULL != contents, "LocalLock whines\n" );
 
     ok( bytes_written == _hread( filehandle, contents, bytes_written), "read length differ from write length\n" );
@@ -2094,7 +2094,7 @@ static void test_FindFirstFileA(void)
     strcat(buffer2, "nul");
     handle = FindFirstFileA(buffer2, &data);
     err = GetLastError();
-    ok( handle != INVALID_HANDLE_VALUE, "FindFirstFile on %s failed\n", buffer2 );
+    ok( handle != INVALID_HANDLE_VALUE, "FindFirstFile on %s failed: %d\n", buffer2, err );
     ok( 0 == lstrcmpiA(data.cFileName, "nul"), "wrong name %s\n", data.cFileName );
     ok( FILE_ATTRIBUTE_ARCHIVE == data.dwFileAttributes ||
         FILE_ATTRIBUTE_DEVICE == data.dwFileAttributes /* Win9x */,
@@ -2114,7 +2114,7 @@ static void test_FindFirstFileA(void)
     strcpy(buffer2, "lpt1");
     handle = FindFirstFileA(buffer2, &data);
     err = GetLastError();
-    ok( handle != INVALID_HANDLE_VALUE, "FindFirstFile on %s failed\n", buffer2 );
+    ok( handle != INVALID_HANDLE_VALUE, "FindFirstFile on %s failed: %d\n", buffer2, err );
     ok( 0 == lstrcmpiA(data.cFileName, "lpt1"), "wrong name %s\n", data.cFileName );
     ok( FILE_ATTRIBUTE_ARCHIVE == data.dwFileAttributes ||
         FILE_ATTRIBUTE_DEVICE == data.dwFileAttributes /* Win9x */,




More information about the wine-cvs mailing list