Fixed latency calculation.

 - replaced more __attribute__((warn_unused_result)) with WARN_UNUSED_RESULT
 - do not use remote last_p(i|o)ng_recv for latency calculation is that make no sense at all
 - static' some functions

Signed-off-by: Toni Uhlig <matzeton@googlemail.com>

1 files changed, 4 insertions, 3 deletions

diff --git a/client.c b/client.c
index 7902165..bf7ef23 100644
--- a/client.c
+++ b/client.c
@@ -110,9 +110,6 @@ enum recv_return protocol_request_ping(struct connection * const state,
     strftime_local(ts, ts_str, sizeof(ts_str));
     LOG(NOTICE, "Received PING with timestamp %.09lfs: %s / %lluns",
         ts, ts_str, extract_nsecs(ts));
-    if (state->latency > 0.0f) {
-        LOG(NOTICE, "PING-PONG latency: %.09lfs", state->latency);
-    }
 
     if (ev_protocol_pong(state) != 0) {
         return RECV_FATAL_CALLBACK_ERROR;
@@ -134,6 +131,10 @@ enum recv_return protocol_request_pong(struct connection * const state,
     LOG(NOTICE, "Received PONG with timestamp %.09lfs: %s / %lluns / %zu outstanding PONG's",
         ts, ts_str, extract_nsecs(ts), state->awaiting_pong);
 
+    if (state->latency > 0.0) {
+        LOG(NOTICE, "PING-PONG latency: %.09lfs", state->latency);
+    }
+
     if (state->awaiting_pong > 3) {
         LOG(ERROR, "Waiting for more than 3 PONG's, disconnecting..");
         return RECV_FATAL_CALLBACK_ERROR;