src: modernize use-equals-default

PR-URL: nodejs#48735
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>

Author: CGQAQ

src/api/environment.cc

@@ -408,8 +408,6 @@ void FreeIsolateData(IsolateData* isolate_data) {
   delete isolate_data;
 }
 
-InspectorParentHandle::~InspectorParentHandle() {}
-
 // Hide the internal handle class from the public API.
 #if HAVE_INSPECTOR
 struct InspectorParentHandleImpl : public InspectorParentHandle {

src/blob_serializer_deserializer.h

@@ -39,7 +39,7 @@ class BlobDeserializer : public BlobSerializerDeserializer {
  public:
   explicit BlobDeserializer(bool is_debug_v, std::string_view s)
       : BlobSerializerDeserializer(is_debug_v), sink(s) {}
-  ~BlobDeserializer() {}
+  ~BlobDeserializer() = default;
 
   size_t read_total = 0;
   std::string_view sink;
@@ -85,7 +85,7 @@ class BlobSerializer : public BlobSerializerDeserializer {
  public:
   explicit BlobSerializer(bool is_debug_v)
       : BlobSerializerDeserializer(is_debug_v) {}
-  ~BlobSerializer() {}
+  ~BlobSerializer() = default;
 
   Impl* impl() { return static_cast<Impl*>(this); }
   const Impl* impl() const { return static_cast<const Impl*>(this); }

src/cleanup_queue.h

@@ -18,7 +18,7 @@ class CleanupQueue : public MemoryRetainer {
  public:
   typedef void (*Callback)(void*);
 
-  CleanupQueue() {}
+  CleanupQueue() = default;
 
   // Not copyable.
   CleanupQueue(const CleanupQueue&) = delete;

src/crypto/crypto_ec.cc

@@ -130,8 +130,6 @@ void ECDH::MemoryInfo(MemoryTracker* tracker) const {
   tracker->TrackFieldWithSize("key", key_ ? kSizeOf_EC_KEY : 0);
 }
 
-ECDH::~ECDH() {}
-
 void ECDH::New(const FunctionCallbackInfo<Value>& args) {
   Environment* env = Environment::GetCurrent(args);
 

src/crypto/crypto_ec.h

@@ -20,7 +20,7 @@ int GetOKPCurveFromName(const char* name);
 
 class ECDH final : public BaseObject {
  public:
-  ~ECDH() override;
+  ~ECDH() override = default;
 
   static void Initialize(Environment* env, v8::Local<v8::Object> target);
   static void RegisterExternalReferences(ExternalReferenceRegistry* registry);

src/inspector_agent.cc

@@ -687,7 +687,7 @@ Agent::Agent(Environment* env)
       debug_options_(env->options()->debug_options()),
       host_port_(env->inspector_host_port()) {}
 
-Agent::~Agent() {}
+Agent::~Agent() = default;
 
 bool Agent::Start(const std::string& path,
                   const DebugOptions& options,

src/js_native_api_v8.h

@@ -10,8 +10,8 @@ namespace v8impl {
 
 class RefTracker {
  public:
-  RefTracker() {}
-  virtual ~RefTracker() {}
+  RefTracker() = default;
+  virtual ~RefTracker() = default;
   virtual void Finalize() {}
 
   typedef RefTracker RefList;

src/json_parser.h

@@ -16,7 +16,7 @@ namespace node {
 class JSONParser {
  public:
   JSONParser();
-  ~JSONParser() {}
+  ~JSONParser() = default;
   bool Parse(const std::string& content);
   std::optional<std::string> GetTopLevelStringField(std::string_view field);
   std::optional<bool> GetTopLevelBoolField(std::string_view field);

src/node.cc

@@ -1152,9 +1152,6 @@ void TearDownOncePerProcess() {
   }
 }
 
-InitializationResult::~InitializationResult() {}
-InitializationResultImpl::~InitializationResultImpl() {}
-
 ExitCode GenerateAndWriteSnapshotData(const SnapshotData** snapshot_data_ptr,
                                       const InitializationResultImpl* result) {
   ExitCode exit_code = result->exit_code_enum();

src/node.h

@@ -289,7 +289,7 @@ enum Flags : uint32_t {
 
 class NODE_EXTERN InitializationResult {
  public:
-  virtual ~InitializationResult();
+  virtual ~InitializationResult() = default;
 
   // Returns a suggested process exit code.
   virtual int exit_code() const = 0;
@@ -658,7 +658,7 @@ enum Flags : uint64_t {
 }  // namespace EnvironmentFlags
 
 struct InspectorParentHandle {
-  virtual ~InspectorParentHandle();
+  virtual ~InspectorParentHandle() = default;
 };
 
 // TODO(addaleax): Maybe move per-Environment options parsing here.