[svn-r3199]

Purpose: bug fix

Description:
	I found a couple of places where virtual destructors were missing
	and could cause small memory leaks.  Also, some destructors were
	not virtual when they should be.

Solution:
	- added virtual destructors, which also free dynamically allocated
	  memory
	- added virtual to several destructors
	- also, fixed several typos

Platforms tested:
	Solaris 2.7 (arabica)

1 files changed, 9 insertions, 7 deletions

diff --git a/c++/src/H5DataSet.h b/c++/src/H5DataSet.h
index 8a59234..e094159 100644
--- a/c++/src/H5DataSet.h
+++ b/c++/src/H5DataSet.h
@@ -1,3 +1,6 @@
+
+// Class DataSet inherits from AbstractDs and provides accesses to a dataset.
+
 #ifndef _H5DataSet_H
 #define _H5DataSet_H
 
@@ -6,14 +9,7 @@ namespace H5 {
 #endif
 
 class DataSet : public AbstractDs {
-
    public:
-	// Default constructor
-	DataSet();
-
-	// Copy constructor
-	DataSet( const DataSet& original );
-
 	// Gets the dataspace of this dataset.
 	virtual DataSpace getSpace() const;
 
@@ -52,6 +48,12 @@ class DataSet : public AbstractDs {
 	// Used by the API to appropriately close a dataset
 	virtual void p_close() const;
 
+	// Default constructor
+	DataSet();
+
+	// Copy constructor
+	DataSet( const DataSet& original );
+
 	virtual ~DataSet();
 
    private: