André Hentschel : msvcrt/tests: Don' t test function directly when reporting errno.
Alexandre Julliard
julliard at winehq.org
Fri Feb 25 11:19:44 CST 2011
Module: wine
Branch: master
Commit: 188b3ae1b1c403363ddcca6028bc8d4c20d47240
URL: http://source.winehq.org/git/wine.git/?a=commit;h=188b3ae1b1c403363ddcca6028bc8d4c20d47240
Author: André Hentschel <nerv at dawncrow.de>
Date: Thu Feb 24 21:14:37 2011 +0100
msvcrt/tests: Don't test function directly when reporting errno.
---
dlls/msvcrt/tests/file.c | 31 +++++++++++++++++++++----------
1 files changed, 21 insertions(+), 10 deletions(-)
diff --git a/dlls/msvcrt/tests/file.c b/dlls/msvcrt/tests/file.c
index 33fe128..f70285b 100644
--- a/dlls/msvcrt/tests/file.c
+++ b/dlls/msvcrt/tests/file.c
@@ -1316,13 +1316,15 @@ static void test_stat(void)
{
int fd;
int pipes[2];
+ int ret;
struct stat buf;
/* Tests for a file */
fd = open("stat.tst", O_WRONLY | O_CREAT | O_BINARY, _S_IREAD |_S_IWRITE);
if (fd >= 0)
{
- ok(fstat(fd, &buf) == 0, "fstat failed: errno=%d\n", errno);
+ ret = fstat(fd, &buf);
+ ok(!ret, "fstat failed: errno=%d\n", errno);
ok((buf.st_mode & _S_IFMT) == _S_IFREG, "bad format = %06o\n", buf.st_mode);
ok((buf.st_mode & 0777) == 0666, "bad st_mode = %06o\n", buf.st_mode);
ok(buf.st_dev == 0, "st_dev is %d, expected 0\n", buf.st_dev);
@@ -1330,7 +1332,8 @@ static void test_stat(void)
ok(buf.st_nlink == 1, "st_nlink is %d, expected 1\n", buf.st_nlink);
ok(buf.st_size == 0, "st_size is %d, expected 0\n", buf.st_size);
- ok(stat("stat.tst", &buf) == 0, "stat failed: errno=%d\n", errno);
+ ret = stat("stat.tst", &buf);
+ ok(!ret, "stat failed: errno=%d\n", errno);
ok((buf.st_mode & _S_IFMT) == _S_IFREG, "bad format = %06o\n", buf.st_mode);
ok((buf.st_mode & 0777) == 0666, "bad st_mode = %06o\n", buf.st_mode);
ok(buf.st_dev == buf.st_rdev, "st_dev (%d) and st_rdev (%d) differ\n", buf.st_dev, buf.st_rdev);
@@ -1346,7 +1349,8 @@ static void test_stat(void)
/* Tests for a char device */
if (_dup2(0, 10) == 0)
{
- ok(fstat(10, &buf) == 0, "fstat(stdin) failed: errno=%d\n", errno);
+ ret = fstat(10, &buf);
+ ok(!ret, "fstat(stdin) failed: errno=%d\n", errno);
if ((buf.st_mode & _S_IFMT) == _S_IFCHR)
{
ok(buf.st_mode == _S_IFCHR, "bad st_mode=%06o\n", buf.st_mode);
@@ -1364,7 +1368,8 @@ static void test_stat(void)
/* Tests for pipes */
if (_pipe(pipes, 1024, O_BINARY) == 0)
{
- ok(fstat(pipes[0], &buf) == 0, "fstat(pipe) failed: errno=%d\n", errno);
+ ret = fstat(pipes[0], &buf);
+ ok(!ret, "fstat(pipe) failed: errno=%d\n", errno);
ok(buf.st_mode == _S_IFIFO, "bad st_mode=%06o\n", buf.st_mode);
ok(buf.st_dev == pipes[0], "st_dev is %d, expected %d\n", buf.st_dev, pipes[0]);
ok(buf.st_rdev == pipes[0], "st_rdev is %d, expected %d\n", buf.st_rdev, pipes[0]);
@@ -1394,7 +1399,8 @@ static void test_pipes_child(int argc, char** args)
}
fd=atoi(args[3]);
- ok(close(fd) == 0, "unable to close %d: %d\n", fd, errno);
+ i=close(fd);
+ ok(!i, "unable to close %d: %d\n", fd, errno);
fd=atoi(args[4]);
@@ -1406,7 +1412,8 @@ static void test_pipes_child(int argc, char** args)
Sleep(100);
}
- ok(close(fd) == 0, "unable to close %d: %d\n", fd, errno);
+ i=close(fd);
+ ok(!i, "unable to close %d: %d\n", fd, errno);
}
static void test_pipes(const char* selfname)
@@ -1434,7 +1441,8 @@ static void test_pipes(const char* selfname)
arg_v[4] = str_fdw; sprintf(str_fdw, "%d", pipes[1]);
arg_v[5] = NULL;
proc_handles[0] = (HANDLE)_spawnvp(_P_NOWAIT, selfname, arg_v);
- ok(close(pipes[1]) == 0, "unable to close %d: %d\n", pipes[1], errno);
+ i=close(pipes[1]);
+ ok(!i, "unable to close %d: %d\n", pipes[1], errno);
for (i=0; i<N_TEST_MESSAGES; i++) {
r=read(pipes[0], buf, sizeof(buf)-1);
@@ -1446,7 +1454,8 @@ static void test_pipes(const char* selfname)
r=read(pipes[0], buf, sizeof(buf)-1);
ok(r == 0, "expected to read 0 bytes, got %d\n", r);
- ok(close(pipes[0]) == 0, "unable to close %d: %d\n", pipes[0], errno);
+ i=close(pipes[0]);
+ ok(!i, "unable to close %d: %d\n", pipes[0], errno);
/* Test reading from a pipe with fread() */
if (_pipe(pipes, 1024, O_BINARY) < 0)
@@ -1462,7 +1471,8 @@ static void test_pipes(const char* selfname)
arg_v[4] = str_fdw; sprintf(str_fdw, "%d", pipes[1]);
arg_v[5] = NULL;
proc_handles[1] = (HANDLE)_spawnvp(_P_NOWAIT, selfname, arg_v);
- ok(close(pipes[1]) == 0, "unable to close %d: %d\n", pipes[1], errno);
+ i=close(pipes[1]);
+ ok(!i, "unable to close %d: %d\n", pipes[1], errno);
file=fdopen(pipes[0], "r");
/* In blocking mode, fread will keep calling read() until it gets
@@ -1485,7 +1495,8 @@ static void test_pipes(const char* selfname)
ok(ferror(file) == 0, "got ferror() = %d\n", ferror(file));
ok(feof(file), "feof() is false!\n");
- ok(fclose(file) == 0, "unable to close the pipe: %d\n", errno);
+ i=fclose(file);
+ ok(!i, "unable to close the pipe: %d\n", errno);
}
static void test_unlink(void)
More information about the wine-cvs
mailing list