diff --git a/src/base_object.h b/src/base_object.h
index 090bb70aebeaf8..bc4c2aaf039547 100644
--- a/src/base_object.h
+++ b/src/base_object.h
@@ -40,6 +40,8 @@ class BaseObject : public MemoryRetainer {
   inline BaseObject(Environment* env, v8::Local<v8::Object> object);
   inline ~BaseObject() override;
 
+  BaseObject() = delete;
+
   // Returns the wrapped object.  Returns an empty handle when
   // persistent.IsEmpty() is true.
   inline v8::Local<v8::Object> object() const;
@@ -74,8 +76,6 @@ class BaseObject : public MemoryRetainer {
       Environment* env);
 
  private:
-  BaseObject();
-
   v8::Local<v8::Object> WrappedObject() const override;
   bool IsRootNode() const override;
   static void DeleteMe(void* data);
diff --git a/src/cares_wrap.cc b/src/cares_wrap.cc
index 12d712811dc28b..aeb525b33e7efa 100644
--- a/src/cares_wrap.cc
+++ b/src/cares_wrap.cc
@@ -296,18 +296,13 @@ void node_ares_task::MemoryInfo(MemoryTracker* tracker) const {
 node_ares_task* ares_task_create(ChannelWrap* channel, ares_socket_t sock) {
   auto task = new node_ares_task();
 
-  if (task == nullptr) {
-    /* Out of memory. */
-    return nullptr;
-  }
-
   task->channel = channel;
   task->sock = sock;
 
   if (uv_poll_init_socket(channel->env()->event_loop(),
                           &task->poll_watcher, sock) < 0) {
     /* This should never happen. */
-    free(task);
+    delete task;
     return nullptr;
   }
 
diff --git a/src/env.cc b/src/env.cc
index 768e14d796d5bf..697a03ae2a226f 100644
--- a/src/env.cc
+++ b/src/env.cc
@@ -8,7 +8,6 @@
 #include "node_internals.h"
 #include "node_native_module.h"
 #include "node_options-inl.h"
-#include "node_platform.h"
 #include "node_process.h"
 #include "node_v8_platform-inl.h"
 #include "node_worker.h"
diff --git a/src/heap_utils.cc b/src/heap_utils.cc
index 6690abc78d6c67..340960359d1475 100644
--- a/src/heap_utils.cc
+++ b/src/heap_utils.cc
@@ -280,7 +280,6 @@ class HeapSnapshotStream : public AsyncWrap,
 
   int DoShutdown(ShutdownWrap* req_wrap) override {
     UNREACHABLE();
-    return 0;
   }
 
   int DoWrite(WriteWrap* w,
@@ -288,7 +287,6 @@ class HeapSnapshotStream : public AsyncWrap,
               size_t count,
               uv_stream_t* send_handle) override {
     UNREACHABLE();
-    return 0;
   }
 
   bool IsAlive() override { return snapshot_ != nullptr; }
diff --git a/src/module_wrap.h b/src/module_wrap.h
index dc34685fedadbc..0fdd9b39abe296 100644
--- a/src/module_wrap.h
+++ b/src/module_wrap.h
@@ -36,7 +36,6 @@ v8::Maybe<url::URL> Resolve(Environment* env,
 
 class ModuleWrap : public BaseObject {
  public:
-  static const std::string EXTENSIONS[];
   static void Initialize(v8::Local<v8::Object> target,
                          v8::Local<v8::Value> unused,
                          v8::Local<v8::Context> context,