Non-virtual destructors for classes that are unlikely to be base classes
authorGuilherme Maciel Ferreira <guilherme.maciel.ferreira@intra2net.com>
Fri, 15 Apr 2011 12:10:01 +0000 (14:10 +0200)
committerGuilherme Maciel Ferreira <guilherme.maciel.ferreira@intra2net.com>
Fri, 15 Apr 2011 12:10:01 +0000 (14:10 +0200)
src/config/host.h
src/dns/dnsresolver.h
src/dns/hostaddress.h
src/host/hoststatusanalyzer.h
src/host/pinginterval.h
src/host/pingscheduler.h
src/link/linkstatusanalyzer.h

index e2cee88..ef0adf0 100644 (file)
@@ -16,7 +16,7 @@ class Host
 {
 public:
     Host( std::string address );
-    virtual ~Host();
+    ~Host();
 
     std::string get_address() const;
     void set_address( const std::string &address );
index b551b6b..30764b0 100644 (file)
@@ -14,7 +14,7 @@ class DnsResolver
 {
 public:
     explicit DnsResolver( const std::string &dns_address );
-    virtual ~DnsResolver();
+    ~DnsResolver();
 
     bool resolve();
     int get_resolved_ip_count() const;
index 1185fa1..061b798 100644 (file)
@@ -14,7 +14,7 @@ class HostAddress
 public:
     HostAddress();
     HostAddress( std::string ip, int ttl );
-    virtual ~HostAddress();
+    ~HostAddress();
 
     std::string get_ip() const;
     void set_ip( std::string ip );
index 0283cb5..884a8b5 100644 (file)
@@ -24,7 +24,7 @@ public:
             const int ping_fail_percentage_limit,
             const boost::shared_ptr<LinkStatusAnalyzer> link_analyzer
     );
-    virtual ~HostStatusAnalyzer();
+    ~HostStatusAnalyzer();
 
     void set_resolved_ip_count( const int resolved_ip_count );
     bool exceeded_ping_failed_limit() const;
index 0676921..f76272c 100644 (file)
@@ -15,6 +15,7 @@ class PingInterval
 {
 public:
     PingInterval( T interval );
+    ~PingInterval();
 
     operator T();
 
@@ -41,6 +42,11 @@ template<typename T>
     }
 
 template<typename T>
+    PingInterval<T>::~PingInterval()
+    {
+    }
+
+template<typename T>
     PingInterval<T>::operator T()
     {
         return Interval;
index 7f6f90b..14a549d 100644 (file)
@@ -31,7 +31,7 @@ public:
             const int ping_fail_percentage_limit,
             boost::shared_ptr<LinkStatusAnalyzer> link_analyzer
     );
-    virtual ~PingScheduler();
+    ~PingScheduler();
 
     bool start_pinging();
 
index a80f774..3414bc7 100644 (file)
@@ -25,7 +25,7 @@ public:
             const int link_up_interval_in_min,
             const std::string &status_notifier_cmd
     );
-    virtual ~LinkStatusAnalyzer();
+    ~LinkStatusAnalyzer();
 
     void notify_host_up( const std::string &host_address );
     void notify_host_down( const std::string &host_address );