| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change h5diff_exit() routine to always return a zero exit code when running
in parallel, since returning a non-zero exit code can cause MPI implementations
to print output and that can throw off our "expected output" comparisons.
Note that this change only changes the exit code for situations where an
incorrect command-line parameter was given - ph5diff was already returning a
zero exit code for all "normal" executions, including those where a serial
h5diff would return a non-zero exit code.
Tested on:
Mac OSX/64 10.8.3 (amazon) w/parallel
(Too minor to require h5committest)
|
|
|
|
| |
tested on Jam, to minor for committest
|
| |
|
| |
|
|
|
|
| |
tested on Jam - too minor for committest
|
| |
|
|
|
|
|
|
|
|
| |
A "return" was missing from a non-void function. Fixed.
Platforms Tested:
Linux/32 2.6 (jam)
Linux/ppc64 (ostrich)
Asked Allen check Windows.
|
|
|
|
| |
Tested: jam
|
| |
|
|
|
|
|
|
|
|
|
| |
clang++.
Reversed it to use gcc/g++ as the default C and CXX compilers for Darwin 10.X
systems.
Tested: duck (darwin 11), wren (darwin 12) and fred (darwin 10).
|
|
|
|
|
|
|
|
|
| |
Description:
Added wrappers for H5Aexists.
Platforms Tested:
Linux/32 2.6 (jam)
Linux/64 2.6 (koala)
Linux/ppc64 (ostrich)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use Clang as the compiler for the Mac platform.
Changed all Mac to use clang/clang++ as the default C and CXX compilers if
not provided by the user. Still use gfortran as the default fortran
compiler since Apple does not provide any fortran compiler.
Tested: passed all three macs OS, snow leapord (fred), lion (duck) and
mountain lion (wren). Also passed test using Apple gcc, Gnu gcc, Intel
as the C compilers and all corresponding CXX compilers fits in.
Did not run h5committest or other non-mac platform since this change
affect Apple platforms only.
|
|
|
|
|
|
|
| |
Compile scripts will add paths for external libraries linked to executable files that they create.
Eliminates need for LD_LIBRARY_PATH for szip in locations not known to ld.
Teted on platypus and emu (64bit) in addition to h5committest.
|
|
|
|
|
|
|
|
| |
Changes default value for USE_SHARED_LIB in compile scripts to yes when static is disabled.
Comments in scripts about the order of flag variables containing library paths was also corrected.
Tested with h5committest on duck, jam, koala, and ostrich in addition to specific testing for the
compile script change.
|
|
|
|
|
|
| |
Fixed bug HDFFV-4272. Just typos in documentation.
Platform tested:
Jam
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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)
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
being read
during variable-length type conversion. All variable-length type conversion
buffers are now initialized to contain 0s on allocation.
Tested on: jam(w/ fortran, C++, parallel, parallel fortran)
ostrich
This is a very small change (basially changing malloc to calloc) and
is unlikely to cause issues on other platforms.
|
|
|
|
|
|
|
| |
Avoid using fpclassify() for detecting zero/non-zero values.
Tested on:
Mac OSX/64 10.8.3 (amazon) w/debug
|
| |
|
|
|
|
| |
Tested: local linux
|
|
|
|
|
|
|
|
|
|
|
| |
reference to __h5r_MOD_h5rget_region_region_f
This turned out being an issue with configure. The reporter submitted a patch which fixed the fact that we should not be
setting AM_FCFLAGS (an automake variable) with FFLAGS (a user variable).
I removed this, and we now only set FFLAGS if the environment variable is set, otherwise we don't.
Tested: jam (gnu)
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
HDFFV-8214 - h5repack failed converting small chunked dataset (size < 1K) to contiguous layout.
Description:
h5repack failed converting small chunked dataset (size < 1K) to contiguous layout.
The first case was when chunk dim is bigger than the dataset dim (at leat one), h5repack failed with displaying error stacks.
The other case is when chunk dim is smaller than the dataset dim, h5repack failed to change layout.
Tested:
jam (linux32-LE), koala (linux64-LE), ostrich (linuxppc64-BE), emu (solaris-BE),fred (mac64-LE), Windows (32-LE cmake), cmake (jam)
|
|
|
|
|
|
|
|
|
|
|
|
| |
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)
|
|
|
|
| |
Tested: local linux
|
|
|
|
|
|
|
|
| |
and .f08 as source files
added missing extensions.
Tested: jam (gnu)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
from the
library. The intended functionality for them was never fully implemented
and they are fundamentally broken.
The functions were removed from the C and C++ interfaces. They were not
exported in the Fortran interface.
Tested on: jam(c++/fortran/hl)
I tested using check-vfd to ensure the multi VFD still works properly.
There is no reason to test this on other machines or on parallel.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
comment.
Tested: no need since only comments are changed.
|
| |
|
|
|
|
|
|
| |
excluding and running long running tests separately.
Tested: local linux
|
|
|
|
|
|
|
|
| |
Some systems, like Mac, strings inspect library files and older versions
of strings maybe know newer library format, resulting in errors.
Make it read the file as stdin, avoiding this problem.
Tested: h5committest. all passed.
|
| |
|
|
|
|
| |
options using only.
|
|
|
|
|
|
|
|
|
|
|
| |
------------
Fix for:
HDFFV-8149: h5pset_external_f API - the offset is declared integer which can cause problem if use 64-bit integer
Added new type off_t, modified source files and added a new test.
Tested (jam, intel, gnu)
|
| |
|
|
|
|
| |
Tested: local linux
|
|
|
|
| |
Tested: local linux
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bring changes from Coverity branch to the trunk:
r20612:
Changed string functions to versions with string length to fix coverity issues
922, 942 and 943.
r20614:
Use HDsnprintf. --gh
r20675:
Fix for coverity #1714 due to the fix for #810.
Use HDfree() instead of H5MM_xfree().
r20678:
Repaired coverity issue #598 -- failure to check return value from a
call to fstat(), or to tidy up in the event of failure.
r20679:
Use HDstrncpy. --gh
r20681:
Use HDstrncat and HDstrncpy. --gh
Tested on:
Mac OSX/64 10.8.2 (amazon) w/Debug, etc.
(too minor to require h5committest)
|
| |
|
| |
|
|
|
|
| |
Tested: local linux
|
|
|
|
|
|
| |
Remove h5ls references
Tested: local linux
|
|
|
|
| |
Tested: local linux
|