[trafficserver] branch master updated: Ubuntu build problem fix. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66425 For the record, I think (void) not suppressing "warning unused" is a poor choice.

[am...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

amc pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  222ea02   Ubuntu build problem fix. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66425 For the record, I think (void) not suppressing "warning unused" is a poor choice.
222ea02 is described below

commit 222ea02ed938733abec4079c2a3d050c9df6d938
Author: Alan M. Carroll <am...@apache.org>
AuthorDate: Sat Feb 25 18:15:47 2017 -0600

    Ubuntu build problem fix.
    See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66425
    For the record, I think (void) not suppressing "warning unused" is a poor choice.
---
 lib/ts/signals.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/ts/signals.cc b/lib/ts/signals.cc
index 1beddc7..3980d27 100644
--- a/lib/ts/signals.cc
+++ b/lib/ts/signals.cc
@@ -170,7 +170,8 @@ signal_format_siginfo(int signo, siginfo_t *info, const char *msg)
   snprintf(buf, sizeof(buf), "%s: received signal %d\n", msg, signo);
 #endif
 
-  (void)write(STDERR_FILENO, buf, strlen(buf));
+  ssize_t ignored = write(STDERR_FILENO, buf, strlen(buf));
+  (void)ignored; // because gcc and glibc are stupid, "(void)write(...)" doesn't suffice.
 }
 
 void

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].