chore: use final for overrides and remove virtual for non base classes

src/lib/arch/unix/ArchMultithreadPosix.h

@@ -63,10 +63,10 @@ public:
   void closeMutex(ArchMutex) override;
   void lockMutex(ArchMutex) override;
   void unlockMutex(ArchMutex) override;
-  ArchThread newThread(ThreadFunc, void *) override;
+  ArchThread newThread(ThreadFunc, void *) final;
   ArchThread newCurrentThread() override;
   ArchThread copyThread(ArchThread) override;
-  void closeThread(ArchThread) override;
+  void closeThread(ArchThread) final;
   void cancelThread(ArchThread) override;
   void setPriorityOfThread(ArchThread, int n) override;
   void testCancelThread() override;

src/lib/client/Client.h

@@ -130,7 +130,7 @@ public:
   //@}
 
   // IScreen overrides
-  void *getEventTarget() const override;
+  void *getEventTarget() const final;
   bool getClipboard(ClipboardID id, IClipboard *) const override;
   void getShape(int32_t &x, int32_t &y, int32_t &width, int32_t &height) const override;
   void getCursorPos(int32_t &x, int32_t &y) const override;

src/lib/deskflow/Clipboard.h

@@ -43,9 +43,9 @@ public:
   //@}
 
   // IClipboard overrides
-  bool empty() override;
+  bool empty() final;
   void add(EFormat, const std::string &data) override;
-  bool open(Time) const override;
+  bool open(Time) const final;
   void close() const override;
   Time getTime() const override;
   bool has(EFormat) const override;

src/lib/deskflow/Screen.h

@@ -46,14 +46,14 @@ public:
   For a secondary screen it also means disabling the screen saver if
   synchronizing it and preparing to synthesize events.
   */
-  virtual void enable();
+  void enable();
 
   //! Deactivate screen
   /*!
   Undoes the operations in activate() and events are no longer
   reported.  It also releases keys that are logically pressed.
   */
-  virtual void disable();
+  void disable();
 
   //! Enter screen
   /*!
@@ -168,14 +168,14 @@ public:
   /*!
   Resets all options to their default values.
   */
-  virtual void resetOptions();
+  void resetOptions();
 
   //! Notify of options changes
   /*!
   Set options to given values.  Ignores unknown options and doesn't
   modify options that aren't given in \c options.
   */
-  virtual void setOptions(const OptionsList &options);
+  void setOptions(const OptionsList &options);
 
   //! Set clipboard sequence number
   /*!

src/lib/platform/EiScreen.h

@@ -35,7 +35,7 @@ public:
   ~EiScreen() override;
 
   // IScreen overrides
-  void *getEventTarget() const override;
+  void *getEventTarget() const final;
   bool getClipboard(ClipboardID id, IClipboard *) const override;
   void getShape(std::int32_t &x, std::int32_t &y, std::int32_t &width, std::int32_t &height) const override;
   void getCursorPos(std::int32_t &x, std::int32_t &y) const override;

src/lib/platform/XWindowsKeyState.h

@@ -96,7 +96,7 @@ public:
   // IKeyState overrides
   bool fakeCtrlAltDel() override;
   KeyModifierMask pollActiveModifiers() const override;
-  int32_t pollActiveGroup() const override;
+  int32_t pollActiveGroup() const final;
   void pollPressedKeys(KeyButtonSet &pressedKeys) const override;
 
 protected:

src/lib/platform/XWindowsScreen.h

@@ -42,7 +42,7 @@ public:
   //@}
 
   // IScreen overrides
-  void *getEventTarget() const override;
+  void *getEventTarget() const final;
   bool getClipboard(ClipboardID id, IClipboard *) const override;
   void getShape(int32_t &x, int32_t &y, int32_t &width, int32_t &height) const override;
   void getCursorPos(int32_t &x, int32_t &y) const override;

src/lib/server/InputFilter.h

@@ -357,7 +357,7 @@ public:
 
   explicit InputFilter(IEventQueue *events);
   InputFilter(const InputFilter &);
-  virtual ~InputFilter();
+  ~InputFilter();
 
   InputFilter &operator=(const InputFilter &);
 
@@ -372,7 +372,7 @@ public:
 
   // enable event filtering using the given primary client.  disable
   // if client is nullptr.
-  virtual void setPrimaryClient(PrimaryClient *client);
+  void setPrimaryClient(PrimaryClient *client);
 
   // convert rules to a string
   std::string format(const std::string_view &linePrefix) const;