diff options
| author | Raymond Lu <songyulu@hdfgroup.org> | 2005-06-29 16:01:09 (GMT) |
|---|---|---|
| committer | Raymond Lu <songyulu@hdfgroup.org> | 2005-06-29 16:01:09 (GMT) |
| commit | 7b193be887d3d84adddc5c1738799e53d1fc2754 (patch) | |
| tree | e30adc925395f1d3eac263e15589c815ea596749 /src | |
| parent | 1e1e2578f66a7346aef55b33f44c2d27853c7da7 (diff) | |
| download | hdf5-7b193be887d3d84adddc5c1738799e53d1fc2754.zip hdf5-7b193be887d3d84adddc5c1738799e53d1fc2754.tar.gz hdf5-7b193be887d3d84adddc5c1738799e53d1fc2754.tar.bz2 | |
[svn-r10998] Purpose: Bug fix.
Description:
For the definition of the macro H5T_CONV_Xx_CORE, added a condition branch
(else if (*((ST*)S) == (DT)(D_MAX))) which seems redundant.
It handles a special situation when the source is "float" and assigned the value
of "INT_MAX". A compiler may do roundup making this value "INT_MAX+1". However,
when do comparison "if (*((ST*)S) > (DT)(D_MAX))", the compiler may consider them
equal.
Platforms tested: fuss - simple change.
Diffstat (limited to 'src')
| -rw-r--r-- | src/H5Tconv.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/H5Tconv.c b/src/H5Tconv.c index 0386b8c..4690857 100644 --- a/src/H5Tconv.c +++ b/src/H5Tconv.c @@ -183,6 +183,12 @@ H5FL_BLK_DEFINE_STATIC(array_seq); *((DT*)D) = (DT)(*((ST*)S)); \ } +/* Added a condition branch(else if (*((ST*)S) == (DT)(D_MAX))) which seems redundant. + * It handles a special situation when the source is "float" and assigned the value + * of "INT_MAX". A compiler may do roundup making this value "INT_MAX+1". However, + * when do comparison "if (*((ST*)S) > (DT)(D_MAX))", the compiler may consider them + * equal. SLU - 2005/06/29 + */ #define H5T_CONV_Xx_CORE(S,D,STYPE,DTYPE,ST,DT,D_MIN,D_MAX) { \ if (*((ST*)S) > (DT)(D_MAX)) { \ if(cb_struct.func) { \ @@ -199,6 +205,8 @@ H5FL_BLK_DEFINE_STATIC(array_seq); } \ else \ *((DT*)D) = (D_MAX); \ + } else if (*((ST*)S) == (DT)(D_MAX)) { \ + *((DT*)D) = (D_MAX); \ } else if (*((ST*)S) < (D_MIN)) { \ if(cb_struct.func) { \ H5T_conv_ret_t except_ret; /*callback return*/ \ @@ -214,7 +222,7 @@ H5FL_BLK_DEFINE_STATIC(array_seq); } \ else \ *((DT*)D) = (D_MIN); \ - } else \ + } else \ *((DT*)D) = (DT)(*((ST*)S)); \ } |