From 47f7797011c7c4f9d7bc5d06837c4a7891941ac0 Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Fri, 14 Dec 2012 10:29:38 +1000 Subject: process: drop a few SCOPED_TRACE SCOPED_TRACE will only print if the current scope fails and this can never happen here. Change one message to standard printf, drop the others. Signed-off-by: Peter Hutterer --- src/process.cpp | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/process.cpp b/src/process.cpp index 43ac0d6..af8b0f8 100644 --- a/src/process.cpp +++ b/src/process.cpp @@ -155,18 +155,15 @@ bool xorg::testing::Process::WaitForExit(unsigned int timeout) { struct timespec sig_timeout = {timeout / 1000, (timeout % 1000) * 1000000L}; - if (sigtimedwait(&sig_mask, NULL, &sig_timeout) != SIGCHLD && errno != EAGAIN) { - SCOPED_TRACE("INFO: Failure waiting for SIGCHLD: " + - std::string(strerror(errno)) + ". I slept instead."); + if (sigtimedwait(&sig_mask, NULL, &sig_timeout) != SIGCHLD && errno != EAGAIN) usleep(timeout * 1000); - } if (!sigismember(&sig_mask, SIGCHLD)) { if (sigprocmask(SIG_UNBLOCK, &sig_mask, NULL) == -1) - SCOPED_TRACE("WARNING: Failed to unblock SIGCHLD. Tests may behave funny.\n"); + std::cout << "WARNING: Failed to unblock SIGCHLD. Tests may behave funny.\n"; } + } else { /* oops, can't wait for SIGCHLD, sleep instead */ - SCOPED_TRACE("INFO: Failed to set SIGCHLD mask, sleeping instead.\n"); usleep(timeout * 1000); } -- cgit v1.2.3