| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
check code warrior changes at HDF5 branch
Description:
1. add a maro at H5config.h and H5pubconf.h to maintain the correct printf
output in code warrior.
2. update code warrior project changes.
Solution:
Platforms tested:
tested by pedro on windows ?
|
|
|
|
|
|
|
|
| |
Add regression tests
Description:
Added regression test for using a VL-datatype in two separate files.
Platforms tested:
FreeBSD 4.5 (sleipnir)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug fixes
Description:
This checkin addresses two separate bugs:
#1 - When a compound datatype with a VL datatype as a field is used
to read or write data, the datatype conversion paths added to
the table of paths were treating the VL datatypes somewhat too
genericly - they weren't dicriminating between VL datatypes used
in different files and therefore there was the possibility that
a path in the table having information for one file could get used
with a second file, causing errors or core dumps (if the first
file was closed before the later file used the path).
#2 - When composite (compound, array or VL) datatype paths in the datatype
conversion table are being tested to see if they can be used for
the current datatypes being converted, they can cause additional
conversion paths to be registered in the conversion path table.
Since this causes the global table to change size and entries to
move around, it is possible that the local variables tracking a
potential path could become incorrect as the global table was
changed out from underneath them.
Both bugs fixed are described in bug #703
Solution:
Two separate fixes:
#1 - Changed H5T_cmp to differentiate between VL datatypes in different
files and not to return datatypes in two different files as equal.
#2 - Note size of global table before evaluating datatype paths. Then,
after the appropriate path has been chosen, check if the size of
the global table has changed and recompute the position of the path
chosen in the table if necessary.
Platforms tested:
FreeBSD 4.5 (sleipnir)
|
|
|
|
|
|
|
|
| |
Code cleanup
Description:
Added some more assertions to help debug in the future...
Platforms tested:
FreeBSD 4.5 (sleipnir)
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Maintenance
Description:
Several compiler's warnings caused Hitachi SR8000
F90 and C compilers to fail.
Solution:
Fixed the code to eliminate warnings.
Platforms tested:
O2K (modi4) and Solaris 2.7 (arabica)
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
More optimizing for byte order conversion. Mostly just making code
easier to follow by simplifying Duff's device coding of the loops.
I also split the conversion function into two functions with different
names so output from H5T debugging indicates whether the optimized or
unoptimized case was invoked.
2002-01-25 10:48:34 Robb Matzke <matzke@arborea.spizella.com>
*: Added prototype for H5T_conv_order_opt().
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
More optimizing for byte order conversion. Mostly just making code
easier to follow by simplifying Duff's device coding of the loops.
I also split the conversion function into two functions with different
names so output from H5T debugging indicates whether the optimized or
unoptimized case was invoked.
2002-01-25 10:48:54 Robb Matzke <matzke@arborea.spizella.com>
* H5T_conv_order: Removed Duff's device consisting of >500 lines of code. Unrolled
two loops by hand resulting in <300 lines of code
which is a few percent faster and far easier to read.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
More optimizing for byte order conversion. Mostly just making code
easier to follow by simplifying Duff's device coding of the loops.
I also split the conversion function into two functions with different
names so output from H5T debugging indicates whether the optimized or
unoptimized case was invoked.
2002-01-25 10:47:13 Robb Matzke <matzke@arborea.spizella.com>
* H5T_init_interface: Registered conversion function H5T_conv_order_opt() under two
names. H5T debugging will report the conversion
function as either "ibo(opt)" or "fbo(opt)".
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Add "Release 1.5" section.
Solution:
Added outline for Release 1.5 API changes.
And corrected a spelling error.
Platforms tested:
IE 5
|
|
|
|
| |
Update manifest with new test file.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug Fix
Description:
When file space was returned to the file space free-list for reuse,
occasionally raw data allocations which used space from the free-list
would overlap with the metadata accumulator and get over-written with
the cached information in the accumulator, corrupting the data.
Solution:
Check if the space about to be recycled on the free-list is going to be
used for raw data and also overlaps with the metadata accumulator cache,
avoiding using space that fits those criteria.
This fixes bug #701
Platforms tested:
FreeBSD 4.5 (sleipnir)
|
|
|
|
|
|
|
|
| |
Purpose:
Fix
Description:
Needed to use a different autoconf like I did at a previous time for
generating the configure file.
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Fix fix
Description:
Moved Binh-Minh's fix of the lack of support for C++ in libtool by
adding the DEFAULT_LIBS macro into the configure.in and
config/commence.in files.
Solution:
Put it into the configure.in and commence.in files. Reran autoconf.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Improvement
Description:
Using LIBS for c++ default library resulted in the library showed up
in two many places.
Solution:
Added new macro DEFAULT_LIBS to specify default library for the
C++ API tests
Platforms tested:
IRIX64 (modi4)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Added platform support
Description:
The current libtool doesn't fully support C++ yet, so until it
does, the default libraries have to be manually specify.
Some minor cleanups needed.
Solution:
Added DEFAULT_LIBS=@DEFAULT_LIBS@ to use new macro for default library.
Added $(DEFAULT_LIBS) to link lines.
Removed inappropriate comments.
Platforms tested:
IRIX64 (modi4)
I'll check out again to make sure I haven't broken anything else.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Added platform support
Description:
The current libtool doesn't fully support C++ yet, so until it
does, the default libraries have to be manually specify.
Some minor cleanups needed.
Solution:
Added DEFAULT_LIBS=@DEFAULT_LIBS@ to use new macro for default library
Added $(DEFAULT_LIBS) to link lines
Removed inappropriate comments, and this extra set, it occurs twice:
## These are our main targets. They should be listed in the order to be
## executed, generally most specific tests to least specific tests.
RUNTEST=$(LT_RUN)
although I'm not sure if it even should be there, so I should leave
it for Bill.
Platforms tested:
IRIX64 (modi4)
I'll check out again to make sure I haven't broken anything else.
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Added platform support
Description:
Added new macro DEFAULT_LIBS to specify default library for the
C++ API tests
Platforms tested:
IRIX 6.5 (modi4)
|
|
|
|
|
|
|
| |
Purpose:
Updated MANIFEST
Description:
Added file c++/config/irix6.x for modi4
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Adding new supported platform
Description:
Added this config file for modi4. Note that, when libtool v1.5
is released and we start using it, the default library libCio can
and should be removed from LIBS. It is there now because previous
versions of libtool do not fully support c++ yet and default
libraries must be manually added to the link line.
Platforms tested:
IRIX64 6.5 (modi4)
|
|
|
|
|
|
|
|
|
|
|
|
| |
Feature improvement
Description:
Re-write how the free-list headers were used, to reduce the amount of space
added to each malloc request. Reduced header for array and block free
list items from 24 bytes to 8 bytes and eliminated the header for fixed-size
free list items entirely. This should reduce the amount of memory that the
library uses.
Platforms tested:
FreeBSD 4.5 (sleipnir) & IRIX64 6.5 (modi4)
|
|
|
|
|
|
|
|
|
|
| |
Code improvement
Description:
Take advantage of the "MALLOC_WORKS" flag to not force malloced memory
pieces to be at least 1 byte when the malloc implementation allows
malloc(0) to work.
Platforms tested:
FreeBSD 4.5 (sleipnir) & Cray SV1 (killeen)
|
|
|
|
|
|
|
|
|
|
| |
New feature check
Description:
Added check to determine if malloc(0) returns a valid pointer for a
particular architecture.
Platforms tested:
FreeBSD 4.5 (sleipnir) where malloc(0) works
Cray SV1 (killeen) where malloc(0) doesn't work
|
|
|
|
|
|
|
|
|
|
| |
Bug fix
Description:
Missed CVS conflict in ifdef'd code
Solution:
Editted to remove CVS conflicted code.
Platforms tested:
FreeBSD 4.5 (sleipnir)
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bugfix. Copy edits and clarification.
Description:
Added IN/OUT designations to parameters.
Copy edits and clarification.
Removed () from public H5 functions in running text.
Platforms tested:
IE 5
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bugfix. Copy edits.
Description:
Added IN/OUT designations to parameters.
Copy edits. Removed () from public H5 functions in running text.
Platforms tested:
IE 5
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bugfix.
Description:
H5Sget_simple_extent_npoints: Corrected return type to hssize_t.
Fixes bug #640 in development branch.
Platforms tested:
IE 5
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bugfix. Clarification. Copy edits and minor formatting.
Description:
H5Gunlink: Corrected loc_id parameter to be 'file or group'
rather than just 'file'.
Added IN/OUT designations to parameters.
Minor formatting. Removed () from public H5 functions in running text.
Platforms tested:
IE 5
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bugfix.
Description:
h5pset/get_fapl_mpio_f & h5pset/get_dxpl_mpio_f:
Corrected function names (mpi --> mpio)
Platforms tested:
IE 5
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bugfix. Clarification & extension. Removed old function.
Description:
h5pset/get_fapl_mpio_f & h5pset/get_dxpl_mpio_f:
Corrected function names (mpi --> mpio)
H5Pset_fapl_dpss:
Removed function from RM.
H5Pget_driver:
Corrected return type.
Added list of valid driver identifiers and note re custom drivers.
H5Pcreate:
Added note, per user complaint, that the property must eventually
be closed with H5Pclose, else errors are likely to occur.
Platforms tested:
IE 5
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bugfix and clarification.
Description:
H5Aclose -- Changed statement regarding future use of a released
attribute identifier.
H5Acreate -- Corrected create_plist_id reference and listed H5P_DEFAULT
as only currently-accepted value.
Platforms tested:
IE 5
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
update testhdf5dll.dsp for release dll settings
Description:
previous testhdf5dll failed.
Solution:
Platforms tested:
windows 98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
a bug fix for release dll
Description:
change the project setting for release dll from single-threaded to
multithreaded DLL seems to make all tests pass.
For DLL testing, it should be set to multithreaded DLL anyway.
Solution:
Platforms tested:
windows 2000
[machines you have tested the changed version. This is absolute
important. Test it out on at least two or three different platforms
such as Big-endian-32bit (SUN/IRIX), little-endian-32(LINUX) and
64-bit (IRIX64/UNICOS/DEC-ALPHA) would be good.]
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Small Fix
Description:
Fixed the Min/Max/Average accumlation stuff...
Solution:
Actually thought about the code and made it accumulate the
information in the correct way.
Platforms tested:
Linux
|
|
|
|
|
|
|
| |
The code was doing too many MPI_Send for the gathering.
Changed the get_minmax() to use the MPI_Allreduce routine.
Platforms tested:
modi4
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Feature Fix
Description:
Added timer from open to close for write. Changed reporting of time
for write and read to only measure the actual read/write instead of
the open/close as well...
Platforms tested:
Linux(pp)
|
|
|
|
|
|
|
|
|
|
|
| |
Bug fix
Description:
HDfprintf was not handling Microsoft's "%I64d" extension to printf for
printing thier '__int64' type.
Solution:
Added code to properly detect and use this extension.
Platforms tested:
None! (Kent will be testing shortly)
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Feature Add
Description:
Added the time it takes from opening the file to closing the file
(over all files).
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Feature Fix
Description:
Instead of outputting the # of kilobytes in a transfer size, output
the total bytes. This helps when you're interested in just doing a
copy-paste type of thing for the transfer buffer size.
Solution:
Changed the output report
Platforms tested:
Linux(pp)
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bug fix
Description:
In several if statements, '>' was used in place of '<='. This
caused exceptions thrown inappropriately. Replaced '>' by '<='
where applicable.
Platforms tested:
SunOS 5.7 (arabica)
Windows 98
|
|
|
|
|
|
|
|
|
| |
Purpose:
H5FDdpss test no longer exists, modify project file at 1.5 branch.
Description:
Solution:
Platforms tested:
windows 2000
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Feature Add
Description:
Added output which tells how large the file is (that is, the number
of dsets * number of elements in a dset * sizeof(int)).
Platforms tested:
Linux(pp)
|
|
|
|
|
|
|
|
| |
The default minimum xfer size of 1K was way too slow for
parallel file system like the PFS of Tflops. Set it to
128K to complete soon for default settings.
Platforms tested:
Tflops, modi4.
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
somehow the updated H5pubconf.h and H5config.h missed from all zip files.
recheck in.
Description:
Solution:
Platforms tested:
windows 2000
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug fix
Description:
The value of H5_SIZEOF_SSIZE_T is not being updated to reflect the correct
size of the ssize_t typedef, if we have to define it ourselves.
Solution:
Undef H5_SIZEOF_SSIZE_T at beginning of block where we define the typedef
for ssize_t and then re-define it to the correct size when we've chosen a
size.
Platforms tested:
FreeBSD 4.5 (sleipnir)
|
|
|
|
|
|
|
|
|
|
| |
Extension
Description:
int64_t type is not available everywhere.
#include H5private.h which has platform dependent hooks
to define int64_t to something available.
Platforms tested:
Tflops
|
|
|
|
|
|
|
|
|
|
|
| |
Bug fix.
Description:
__int64 is not a legal type in Tflops.
Set its sizeof to 0 in the cached values.
This is a temporary fix since the real problem is
in configure which hardset __int64 to 8 for cross-compiling cases.
Platforms tested:
Tflops
|
|
|
|
|
|
|
|
| |
Document bug
Description:
Mention that h5dump and h5ls are not displaying variable-length string
datatype information correctly. I've also entered a more detailed bug into
the bug database.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Small Fix
Description:
Changed how the list of drivers were listed from:
{ "driver1",
"driver2",
/* ... */
"drivern",
#ifdef FOO
"driverm"
#endif
};
to
{ "driver1"
,"driver2"
/* ... */
,"drivern"
#ifdef FOO
,"driverm"
#endif
};
since it's a nicer way of doing the same thing without the annoying
warning of an extraneous , if FOO isn't defined.
Platforms tested:
Dangermouse, Modi4, Kelgia
|