Merge pull request #1910 from c1728p9/add_virtual_destructors

Add virtual destructor to classes

hal/api/AnalogIn.h

@@ -103,6 +103,10 @@ public:
     }
 #endif
 
+    virtual ~AnalogIn() {
+        // Do nothing
+    }
+
 protected:
 
     virtual void lock() {

hal/api/AnalogOut.h

@@ -122,6 +122,10 @@ public:
     }
 #endif
 
+    virtual ~AnalogOut() {
+        // Do nothing
+    }
+
 protected:
 
     virtual void lock() {

hal/api/I2C.h

@@ -145,6 +145,10 @@ public:
      */
     virtual void unlock(void);
 
+    virtual ~I2C() {
+        // Do nothing
+    }
+
 #if DEVICE_I2C_ASYNCH
 
     /** Start non-blocking I2C transfer.

hal/api/InterruptManager.h

@@ -113,14 +113,13 @@ public:
      */
     bool remove_handler(pFunctionPointer_t handler, IRQn_Type irq);
 
-protected:
-    virtual void lock();
-    virtual void unlock();
-
 private:
     InterruptManager();
     ~InterruptManager();
 
+    void lock();
+    void unlock();
+
     // We declare the copy contructor and the assignment operator, but we don't
     // implement them. This way, if someone tries to copy/assign our instance,
     // he will get an error at compile time.