diff options
| author | David Young <dyoung@hdfgroup.org> | 2020-02-17 16:46:52 (GMT) |
|---|---|---|
| committer | David Young <dyoung@hdfgroup.org> | 2020-02-17 16:46:52 (GMT) |
| commit | fa2b89c6b0c894915f12a1df594ef8c427adbc2d (patch) | |
| tree | fea728f0380438d459ba7c785e3d66da6b1ee4ce /src/H5Tnative.c | |
| parent | ddf139aed55ea130eebff56d5385351a01d46508 (diff) | |
| parent | 3f903a441ad84001ea66589728bd8b036b6fdfca (diff) | |
| download | hdf5-fa2b89c6b0c894915f12a1df594ef8c427adbc2d.zip hdf5-fa2b89c6b0c894915f12a1df594ef8c427adbc2d.tar.gz hdf5-fa2b89c6b0c894915f12a1df594ef8c427adbc2d.tar.bz2 | |
Merge pull request #2325 in HDFFV/hdf5 from ~DYOUNG/werror:h5t_copy to develop
Merged per discussion with Elena.
* commit '3f903a441ad84001ea66589728bd8b036b6fdfca':
Take out the temporary performance tests.
Make calls through a function pointer. Use the same number of arguments, always.
Increase iterations, provide a baseline for no-op, simplify the overhead case a bit.
Temporarily add some code that measures the time to run the simplest possible H5T__copy_all()-like routine 10 million times and then measures the version with FUNC_ENTER_STATIC/_LEAVE_NOAPI and a HGOTO_ERROR() statement.
H5T_copy() constification plus Quincey's contributions.
Diffstat (limited to 'src/H5Tnative.c')
| -rw-r--r-- | src/H5Tnative.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/H5Tnative.c b/src/H5Tnative.c index 8859102..357bb08 100644 --- a/src/H5Tnative.c +++ b/src/H5Tnative.c @@ -272,7 +272,7 @@ H5T__get_native_type(H5T_t *dtype, H5T_direction_t direction, size_t *struct_ali /* Construct child compound type and retrieve a list of their IDs, offsets, total size, and alignment for compound type. */ for(u = 0; u < nmemb; u++) { - if(NULL == (memb_type = H5T_get_member_type(dtype, u, H5T_COPY_TRANSIENT))) + if(NULL == (memb_type = H5T_get_member_type(dtype, u))) HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, NULL, "member type retrieval failed") if(NULL == (comp_mname[u] = H5T__get_member_name(dtype, u))) |