Add virtual destructors to inheritance roots

src/NH/Bass/Command.h

@@ -22,8 +22,8 @@ namespace NH::Bass
 
     struct Command
     {
-
         virtual CommandResult Execute(Engine& engine) = 0;
+        virtual ~Command() = default;
     };
 
     class FunctionCommand : public Command

src/NH/Bass/EventManager.h

@@ -32,6 +32,7 @@ namespace NH::Bass
 
         Event() = delete;
         Event(EventType type, DataType data) : Type(type), Data(data) {}
+        virtual ~Event() = default;
     };
 
     using EventSubscriberFn = void (*)(const Event&, void*);

src/NH/Bass/IChannel.h

@@ -32,6 +32,8 @@ namespace NH::Bass
         template<typename T>
         using Result = std::expected<T, Error>;
 
+        virtual ~IChannel() = default;
+
         virtual Result<void> PlayInstant(const AudioFile& audioFile) = 0;
         virtual void StopInstant() = 0;
 

src/NH/Bass/IEngine.h

@@ -10,6 +10,8 @@ namespace NH::Bass
 
     struct IEngine
     {
+        virtual ~IEngine() = default;
+
         /**
          * Get a free channel and mark it as busy
          * @return shared_ptr to a free channel (or null if all channels are busy)

src/NH/Executor.h

@@ -17,6 +17,7 @@ namespace NH
     {
         virtual void AddTask(TaskFN&& task) = 0;
         virtual void AddTask(const TaskFN& task) = 0;
+        virtual ~Executor() = default;
     };
 
     class InstantExecutor : public Executor

src/NH/Logger.h

@@ -42,6 +42,7 @@ namespace NH
         LoggerLevel m_Level;
 
         explicit ILoggerAdapter(LoggerLevel level = LoggerLevel::Debug) : m_Level(level) {}
+        virtual ~ILoggerAdapter() = default;
 
         bool CanLog(LoggerLevel level) const { return level <= m_Level; }