diff options
| author | Jordan Henderson <jhenderson@hdfgroup.org> | 2018-05-17 16:14:52 (GMT) |
|---|---|---|
| committer | M. Scot Breitenfeld <brtnfld@hdfgroup.org> | 2018-07-27 22:29:58 (GMT) |
| commit | 5c51cd4212d5597f63f03e13621aafcf8fc8b4c4 (patch) | |
| tree | cc67dad149c7d8f24a724d0e8c24ab3df89c29c5 /src | |
| parent | 44507db95d1b2d660238aac06363f17133f8a1b9 (diff) | |
| download | hdf5-5c51cd4212d5597f63f03e13621aafcf8fc8b4c4.zip hdf5-5c51cd4212d5597f63f03e13621aafcf8fc8b4c4.tar.gz hdf5-5c51cd4212d5597f63f03e13621aafcf8fc8b4c4.tar.bz2 | |
Remove unused local variable
Ensure frees are done in a more heap-fragmentation friendly order
Diffstat (limited to 'src')
| -rw-r--r-- | src/H5Dmpio.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/src/H5Dmpio.c b/src/H5Dmpio.c index d496e36..cc87a1a 100644 --- a/src/H5Dmpio.c +++ b/src/H5Dmpio.c @@ -2986,7 +2986,6 @@ H5D__filtered_collective_chunk_entry_io(H5D_filtered_collective_io_info_t *chunk H5D_chunk_info_t *chunk_info = NULL; H5S_sel_iter_t *mem_iter = NULL; /* Memory iterator for H5D__scatter_mem/H5D__gather_mem */ H5S_sel_iter_t *file_iter = NULL; - unsigned char *mod_data = NULL; /* Chunk modification data sent by a process to a chunk's owner */ H5Z_EDC_t err_detect; /* Error detection info */ H5Z_cb_t filter_cb; /* I/O filter callback function */ unsigned filter_mask = 0; @@ -3204,18 +3203,16 @@ done: H5MM_free(chunk_entry->async_info.receive_buffer_array); if (chunk_entry->async_info.receive_requests_array) H5MM_free(chunk_entry->async_info.receive_requests_array); - if (mod_data) - H5MM_free(mod_data); if (tmp_gath_buf) H5MM_free(tmp_gath_buf); - if (mem_iter_init && H5S_SELECT_ITER_RELEASE(mem_iter) < 0) - HDONE_ERROR(H5E_DATASET, H5E_CANTFREE, FAIL, "couldn't release selection iterator") - if (mem_iter) - H5MM_free(mem_iter); if (file_iter_init && H5S_SELECT_ITER_RELEASE(file_iter) < 0) HDONE_ERROR(H5E_DATASET, H5E_CANTFREE, FAIL, "couldn't release selection iterator") if (file_iter) H5MM_free(file_iter); + if (mem_iter_init && H5S_SELECT_ITER_RELEASE(mem_iter) < 0) + HDONE_ERROR(H5E_DATASET, H5E_CANTFREE, FAIL, "couldn't release selection iterator") + if (mem_iter) + H5MM_free(mem_iter); if (dataspace) if (H5S_close(dataspace) < 0) HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "can't close dataspace") |