From: Bruno Haible <bruno@clisp.org>
Date: Fri, 8 Jul 2011 23:59:31 +0000 (+0200)
Subject: sigprocmask tests: A better way to avoid a compiler warning.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e82dcbb7d0a77ffe321b6ad839b08b2ddb2b01c2;p=pspp

sigprocmask tests: A better way to avoid a compiler warning.

* tests/test-sigprocmask.c: Don't include "ignore-value.h".
(main): Complain if system() returns non-zero.
* modules/sigprocmask-tests (Depends-on): Remove ignore-value.
---

diff --git a/ChangeLog b/ChangeLog
index cb12185916..919c98f2aa 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2011-07-08  Bruno Haible  <bruno@clisp.org>
+
+	sigprocmask tests: A better way to avoid a compiler warning.
+	* tests/test-sigprocmask.c: Don't include "ignore-value.h".
+	(main): Complain if system() returns non-zero.
+	* modules/sigprocmask-tests (Depends-on): Remove ignore-value.
+
 2011-07-08  Bruno Haible  <bruno@clisp.org>
 
 	pthread_sigmask: Work around IRIX bug.
diff --git a/modules/sigprocmask-tests b/modules/sigprocmask-tests
index f6c0410bea..80fcd81fea 100644
--- a/modules/sigprocmask-tests
+++ b/modules/sigprocmask-tests
@@ -4,7 +4,6 @@ tests/signature.h
 tests/macros.h
 
 Depends-on:
-ignore-value
 sleep
 
 configure.ac:
diff --git a/tests/test-sigprocmask.c b/tests/test-sigprocmask.c
index 973c5d88e1..8b0657ef6a 100644
--- a/tests/test-sigprocmask.c
+++ b/tests/test-sigprocmask.c
@@ -28,7 +28,6 @@ SIGNATURE_CHECK (sigprocmask, int, (int, const sigset_t *, sigset_t *));
 #include <stdlib.h>
 #include <unistd.h>
 
-#include "ignore-value.h"
 #include "macros.h"
 
 #if !((defined _WIN32 || defined __WIN32__) && !defined __CYGWIN__)
@@ -62,7 +61,7 @@ main (int argc, char *argv[])
 
   /* Request a SIGINT signal from outside.  */
   sprintf (command, "sh -c 'sleep 1; kill -%d %d' &", SIGINT, pid);
-  ignore_value (system (command));
+  ASSERT (system (command) == 0);
 
   /* Wait.  */
   sleep (2);