summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [svn-r18198] Description:Quincey Koziol2010-01-30178-11635/+11635
| | | | | | | | | Trim trailing whitespace from source code files, with following command: find . \( -name "*.[ch]" -or -name "*.cpp" -or -name "*.f90" \) -print |xargs -n 1 sed -i "" 's/[[:blank:]]*$//' Tested on: None - eyeballed only
* [svn-r18189] I added a test for copying an existing DCPL and using the copy ↵Raymond Lu2010-01-294-7/+145
| | | | | | | | | | to create a new dataset. There's a bug report (#1608) saying h5repack failed to do that sometimes. But I couldn't reproduce it. So I added this test and closed the report. The test is in the function test_copy_dcpl in dsets.c. Tested on jam, amani, and linew.
* [svn-r18173] Bug ID 1753: added macro version of GetTestVerbosity.Albert Cheng2010-01-272-20/+21
| | | | | | | | | | | | The previous implementation did not work for WIndows. Moved the macro definition of HDGetTestVerbosity from testhdf5.h to h5test.h. Removed as much as possible macro definitions that use HDGetTestVerbosity directly. Tested: Jam (serial and parallel). Did not run h5committest since this is a trival change that is portable.
* [svn-r18172] Description:Quincey Koziol2010-01-2736-2337/+2445
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring r18171 from trunk to 1.8 branch (with appropriate tweaks): Bring r18169 from 'merge_metadata_journaling' branch back to trunk: Converge changes on metadata_journaling branch and trunk: - Remove H5F_t* parameter from H5HL_unprotect() - Remove H5F_t* parameter from H5HL_dirty() - Remove H5F_t* parameter from H5O_unprotect() - Bring changes to metadata caching routines back: - H5AC_pin_protected_entry() - H5AC_resize_pinned_entry() - H5AC_unpin_entry() - H5AC_mark_pinned_entry_dirty() - H5AC_mark_pinned_or_protected_entry_dirty() - Revise internal H5C routines to drop H5F_t* and/or H5C_t* as parameter where possible - Revise tests to follow internal routine changes Also, other minor changes to speedup the 'cache' test. Tested on: FreeBSD/32 6.3 (duty) in debug mode FreeBSD/64 6.3 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (jam) w/PGI compilers, w/default API=1.8.x, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (amani) w/Intel compilers, w/default API=1.6.x, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.4 (tg-login3) w/parallel, w/FORTRAN, in debug mode Linux/64-amd64 2.6 (abe) w/parallel, w/FORTRAN, in production mode Mac OS X/32 10.6.2 (amazon) in debug mode Mac OS X/32 10.6.2 (amazon) w/C++ & FORTRAN, w/threadsafe,
* [svn-r18161] Snapshot version 1.8 release 4 (snap7)HDF Tester2010-01-257-22/+22
|
* [svn-r18160] Description:Quincey Koziol2010-01-2318-418/+456
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring r18159 from trunk to 1.8 branch: Bring Coverity fixes from 1/22/10 session to trunk: r18137: 219: Initialized hid_t to -1 and added close to error block. 189-191: Initialized line to NULL and added free line, and close fp to error block. r18138: 19: Moved code block for printing that the number of enums is empty to the error block. (Would never have been executed otherwise) r18139: Fix coverity item 58. Moved code related to displaying the parent of a repeated group to the else(isRoot) section, as the root group has no parent. r18140: 218: Initialized ret_value variable to -1. Because of throw Exception in default case of switch, the coverity problem would not have executed anyway. Good pratice is to initialize variables. r18141: Fix coverity item 92. Added code to H5E_register_class to free cls in case of an error. r18142: Fix coverity item 91. Added code to H5E_create_msg to free msg in case of an error. r18143: fixed issue 14, took away "if" and used #ifndef_xxx. r18144: Fix coverity item 110. Added code to H5Eget_minor to free msg_str in case of an error. r18145: fixed coverity #18 removed "aligned", it is always NULL. r18146: Fix coverity item 109. Added code to H5Eget_major to free msg_str in case of an error. r18147: Fixed coverity #81 and #82, Check for bad pointer(s), but can't issue error, just leave r18148: Fix coverity item 97. Added code to H5FD_fapl_open to free copied_driver_info in case of an error. r18149: Fix coverity item 96. Added code to H5FD_dxpl_open to free copied_driver_info in case of an error. r18150: Fix Coverity issue #29: Protected cache_ptr dereferences with "if(pass)" block r18151: Fix coverity item 93. Added code to H5FL_fac_init to free factory and new_node in case of an error. r18152: Fix coverity items 98 and 99. Added code free allocated space in case of error. r18155: 124: Freed head pointer before jumping to done. There was no error handling block and normal exit used same path out. 120-123: Freed list of lists in error handling block. r18156: Fix coverity issues 179, 180, 181, 182, 183, 184, 186, 320, 407. These were resource leak issues where allocated memory was not freed, generally in the case of tests that failed. Tested on: Mac OS X/32 10.6.2 (amazon) debug & production
* [svn-r18158] Description:Quincey Koziol2010-01-2312-1103/+826
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring r18157 from trunk to 1.8 branch: Bring back changes from Coverity session on 1/15/10: r18111: Fix Coverity issue #130: make certain that the cache gets freed on error. r18112: Fix Coverity issue #43 by making cache testing calls protected by 'pass' variable. r18113: Fix Coverity issue #129 by releasing the cache on error. r18115: Coverity #45 fix: patched an error check in H5Screate_simple to prevent future dereferencing of a NULL point. Added a verification in test/th5s.c. r18116: Fix Coverity issue #43 by releasing cache on error. r18117: Coverity #362,363 by adding HGOTO_DONE, freeing allocations and associated changes. REsolving coverity results #364-368, 369, 370-372, 377, 379, and 380. r18118: Fix Coverity issue #42: assert that cache & test specification pointer are valid. r18122: Coverity #362,363 by adding HGOTO_DONE and freeing allocations. This also takes care of #357,358. r18123: Coverity #359-361, 373-376: Added HGOTO_DONE(FAIL) statement after checking allocation for NULL. Verified allocation is freed in done block. r18128: Fixed coverity issue #10 -- removed dead code. Tested on: Mac OS X/32 10.6.2 (amazon)
* [svn-r18153] ID 1753: added macro version of GetTestVerbosity.Albert Cheng2010-01-222-13/+20
| | | | | | | | GetTestVerbosity() is called many times during tests. Implemented a more efficient macro version HDGetTestVerbosity. Tested: Jam (serial and parallel). Did not run h5committest since this is a trival change that is portable, I am 99% sure.
* [svn-r18135] Purpose:Jonathan Kim2010-01-2112-16/+8
| | | | | | | | | | | | Additional fix relted to the fix of bug1672. Description: The fix of bug1672 caused some changes of output which required modifying some test cases. After some discussion, restoring the changes of the test cases was decided. After many experiments for the solution, this fix was made so the code which improved performance can stay. Tested on Jam.
* [svn-r18132] Add test files for h5diff relate to r18130 commit.Jonathan Kim2010-01-211-0/+2
|
* [svn-r18130] Purpose:Jonathan Kim2010-01-213-2/+13
| | | | | | | Add test case relate to bug1749. Also corrected command echo line which is not match with actual test. Tested on Jam.
* [svn-r18127] Purpose:Jonathan Kim2010-01-201-12/+2
| | | | | | | | | | | | | Fix for bug1749: Incorrect code in diff() function for links compare Also removed duplicated redundant code, which improves performance. Description: When comparing file1 to file2 and both files contain same soft-links and/or external-links, if file2's target object path name is longer than file1's target object path name, the current code allocates memory buffer incorrectly, so it could end up buffer overflow or wrong result. Tested on Jam.
* [svn-r18124] I added the support of bitfield in H5Tget_native_type. ↵Raymond Lu2010-01-203-114/+209
| | | | | | | | Bitfield is similar to unsigned integer. I also added a test. Tested on amani, jam, linew.
* [svn-r18121] Snapshot version 1.8 release 4 (snap6)HDF Tester2010-01-177-21/+21
|
* [svn-r18110] Description:Quincey Koziol2010-01-1522-553/+340
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring r18109 from trunk to 1.8 branch: Bring r18076-18096 from hdf5_1_8_coverity branch to trunk: r18076: Correct Coverity issue #1 by removing dead code r18077: Fix coverity item 142. When an error occurred while copying a linked list in H5S_point_copy, the library would not free the partially allocated list. Added code to free the list in this case. r18078: Correct Coverity issue #2 by removing impossible to reach code. r18079: Correct #3 by removing impossible to reach code. r18080: Correct Coverity issue #4 by removing impossible to reach code. r18081: fix coverity 26 , check (dblik->parent) before calls H5HF_man_iblock_detach(). r18082: Fixed coverity issues 321 and 316. 321: freed sm_buf in error handling to remove resource leak. Also set sm_buf to NULL after other instances in which it is freed to prevent double free. 316: initialized nmembs to 0. r18083: Correct Coverity issue #6 by removing debugging knob from error reporting code. r18084: Fix coverity item 269 + others. When a error occurred in a function using the h5tools error framework, the "past_catch" variable would not be set to true because that statement was before the label that goto jumped to. This could cause a failure in the cleanup section to go back to the start of the section, freeing variables twice, etc. Moved the label infront of past_catch=TRUE. r18085: fixed coverity #27, check if (heap) before use heap->obj.... r18086: fixed coverity #28, check curr_span not null before use it at if(curr_span && (io_bytes_left==0 || curr_seq>=maxseq)) r18087: Correct Coverity issue #7 by cleaning up correctly on error r18088: Correct Coverity #8 by removing unchanged variable checking code. r18089: Correct Coverity issue #9 - remove impossible to reach code. r18090: Correct Coverity issue #11 by removing impossible to reach code. Also clean up some minor style issues. r18091: Fix coverity items 314 and 318. Changed the improper assertion of the return value of a library function to a check, and a return(void) on failure. r18092: Fix coverity item 70. Changed the improper assertion of the return value of a library function to a check, and a return(void) on failure. r18093: Correct Coverity issue #12 by removing dead code. r18094: Correct Coverity issue #16 by removing debugging code. r18095: Fixed coverity issue # 271. Removed redundant checking and freeing of sm_buf1 and sm_buf2. r18096: Correct Coverity issue #17 by refactoring test to remove dead code. Also, removed previous "coverity" statements in comments, we'll review those issues again and see if we can figure them out, now that we have more experience with Coverity. Tested on: Mac OS X/32 10.6.2 (amazon) debug & prod
| * [svn-r18096] Description:Quincey Koziol2010-01-081-122/+107
| | | | | | | | | | | | | | Correct Coverity issue #17 by refactoring test to remove dead code. Tested on: Mac OS X/32 10.6.2 (amazon) w/debug
| * [svn-r18095] Fixed coverity issue # 271.Mike McGreevy2010-01-081-5/+0
| | | | | | | | Removed redundant checking and freeing of sm_buf1 and sm_buf2.
| * [svn-r18094] Description:Quincey Koziol2010-01-081-115/+0
| | | | | | | | | | | | | | Correct Coverity issue #16 by removing debugging code. Tested on: Mac OS X/32 10.6.2 (amazon) w/debug
| * [svn-r18093] Description:Quincey Koziol2010-01-081-34/+35
| | | | | | | | | | | | | | Correct Coverity issue #12 by removing dead code. Tested on: Mac OS X/32 10.6.3 (amazon) w/debug
| * [svn-r18092] Fix coverity item 70. Changed the improper assertion of the ↵Neil Fortner2010-01-081-1/+2
| | | | | | | | | | | | | | | | return value of a library function to a check, and a return(void) on failure. Tested: Fedora
| * [svn-r18091] Fix coverity items 314 and 318. Changed the improper assertion ↵Neil Fortner2010-01-081-2/+4
| | | | | | | | | | | | | | | | of the return value of a library function to a check, and a return(void) on failure. Tested: Fedora
| * [svn-r18090] Description:Quincey Koziol2010-01-084-15/+7
| | | | | | | | | | | | | | | | Correct Coverity issue #11 by removing impossible to reach code. Also clean up some minor style issues. Tested on: Mac OS X/32 10.6.3 (amazon) w/debug
| * [svn-r18089] Description:Quincey Koziol2010-01-081-6/+0
| | | | | | | | | | | | | | Correct Coverity issue #9 - remove impossible to read code. Tested on: Mac OS X/32 10.6.3 (amazon)
| * [svn-r18088] Description:Quincey Koziol2010-01-081-41/+1
| | | | | | | | | | | | | | Correct Coverity #8 by removing unchanged variable checking code. Tested on: Mac OS X/32 10.6.3 (amazon) w/debug
| * [svn-r18087] Description:Quincey Koziol2010-01-081-23/+28
| | | | | | | | | | | | | | Correct Coverity issue #7 by cleaning up correctly on error Tested on: Mac OS X/32 10.6.3 (amazon) w/debug
| * [svn-r18086] fixed coverity #28, check curr_span not null before use it at ↵Peter Cao2010-01-081-1/+1
| | | | | | | | if(curr_span && (io_bytes_left==0 || curr_seq>=maxseq))
| * [svn-r18085] Peter Cao2010-01-081-1/+2
| | | | | | | | fixed coverity #27, check if (heap) before use heap->obj....
| * [svn-r18084] Fix coverity item 269 + others. When a error occurred in a ↵Neil Fortner2010-01-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | function using the h5tools error framework, the "past_catch" variable would not be set to true because that statement was before the label that goto jumped to. This could cause a failure in the cleanup section to go back to the start of the section, freeing variables twice, etc. Moved the label infront of past_catch=TRUE. Tested: Fedora
| * [svn-r18083] Description:Quincey Koziol2010-01-081-29/+14
| | | | | | | | | | | | | | | | Correct Coverity issue #6 by removing debugging knob from error reporting code. Tested on: Mac OS X/32 10.6.3 (amazon) w/debug
| * [svn-r18082] Fixed coverity issues 321 and 316.Mike McGreevy2010-01-081-1/+6
| | | | | | | | | | | | | | | | 321: freed sm_buf in error handling to remove resource leak. Also set sm_buf to NULL after other instances in which it is freed to prevent double free. 316: initialized nmembs to 0.
| * [svn-r18081] fix coverity 26 , check (dblik->parent) before calls ↵Peter Cao2010-01-081-2/+5
| | | | | | | | | | | | | | | | H5HF_man_iblock_detach(). line, and those below, will be ignored-- M src/H5HFdblock.c
| * [svn-r18080] Description:Quincey Koziol2010-01-081-4/+0
| | | | | | | | | | | | | | Correct Coverity issue #4 by removing impossible to reach code. Tested on: Mac OS X/32 10.6.3 (amazon) w/debug
| * [svn-r18079] Description:Quincey Koziol2010-01-081-9/+0
| | | | | | | | | | | | | | Correct #3 by removing impossible to reach code. Tested on: Mac OS X/32 10.6.2 (amazon) w/debug
| * [svn-r18078] Description:Quincey Koziol2010-01-081-8/+1
| | | | | | | | | | | | | | Correct Coverity issue #2 by removing impossible to reach code. Tested on: Mac OS X/32 10.6.2 (amazon) w/debug
| * [svn-r18077] Fix coverity item 142. When an error occurred while copying a ↵Neil Fortner2010-01-081-8/+27
| | | | | | | | | | | | | | | | | | linked list in H5S_point_copy, the library would not free the partially allocated list. Added code to free the list in this case. Tested: Fedora
| * [svn-r18076] Description:Quincey Koziol2010-01-081-8/+1
| | | | | | | | | | | | | | Correct Coverity issue #1 by removing dead code Tested on: Mac OS X/32 (amazon) debug
* | [svn-r18106] bug 1673:Albert Cheng2010-01-1244-165/+165
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The mixed use of RUNTEST (original) and RUNTESTS (new) caused confusion. E.g., the timings in test/ was still using the old $RUNTEST. It made more sense to use $RUNTEST which is used by the dejagnu feature of automake. So, I changed all $RUNTEST or $RUNTESTS to $RUNEXEC. config/commence.am & config/conclude.am are the two files that got changes. Also fixed an error in test/Makefile.am. The rest are changed by bin/reconfigure. Tested: h5committested.
* | [svn-r18103] Added new H5HLint.c file. Corrected nodebug parameter to check.Allen Byrne2010-01-113-3/+11
| | | | | | | | Tested: windows
* | [svn-r18101] SnapshotHDF Tester2010-01-107-21/+21
| |
* | [svn-r18099] Description:Scot Breitenfeld2010-01-084-55/+68
| | | | | | | | | | | | | | | | | | Merged changes from trunk/fortran into the branch used: svn merge -r 17682:18098 https://svn.hdfgroup.uiuc.edu/hdf5/trunk/fortran Tested on: jam (intel) amani (pgi)
* | [svn-r18073] Description:Quincey Koziol2010-01-0724-559/+1622
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring r18072 from trunk to 1.8 branch: Bring r18071 from metadata journaling merge branch: Refactor local heap routines with changes from metadata journaling branch, along with other misc. changes as the changes on the metadata journaling branch are being converged with the current state of the trunk. Tested on: Mac OS X/32 10.6.2 (amazon) debug & prod Linux/64 2.6 (amani) prod FreeBSD/64 (liberty) prod
* | [svn-r18069] Added entry to MANIFEST for new file config/ibm-aix.Larry Knox2010-01-061-0/+1
| | | | | | | | Tested chkmanifest on jam.
* | [svn-r18064] Purpose:Albert Cheng2010-01-051-0/+170
| | | | | | | | | | | | | | Port to AIX 6.1 for RS6000. Should be the same as PowerPC based AIX. Tested: Blue-print of NCSA.
* | [svn-r18063] Fix bug in links.c which occurs when zlib is disabled. This ↵Neil Fortner2010-01-041-1/+1
| | | | | | | | | | | | | | | | | | | | was fixed in the trunk in r17751, but was not ported to 1.8 because that feature (compress group fheap) had not beed added to 1.8 yet. This fix should have been ported to the cgfh branch at the time, but it was overlooked. Tested: Fedora (too small for commit test)
* | [svn-r18062] SnapshotHDF Tester2010-01-047-22/+22
| |
* | [svn-r18058] SnapshotHDF Tester2009-12-277-22/+22
| |
* | [svn-r18056] Bug fix: 1192Albert Cheng2009-12-268-26/+60
| | | | | | | | | | | | | | | | Description: Fixed exit code (sometimes return code in Main) to follow the HDF5 standards. Tested: Jam, both serial and parallel.
* | [svn-r18054] Updated for the moves of the testfiles.Albert Cheng2009-12-261-4/+2
| | | | | | | | Checked by bin/chkmanifest.
* | [svn-r18053] Bug fix: 1192Albert Cheng2009-12-2618-101/+155
| | | | | | | | | | | | | | | | Description: Fixed exit code (sometimes return code in Main) to follow the HDF5 standards. Tested: H5committested plus serial test in Jam.
* | [svn-r18052] Moved gif2h5 tools tesfiles from hl/tools/testfiles to ↵Albert Cheng2009-12-253-2/+2
| | | | | | | | | | | | | | | | | | hl/tools/gif2h5/testfiles so that it has its own testfiles directory. Tested: h5committested.