From 613a7e2a7dc7f7e9f215390fd5138187c7d17b3c Mon Sep 17 00:00:00 2001 From: Yangster Date: Tue, 8 May 2018 15:12:30 -0700 Subject: [PATCH] Should not use the absolute value when the value does not change. Test: statsd test BUG: b/79265262 Change-Id: Ic654e1a07a505ec0e073a41fb41bcd38a29b125e --- cmds/statsd/src/metrics/ValueMetricProducer.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmds/statsd/src/metrics/ValueMetricProducer.cpp b/cmds/statsd/src/metrics/ValueMetricProducer.cpp index 69330ba2a0c7f..df8763cda244c 100644 --- a/cmds/statsd/src/metrics/ValueMetricProducer.cpp +++ b/cmds/statsd/src/metrics/ValueMetricProducer.cpp @@ -396,7 +396,7 @@ void ValueMetricProducer::onMatchedLogEventInternalLocked( // If not, there was a reset event. We take the absolute value as // diff in this case. if (interval.startUpdated) { - if (value > interval.start) { + if (value >= interval.start) { interval.sum += (value - interval.start); } else { interval.sum += value;