Call capture_and_reset_value() instead of having reset behavior inside of

value()

Author: cgilmour

src/datadog/metrics.cpp

@@ -12,7 +12,8 @@ const std::string Metric::name() { return name_; }
 const std::string Metric::type() { return type_; }
 const std::vector<std::string> Metric::tags() { return tags_; }
 bool Metric::common() { return common_; }
-uint64_t Metric::value() { return value_.exchange(0); }
+uint64_t Metric::value() { return value_; }
+uint64_t Metric::capture_and_reset_value() { return value_.exchange(0); }
 
 CounterMetric::CounterMetric(const std::string name,
                              const std::vector<std::string> tags, bool common)

src/datadog/metrics.h

@@ -34,6 +34,7 @@ class Metric {
   const std::vector<std::string> tags();
   bool common();
   uint64_t value();
+  uint64_t capture_and_reset_value();
 };
 
 class CounterMetric : public Metric {

src/datadog/tracer_telemetry.cpp

@@ -102,7 +102,7 @@ void TracerTelemetry::capture_metrics() {
                          clock_().wall.time_since_epoch())
                          .count();
   for (auto& m : metrics_snapshots_) {
-    auto value = m.first.get().value();
+    auto value = m.first.get().capture_and_reset_value();
     m.second.emplace_back(timepoint, value);
   }
 }