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/H5DataType.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/H5DataType.cpp')
-rw-r--r-- | c++/src/H5DataType.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/c++/src/H5DataType.cpp b/c++/src/H5DataType.cpp index 1502033..3e5ad0f 100644 --- a/c++/src/H5DataType.cpp +++ b/c++/src/H5DataType.cpp @@ -362,10 +362,10 @@ void DataType::commit(H5Location& loc, const H5std_string& name) bool DataType::committed() const { // Call C function to determine if a datatype is a named one - htri_t committed = H5Tcommitted( id ); - if( committed > 0 ) + htri_t is_committed = H5Tcommitted( id ); + if (is_committed > 0) return true; - else if( committed == 0 ) + else if (is_committed == 0) return false; else { |