From d953fa3038c702d6fcec062864b7a5965d6470d9 Mon Sep 17 00:00:00 2001 From: Yash Ladha Date: Mon, 17 Aug 2020 15:09:53 +0530 Subject: [PATCH] src: usage of modernize-use-equals-default Update the destructor and constructor calls to use the default member function. This will bascially enable the compiler to do better optimization as the functions as explicitly defined as trivial. Refs: https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-equals-default.html PR-URL: https://github.com/nodejs/node/pull/34807 Reviewed-By: Anna Henningsen Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Rich Trott --- src/node_file.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/node_file.cc b/src/node_file.cc index 3583456bbcc506..3ec72120d66201 100644 --- a/src/node_file.cc +++ b/src/node_file.cc @@ -126,9 +126,9 @@ void FSContinuationData::MemoryInfo(MemoryTracker* tracker) const { tracker->TrackField("paths", paths_); } -FileHandleReadWrap::~FileHandleReadWrap() {} +FileHandleReadWrap::~FileHandleReadWrap() = default; -FSReqBase::~FSReqBase() {} +FSReqBase::~FSReqBase() = default; void FSReqBase::MemoryInfo(MemoryTracker* tracker) const { tracker->TrackField("continuation_data", continuation_data_);