commit 7308f16b3255ad9a0e9efe87554bd2949a2c8852
author Rein Klazes <wijn@wanadoo.nl> Thu Mar 31 19:07:08 2005 +0000
committer Alexandre Julliard <julliard@winehq.org> Thu Mar 31 19:07:08 2005 +0000
tree cc3afbdc439ce5edc919fb7617ae6a4c4ad16f84
parent a19bf290f0b3a352b7c09c55eb1b6f3f6afeeb11

Fix some incorrect uses of the ok macro where the result depends on
the evaluation order of its arguments.

dlls/kernel/tests/file.c

diff --git a/dlls/kernel/tests/file.c b/dlls/kernel/tests/file.c
index 3c1d17b..92ae9bf 100644
--- a/dlls/kernel/tests/file.c
+++ b/dlls/kernel/tests/file.c
@@ -571,9 +571,11 @@
        "WriteFile error %ld\n", GetLastError());
     ok(GetFileSize(hfile, NULL) == sizeof(prefix), "source file has wrong size\n");
     /* get the file time and change it to prove the difference */
-    ok(GetFileTime(hfile, NULL, NULL, &ft1), "GetFileTime error %ld\n", GetLastError());
+    ret = GetFileTime(hfile, NULL, NULL, &ft1);
+    ok( ret, "GetFileTime error %ld\n", GetLastError());
     ft1.dwLowDateTime -= 600000000; /* 60 second */
-    ok(SetFileTime(hfile, NULL, NULL, &ft1), "SetFileTime error %ld\n", GetLastError());
+    ret = SetFileTime(hfile, NULL, NULL, &ft1);
+    ok( ret, "SetFileTime error %ld\n", GetLastError());
     GetFileTime(hfile, NULL, NULL, &ft1);  /* get the actual time back */
     CloseHandle(hfile);
 
@@ -595,7 +597,8 @@
     ok(ret == sizeof(prefix), "destination file has wrong size %ld\n", ret);
 
     /* make sure that destination has the same filetime */
-    ok(ret = GetFileTime(hfile, NULL, NULL, &ft2), "GetFileTime error %ld\n", GetLastError());
+    ret = GetFileTime(hfile, NULL, NULL, &ft2);
+    ok( ret, "GetFileTime error %ld\n", GetLastError());
     ok(CompareFileTime(&ft1, &ft2) == 0, "destination file has wrong filetime\n");
 
     SetLastError(0xdeadbeef);