| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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)
|
|
|
|
|
|
|
|
|
| |
double with GCC 4.8. A user in the forum found the
problem in H5detect.c (uninitialized buffer) and provided the solution. I checked it in yesterday. But it
turned out to be a false fix. So I'm back it out.
Tested on jam - reverse of changes.
|
| |
|
|
|
|
|
|
|
|
| |
double with GCC 4.8. A user in the forum found the
problem in H5detect.c (uninitialized buffer) and provided the solution. Allen and I verified it.
Tested with h5committest and CMake on jam, koala, ostrich, playtus, and Allen's virtual machine.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Count).
This is needed to avoid a conflict with the Read Context object to be
introduced in another Project.
No impact on users, since this is a private module, and low for
developers as the RC module use is very sparse in the library.
Tested with h5committest.
|
| |
|
|
|
|
|
|
| |
from -noshlib to -shlib when hdf5 is built without static libraries (--disable-static).
Tested with h5committest on jam, koala, ostrich and platypus.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
--boot-block and --superblock.
Tested: local linux
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Reverted code change and changed default to 0 from 1024. Changed limit test to use h5dump to compare repacked file instead of h5diff.
Corrected test scripts for test folder path
Tested: h5committest and local linux with CMake
|
|
|
|
|
|
| |
Reverted code change and changed default to 0 from 1024. Changed limit test to use h5dump to compare repacked file instead of h5diff.
Tested local linux with CMake
|
|
|
|
|
|
| |
will generate correct H5Tinit.c file.
Tested: local gcc 4.8.2 linux
|
|
|
|
|
|
|
|
| |
Clean up more compiler warnings.
Tested on:
Mac OSX/64 10.8.4 (amazon) w/gcc, C++ & FORTRAN
(too minor to require h5committest)
|
|
|
|
|
|
| |
developers.
Tested: local linux
|
|\
| |
| |
| |
| | |
This is the fix for "switch missing default case" warnings in src.
Mohamad & Vailin worked on this patch.
|
| |
| |
| |
| |
| |
| |
| | |
Rename static functions in H5Adense.c, H5B2cache.c, H5Bcache.c
Switch these functions to use FUNC_ENTER_STATIC* macros
Tested: koala
|
| | |
|
| |
| |
| |
| |
| |
| | |
script.
Tested: local linux
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Clean up compiler warnings
Tested on:
Mac OSX/64 10.8.4 (amazon) w/C++ & FORTRAN
(too minor to require h5committest)
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The new gcc v4.8.* compilers does not work well with dt_arith which failed
in production mode.
Solution:
A temporary patch by removing any optimization (-O*) from the PROD_CFLAGS
so that dt_arith will pass for now. A more through investigation is needed.
Tested: Jam and Koala where it failed and now passes.
|
| |
| |
| |
| |
| |
| |
| | |
bug fix HDFFV-1238:
Fix a problem when using opt_arg that is NULL for the new options added: -l N, -m N, -a N.
h5committested.
|
| |
| |
| |
| |
| |
| |
| |
| | |
Clean up a few warnings
Tested on:
Mac OSX 10.8.4 (amazon) w/C++ & FORTRAN
(too minor to require h5committest)
|
| |
| |
| |
| | |
Tested: cygwin
|
| | |
|
| |
| |
| |
| |
| |
| | |
Also remove GetConsoleScreenInfo check from cygwin configure.
Tested: h5committest
|
| |
| |
| |
| | |
Updated CDash site name in cmake script.
|
| |
| |
| |
| |
| |
| | |
correct math of adding a NULL to a string.
Tested: local linux (valgrind) and h5committest
|
| | |
|
| |
| |
| |
| |
| |
| | |
New test added, scripts updated, help text updated.
Tested: local linux
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
Make compiler happier and issue less warnings.
Tested on:
Mac OSX/64 10.8.4 (amazon) w/gcc 4.8
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Tested: jam, koala, ostrich (h5committest)
Log from r23939:
Fix unused variables warnings in trunk
Rename static functions in H5Adense.c, H5B2cache.c, H5Bcache.c
Switch these functions to use FUNC_ENTER_STATIC* macros
Tested: koala
|
|\ \
| |/ |
|
| |
| |
| |
| | |
H5HF_sect_single_dblock_info prototype
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
test directory in files:
dt_arith.c
filter_fail.c
big.c
dynlib2.c
dynlib3.c
Also an HDassert(0 && "Unknown type"); statement was added to all 29 combinations of
case OTHER:
default:
See crucible review HDF5-145 for more details.
|
| | |
|
| |
| |
| |
| |
| |
| | |
Remove unused param (src) and unused var (hl) warnings
Tested on koala
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Changes applied to eliminate several compiler warnings in the test directory in
files:
dt_arith.c
filter_fail.c
big.c
dynlib2.c
dynlib3.c
Also an HDassert(0 && "Unknown type"); statement was added to all 29 combination
s of
case OTHER:
default:
See crucible review HDF5-145 for more details.
Tested with h5committest.
|
| | |
|
| | |
|
| | |
|