From 1f2e75ecea244c9a7e7084489f12d98237394095 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Wed, 12 Jun 2019 16:09:57 -0400 Subject: [PATCH] kernel/vm_manager: Remove redundant Reset call in destructor This is performing more work than would otherwise be necessary during VMManager's destruction. All we actually want to occur in this scenario is for any allocated memory to be freed, which will happen automatically as the VMManager instance goes out of scope. Anything else being done is simply unnecessary work. --- src/core/hle/kernel/vm_manager.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/core/hle/kernel/vm_manager.cpp b/src/core/hle/kernel/vm_manager.cpp index c7cc36a3c..3280f99e9 100644 --- a/src/core/hle/kernel/vm_manager.cpp +++ b/src/core/hle/kernel/vm_manager.cpp @@ -40,9 +40,7 @@ VMManager::VMManager(Memory::MemorySystem& memory) : memory(memory) { Reset(); } -VMManager::~VMManager() { - Reset(); -} +VMManager::~VMManager() = default; void VMManager::Reset() { vma_map.clear();