diff options
author | Lenny Maiorani <lenny@serenityos.org> | 2022-03-16 13:15:15 -0600 |
---|---|---|
committer | Brian Gianforcaro <b.gianfo@gmail.com> | 2022-03-17 00:51:36 -0700 |
commit | 190cf1507b88d25d04148d983d72a378035b05e9 (patch) | |
tree | a0aa522b2a47f022b3b791b14e3b6902f3df29d6 /Kernel/Memory | |
parent | 68f75ab98e4d8a7e48de48c2497a8395b79a4ba5 (diff) | |
download | serenity-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.cpp | 4 | ||||
-rw-r--r-- | Kernel/Memory/AnonymousVMObject.cpp | 8 | ||||
-rw-r--r-- | Kernel/Memory/InodeVMObject.cpp | 4 | ||||
-rw-r--r-- | Kernel/Memory/MemoryManager.cpp | 4 | ||||
-rw-r--r-- | Kernel/Memory/PageDirectory.cpp | 4 | ||||
-rw-r--r-- | Kernel/Memory/PhysicalRegion.cpp | 4 | ||||
-rw-r--r-- | Kernel/Memory/PrivateInodeVMObject.cpp | 4 |
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; } |