summaryrefslogtreecommitdiff
path: root/Kernel/Memory
diff options
context:
space:
mode:
authorLenny Maiorani <lenny@serenityos.org>2022-03-16 13:15:15 -0600
committerBrian Gianforcaro <b.gianfo@gmail.com>2022-03-17 00:51:36 -0700
commit190cf1507b88d25d04148d983d72a378035b05e9 (patch)
treea0aa522b2a47f022b3b791b14e3b6902f3df29d6 /Kernel/Memory
parent68f75ab98e4d8a7e48de48c2497a8395b79a4ba5 (diff)
downloadserenity-190cf1507b88d25d04148d983d72a378035b05e9.zip
Kernel: Use default constructors/destructors
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#cother-other-default-operation-rules "The compiler is more likely to get the default semantics right and you cannot implement these functions better than the compiler."
Diffstat (limited to 'Kernel/Memory')
-rw-r--r--Kernel/Memory/AddressSpace.cpp4
-rw-r--r--Kernel/Memory/AnonymousVMObject.cpp8
-rw-r--r--Kernel/Memory/InodeVMObject.cpp4
-rw-r--r--Kernel/Memory/MemoryManager.cpp4
-rw-r--r--Kernel/Memory/PageDirectory.cpp4
-rw-r--r--Kernel/Memory/PhysicalRegion.cpp4
-rw-r--r--Kernel/Memory/PrivateInodeVMObject.cpp4
7 files changed, 8 insertions, 24 deletions
diff --git a/Kernel/Memory/AddressSpace.cpp b/Kernel/Memory/AddressSpace.cpp
index f976bb6a8b..d5fdd97e05 100644
--- a/Kernel/Memory/AddressSpace.cpp
+++ b/Kernel/Memory/AddressSpace.cpp
@@ -29,9 +29,7 @@ AddressSpace::AddressSpace(NonnullRefPtr<PageDirectory> page_directory)
{
}
-AddressSpace::~AddressSpace()
-{
-}
+AddressSpace::~AddressSpace() = default;
ErrorOr<void> AddressSpace::unmap_mmap_range(VirtualAddress addr, size_t size)
{
diff --git a/Kernel/Memory/AnonymousVMObject.cpp b/Kernel/Memory/AnonymousVMObject.cpp
index 1625c677e8..43b0c7cd64 100644
--- a/Kernel/Memory/AnonymousVMObject.cpp
+++ b/Kernel/Memory/AnonymousVMObject.cpp
@@ -166,9 +166,7 @@ AnonymousVMObject::AnonymousVMObject(AnonymousVMObject const& other, NonnullRefP
{
}
-AnonymousVMObject::~AnonymousVMObject()
-{
-}
+AnonymousVMObject::~AnonymousVMObject() = default;
size_t AnonymousVMObject::purge()
{
@@ -370,9 +368,7 @@ AnonymousVMObject::SharedCommittedCowPages::SharedCommittedCowPages(CommittedPhy
{
}
-AnonymousVMObject::SharedCommittedCowPages::~SharedCommittedCowPages()
-{
-}
+AnonymousVMObject::SharedCommittedCowPages::~SharedCommittedCowPages() = default;
NonnullRefPtr<PhysicalPage> AnonymousVMObject::SharedCommittedCowPages::take_one()
{
diff --git a/Kernel/Memory/InodeVMObject.cpp b/Kernel/Memory/InodeVMObject.cpp
index a072968d85..77cacf7e53 100644
--- a/Kernel/Memory/InodeVMObject.cpp
+++ b/Kernel/Memory/InodeVMObject.cpp
@@ -25,9 +25,7 @@ InodeVMObject::InodeVMObject(InodeVMObject const& other, FixedArray<RefPtr<Physi
m_dirty_pages.set(i, other.m_dirty_pages.get(i));
}
-InodeVMObject::~InodeVMObject()
-{
-}
+InodeVMObject::~InodeVMObject() = default;
size_t InodeVMObject::amount_clean() const
{
diff --git a/Kernel/Memory/MemoryManager.cpp b/Kernel/Memory/MemoryManager.cpp
index 7bbed63135..3d5a27b027 100644
--- a/Kernel/Memory/MemoryManager.cpp
+++ b/Kernel/Memory/MemoryManager.cpp
@@ -94,9 +94,7 @@ UNMAP_AFTER_INIT MemoryManager::MemoryManager()
m_lazy_committed_page = committed_pages.take_one();
}
-UNMAP_AFTER_INIT MemoryManager::~MemoryManager()
-{
-}
+UNMAP_AFTER_INIT MemoryManager::~MemoryManager() = default;
UNMAP_AFTER_INIT void MemoryManager::protect_kernel_image()
{
diff --git a/Kernel/Memory/PageDirectory.cpp b/Kernel/Memory/PageDirectory.cpp
index aa59b8c568..faa21f8379 100644
--- a/Kernel/Memory/PageDirectory.cpp
+++ b/Kernel/Memory/PageDirectory.cpp
@@ -123,9 +123,7 @@ ErrorOr<NonnullRefPtr<PageDirectory>> PageDirectory::try_create_for_userspace(Vi
return directory;
}
-PageDirectory::PageDirectory()
-{
-}
+PageDirectory::PageDirectory() = default;
UNMAP_AFTER_INIT void PageDirectory::allocate_kernel_directory()
{
diff --git a/Kernel/Memory/PhysicalRegion.cpp b/Kernel/Memory/PhysicalRegion.cpp
index 9c4edfbed8..7f7409ee98 100644
--- a/Kernel/Memory/PhysicalRegion.cpp
+++ b/Kernel/Memory/PhysicalRegion.cpp
@@ -27,9 +27,7 @@ static constexpr u32 next_power_of_two(u32 value)
return value;
}
-PhysicalRegion::~PhysicalRegion()
-{
-}
+PhysicalRegion::~PhysicalRegion() = default;
PhysicalRegion::PhysicalRegion(PhysicalAddress lower, PhysicalAddress upper)
: m_lower(lower)
diff --git a/Kernel/Memory/PrivateInodeVMObject.cpp b/Kernel/Memory/PrivateInodeVMObject.cpp
index 623e02776f..249a9eb8ea 100644
--- a/Kernel/Memory/PrivateInodeVMObject.cpp
+++ b/Kernel/Memory/PrivateInodeVMObject.cpp
@@ -33,8 +33,6 @@ PrivateInodeVMObject::PrivateInodeVMObject(PrivateInodeVMObject const& other, Fi
{
}
-PrivateInodeVMObject::~PrivateInodeVMObject()
-{
-}
+PrivateInodeVMObject::~PrivateInodeVMObject() = default;
}