Clang deduced that if ENABLE_DEBUGGING is undefined, then err was a dead assignment. But in the EAI_SYSTEM case, we fail the test, so the output should be unconditional.
* tests/test-getaddrinfo.c (simple): Use err outside of dbprintf. Signed-off-by: Eric Blake <ebl...@redhat.com> --- ChangeLog | 5 +++++ tests/test-getaddrinfo.c | 2 +- 2 files changed, 6 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index 8f96680..5df0852 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2011-05-03 Eric Blake <ebl...@redhat.com> + + test-getaddrinfo: report error information + * tests/test-getaddrinfo.c (simple): Use err outside of dbprintf. + 2011-05-03 Jim Meyering <meyer...@redhat.com> bootstrap: avoid build failure when $GZIP is set diff --git a/tests/test-getaddrinfo.c b/tests/test-getaddrinfo.c index 4826989..d2984e2 100644 --- a/tests/test-getaddrinfo.c +++ b/tests/test-getaddrinfo.c @@ -107,7 +107,7 @@ simple (char const *host, char const *service) #endif /* Provide details if errno was set. */ if (res == EAI_SYSTEM) - dbgprintf ("system error: %s\n", strerror (err)); + fprintf (stderr, "system error: %s\n", strerror (err)); return 1; } -- 1.7.4.4