diff options
author | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2016-03-08 12:23:54 (GMT) |
---|---|---|
committer | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2016-03-08 12:23:54 (GMT) |
commit | 116eed3ebd48bd96292af786a09bb67c124bb3e9 (patch) | |
tree | d1482a87cc7277c9705e5f53db16607fbe743e2a /c++/src/H5ArrayType.cpp | |
parent | 1b7b09b08104f05a65a0269c5c88a342dba8e972 (diff) | |
download | hdf5-116eed3ebd48bd96292af786a09bb67c124bb3e9.zip hdf5-116eed3ebd48bd96292af786a09bb67c124bb3e9.tar.gz hdf5-116eed3ebd48bd96292af786a09bb67c124bb3e9.tar.bz2 |
[svn-r29340] Purpose: Code cleanup
Description:
- Removed many warnings:
warning: use of old-style cast
warning: enumeration value ‘H5D_VIRTUAL’ not handled in switch
warning: comparison between signed and unsigned
There are others of the same warnings and they will be taken care of
in the next release.
- Made some code reuse between overloads
Platforms tested:
Linux/32 2.6 (jam)
Linux/64 (platypus)
Darwin (osx1010test)
Diffstat (limited to 'c++/src/H5ArrayType.cpp')
-rw-r--r-- | c++/src/H5ArrayType.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/c++/src/H5ArrayType.cpp b/c++/src/H5ArrayType.cpp index 5792467..9731a13 100644 --- a/c++/src/H5ArrayType.cpp +++ b/c++/src/H5ArrayType.cpp @@ -54,11 +54,8 @@ ArrayType::ArrayType( const hid_t existing_id ) : DataType( existing_id ) ///\brief Copy constructor: makes a copy of the original ArrayType object. // Programmer Binh-Minh Ribler - May 2004 //-------------------------------------------------------------------------- -ArrayType::ArrayType( const ArrayType& original ) : DataType( original ) +ArrayType::ArrayType( const ArrayType& original ) : DataType( original ), rank(original.rank) { - // Copy the rank of the original array - rank = original.rank; - // Allocate space then copy the dimensions from the original array dimensions = new hsize_t[rank]; for (int i = 0; i < rank; i++) |