final-ize some derived types

src/clientimpl.h

@@ -81,7 +81,7 @@ struct RequestInfo {
     RequestInfo(uint32_t sid, uint32_t ioid, std::shared_ptr<OperationBase>& handle);
 };
 
-struct Connection : public ConnBase, public std::enable_shared_from_this<Connection> {
+struct Connection final : public ConnBase, public std::enable_shared_from_this<Connection> {
     const std::shared_ptr<ContextImpl> context;
 
     // While HoldOff, the time until re-connection
@@ -151,7 +151,7 @@ protected:
     static void tickEchoS(evutil_socket_t fd, short evt, void *raw);
 };
 
-struct ConnectImpl : public Connect
+struct ConnectImpl final : public Connect
 {
     const evbase loop;
     std::shared_ptr<Channel> chan;
@@ -223,7 +223,7 @@ struct Channel {
                                    const std::string& server);
 };
 
-struct Discovery : public OperationBase
+struct Discovery final : public OperationBase
 {
     const std::shared_ptr<ContextImpl> context;
     std::function<void(const Discovered &)> notify;

src/clientmon.cpp

@@ -31,7 +31,7 @@ struct Entry {
 };
 }
 
-struct SubscriptionImpl : public OperationBase, public Subscription
+struct SubscriptionImpl final : public OperationBase, public Subscription
 {
     // for use in log messages, even after cancel()
     std::string channelName;

src/evhelper.cpp

@@ -107,7 +107,7 @@ struct ThreadEvent
     inline epicsEvent* operator->() { return get(); }
 };
 
-struct evbase::Pvt : public epicsThreadRunable
+struct evbase::Pvt final : public epicsThreadRunable
 {
     SockAttach attach;
 

src/serverconn.h

@@ -108,7 +108,7 @@ struct ServerChan
     void cleanup();
 };
 
-struct ServerConn : public ConnBase, public std::enable_shared_from_this<ServerConn>
+struct ServerConn final : public ConnBase, public std::enable_shared_from_this<ServerConn>
 {
     ServIface* const iface;
     const size_t tcp_tx_limit;

src/sharedpv.cpp

@@ -467,7 +467,7 @@ Value SharedPV::fetch() const
     }
 }
 
-struct StaticSource::Impl : public Source
+struct StaticSource::Impl final : public Source
 {
     mutable RWLock lock;
 

src/udp_collector.cpp

@@ -31,8 +31,8 @@ namespace pvxs {namespace impl {
 DEFINE_LOGGER(logio, "pvxs.udp.io");
 DEFINE_LOGGER(logsetup, "pvxs.udp.setup");
 
-struct UDPCollector : public UDPManager::Search,
-                      public std::enable_shared_from_this<UDPCollector>
+struct UDPCollector final : public UDPManager::Search,
+                            public std::enable_shared_from_this<UDPCollector>
 {
     UDPManager::Pvt* const manager;
     SockAddr bind_addr; // address our socket is bound to

src/util.cpp

@@ -246,7 +246,7 @@ struct SocketPair {
 
 } // namespace
 
-struct SigInt::Pvt : public epicsThreadRunable {
+struct SigInt::Pvt final : private epicsThreadRunable {
     void (*prevINT)(int);
     void (*prevTERM)(int);
     const std::function<void()> handler;