From 7912343ae55f0ada4a0b77c1dbeb4602f13b1988 Mon Sep 17 00:00:00 2001
From: Wolfgang Merkt <w.merkt@gmail.com>
Date: Thu, 13 Oct 2022 17:13:38 +0100
Subject: [PATCH] [socketcan_interface] Address inconsistent missing overrides

---
 socketcan_interface/include/socketcan_interface/threading.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/socketcan_interface/include/socketcan_interface/threading.h b/socketcan_interface/include/socketcan_interface/threading.h
index 58a796ce9..62eb45c2f 100644
--- a/socketcan_interface/include/socketcan_interface/threading.h
+++ b/socketcan_interface/include/socketcan_interface/threading.h
@@ -49,7 +49,7 @@ template<typename WrappedInterface> class ThreadedInterface : public WrappedInte
         }
     }
 public:
-    [[deprecated("provide settings explicitly")]] virtual bool init(const std::string &device, bool loopback) override {
+    [[deprecated("provide settings explicitly")]] bool init(const std::string &device, bool loopback) override {
         #pragma GCC diagnostic push
         #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
         if(!thread_ && WrappedInterface::init(device, loopback)){
@@ -60,7 +60,7 @@ template<typename WrappedInterface> class ThreadedInterface : public WrappedInte
         return WrappedInterface::getState().isReady();
         #pragma GCC diagnostic pop
     }
-    virtual bool init(const std::string &device, bool loopback, SettingsConstSharedPtr settings) override {
+    bool init(const std::string &device, bool loopback, SettingsConstSharedPtr settings) override {
         if(!thread_ && WrappedInterface::init(device, loopback, settings)){
             StateWaiter waiter(this);
             thread_.reset(new boost::thread(&ThreadedInterface::run_thread, this));
@@ -69,7 +69,7 @@ template<typename WrappedInterface> class ThreadedInterface : public WrappedInte
         return WrappedInterface::getState().isReady();
     }
 
-    virtual void shutdown(){
+    void shutdown() override{
         WrappedInterface::shutdown();
         shutdown_internal();
     }