Deinline as compiler makes the best choices.

2 files changed, 7 insertions, 6 deletions

diff --git a/storageframework/src/vespa/storageframework/defaultimplementation/memory/nomemorymanager.cpp b/storageframework/src/vespa/storageframework/defaultimplementation/memory/nomemorymanager.cpp
index 36f85d25ff1..3480c5db4bf 100644
--- a/storageframework/src/vespa/storageframework/defaultimplementation/memory/nomemorymanager.cpp
+++ b/storageframework/src/vespa/storageframework/defaultimplementation/memory/nomemorymanager.cpp
@@ -3,9 +3,9 @@
 #include "nomemorymanager.h"
 #include <vespa/vespalib/util/exceptions.h>
 
-namespace storage {
-namespace framework {
-namespace defaultimplementation {
+namespace storage::framework::defaultimplementation {
+
+NoMemoryManager::~NoMemoryManager() {}
 
 const MemoryAllocationType&
 NoMemoryManager::registerAllocationType(const MemoryAllocationType& type)
@@ -42,6 +42,5 @@ NoMemoryManager::getAllocationTypes() const
     return types;
 }
 
-} // defaultimplementation
-} // framework
-} // storage
+}
+
diff --git a/storageframework/src/vespa/storageframework/defaultimplementation/memory/nomemorymanager.h b/storageframework/src/vespa/storageframework/defaultimplementation/memory/nomemorymanager.h
index 43669eac603..452dd3a2444 100644
--- a/storageframework/src/vespa/storageframework/defaultimplementation/memory/nomemorymanager.h
+++ b/storageframework/src/vespa/storageframework/defaultimplementation/memory/nomemorymanager.h
@@ -41,6 +41,8 @@ class NoMemoryManager : public MemoryManagerInterface
 public:
     typedef std::unique_ptr<NoMemoryManager> UP;
 
+    ~NoMemoryManager();
+
     void setMaximumMemoryUsage(uint64_t) override {}
     const MemoryAllocationType & registerAllocationType(const MemoryAllocationType& type) override;
     const MemoryAllocationType & getAllocationType(const std::string& name) const override;