Remove virtual destructor from OptionManagerWatcher

We should never destruct anything through an OptionManagerWatcher
pointer, so having all those destructor virtual makes no sense.

1 files changed, 1 insertions, 3 deletions

diff --git a/src/option_manager.hh b/src/option_manager.hh
index f68f6662..2970d4e2 100644
--- a/src/option_manager.hh
+++ b/src/option_manager.hh
@@ -70,8 +70,6 @@ protected:
 class OptionManagerWatcher
 {
 public:
-    virtual ~OptionManagerWatcher() = default;
-
     virtual void on_option_changed(const Option& option) = 0;
 };
 
@@ -79,7 +77,7 @@ class OptionManager : private OptionManagerWatcher
 {
 public:
     OptionManager(OptionManager& parent);
-    ~OptionManager() override;
+    ~OptionManager();
 
     Option& operator[] (StringView name);
     const Option& operator[] (StringView name) const;