diff options
| author | Raymond Lu <songyulu@hdfgroup.org> | 2011-08-18 21:11:14 (GMT) |
|---|---|---|
| committer | Raymond Lu <songyulu@hdfgroup.org> | 2011-08-18 21:11:14 (GMT) |
| commit | a5522454a560673856ef525066da5e8f5a90b9a9 (patch) | |
| tree | ffc9932979676ead592067f7fb8dd9849ee641d8 /test | |
| parent | 215c872226bc099bb216b6f141dfd3706a4ab4c0 (diff) | |
| download | hdf5-a5522454a560673856ef525066da5e8f5a90b9a9.zip hdf5-a5522454a560673856ef525066da5e8f5a90b9a9.tar.gz hdf5-a5522454a560673856ef525066da5e8f5a90b9a9.tar.bz2 | |
[svn-r21258] Issue 7674 - clang compiler reported an error with line 334:
temp_point->l = (unsigned long long)((i * 100 + j * 1000) * n);
The value can overflow the signed int before being converted to unsigned long long. So I changed it to
temp_point->l = (unsigned long long)((i * 40 + j * 400) * n);
to keep it under the maximal value.
Tested on jam. Simple change.
Diffstat (limited to 'test')
| -rw-r--r-- | test/ntypes.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/ntypes.c b/test/ntypes.c index 03a343f..165d21b 100644 --- a/test/ntypes.c +++ b/test/ntypes.c @@ -331,7 +331,7 @@ test_compound_dtype2(hid_t file) temp_point->st.c2 = (short)(i + j); temp_point->st.l2 = (i * 5 + j * 50) * n; temp_point->st.ll2 = (i * 10 + j * 100) * n; - temp_point->l = (unsigned long long)((i * 100 + j * 1000) * n); + temp_point->l = (unsigned long long)((i * 40 + j * 400) * n); } /* end for */ } /* end for */ |