From: Florian Forster <octo@collectd.org>
Date: Fri, 26 Jul 2013 13:18:43 +0000 (+0200)
Subject: stats plugin: Don't check the value of '(value_t).derive'.
X-Git-Tag: collectd-5.4.0~13^2
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=73918b4759930c3420b8082342847e92a0cb44c5;p=collectd.git

stats plugin: Don't check the value of '(value_t).derive'.

statsd_parse_value() will write to (value_t).gauge, so don't read from another
member of the union. This also fixes problems with counter values being zero,
for whatever that is supposed to be useful for.

Github: #358
---

diff --git a/src/statsd.c b/src/statsd.c
index b697a347..5af34833 100644
--- a/src/statsd.c
+++ b/src/statsd.c
@@ -218,9 +218,6 @@ static int statsd_handle_counter (char const *name, /* {{{ */
   if (status != 0)
     return (status);
 
-  if (value.derive < 1)
-    return (-1);
-
   return (statsd_metric_add (name, (double) (value.gauge / scale.gauge),
         STATSD_COUNTER));
 } /* }}} int statsd_handle_counter */