Avoid expensive copies of Hooks in HookManager::run_hooks

Use a deferred deletion mechanism to ensure hooks are kept alive
for the duration of run_hooks.

1 files changed, 3 insertions, 1 deletions

diff --git a/src/hook_manager.hh b/src/hook_manager.hh
index ae27b352..e2e238ea 100644
--- a/src/hook_manager.hh
+++ b/src/hook_manager.hh
@@ -35,8 +35,10 @@ private:
     };
 
     SafePtr<HookManager> m_parent;
-    HashMap<String, Vector<Hook, MemoryDomain::Hooks>, MemoryDomain::Hooks> m_hooks;
+    HashMap<String, Vector<std::unique_ptr<Hook>, MemoryDomain::Hooks>, MemoryDomain::Hooks> m_hooks;
+
     mutable Vector<std::pair<StringView, StringView>, MemoryDomain::Hooks> m_running_hooks;
+    mutable Vector<std::unique_ptr<Hook>, MemoryDomain::Hooks> m_hooks_trash;
 };
 
 }