summaryrefslogtreecommitdiffstats
path: root/tools/lib/h5diff_array.c
Commit message (Collapse)AuthorAgeFilesLines
* [svn-r27626] Various minor warning fixes before major SWMR and VDS merges.Dana Robinson2015-08-311-20/+20
| | | | | | | | | | | gcc 4.9.2 was used to create the warning list - implicit casts - shadowed variables - various enum issues - other minor fixes (comments, unused macros, etc.) Tested on: h5committest
* [svn-r27572] Removed VMS-specific code from the library.Dana Robinson2015-08-241-22/+0
| | | | | | | | | The only remaining code consists of a few floating-point tests that rely on pre-generated and checked-in VMS files. These have been left alone, even though they will not be possible to recreate, since testing VMS float behavior is still important. Tested on: h5committest
* [svn-r24224] HDFFV-8529: Space separate define stringsAllen Byrne2013-09-301-6/+6
| | | | Tested: h5committest
* [svn-r24138] Cast constants to specified type. Allen Byrne2013-09-131-5/+5
| | | | | | | | Cast smaller vars to larger type. Cast calculation result to correct type. Changed int member to hbool_t Tested: local linux
* [svn-r24101] Description:Quincey Koziol2013-09-051-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clean up warnings, enable new compiler warning flag(s) and bring back changes from Coverity branch: r20813: Remove the dead code as listed for coverity bug #1722. h5committested. r20814: Issue 69: Check return value and throw error if negative return. Also free datatype id on error r20815: Use HDstrncpy. --gh r20816: Replaced one last HDstrcat call with HDstrncat to resolve coverity issue 832. r20817: Use HDstrncpy and HDstrncat. --gh r20818: Purpose: Fix valgrind issues with h5jam Description: Modified h5jam to free strings strdup'd in parse_command_line before exit. Note that they may still not be freed in case of error, due to the widespread use of exit(). r20819: Issue 80: change loop to use int as loop index. r20820: Maintenance: Fixed the bug found by coverity CID 788 There were two problems with this function: 1) it tried to unnecessary free NULL pointer 2) it tried to allocate c_name buffer that is done by H5Pget_class_name Tested on: Mac OSX 10.8.4 (amazon) w/gcc 4.8.1, C++ & FORTRAN (too minor to require h5committest)
* [svn-r23695] Description:Quincey Koziol2013-05-111-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clean up warnings in H5Tconv.c (down to _only_ 9000 lines of output now!) Merge changes from Coverity branch back to trunk: r20684: Fix for coverity bug #1721 which was due to the fix for coverity bug #943. r20685: Use HDstrncpy. --gh r20761: Purpose: Fix valgrind issues Description: Free image_data and data as appropriate in test_image. r20762: Purpose: Fix coverity issue 600 Description: Add check for return value of H5O_close in H5Ocopy. Also cleaned up various warnings. r20763: Purpose: Fix valgrind issues with h5stat Description: Modified h5stat to free "iter" before exit, and free "hand" before exit if parse_command_line exits directly. r20764: fixed coverity issues: 69, 327, 614, 684, 685, 696, 697, 1681, 967, 826, 660, 80 r20765: Fixed coverity bug 668 Pulled x * y * z multiply out of malloc operand into a separate n_elements variable to quiet integer overflow warning. No actual integer overflow tests are performed since it's just a test program but I did add a check that n_elements is >= 1. I also changed an error condition from doing its own close and returning -1 to "goto out;" like the rest of the program. r20766: Fixed coverity bug 667 Pulled x * y * z multiply out of malloc operand into a separate n_elements variable to quiet integer overflow warning. No actual integer overflow tests are performed since it's just a test program. Tested on: Mac OSX/64 10.8.3 (amazon) w/C++ & FORTRAN (h5committest upcoming)
* [svn-r23415] Description:Quincey Koziol2013-03-211-2/+2
| | | | | | | Avoid using fpclassify() for detecting zero/non-zero values. Tested on: Mac OSX/64 10.8.3 (amazon) w/debug
* [svn-r23395] Description:Quincey Koziol2013-03-201-259/+430
| | | | | | | | | | | | Bring Coverity changes from branch to trunk: r20682: fixed coverity: 76, 77, 635, 636, 1164, 1165, 1166, 1121, 1122, 1117, 1343 Tested on: Mac OSX/64 10.8.3 (amazon) w/C++ & FORTRAN (too minor to require h5committest)
* [svn-r22625] HDFFV-8111: h5diff compare file to itself - removed path shortcutAllen Byrne2012-08-031-0/+45
|
* [svn-r21980] Correct HD prefix in tools for *allocAllen Byrne2012-02-241-1/+1
| | | | Checked for HD support.
* [svn-r21979] Correct HD prefix in tools for strlen and memcpyAllen Byrne2012-02-241-136/+136
| | | | Checked for HD support.
* [svn-r21974] Update HSIZE_T print statements to use correct macro.Allen Byrne2012-02-231-32/+32
| | | | | | Fix some more HD prefix issues in tools. Tested: local linux
* [svn-r21358] Purpose:Jonathan Kim2011-09-021-30/+40
| | | | | | | | | | | HDFFV-7712 - h5diff: segfault over combinations of complex container types (compound, array, vlen) Description: - Fixed segfault over dataset with container types (array,lven) with multiple nested compound types. (ex: compound->array->compound, compound->vlen->compound) Tested: jam (linux32-LE), koala (linux64-LE), heiwa (linuxppc64-BE), tejeda (mac32-LE), linew (solaris-BE), Cmake (jam)
* [svn-r21121] Description:Quincey Koziol2011-07-181-4/+4
| | | | | | | | | Switch from H5P_DATASET_ACCESS_DEFAULT to H5P_DEFAULT for calls to H5Rdereference2(). Tested on: Mac OS X/32 10.6.8 (amazon) w/debug (too minor to require h5committest)
* [svn-r21117] Issue 2763 - I added a new parameter of object access property ↵Raymond Lu2011-07-181-4/+4
| | | | | | list to the function H5Rdereference. It's called H5Rdereference2 now. H5Rdereference function has been deprecated to H5Rdereference1. I also added some test cases in trefer.c. Tested on jam, heiwa, and koala.
* [svn-r21105] Fixed two bugs:Peter Cao2011-07-151-52/+80
| | | | | | | - h5repack: h5repack failed to copy dataset if the layout is changed from chunked with unlimited dims to contiguous. (PC -- 2011/07/15) - h5diff: "--delta" option considers two NaN of the same type are different, which is wrong based on http://www.hdfgroup.org/HDF5/doc/RM/Tools.html#Tools-Diff. (PC -- 2011/07/15)
* [svn-r20724] Purpose: Fix a bug in h5diff when enum values are compared that doDana Robinson2011-05-041-2/+33
| | | | | | | | | | | | | | | | | not represent a valid enum value. Description: The h5diff code compares enum values by converting them to strings and then comparing them. When the enum value is out of range and can't be converted to an enum string representation, the comparison was skipped. The code now flags differences when one of the two enum values is out of range (two out of range values are compared in memory). A test has been added to the tools test script. This fixes JIRA HDFFV-7527 Tested on: jam, koala, heiwa (h5committest)
* [svn-r20384] Purpose:Jonathan Kim2011-03-311-3/+3
| | | | | | | | | | | | | | | | | | Fixed Bug 2184 - GMQS: h5diff - incorrect calculation code for --use-system-epsilon option Description: Merged from HDF5 1.8 branch r20369. Fixed h5diff for --use-system-epsilon option: the calculation changed from ( |a - b| / b ) to ( |a - b| ). This was decided for better performance and was corrected only in HDF5 trunk, so 1.8 got updated. Also comments for equal_XXX() function were updated correctly. Also help page and RM got updated correctly. Also add test cases for testing the differences w/wo the option. Tested: jam (linux32-LE), amani (linux64-LE), heiwa (linuxppc64-BE), tejeda (mac32-LE), linew (solaris-BE), Windows (32-LE)
* [svn-r20070] Description:Quincey Koziol2011-02-091-1/+1
| | | | | | | Switch order of system includes, to correct error on FreeBSD machines. Tested on: FreeBSD/32 6.3 (duty) w/debug
* [svn-r20065] Description:Quincey Koziol2011-02-081-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring changes from Coverity branch to trunk: r19975: Fixed potential mem leak at H5O_attr_open_by_name r19980: Fix coverity issue 792. Free tmp_env_prefix in H5Lexternal.c line 365 if it is not NULL but its contents are 0 when it goes out of scope. r20039: Eliminate warnings about nested extern and implicit declarations of parallel_print and address Coverity defects 712-781 by #including h5tools_utils.h in h5diff_array.c, h5diff_attr.c, h5diff_dset.c and h5diff_util.c. r20046: Purpose: Address TOCTOU warnings in h5jam and h5unjam Description: Coverity is afraid that the state of the input file could change between the call to stat() and the call to open(). This is called a time-of- check time-of-use (TOCTOU) vulnerability. Modified stat calls to fstat which uses an open file pointer so it (hopefully) won't complain any more. r20047: Addressed coverity issues 135-137, 462-464. Local pointers that needed to be freed in case of error were moved out of a switch statement in src/H5Tnative.c, set to NULL, and checked before freeing. Tested on: Mac OS X/32 10.6.6 (amazon) w/debug & production (h5committested on Coverity branch)
* [svn-r19892] Purpose:Jonathan Kim2010-12-281-184/+165
| | | | | | | | | | | | | | | | | | | Bug 2089 - GMQS: h5diff segfault on a compound dataset with fixed length + vlen string type order Description: This is fix for the segfault when h5diff compares a compound dataset with combination of fixed length string types and vlen string types in certain orders. Optimized vlariable length string handling codes. The fix is referred from h5dump handling vlen strings. For testing, several compound datasets were added with various combinations. Previous failed cases: - Vlen string, Fixed length string, Vlen string, Fixed length string - Fixed length string, Fixed length string, Vlen string, Vlen string - Fixed length string, Vlen string, Fixed length string, Vlen string Tested: jam (linux32-LE), amani (linux64-LE), heiwa (linuxppc64-BE), tejeda (mac32-LE), linew (solaris-BE)
* [svn-r19759] Purpose:Jonathan Kim2010-11-111-4/+12
| | | | | | | | | | | | | | | | | | Fix compile error for Windows from previous checkin r19757. Description: Previous log: Improve h5diff performance. The following changes for improving h5diff performance: 1) use HDmemcmp() before comparing each elements (memcmp() is very fast at both linew and jam) 2) replace the expensive H5Tequals() calls 3) retrieve datatype information at dataset level not each element level for compound datasets Tested: jam (linux32-LE), Windows
* [svn-r19757] Purpose:Jonathan Kim2010-11-111-35/+118
| | | | | | | | | | | | | | | | Improve h5diff performance. Description: The following changes for improving h5diff performance: 1) use HDmemcmp() before comparing each elements (memcmp() is very fast at both linew and jam) 2) replace the expensive H5Tequals() calls 3) retrieve datatype information at dataset level not each element level for compound datasets Tested: jam (linux32-LE), amani (linux64-LE), heiwa (linuxppc64-BE), tejeda (mac32-LE), linew (solaris-BE)
* [svn-r19712] Purpose:Jonathan Kim2010-11-021-30/+76
| | | | | | | | | | | | | | | Fixed h5diff to handle variable-length strings in a compound dataset correctly. Also variable-length string array in a compound dataset. Bug #1989. Description: Garbage values were displayed when h5diff compared variable-length strings (or string array) in a compound type dataset. Tested: jam (linux32-LE), amani (linux64-LE), heiwa (linuxppc64-BE), tejeda (Mac32)
* [svn-r19237] Description:Quincey Koziol2010-08-121-6/+2
| | | | | | | | | | Clean up misc. compiler warnings. Tested on: FreeBSD/32 6.3 (duty) in debug mode Mac OS X/32 10.6.4 (amazon) w/C++ & FORTRAN, w/threadsafe, in production mode Mac OS X/32 10.6.4 (amazon) w/parallel, in debug mode
* [svn-r18425] Purpose:Jonathan Kim2010-03-181-1/+2
| | | | | | | | | | | | Fix for bug1814 - NPOESS: h5repack doesn't handle references to the groups as an element of a dataset Description: handles object reference to named-datatype as well. Add test cases. Tested: jam, amani, linew
* [svn-r18197] Description:Quincey Koziol2010-01-301-39/+39
| | | | | | | | | Trim trailing whitespace from source code files with this command: find . \( -name "*.[ch]" -or -name "*.cpp" -or -name "*.f90" \) -print |xargs -n 1 sed -i "" 's/[[:blank:]]*$//' Tested on: None - eyeballed only
* [svn-r17981] Description:Quincey Koziol2009-12-101-50/+47
| | | | | | | | Abstract "print header" code into separate routine. Tested on: Mac OS X/32 10.6.2 (amazon) w/debug (too minor to require h5committest)
* [svn-r17680] Use absolute comparison for use-system-epsilon and default option.Peter Cao2009-10-191-2/+2
|
* [svn-r17466] " Use strict equality as defaultPeter Cao2009-09-111-50/+26
| | | | | | " Use "--use-system-epsilon" for system EPSILON " Use "-p" or "-d" for whatever user's choice of epsilon " Use "-p 0" or "-d 0" for strict equality (same as default)
* [svn-r17254] Fixed Bug 1563 - h5diff and Infinity Peter Cao2009-07-281-5/+10
| | | | Added test cases
* [svn-r16785] added missing test and compare cases inside compounds for nanPedro Vicente Nunes2009-04-181-172/+483
| | | | tested: linux
* [svn-r16502] bug fix: a comma accidently left on a conditional typedef was ↵Pedro Vicente Nunes2009-02-201-4/+13
| | | | | | causing compilition error on AIX tested: h5commitest
* [svn-r16501] remove a strange character from a commentPedro Vicente Nunes2009-02-201-2/+2
| | | | tested: h5committest
* [svn-r16489] Fixed bug #1459 by eliminating the macro long_long and ↵Allen Byrne2009-02-181-57/+57
| | | | | | | | | | | replacing all instances with long long. Tested: h5comittest fedora 10 x64 Vista 32, VS2005, IVF101 XP32, Cygwin
* [svn-r16373] replaced some printf statements with parallel_printPedro Vicente Nunes2009-01-291-1/+0
| | | | tested: linux serial and parallel
* [svn-r16348] #1368 (E1) h5diff: implement "not comparable" messages. ↵Pedro Vicente Nunes2009-01-261-52/+2
| | | | | | | | Implemented RFC. The new option is <-c, --compare List objects that are not comparable> added some test cases tested: windows, linux
* [svn-r16319] add conditinal nan detection to some missing placesPedro Vicente Nunes2009-01-141-32/+68
| | | | tested: windows, linux
* [svn-r16300] Added an option to avoid dealing with NaNs Pedro Vicente Nunes2009-01-131-95/+111
| | | | | | | -N, --nan Avoid NaNs detection Note: there is no shell script run for datasets with NaN because the output is non portable (different results and NaN strings for different systems) Tested: windows, linux
* [svn-r16217] Bug fixPedro Vicente Nunes2008-12-231-5/+13
| | | | | | | PG compiler complains about array out of bounds (a rank of zero was not checked) Adding a scalar dataset to the test generator program. this case is run on a previous existing run, the case was added to 2 existing files Tested: windows, linux
* [svn-r16024] For scalar string datasets print the character position when a ↵Pedro Vicente Nunes2008-11-041-91/+226
| | | | | | difference is found instead of a non-existing array position Tested: windows
* [svn-r15989] Merge with 1.8 up to rev 15945Pedro Vicente Nunes2008-10-291-44/+4
| | | | | | Regarding use of unsigned long long print format Tested: linux
* [svn-r15977] merge rev 15866 from 1.8 branch into trunk (diffs for unsigned ↵Pedro Vicente Nunes2008-10-281-114/+506
| | | | | | long long) tested: windows, linux
* [svn-r15628] Description:Quincey Koziol2008-09-161-601/+601
| | | | | | | | | | | | | Remove trailing whitespace from C/C++ source files, with the following script: foreach f (*.[ch] *.cpp) sed 's/[[:blank:]]*$//' $f > sed.out && mv sed.out $f end Tested on: Mac OS X/32 10.5.5 (amazon) No need for h5committest, just whitespace changes...
* [svn-r14903] Undoing change committed in r14902.Scot Breitenfeld2008-04-301-116/+160
|
* [svn-r14902] Merged fortran_1_8 branch changes r14505:14901 into the trunk. ↵Scot Breitenfeld2008-04-301-160/+116
| | | | New fortran wrappers added.
* [svn-r14700] bug fix: the function equal_double that compares double type ↵Pedro Vicente Nunes2008-03-041-69/+119
| | | | | | did not had nan logic for the cases of options -d and -p tested: windows, linux
* [svn-r14690] Description:Quincey Koziol2008-02-281-47/+41
| | | | | | | Handle comparing datasets & attributes w/variable-length strings properly. Tested on: Linux/64 2.6.9 (chicago)
* [svn-r14382] bug fix: enable NaN detection for double typePedro Vicente Nunes2008-01-081-1952/+2178
| | | | | | note: RFC at http://www.hdfgroup.uiuc.edu/RFC/HDF5/NaNs_and_HDF5/ tested: windows, linux, solaris
* [svn-r14256] bug fix: h5diff did not handle NaNsPedro Vicente Nunes2007-11-131-2/+228
| | | | | | | solution: added a NaN detection function according to RFC http://www.hdfgroup.uiuc.edu/RFC/HDF5/NaNs_and_HDF5/ tested: windows XP Vs6, linux, freeBsd, solaris 5.10