NoopStats non copyable (#105)

* NoopStats non copyable

* movable constructors etc

Author: yschimke

src/Stats.cpp

@@ -19,10 +19,24 @@ class NoopStats : public Stats {
   void bytesRead(size_t bytes) override{};
   void frameWritten(const std::string& frameType) override{};
   void frameRead(const std::string& frameType) override{};
+
+  static NoopStats& instance(void) {
+    static NoopStats singleton;
+    return singleton;
+  }
+
+ protected:
+  NoopStats() = default;
+  ~NoopStats() = default;
+
+ private:
+  NoopStats(const NoopStats& other) = delete; // non construction-copyable
+  NoopStats& operator=(const NoopStats&) = delete; // non copyable
+  NoopStats& operator=(const NoopStats&&) = delete; // non movable
+  NoopStats(const NoopStats&&) = delete; // non construction-movable
 };
 
 Stats& Stats::noop() {
-  static NoopStats noop_;
-  return noop_;
+  return NoopStats::instance();
 };
 }

test/InlineConnection.cpp

@@ -20,8 +20,7 @@ InlineConnection::InlineConnection()
 
 InlineConnection::~InlineConnection() {}
 
-void InlineConnection::connectTo(
-    InlineConnection& other) {
+void InlineConnection::connectTo(InlineConnection& other) {
   ASSERT_FALSE(other_);
   ASSERT_FALSE(other.other_);
   other.other_ = this;