Ensure virtual class has virtual destructor.

4 files changed, 11 insertions, 0 deletions

diff --git a/ChangeLog b/ChangeLog
index b9830fb..8c38167 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2007-01-11  Steve M. Robbins  <smr@sumost.ca>
 
+	* include/cppunit/plugin/TestPlugIn.h (struct CppUnitTestPlugIn): 
+	* include/cppunit/extensions/TestFixtureFactory.h (class TestFixtureFactory): 
+	* include/cppunit/XmlOutputterHook.h (XmlOutputterHook): Add
+	virtual destructor to virtual class.
+
 	* examples/cppunittest/TestAssertTest.cpp: Put a C++ statement in
 	the first argument of CPPUNIT_ASSERT_THROW() and
 	CPPUNIT_ASSERT_NO_THROW().
diff --git a/include/cppunit/XmlOutputterHook.h b/include/cppunit/XmlOutputterHook.h
index 4a2de7e..5ded3b1 100644
--- a/include/cppunit/XmlOutputterHook.h
+++ b/include/cppunit/XmlOutputterHook.h
@@ -153,6 +153,8 @@ public:
    */
   virtual void statisticsAdded( XmlDocument *document,
                                 XmlElement *statisticsElement );
+
+  virtual ~XmlOutputterHook() {}
 };
 
 
diff --git a/include/cppunit/extensions/TestFixtureFactory.h b/include/cppunit/extensions/TestFixtureFactory.h
index 1e3e803..45354c6 100644
--- a/include/cppunit/extensions/TestFixtureFactory.h
+++ b/include/cppunit/extensions/TestFixtureFactory.h
@@ -18,6 +18,8 @@ class TestFixtureFactory
 public:
   //! Creates a new TestFixture instance.
   virtual TestFixture *makeFixture() =0;
+
+  virtual ~TestFixtureFactory() {}
 };
 
 
diff --git a/include/cppunit/plugin/TestPlugIn.h b/include/cppunit/plugin/TestPlugIn.h
index 5692fa3..1c9b929 100644
--- a/include/cppunit/plugin/TestPlugIn.h
+++ b/include/cppunit/plugin/TestPlugIn.h
@@ -92,6 +92,8 @@ struct CppUnitTestPlugIn
    * unregistered.
    */
   virtual void uninitialize( CPPUNIT_NS::TestFactoryRegistry *registry ) =0;
+
+  virtual ~CppUnitTestPlugIn() {}
 };