| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adding tests to the C++ API
Description:
The C++ API has no formal testing yet.
Solution:
Added tests for file and dataset interfaces. I'm still working on
other tests.
Platforms tested:
Linux (gcc version egcs-2.91.66)
I temporarily modified the Makefile on my local Linux machine and these
tests work. I need Bill to help adding them permanently before I can
test on an NCSA machine. I checked the files in now so Bill can do that.
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
| |
Fix again
Description:
Needed to change the .C to .cpp in this depend module.
|
|
|
|
|
|
|
|
| |
Fix
Description:
Hopefully the last fix in the .C -> .cpp suffix change saga.
Forgot to tell the commence/conclude files that the suffix has
changed
|
|
|
|
|
|
|
|
|
|
| |
Fix
Description:
Needed to change the extention to the file configure expects to
be in the source directory when Binh-Minh changed the extensions
for all the files.
Solution:
Did just that.
|
|
|
|
|
|
|
|
|
|
|
| |
Fix
Description:
Windows couldn't handle the .C extension. The filenames had to be
changed to *.cpp instead.
Solution:
Changed in the Makefiles to accomodate said change.
Platforms tested:
Eyeballed.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
Renaming source files for portability
Description:
Removed *.C files after adding the corresponding *.cpp files.
Platforms tested:
|
|
|
|
|
|
|
|
| |
Purpose:
Renaming source files for portability
Description:
Removed *.C files after adding the corresponding *.cpp files.
Platforms tested:
|
|
|
|
|
|
|
|
|
|
|
| |
Purpose:
Support portability
Description:
I forgot that source file extension .C will not work on Windows.
Solution:
Changed all source file from *.C to *.cpp for portability.
Platforms tested:
arabica (sparc-sun-solaris 2.7)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Description:
The prototype of two C API functions, H5Tget_offset and
H5Sget_simple_extent_npoints was changed to fix bug #446,
resulting in the need for updating the two corresponding
C++ API functions.
Solution:
- Changed the return type of AtomType::getOffset from size_t
to int, and the error return value from 0 to -1.
- Changed the return type of DataSpace::getSimpleExtentNpoints
from hsize_t to hssize_t, and the error value from 0 to -1.
Platforms tested:
arabica (sparc-sun-solaris 2.7)
|
|
|
|
|
|
|
| |
New Flag
Description:
Added a test to see if the C++ compiler can handle the bool
datatype. If not, then it defines the BOOL_NOTDEFINED macro.
|
|
|
|
|
|
|
|
|
|
|
| |
Improvement
Description:
Made the Makefiles better.
Solution:
Added standard macros concerning where things are place. Like
libraries and so forth.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
| |
YABF
Description:
Needed to add the source directory for the built HDF5 library.
Solution:
Added -I../../src to the compile flags.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
| |
Refix
Description:
The C++ stuff also needs the main HDF5 headers.
Solution:
Added the flag to do just that.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug fix
Description:
When building in a different directory, we weren't picking up the
header files correctly. The -I flag wasn't pointing to the
correct place.
Solution:
Changed the -I flag to point to the C++ source directory.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reversal of Patch
Description:
I thought that the solaris boxen could handle the -Wc,<flag> flag
(or, rather, that libtool would do the correct thing with their
compilers). However, this isn't the case, apparently.
Solution:
Removed the ``-Wc,-LANG:std'' flag and put it in the CPPFLAGS
macro instead. This should cause it to be passed to the compiler
only...one hopes...
|
|
|
|
|
|
|
|
|
| |
Argh!
Description:
Again the compiler doesn't like the -dlopen self flag when
compiling...
Platforms tested:
Linux
|
|
|
|
|
|
|
| |
Oops.
Description:
Forgot to make the libraries static since we're not certain that
they can be shared...
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug-ish fix
Description:
Spurious warnings would show up if libtool was trying to pass the
-LANG:std flag to the compile line.
Also, if the library is compiled with -all-static, then it would
complain about not finding dlopen and friends.
Solution:
Used the libtool -Wc,<flag> option to pass the -LANG:std flag to
the compile line.
Used ``-dlopen self'' when compiling.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug
Description:
Compiler needed both the -instances=global and the -LANG:std flag
to function.
libtools spits out a warning about not being able to find the
ANG:std library directory, but that's okay...it passes it to the
linker anyway.
Platforms tested:
Solaris (Arabica)
|
|
|
|
|
|
|
|
|
| |
Bug
Description:
When copying the --enable-production stuff, I forgot to change
the CFLAGS to CXXFLAGS...
Platforms tested:
Solaris
|
|
|
|
|
|
|
| |
Feature Update
Description:
Added ability to compile code as production, development, or
profile with the appropriate flags...
|
|
|
|
|
|
|
|
|
|
|
| |
Bug
Description:
Solaris needs the ``-instances=global'' flag to make templates
open to everyone.
Solution:
Added
Platforms tested:
Sun (Arabica)
|
|
|
|
|
|
|
|
| |
Changed to handle tracing in C++ files.
Description:
Pointed to the correct ``trace'' program (the one in hdf5/bin/)
Platforms tested:
Linux
|
|
|
|
|
|
|
|
| |
Changed to handle tracing in C++ files.
Description:
Added case statements around the tracing logic.
Platforms tested:
Linux
|
|
|
|
| |
Needed Dependencies file
|
|
|
|
|
|
|
|
|
|
|
| |
Bug fix
Description:
distclean was removing .C files.
Solution:
It was trying to remove the .f90 extension, of which there wasn't
one.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
|
| |
Buglet fix...
Description:
When configuring, configure would complain about there not being
a confdefs.h file.
Solution:
Added a src/H5config.h file which configure can generate. This
can be used for configure stuff later, but is empty now.
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
| |
Bug fix
Description:
It would be nice if I'd done this correctly the first time.
Solution:
Doh! NEeded to use the ``using'' keyword...
Platforms tested:
Linux
|
|
|
|
|
|
|
|
|
|
|
| |
Bug fix
Description:
Namespace wasn't being detected properly.
Solution:
Changed it to do it properly (not putting the main() function in
the namespace);
Platforms tested:
Linux
|
|
|
|
|
|
|
| |
include file
Description:
Added header file to the PUB_HDR macro so that it'll be
installed with the library.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bug fix
Description:
libtool complained that the library name didn't have a ``lib''
prefix. Also, the $(top_srcdir) macro points to the top of the
c++/ directory, which was messing up the -I$(top_srcdir)/src
stuff. It needed to be changed to the correct value.
Solution:
Made it so
Platforms tested:
Linux
|