diff options
| author | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2016-04-22 18:58:08 (GMT) |
|---|---|---|
| committer | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2016-04-22 18:58:08 (GMT) |
| commit | 58803d08320c0f704bc4cca600e9701572ddc1b4 (patch) | |
| tree | 54f45208d66d18442fd5c588e4a676b3a203a072 /c++/src/H5DataType.cpp | |
| parent | 38687f3498d0878c1c28cb52b8b9e070fcb45da8 (diff) | |
| download | hdf5-58803d08320c0f704bc4cca600e9701572ddc1b4.zip hdf5-58803d08320c0f704bc4cca600e9701572ddc1b4.tar.gz hdf5-58803d08320c0f704bc4cca600e9701572ddc1b4.tar.bz2 | |
[svn-r29765] 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
Merged from trunk r29340.
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 3b5d05b..08b317e 100644 --- a/c++/src/H5DataType.cpp +++ b/c++/src/H5DataType.cpp @@ -360,10 +360,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 { |