diff options
author | Raymond Lu <songyulu@hdfgroup.org> | 2011-02-18 20:18:01 (GMT) |
---|---|---|
committer | Raymond Lu <songyulu@hdfgroup.org> | 2011-02-18 20:18:01 (GMT) |
commit | 13edfb217585eee6fe9f9f20d0b3198c1bb5af98 (patch) | |
tree | 411d76b3e9e3d121d7c3380276b578b0fc697517 /test | |
parent | c620068cb6bc26edf4f37feb41e825dac3c9a9ce (diff) | |
download | hdf5-13edfb217585eee6fe9f9f20d0b3198c1bb5af98.zip hdf5-13edfb217585eee6fe9f9f20d0b3198c1bb5af98.tar.gz hdf5-13edfb217585eee6fe9f9f20d0b3198c1bb5af98.tar.bz2 |
[svn-r20125] Bug 1586 - the datatype handler created with H5Tencode/decode had the reference count 0. I fixed it by changing the APP_REF parameter of H5I_register from FALSE to TRUE and added a test case in dtypes.c. The changes to configture.in, tools/misc, tools/lib, config, c++/test, Makefile.am, and fortran are only property changes when I merged from the trunk.
Tested on jam. But I tested the same fix in the trunk with h5committest.
Diffstat (limited to 'test')
-rw-r--r-- | test/dtypes.c | 82 |
1 files changed, 71 insertions, 11 deletions
diff --git a/test/dtypes.c b/test/dtypes.c index cbf5fa9..c4871d1 100644 --- a/test/dtypes.c +++ b/test/dtypes.c @@ -4901,6 +4901,10 @@ opaque_funcs(void) * Modifications: Raymond Lu * July 13, 2009 * Added the test for VL string types. + * + * Raymond Lu + * 17 February 2011 + * I added the test of reference count for decoded datatypes. *------------------------------------------------------------------------- */ static int @@ -5329,37 +5333,93 @@ test_encode(void) } /* end if */ /*----------------------------------------------------------------------- - * Close and release + * Test the reference count of the decoded datatypes *----------------------------------------------------------------------- */ - /* Close datatype and file */ - if(H5Tclose(tid1) < 0) { + + /* Make sure the reference counts for the decoded datatypes are one. */ + if(H5Iget_ref(decoded_tid1) != 1) { H5_FAILED(); - printf("Can't close datatype\n"); + printf("Decoded datatype has incorrect reference count\n"); goto error; } /* end if */ - if(H5Tclose(tid2) < 0) { + + if(H5Iget_ref(decoded_tid2) != 1) { H5_FAILED(); - printf("Can't close datatype\n"); + printf("Decoded datatype has incorrect reference count\n"); goto error; } /* end if */ - if(H5Tclose(tid3) < 0) { + + if(H5Iget_ref(decoded_tid3) != 1) { H5_FAILED(); - printf("Can't close datatype\n"); + printf("Decoded datatype has incorrect reference count\n"); goto error; } /* end if */ - if(H5Tclose(decoded_tid1) < 0) { + /* Make sure the reference counts for the decoded datatypes can be + * decremented and the datatypes are closed. */ + if(H5Idec_ref(decoded_tid1) != 0) { + H5_FAILED(); + printf("Decoded datatype can't close\n"); + goto error; + } /* end if */ + + if(H5Idec_ref(decoded_tid2) != 0) { + H5_FAILED(); + printf("Decoded datatype can't close\n"); + goto error; + } /* end if */ + + if(H5Idec_ref(decoded_tid3) != 0) { + H5_FAILED(); + printf("Decoded datatype can't close\n"); + goto error; + } /* end if */ + + /* Make sure the decoded datatypes are already closed. */ + H5E_BEGIN_TRY { + ret = H5Tclose(decoded_tid1); + } H5E_END_TRY; + if(ret!=FAIL) { + H5_FAILED(); + printf("Decoded datatype should have been closed\n"); + goto error; + } + + H5E_BEGIN_TRY { + ret = H5Tclose(decoded_tid2); + } H5E_END_TRY; + if(ret!=FAIL) { + H5_FAILED(); + printf("Decoded datatype should have been closed\n"); + goto error; + } + + H5E_BEGIN_TRY { + ret = H5Tclose(decoded_tid3); + } H5E_END_TRY; + if(ret!=FAIL) { + H5_FAILED(); + printf("Decoded datatype should have been closed\n"); + goto error; + } + + /*----------------------------------------------------------------------- + * Close and release + *----------------------------------------------------------------------- + */ + /* Close datatype and file */ + if(H5Tclose(tid1) < 0) { H5_FAILED(); printf("Can't close datatype\n"); goto error; } /* end if */ - if(H5Tclose(decoded_tid2) < 0) { + if(H5Tclose(tid2) < 0) { H5_FAILED(); printf("Can't close datatype\n"); goto error; } /* end if */ - if(H5Tclose(decoded_tid3) < 0) { + if(H5Tclose(tid3) < 0) { H5_FAILED(); printf("Can't close datatype\n"); goto error; |