Use local buffer

Author: AadityaRavindran

include/cpp-statsd-client/StatsdClient.hpp

@@ -6,7 +6,6 @@
 #include <cstdio>
 #include <iomanip>
 #include <memory>
-#include <mutex>
 #include <random>
 #include <sstream>
 #include <string>
@@ -163,12 +162,6 @@ class StatsdClient {
     //! The random number generator for handling sampling
     mutable std::mt19937 m_randomEngine;
 
-    //! The buffer string format our stats before sending them
-    mutable std::string m_buffer;
-
-    //! The mutex to lock m_buffer
-    mutable std::mutex m_buffer_mutex;
-
     //! Fixed floating point precision of gauges
     int m_gaugePrecision;
 };
@@ -200,8 +193,6 @@ inline StatsdClient::StatsdClient(const std::string& host,
       m_gaugePrecision(gaugePrecision) {
     // Initialize the random generator to be used for sampling
     seed();
-    // Avoid re-allocations by reserving a generous buffer
-    m_buffer.reserve(256);
 }
 
 inline void StatsdClient::setConfig(const std::string& host,
@@ -295,36 +286,36 @@ inline void StatsdClient::send(const std::string& key,
     std::stringstream valueStream;
     valueStream << std::fixed << std::setprecision(m_gaugePrecision) << value;
 
-    std::lock_guard<std::mutex> buffer_lock(m_buffer_mutex);
-    m_buffer.clear();
+    std::string buffer;
+    buffer.reserve(256);
 
-    m_buffer.append(m_prefix);
+    buffer.append(m_prefix);
     if (!m_prefix.empty() && !key.empty()) {
-        m_buffer.push_back('.');
+        buffer.push_back('.');
     }
 
-    m_buffer.append(key);
-    m_buffer.push_back(':');
-    m_buffer.append(valueStream.str());
-    m_buffer.push_back('|');
-    m_buffer.append(type);
+    buffer.append(key);
+    buffer.push_back(':');
+    buffer.append(valueStream.str());
+    buffer.push_back('|');
+    buffer.append(type);
 
     if (frequency < 1.f) {
-        m_buffer.append("|@0.");
-        m_buffer.append(std::to_string(static_cast<int>(frequency * 100)));
+        buffer.append("|@0.");
+        buffer.append(std::to_string(static_cast<int>(frequency * 100)));
     }
 
     if (!tags.empty()) {
-        m_buffer.append("|#");
+        buffer.append("|#");
         for (const auto& tag : tags) {
-            m_buffer.append(tag);
-            m_buffer.push_back(',');
+            buffer.append(tag);
+            buffer.push_back(',');
         }
-        m_buffer.pop_back();
+        buffer.pop_back();
     }
 
     // Send the message via the UDP sender
-    m_sender->send(m_buffer);
+    m_sender->send(buffer);
 }
 
 inline void StatsdClient::seed(unsigned int seed) noexcept {