Shutting down also when moving

Author: pablomartinezbernardo

src/datadog/telemetry/telemetry_impl.cpp

@@ -230,7 +230,6 @@ void Telemetry::schedule_tasks() {
 Telemetry::~Telemetry() {
   shutting_down_.store(true, std::memory_order_release);
   if (!tasks_.empty()) {
-    logger_->log_error("Tasks wasnt empty");
     cancel_tasks(tasks_);
     capture_metrics();
     // The app-closing message is bundled with a message containing the
@@ -260,6 +259,7 @@ Telemetry::Telemetry(Telemetry&& rhs)
       host_info_(std::move(rhs.host_info_)) {
   logger_->log_error("Calling move constructor");
   cancel_tasks(rhs.tasks_);
+  rhs.shutting_down_.store(true, std::memory_order_release);
   schedule_tasks();
   logger_->log_error("Move constructor finished");
 }