fix: add missing special functions for classes with destructors

Author: aminya

src/closable.h

@@ -4,7 +4,13 @@
    up ZMQ resources at agent exit. */
 namespace zmq {
 struct Closable {
+    Closable() = default;
+    Closable(const Closable&) = default;
+    Closable(Closable&&) = default;
+    Closable& operator=(const Closable&) = default;
+    Closable& operator=(Closable&&) = default;
     virtual ~Closable() = default;
+
     virtual void Close() = 0;
 };
-}
+}  // namespace zmq

src/context.h

@@ -12,10 +12,12 @@ class Context : public Napi::ObjectWrap<Context>, public Closable {
     static void Initialize(Module& module, Napi::Object& exports);
 
     explicit Context(const Napi::CallbackInfo& info);
-    ~Context() override;
 
+    Context(const Context&) = delete;
+    Context& operator=(const Context&) = delete;
     Context(Context&&) = delete;
     Context& operator=(Context&&) = delete;
+    ~Context() override;
 
     void Close() override;
 

src/incoming_msg.h

@@ -12,6 +12,8 @@ class IncomingMsg {
 
     IncomingMsg(const IncomingMsg&) = delete;
     IncomingMsg& operator=(const IncomingMsg&) = delete;
+    IncomingMsg(IncomingMsg&&) = delete;
+    IncomingMsg& operator=(IncomingMsg&&) = delete;
 
     Napi::Value IntoBuffer(const Napi::Env& env);
 
@@ -25,6 +27,10 @@ class IncomingMsg {
 
     public:
         Reference();
+        Reference(const Reference&) = delete;
+        Reference(Reference&&) = delete;
+        Reference& operator=(const Reference&) = delete;
+        Reference& operator=(Reference&&) = delete;
         ~Reference();
 
         zmq_msg_t* get() {

src/observer.h

@@ -16,6 +16,11 @@ class Observer : public Napi::ObjectWrap<Observer>, public Closable {
     static void Initialize(Module& module, Napi::Object& exports);
 
     explicit Observer(const Napi::CallbackInfo& info);
+
+    Observer(const Observer&) = delete;
+    Observer(Observer&&) = delete;
+    Observer& operator=(const Observer&) = delete;
+    Observer& operator=(Observer&&) = delete;
     ~Observer() override;
 
     void Close() override;

src/outgoing_msg.h

@@ -17,6 +17,8 @@ class OutgoingMsg {
        nor should we have to copy messages with the right STL containers. */
     OutgoingMsg(const OutgoingMsg&) = delete;
     OutgoingMsg& operator=(const OutgoingMsg&) = delete;
+    OutgoingMsg(OutgoingMsg&&) = delete;
+    OutgoingMsg& operator=(OutgoingMsg&&) = delete;
 
     /* Outgoing message. Takes a string or buffer argument and releases
        the underlying V8 resources whenever the message is sent, or earlier

src/proxy.h

@@ -15,6 +15,11 @@ class Proxy : public Napi::ObjectWrap<Proxy>, public Closable {
     static void Initialize(Module& module, Napi::Object& exports);
 
     explicit Proxy(const Napi::CallbackInfo& info);
+
+    Proxy(const Proxy&) = delete;
+    Proxy(Proxy&&) = delete;
+    Proxy& operator=(const Proxy&) = delete;
+    Proxy& operator=(Proxy&&) = delete;
     ~Proxy() override;
 
     void Close() override;

src/socket.h

@@ -15,6 +15,11 @@ class Socket : public Napi::ObjectWrap<Socket>, public Closable {
     static void Initialize(Module& module, Napi::Object& exports);
 
     explicit Socket(const Napi::CallbackInfo& info);
+
+    Socket(const Socket&) = delete;
+    Socket(Socket&&) = delete;
+    Socket& operator=(const Socket&) = delete;
+    Socket& operator=(Socket&&) = delete;
     ~Socket() override;
 
     void Close() override;

src/util/reaper.h

@@ -21,6 +21,12 @@ class Reaper {
     std::set<T*> pointers;
 
 public:
+    Reaper() = default;
+    Reaper(const Reaper&) = delete;
+    Reaper(Reaper&&) = delete;
+    Reaper& operator=(const Reaper&) = delete;
+    Reaper& operator=(Reaper&&) = delete;
+
     ~Reaper() {
         /* Copy pointers to vector to avoid issues with callbacks deregistering
            themselves from the reaper while we are still iterating. We iterate