src: mark empty destructors as default

Mark empty destructors as having a default no-op implementation.
Remove a few unused constructors and destructors while we are here.

11 files changed, 4 insertions, 28 deletions

diff --git a/src/async-wrap-inl.h b/src/async-wrap-inl.h
index 6fad7eac36..750466b3dd 100644
--- a/src/async-wrap-inl.h
+++ b/src/async-wrap-inl.h
@@ -54,9 +54,6 @@ inline AsyncWrap::AsyncWrap(Environment* env,
 }
 
 
-inline AsyncWrap::~AsyncWrap() {
-}
-
 inline uint32_t AsyncWrap::provider_type() const {
   return provider_type_;
 }
diff --git a/src/async-wrap.h b/src/async-wrap.h
index 1b1802a68f..a8577984ed 100644
--- a/src/async-wrap.h
+++ b/src/async-wrap.h
@@ -61,7 +61,7 @@ class AsyncWrap : public BaseObject {
                    v8::Handle<v8::Object> object,
                    ProviderType provider);
 
-  inline ~AsyncWrap();
+  inline ~AsyncWrap() = default;
 
   inline bool has_async_listener();
 
diff --git a/src/node.cc b/src/node.cc
index 6aaf306827..4bcd62287a 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -159,12 +159,12 @@ class ArrayBufferAllocator : public ArrayBuffer::Allocator {
   // the process.
   static const size_t kMaxLength = 0x3fffffff;
   static ArrayBufferAllocator the_singleton;
-  virtual ~ArrayBufferAllocator() {}
+  virtual ~ArrayBufferAllocator() = default;
   virtual void* Allocate(size_t length) override;
   virtual void* AllocateUninitialized(size_t length) override;
   virtual void Free(void* data, size_t length) override;
  private:
-  ArrayBufferAllocator() {}
+  ArrayBufferAllocator() = default;
   DISALLOW_COPY_AND_ASSIGN(ArrayBufferAllocator);
 };
 
diff --git a/src/node_crypto_clienthello.h b/src/node_crypto_clienthello.h
index 33048c2a60..bbd46e5a31 100644
--- a/src/node_crypto_clienthello.h
+++ b/src/node_crypto_clienthello.h
@@ -47,9 +47,6 @@ class ClientHelloParser {
 
   class ClientHello {
    public:
-    ClientHello() {
-    }
-
     inline uint8_t session_size() const { return session_size_; }
     inline const uint8_t* session_id() const { return session_id_; }
     inline bool has_ticket() const { return has_ticket_; }
diff --git a/src/node_v8_platform.cc b/src/node_v8_platform.cc
index 7439382bca..d71fdd4cc9 100644
--- a/src/node_v8_platform.cc
+++ b/src/node_v8_platform.cc
@@ -35,9 +35,6 @@ using v8::Isolate;
 // The last task to encounter before killing the worker
 class StopTask : public Task {
  public:
-  StopTask() {}
-  ~StopTask() {}
-
   void Run() {}
 };
 
diff --git a/src/process_wrap.cc b/src/process_wrap.cc
index 61cb6cff28..ab7d9bfddb 100644
--- a/src/process_wrap.cc
+++ b/src/process_wrap.cc
@@ -84,9 +84,6 @@ class ProcessWrap : public HandleWrap {
                    AsyncWrap::PROVIDER_PROCESSWRAP) {
   }
 
-  ~ProcessWrap() {
-  }
-
   static void ParseStdioOptions(Environment* env,
                                 Local<Object> js_options,
                                 uv_process_options_t* options) {
diff --git a/src/signal_wrap.cc b/src/signal_wrap.cc
index 1d32f7af0f..fbde1ca526 100644
--- a/src/signal_wrap.cc
+++ b/src/signal_wrap.cc
@@ -80,9 +80,6 @@ class SignalWrap : public HandleWrap {
     CHECK_EQ(r, 0);
   }
 
-  ~SignalWrap() {
-  }
-
   static void Start(const FunctionCallbackInfo<Value>& args) {
     SignalWrap* wrap = Unwrap<SignalWrap>(args.Holder());
     int signum = args[0]->Int32Value();
diff --git a/src/stream_wrap.h b/src/stream_wrap.h
index 3c058d26f8..dbb969a06c 100644
--- a/src/stream_wrap.h
+++ b/src/stream_wrap.h
@@ -72,8 +72,7 @@ class StreamWrapCallbacks {
   explicit StreamWrapCallbacks(StreamWrapCallbacks* old) : wrap_(old->wrap()) {
   }
 
-  virtual ~StreamWrapCallbacks() {
-  }
+  virtual ~StreamWrapCallbacks() = default;
 
   virtual const char* Error();
 
diff --git a/src/timer_wrap.cc b/src/timer_wrap.cc
index dfa551d05c..8ccdca5961 100644
--- a/src/timer_wrap.cc
+++ b/src/timer_wrap.cc
@@ -91,9 +91,6 @@ class TimerWrap : public HandleWrap {
     CHECK_EQ(r, 0);
   }
 
-  ~TimerWrap() {
-  }
-
   static void Start(const FunctionCallbackInfo<Value>& args) {
     TimerWrap* wrap = Unwrap<TimerWrap>(args.Holder());
 
diff --git a/src/udp_wrap.cc b/src/udp_wrap.cc
index 1698ecf645..fa047eccc9 100644
--- a/src/udp_wrap.cc
+++ b/src/udp_wrap.cc
@@ -82,10 +82,6 @@ UDPWrap::UDPWrap(Environment* env, Handle<Object> object)
 }
 
 
-UDPWrap::~UDPWrap() {
-}
-
-
 void UDPWrap::Initialize(Handle<Object> target,
                          Handle<Value> unused,
                          Handle<Context> context) {
diff --git a/src/udp_wrap.h b/src/udp_wrap.h
index 5d9cbef746..fea298031a 100644
--- a/src/udp_wrap.h
+++ b/src/udp_wrap.h
@@ -58,7 +58,6 @@ class UDPWrap: public HandleWrap {
 
  private:
   UDPWrap(Environment* env, v8::Handle<v8::Object> object);
-  virtual ~UDPWrap() override;
 
   static void DoBind(const v8::FunctionCallbackInfo<v8::Value>& args,
                      int family);