[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, 6 insertions, 15 deletions

diff --git a/c++/src/H5DataSpace.h b/c++/src/H5DataSpace.h
index 7f0ca83..5da7299 100644
--- a/c++/src/H5DataSpace.h
+++ b/c++/src/H5DataSpace.h
@@ -10,30 +10,15 @@ class DataSpace : public IdComponent {
 	// Default DataSpace objects
 	static const DataSpace ALL;
 
-	// Default constructor
-	DataSpace();
-
 	// Creates a dataspace object given the space type
 	DataSpace( H5S_class_t type ); // H5Screate
 
-	// Assignement operator
-	//DataSpace& operator=( const DataSpace& rhs );
-
 	// Creates a simple dataspace
 	DataSpace( int rank, const hsize_t * dims, const hsize_t * maxdims = NULL); // H5Screate_simple
 
-	// Copy constructor: makes a copy of the original DataSpace object.
-	DataSpace( const DataSpace& original );
-
 	// Makes copy of an existing dataspace.
 	void copy( const DataSpace& like_space ); // H5Scopy
 
-	// Gets value of member id of this dataspace
-        //hid_t getId () const;
-
-	// Sets value for member id of this dataspace.
-        //void setId( hid_t new_space_id );
-
 	// Determines if this dataspace is a simple one.
 	bool isSimple () const;
 
@@ -103,6 +88,12 @@ class DataSpace : public IdComponent {
 	// Used by the API to close the dataspace 
 	void p_close() const;
 
+	// Default constructor
+	DataSpace();
+
+	// Copy constructor: makes a copy of the original DataSpace object.
+	DataSpace( const DataSpace& original );
+
 	virtual ~DataSpace();
 };
 #ifndef H5_NO_NAMESPACE