Commit Graph

12355 Commits

Author SHA1 Message Date
Raymond Lu
aabfb1d756 [svn-r20934] Issue 4278 - When reading data fails, the error message should say which filter isn't registered. This is the follow-up commit (20929 and 20930). The check-vfd in Makefile has trouble with the file name. Because the test file
is pre-generated, I use the default FAPL to create the full file name to avoid some driver test.  

Tested on jam - simple change.
2011-06-06 12:05:37 -05:00
Raymond Lu
5fe34ffd25 [svn-r20930] Issue 4278 - When reading data fails, the error message should say which filter isn't registered. This is the follow-up commit. When I checked in the fix and test (revision 20929), I forgot to add the data file test/filter_error.h5.
I've tested it in my previous checkin.
2011-06-03 14:59:34 -05:00
Raymond Lu
4bffd76b3c [svn-r20929] Issue 4278 - When reading data fails, the error message should say which filter isn't registered. The fix is simple. Most of the effort is on the test. The file with filter enabled is created in gen_filter.c. The verification of the error message is in test_error.c. The output is compared against the standard output.
Tested on jam, koala, and heiwa.
2011-06-03 10:58:49 -05:00
Jonathan Kim
b71a597e86 [svn-r20927] Purpose:
HDFFV-7607: h5ls - improve efficiency of test scripts - removing top 3 extra lines from expected output files 

Description:
    This is a subtask from 'HDFFV-7514 - gmqs: h5dump - improve effectiveness of testing scripts for handling expected output files'.
    This is prerequisite for 'HDFFV-7602 tools - upgrade tools' test scripts to share test files among tools, providing framework for test file re-usability.'

Tested:
    jam (linux32-LE), koala (linux64-LE), heiwa (linuxppc64-BE), tejeda (mac32-LE), linew (solaris-BE),  Windows (32-LE), Cmake (jam)
2011-06-02 16:35:03 -05:00
Scot Breitenfeld
bf9a02aab0 [svn-r20923] Description: Added test for permutations of attaching and detaching DS, see bug HDFFV-7605.
Tested: Jam (intel and gnu)
2011-05-29 16:43:41 -05:00
HDF Tester
f4ddc227c2 [svn-r20922] Snapshot version 1.9 release 82 2011-05-29 10:53:27 -05:00
Raymond Lu
ca53d64e46 [svn-r20921] Issue 7579 - The overflowing ENUM values are inconsistent. I took out the error report for 2 H5Tclose calls in H5T_conv_enum because they would clear the error stack.
Tested on jam - simple change.
2011-05-27 16:32:39 -05:00
Raymond Lu
157623de6a [svn-r20913] Issue 7579 - The overflowing ENUM values are inconsistent. When no conversion is involved in reading or writing the
data, overflowing values are retained.  When conversion happens, the values become -1. The conversion function puts -1
when overflow happens.  I added two new dataset transfer property to control whether to fill 0xff in the destination 
data or convert to the destination data when overflow happens.  The two new functions are H5Pset(get)_enum_conv_overflow.  I also added test cases in enum.c and dtypes.c.

Tested on jam, koala, and heiwa.
2011-05-27 12:30:24 -05:00
Allen Byrne
de14c332f9 [svn-r20903] Removed PACKED_BITS check to match removal of option in code 2011-05-25 10:17:53 -05:00
Allen Byrne
843879de95 [svn-r20902] Re-ordered routines to match order of 1.8 for easier comparision of code changes 2011-05-25 10:16:01 -05:00
Allen Byrne
1fd585a5a0 [svn-r20901] Remove if() that avoided a copy of the last element to itself. This was to solve an issue of a difference between 1.8 and trunk. 1.8 used the wrong variable in the if() and upon reflection the if() was determined to be unnecessary and complicated the logic. 1.8 tests would have treated the if() as always true with the result being the same as though the if() was absent.
This change reflects that equivalance and will avoid a potential problem in 1.8.

Tested: local linux, reviewed by two developers
2011-05-25 10:14:06 -05:00
Allen Byrne
e082497e0c [svn-r20900] Remove duplicated function lines 2011-05-25 09:58:39 -05:00
Allen Byrne
243cf5ce6c [svn-r20899] Add soversion handling - use config/lt_vers.am file.
Bring r20895 from 1.8
2011-05-25 09:57:16 -05:00
Raymond Lu
48dc2ca2fc [svn-r20896] I added some comments.
Tested on jam - very simple change.
2011-05-24 16:51:41 -05:00
Dana Robinson
0418b80b22 [svn-r20894] Purpose:
Updates configure definitions on linux.

Description:
    We only define _POSIX_SOURCE which only allows for the lowest level of
POSIX support (IEEE 1003.1), which is unsuitable for clock_gettime() functionality on modern linux systems. This symbol was changed to _POSIX_C_SOURCE=199506L, which gives the 1996 version of POSIX support and allows clock_gettime() and CLOCK_MONOTONIC to be found.

    We do not link to the BSD compatibility library, even though we define
_BSD_SOURCE.  -lbsd-compat was added to AM_LDFLAGS in accordance with GNU
recommendations.  This does not appear to change anything in the current
library.

Tested on:
    jam(linux32-LE) This was tested with various configure settings including
parallel, threadsafe, C++ and FORTRAN enabled.
2011-05-23 14:15:39 -05:00
Dana Robinson
3cfec078a6 [svn-r20892] Purpose:
Adds a check for the mach_time header file.

Description:
    The mach time header file is needed for the monotonic timer / log VFD
work.  No code in the trunk depends on this file at this time.

Tested on:
    jam(linux32-LE) tejeda(darwin32 LE) local darwin64 (snow leopard)
2011-05-23 12:29:14 -05:00
Dana Robinson
c8d91d05a4 [svn-r20891] Purpose:
Changes configure/configure.in to use the same CFLAGS and CPPFLAGS in the configure tests that are used to build the library.

Description:
    Only AM_CPPFLAGS was propagated to the CPPFLAGS that were used to build the library.  Now H5_CPPFLAGS, AM_CFLAGS and H5_CFLAGS are used in the configure tests.

Tested on:
    jam(linux32-LE) freedom(bsd63-LE) linew(solaris-BE) tejeda(darwin32 LE)
2011-05-23 12:02:39 -05:00
HDF Tester
0db46e50d2 [svn-r20890] Snapshot version 1.9 release 82 2011-05-23 09:19:26 -05:00
Quincey Koziol
1d9307bd91 [svn-r20885] Description:
Bring Coverity changes back to trunk:

r20340:
Address coverity issue 248.  Set freed pointer to NULL if realloc fails, then check for NULL before attempting to close array of handles stored in memory addressed by pointer.

r20341:
Added initialization udata.path = NULL; udata.visited = NULL; to prevent unintialized use after done: label.

Do C-style initialization AFTER all variable definitions.

r20342:
Fix valgrind error in H5DUMP-XML-tall.h5

r20343:
Addressed Coverity issue 591 - check the return value of H5S_SELECT_ITER_RELEASE.

r20344:
Added initialization udata.path = NULL; to prevent uninitialized used after done: label

r20347:
Purpose: Fix memory leak in overhead.c

Description: Fixed leak in test() in overhead.c where array "had" was never
freed.


r20348:
Addressed Coverity issue 705 - added prototype for resize_configs_are_equal to cache_common.h.

r20392:
Purpose: Fix coverity issues 1677 and 1678

Description: Added return value checking for 4 funciton calls in
H5tools_get_symlink_info().

r20394:
Purpose: Fix coverity issues 1674 and 1675

Description: Modified H5Epush2() and H5E_printf_stack() to keep track of whether
va_start() has been called, and to call va_end() at the end if so.

r20395:
Should be H5MM_calloc(count * sizeof(void*)). Fixed --gh 

r20396:
Should be H5MM_calloc(count * sizeof(void*)). Fixed. --gh

Tested on:
    Mac OSX/32 10.6.7 (amazon) w/debug & production
    (h5committested on branch)
2011-05-20 17:04:03 -05:00
Allen Byrne
3f11adaec2 [svn-r20875] Refactor h5import to close files and free allocations. Also corrected test dependencies so that memcheck would process tests in correct order.
Tested: local linux - confirmed with valgrind
2011-05-20 12:09:57 -05:00
Raymond Lu
2de5fa986f [svn-r20872] Issue 7564 - Allow H5Tcreat to create string type (fixed-length and variable-length). I also added a test case in dtypes.c.
Tested on jam and linew - relatively simple.
2011-05-20 11:14:33 -05:00
Allen Byrne
0982e11bb4 [svn-r20871] Added function to free iter structure allocations.
Tested: local linux - Confirmed by Valgrind
2011-05-20 09:44:55 -05:00
Scot Breitenfeld
f9174cbd9d [svn-r20870] Description: Fixed HDFFV-7605: Dim Scales APIs- error when unattaching dim. sacles
* Fixed Loop counter when removing object in dimension set with more then one scale.
 
Tested: jam (intel and gnu)
2011-05-19 23:32:31 -05:00
Jonathan Kim
a558128a18 [svn-r20869] Purpose:
HDFFV-7514 - GMQS: h5dump - improve effectiveness of testing scripts for handling expected output files

   Description:
       Fist phase for h5dump test update. Next phase will be for h5ls.
       This is prerequisite for 'HDFFV-7602 Tools - upgrade tools' test scripts to share test files among tools, providing framework for test file re-usability.'

   Tested:
       jam (linux32-LE), koala (linux64-LE), heiwa (linuxppc64-BE), linew (solaris-BE),  Windows (32-LE), Cmake (jam)
2011-05-19 17:51:36 -05:00
Allen Byrne
089e81e86a [svn-r20862] Adjust CPack configuration location by removing version extension in folder name. Add install configuration commands if ext libs are packaged with project. 2011-05-18 12:50:32 -05:00
Allen Byrne
976eb5e523 [svn-r20858] Correct state of EXTERNALLY_CONFIGURED when using EXTERNAL_PROJECT for extlibs. 2011-05-18 09:46:50 -05:00
Allen Byrne
aee3e0b047 [svn-r20856] Remove vms_data.hr reference from CMakeLists.txt and update MANIFEST 2011-05-18 08:50:22 -05:00
Raymond Lu
89339ccce5 [svn-r20855] Issue 2599 - Cross-test BE and LE data. I added the test for deflate, szip, fletcher32, shuffle and nbit filter in cross_read.c and gen_cross.c. I also took out the data file for OpenVMS because the data isn't native but IEEE for floating-numbers.
Tested on koala, heiwa, and jam.
2011-05-17 17:18:05 -05:00
Allen Byrne
29411fc602 [svn-r20853] Add CPack commands to properly include extlibs into package
Tested: windows
2011-05-17 17:12:36 -05:00
Allen Byrne
05ee8cdd18 [svn-r20808] Add Using_CMake.txt file to docs and install 2011-05-13 09:11:52 -05:00
Allen Byrne
7f9ac481b7 [svn-r20805] references files for jam tests are in sub-folder - testfiles 2011-05-13 08:47:17 -05:00
Jonathan Kim
b9e1b158a5 [svn-r20804] Purpose:
HDFFV-7594 - GMQS: h5dump shows wrong values for H5T_STD_I8LE dataset on Blue-gene (ppc64 linux BE +@)

Description:
    For the STD_I8 type data, updated to use 'signed char' instead of 'char' as
    'char' could be defined as unsigned on a certain system, which cause h5dump
    to display data values incorrectly.

Tested:
    jam (linux32-LE), koala (linux64-LE), heiwa (linuxppc64-BE), tejeda (mac32-LE)
2011-05-12 18:12:29 -05:00
Allen Byrne
f100b8f6bb [svn-r20800] Correct install path variable 2011-05-12 11:42:05 -05:00
Allen Byrne
00716a21ab [svn-r20798] Correct name of testfile for help tests 2011-05-12 08:18:58 -05:00
Allen Byrne
f49dc38127 [svn-r20796] Refactor print_data_region_blocks/ponts to remove context passed in by value. The print functions needed an independent context with only the current indent level passed in by argument. Also synched the two routines logical flows and corrected a mem_space error.
h5dump (and h5stat, h5jam/h5unjam) help option not tested. Added h5dump parse_command_line technique for h5stat and h5jam/unjam and created testfiles.

Tested: local linux
2011-05-11 15:45:54 -05:00
Allen Byrne
2e7d42a43c [svn-r20785] Revert region data print changes from yesterday 2011-05-10 08:52:13 -05:00
Allen Byrne
3b06122b50 [svn-r20783] correct type of variable for blkindx 2011-05-09 16:50:35 -05:00
Allen Byrne
8e30e8da7b [svn-r20781] correct typo of variable types for indx and jndx 2011-05-09 16:36:34 -05:00
Allen Byrne
67d4653d0f [svn-r20779] Corrected export test to use parameters passed in from ADD_TEST()
Tested: local linux
2011-05-09 16:09:29 -05:00
Raymond Lu
4e5f011c34 [svn-r20778] I changed the macro definition of DIR_SEPC and DIR_SEPS to ']' for OpenVMS in H5private.h. Strictly speaking, it isn't
the directory seperator.  But it can simplify the code and make the H5L_build_name function cleaner.  

Tested on jam because the change only affects VMS.
2011-05-09 15:31:43 -05:00
Allen Byrne
a4c349dece [svn-r20776] Refactor print_data_region_blocks/ponts to remove context passed in by value. The print functions needed an independent context with only the current indent level passed in by argument. Also synched the two routines logical flows and corrected a mem_space error.
Tested: local linux
2011-05-09 14:56:32 -05:00
Albert Cheng
e21992e08f [svn-r20771] Problem:
Test scripts sometimes need to filter some system-specific messages from the
actual output so that it can match the correct expected output. These filtering
functions, ususally called "STDOUT_FILTER()" and "STDERR_FILTER()" were being
repeated in individual test scripts.  This becomes a maintenance problem and
is error prone.

Solution:
Extract the two filter functions code to bin/output_filter.sh and then each
test script sources it in. This allows reuse of coding and is much easier to
maintain and to add new filtering.

Tested:
LLNL Zeus (linux64 cluster) and Dawndev (Blue-Gene cluster), both for serial
mode only.
 --This line, and those below, will be ignored--

M    tools/misc/testh5mkgrp.sh
M    tools/h5dump/testh5dump.sh.in
M    tools/h5diff/testh5diff.sh
M    tools/h5copy/testh5copy.sh
M    tools/h5ls/testh5ls.sh.in
M    MANIFEST
A    bin/output_filter.sh
2011-05-07 12:05:56 -05:00
Jonathan Kim
054ca47350 [svn-r20767] Purpose:
HDFFV-5928 - GMQS: h5diff problem and improvement on comparsing the same objects

Description:
    Improved performance by eliminating duplicated action for getting object 
    information in half from the previous fixe when comparing group vs group. 
    This is addition to the previous commit r20676.

Tested:
  jam (linux32-LE), koala (linux64-LE), heiwa (linuxppc64-BE), tejeda (mac32-LE), linew (solaris-BE), cmake
2011-05-06 17:02:24 -05:00
Raymond Lu
03cc051dde [svn-r20759] I commented out 2 test cases using H5_CANNOT_OPEN_TWICE because they open the same file twice. OpenVMS
can't do it.  Other systems still run them. 

Tested on jam. Simple change.
2011-05-05 15:09:49 -05:00
Dana Robinson
42301788bc [svn-r20756] Fixes h5diff test failure on BE systems.
H5Dread() produces different output for invalid enum type values
on systems of different endiannes.  On BE systems, it always
emits -1.  On LE systems, the invalid value is emitted.  This
difference caused the h5diff test to fail since it tests to
see if different invalid enum values are reported as different
(on BE systems they are not since -1 == -1).

This fix removes the "invalid vs. different invalid" test
while we fix the library bug.  HDFFV-7527 will be reopened
in JIRA until the full test passes.

Tested on: jam, heiwa, linew
2011-05-05 14:37:19 -05:00
Allen Byrne
84effee753 [svn-r20752] Windows does not have snprintf, need to use HDsnprintf. 2011-05-05 10:56:36 -05:00
Allen Byrne
c5e00232ff [svn-r20725] Add overlooked changes from part of coverity fix
Tested: local linux
2011-05-04 15:28:09 -05:00
Dana Robinson
fbba9db955 [svn-r20724] Purpose: Fix a bug in h5diff when enum values are compared that do
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)
2011-05-04 15:07:38 -05:00
Allen Byrne
2ee1841187 [svn-r20721] Refactor ext lib import and install.
Add SOVERSION for linux
2011-05-04 14:42:03 -05:00
Larry Knox
a2f52de36d [svn-r20720] Merge changes to scripts to run installed examples from hdf5 1.8.7 r 20718. The
default prefix in the scripts will use the compile scripts in the installed bin
 directory.
2011-05-04 11:43:16 -05:00