diff options
author | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2013-03-22 17:56:05 (GMT) |
---|---|---|
committer | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2013-03-22 17:56:05 (GMT) |
commit | c8018386da2165a16f5e2d0cb5a391dca07b4ed5 (patch) | |
tree | bd9af146bf98c667e9237afc21bb5f108eb81dbc /c++/test/tcompound.cpp | |
parent | 030a17ca60905ef90cabaa147ee84febb064fc42 (diff) | |
download | hdf5-c8018386da2165a16f5e2d0cb5a391dca07b4ed5.zip hdf5-c8018386da2165a16f5e2d0cb5a391dca07b4ed5.tar.gz hdf5-c8018386da2165a16f5e2d0cb5a391dca07b4ed5.tar.bz2 |
[svn-r23427] Purpose: Fix bug HDFFV-8067
Description:
+ The C++ test failed with the new PGI compilers versions 12.4 and 12.5
+ An exception thrown by an internal function, which was called by
a constructor, was not propagating to the test program during the stack
unwinding, so it couldn't be caught by the test and the program terminated.
+ Various trials and errors indicated that the problem is where an STD string
converted to a char* being passed to the internal function, but confirmation
has not been found yet. It could be a compiler bug.
Solution:
+ Added a try/catch in the constructor around the internal function and
re-throw the exception when it is caught. This is a workaround.
+ Unrelated minor fixes: removed unused variables and MESSAGE's; commented
out tvlstr.cpp/test_read_vl_string_attribute because it may be redundant,
and commented out H5Tpkg.h inclusion because TEST_ALIGNMENT is not added
yet and probably not necessary in the C++ API.
Platforms tested:
Linux/32 2.6 (jam) with PGI compilers
Linux/32 2.6 (jam) with GNU compilers
Linux/64 2.6 (koala)
Diffstat (limited to 'c++/test/tcompound.cpp')
-rw-r--r-- | c++/test/tcompound.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/c++/test/tcompound.cpp b/c++/test/tcompound.cpp index fbb1219..28d8a1e 100644 --- a/c++/test/tcompound.cpp +++ b/c++/test/tcompound.cpp @@ -749,7 +749,6 @@ extern "C" void test_compound() { // Output message about test being performed - //MESSAGE("Testing Compound Data Type operations\n"); MESSAGE(5, ("Testing Compound Data Type operations\n")); test_compound_1(); // various things about compound data types |