| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
Description:
The default constructor of H5File was missing. It's now added.
Platforms:
SunOS 5.7 (arabica)
Linux 6.2 (eirene)
|
|
|
|
|
|
|
|
|
| |
Update
Description:
Updated the h5vers script to also modify the configure.in files and
regenerate the configure file.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
| |
Update
Description:
Updated the way some of the macros were being used in the configure
file. Some of the behavior is deprecated. Mostly has to do with the
AC_OUTPUT and AC_INIT macros...
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Upgrade of autotools...
Description:
Upgraded the configure files to use the newest auto tools. The
directions for regenerating the configure files are the same for the
1.5 branch, you should only regenerate them on sleipnir or
dangermouse for now. Eirene has an older version of the autotools on
it. Burrwhite isn't ready yet, but when it is, it should be upgraded
to the latest/greatest tools as well...
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Adding support for dll
Description:
Added __DLLCPP__ to all public classes and templates.
Added #include "H5Include.h" to H5RefCounter.cpp because of the
use of DLLCPP in .h file and it needs the following chain:
H5Include.h/hdf5.h/H5public.h/H5api_adpt.h
Added #pragma warning(disable: 4251) to H5Exception.h to eliminate
this warning on private data members of type 'string.' This
occurs because 'string' is not yet instantiated at compilation
time; however, since the class is exported, the warning is
harmless.
Changed this member function's parameter to be passed as reference.
from:
void CompType::insertMember( const string name,...
to:
void CompType::insertMember( const string& name,...
Platforms tested:
SunOS 5.7 (arabica)
Linux 6.2 (eirene)
FreeBSD (sleipnir)
Windows 2000
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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)
Linux 6.2 (eirene)
Windows 2000
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Cleanup warnings
Description:
Removed unused variables.
Platforms tested:
SunOS 5.7 (arabica)
Linux 6.2 (eirene)
Windows 2000
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Eliminated warnings
Description:
1. H5CommonFG.cpp:
On IRIX and Windows 98, if a function, that is declared to
return a value, throws an exception in "else" statement, the
compiler still complains that the function doesn't return a value.
2. Others:
H5IdComponent::operator= shouldn't be virtual because the subclass'
operator= have different type for the rhs argument.
Solution:
1.
Replaced:
if (something valid)
return a value
else
throw an exception
with:
if (something invalid)
throw an exception
return a value
2. Removed 'virtual'
Platforms tested:
FreeBSD 4.4 (hawkwind)
SunOS 5.7 (arabica)
Linux 6.2 (eirene)
|
|
|
|
|
|
|
|
|
| |
Purpose:
Maintenance
Description:
Added configuration file for SP3 to support C++
Platforms tested:
NERSC SP3
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug fixing
Description:
+ added operator= to PredType
+ removed 'using namespace std' from H5RefCounter.cpp because it's not
used and leaving it there requires #include some standard header file
+ added 'using namespace std' in H5Exception.cpp
Platforms tested:
SunOS 5.7 (arabica)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bug fix
Description:
Add the overloaded member function Attribute::getName to return
the attribute name's length as in C API. This functionality was
missing. Note that the current getName that returns "string"
is not removed, for different way of using getName.
Platforms tested:
SunOS 5.7 (arabica)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Fixing found bug
Description:
A data file cannot be removed because the corresponding H5File object
is still in existence, which means the data file is still opened.
Solution:
Moved h5_cleanup to outside of the try block so that the H5File object
will go out of scope and be destroyed before h5_cleanup attempts to
remove the corresponding data file.
Platforms tested:
SunOS 5.7 (arabica)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Code cleanup (sorta)
Description:
When the first versions of the HDF5 library were designed, I remembered
vividly the difficulties of porting code from a 32-bit platform to a 16-bit
platform and asked that people use intn & uintn instead of int & unsigned
int, respectively. However, in hindsight, this was overkill and
unnecessary since we weren't going to be porting the HDF5 library to
16-bit architectures.
Currently, the extra uintn & intn typedefs are causing problems for users
who'd like to include both the HDF5 and HDF4 header files in one source
module (like Kent's h4toh5 library).
(Merged from the same changes to development branch)
Solution:
Changed the uintn & intn's to unsigned and int's respectively.
Platforms tested:
FreeBSD 4.4 (hawkwind)
|
|
|
|
|
|
|
| |
Purpose:
Regeneration of Dependencies files
Description:
Regenerated Dependencies files with "make Dependencies"
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Back-port of Bug Fix
Description:
Back port from the 1.5 branch of the .depend/Dependencies files
generation bug. (When the $srcdir et.al. macros began with a ".",
they were lopping off the first character of a header file's path).
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Bug Fix
Description:
We long ago changed the compiler flags to pick out dependencies to
-MM instead of -M so that only #include "filename.h" type headers
will be put into the .depend file.
Solution:
Changed this flag.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Changed due to bug fixed
Description:
The dsets.cpp/test_compression test was disabled because it
failed on eirene - some variable changed to a very large value.
Solution:
The problem was fixed by Bill. So I removed the #if defined (__linux__)
to enable the test_compression again.
Platforms tested:
eirene (linux)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Code Update
Description:
Ported change from the 1.5 branch to the 1.4 branch where all HDF5
include files are in quotes instead of angle brackets:
#include "hdf5_file.h"
instead of
#include <hdf5_file.h>
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
| |
Fix
Description:
Fixed to use the top-level bin/ directory instead of having the files
copied here...
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug Fix
Description:
If we need to specify a -R flag for dynamic libraries (like, in the
case when we specify --with-hdf4), then this flag needs to be added
to the linking line so that it will show up in the generated library
and other programs linking to that library will be able to find the
relevant libraries.
Solution:
Added the DYNAMIC_DIRS macro to the link line.
Platforms tested:
Arabica
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Problem suppressing
Description:
The dsets.cpp/compression (write) test failed on eirene for
1.4 branch. I suspected that the problem had something to do with
the word size (32 vs. 64), but I'm not sure. Elena and Quincey
agreed that we take the test out for now since we don't have
a fix for it at this time.
Solution:
Used #if defined (__linux__) to temporarily disable the
test_compression test for this platform.
Platforms tested:
arabica (solaris 2.7)
eirene (linux 6.2)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Description:
dsets.cpp:
- Added routine's headers to be consistent with the C tests
- Added the following tests (they are not in the C version
test because they are C++ specific):
+ Test copying a user-defined type using DataType::copy
+ Test copying a user-defined type using DataType::operator=
+ Test copying a user-defined int type using DataType::operator=
+ Test copying an integer predefined type using a constructor
+ Test copying an integer predefined type using DataType::operator=
th5s.cpp and tfile.cpp:
- Changed routine's headers to be consistent with the C tests
- Cleanup old couts and statements used during debugging
Platforms tested:
arabica (sparc-sun-solaris 2.7)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Description and Solution:
- Predefined type objects become outdated when the application calls
H5close and then H5open to use the library again.
Solution:
+ Reimplemented PredType using enum type to obtain up-to-date
values for predefined types, i.e. H5T_xxx_yyy's, when they
are used.
+ Added to PredType its own version of getId() to return the
appropriate predefined type id according to the enum value.
+ Made IdComponent::getId() a virtual function
- Since Albert already fixed the "multiply defined RcsId" problem,
removed the now become unnessary "#undef RCSID" from H5Include.h
- Added a check for func_name != NULL in one of the Exception
constructors because seg. fault. might occur in some situations.
- It is more convenient to make a copy of a datatype, dataspace,
or property list by using assignment operator
Solution:
+ Added assignment operator to DataType, DataSpace, and
PropList so that they will invoke H5Tcopy, H5Scopy,
and H5Pcopy, respectively, instead of just simply
copying the data members as for other classes. So,
now a type, space, or property list can be copied by
either :
mytype = original_type, or
mytype.copy(original_type)
Note that copy constructors are still just copying the
data members.
+ Added an assignment operator to DataType that takes a
predefined type.
+ Made IdComponent::operator= a virtual function
Platforms tested:
arabica (sparc-sun-solaris 2.7)
|
|
|
|
|
|
|
|
|
|
|
| |
Improve C++ API test output appearance - 1.4 branch
Description:
The C++ API tests are not easy to be spotted because their output
look similar to part of the C tests.
Solution:
Added "C++ API:" in front of the C++ test titles.
Platforms tested:
arabica (sparc-sun-solaris 2.7)
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Bug Fix
Description:
The test programs weren't compiling correctly. They weren't added to
the TEST_OBJ macro, so make didn't know about them.
Solution:
Added them to the TeST_OBJ macro as in the C tests.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
| |
Update
Description:
Added the th5s test.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
| |
Feature Add
Description:
Added "real" testing to the C++ library.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Adding tests to the C++ API 1.4 branch
Description:
The C++ API in 1.4 branch has no formal testing yet.
Solution:
Added tests for file, dataset, and dataspace interfaces. I'm still
working on other tests.
Platforms tested:
arabica (sparc-sun-solaris 2.7)
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose: Updating as in 1.5 branch
Description:
The C++ API source code was modified in 1.5 branch, mainly to
improve usability in exception handlings. Updated 1.4 C++
source code to match.
Platform:
arabica (sparc-sun-solaris 2.7)
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug Fix
Description:
The RUNSERIAL and RUNPARALLEL macros aren't needed (and aren't set)
for the C++ stuff and were causing errors when people tried to test
the code.
Solution:
Removed them
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
| |
Bug Fix.
Description:
When the user would do a ``make check'' without doing a ``make all''
first, the tests in the examples directory wouldn't build correctly.
Solution:
Added the tests to the TEST_PROGS macro instead of the PROGS macro.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
| |
Bug FIx
Description:
We weren't installing the C++ header files.
Solution:
Added all of the *.h files to the PUB_HDR macro in the Makefile so
that they'd be installed.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Small bug fix
Description:
A stub function has returned type hsize_t but returns -1.
Solution:
Changed returned value from -1 to 0
Platforms tested:
Solaris (arabica)
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixup
Description:
We were specifying a config file for the C++ stuff and it didn't need
one. This would cause problems with compiling the C++ stuff since it
wouldn't get the correct H5config.h file.
Solution:
Removed support for it.
Platforms tested:
Eirene
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose: bug fix
Description:
I found a couple of places where virtual destructors were missing
and could cause small memory leaks. Also, some destructors were
not virtual when they should be.
Solution:
- added virtual destructors, which also free dynamically allocated
memory
- added virtual to several destructors
- also, fixed several typos
Platforms tested:
Solaris 2.7 (arabica)
|
|
|
|
|
|
|
|
|
|
|
| |
Bug Fix?
Description:
Possible that some makes don't like a trailing \ at the end of a
comment.
Solution:
Removed it
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix and improve
Description:
- Put functions that are common to H5File and Group into a
prototype class, CommonFG. I didn't do that before because
of the fear of the consequences of multiple inheritance, since
H5File and Group already inherit from different super classes.
I recently read a C++ book and learned to use MI more safely.
This change reduced some more of code redundancy.
- Added missing const to some function parameters
Platforms tested:
Solaris/CC 5.0 (arabica)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix and improve
Description:
- Put functions that are common to H5File and Group into a
prototype class, CommonFG. I didn't do that before because
of the fear of the consequences of multiple inheritance, since
H5File and Group already inherit from different super classes.
I recently read a C++ book and learned to use MI more safely.
This change reduced some more of code redundancy.
- Added missing const to some function parameters
- Added missing return statements for some functions.
Platforms tested:
Solaris/CC 5.0 (arabica)
|
|
|
|
|
|
|
|
|
|
|
|
| |
Doh!
Description:
When adding the examples directory to the testing, forgot to
remove it from the other scripts in the makefile since it would
then be run twice.
Solution:
Removed it (saved the old code)
Platforms tested:
Hey! Trust me :)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Description:
Dec CC compiler doesn't support some new c++ features.
Some typos caused compilation erros on Dec Alpha.
Solution:
- Added macro BOOL_NOTDEFINED to define bool type on Dec
Alpha (gondolin) since its compiler doesn't support bool.
- Added macro NO_STATIC_CAST to prevent the use of the new
c++ feature static_cast since Dec CC compiler doesn't support.
- Added const to parameters of some functions to match the
functions' declaration and definition. Typos errors and
Dec CC complained.
Platform:
Solaris (arabica). Also tried to build on Dec Alpha but still
not linked due to some missing compiler flags; its compilation
went fine though.
|
|
|
|
|
|
|
| |
Adding Flag
Description:
Testing to see if the C++ compiler can handle the ``static_cast''
keyword.
|
|
|
|
|
|
|
|
|
|
|
| |
Fix
Description:
When not building in the same directory, we needed to tell the
checker wherre to look for files.
Solution:
Put $(srcdir) before things which needed it.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
| |
Add
Description:
Adding a small amount of testing to the C++ library until "real"
testing can be created. (We use the examples).
Platforms tested:
Linux
|
|
|
|
|
|
|
| |
oops
Description:
I somehow checked in some test Makefile.in that I was working on.
Sorry.
|