diff options
author | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2000-12-23 13:17:32 (GMT) |
---|---|---|
committer | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2000-12-23 13:17:32 (GMT) |
commit | d24bacb8a47f945e6919f067e8057ad30012bd5a (patch) | |
tree | 2844b08018435b9d33f68ef4b787d5434db66d5a /c++/src/H5DataSpace.h | |
parent | 45c5dd949f7dd4549dc5996ff8395652ef3d9dda (diff) | |
download | hdf5-d24bacb8a47f945e6919f067e8057ad30012bd5a.zip hdf5-d24bacb8a47f945e6919f067e8057ad30012bd5a.tar.gz hdf5-d24bacb8a47f945e6919f067e8057ad30012bd5a.tar.bz2 |
[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)
Diffstat (limited to 'c++/src/H5DataSpace.h')
-rw-r--r-- | c++/src/H5DataSpace.h | 21 |
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 |