[svn-r139] ./src/*.[ch]

	Removed the interface initialization argument from
	FUNC_ENTER() and made it a locally-defined preprocessor
	symbol, INTERFACE_INIT.

	Changed `offset' to `address' and `length' to `size' in
	documentation so it's more consistent.  `Offset' still appears
	occassionally when it refers to a byte offset within some
	other data structure.

	Moved interface termination function prototypes from public
	header files to .c files and made them static.

./src/H5.c
./src/H5public.h
	Added H5init() because it's possible that the predefined data
	types are not initialized.  This happens only if the first
	call to the hdf5 library passes a predefined data type symbol
	as an argument.  There should be some way to fix this...

./src/H5A.c
./src/H5Aprivate.h
./src/H5Apublic.h
	The free_func returns SUCCEED or FAIL, although the return
	value is ignored by H5A.  This is so we can use the various
	H5*_close() functions to free things.

	H5Ainc_ref() and H5Adec_ref() are no longer public.  Many of
	the other atom functions should also be made private, but I'll
	save that for later...

	Added additional template groups called H5_TEMPLATE_0 through
	H5_TEMPLATE_7 that are used by the various template
	subclasses.

	Increased the number of bits used for atom groups to prevent
	negative atoms.

./src/H5AC.c
./src/H5ACprivate.h
	Changed H5AC_new() to H5AC_create() to make names more consistent.

./src/H5B.c
./src/H5Bprivate.h
	Changed H5B_new() to H5B_create() to make names more consistent.

./src/H5C.c
./src/H5Cprivate.h
./src/H5Cpublic.h
	Now supports multiple subclasses of templates, although it's
	done with big switch statements.  The default values for
	templates are defined in the source file to which that
	template belongs.  This got rid of lots of needless
	preprocessor constants.

	Added H5Ccreate() to create a new template.  Changed
	H5C_release() to H5Cclose() to make the naming more
	consistent.

./src/H5D.c
./src/H5Dprivate.h
./src/H5Dpublic.h
	Enhanced to use the new dataset interface, and uses the enhanced
	data type and data space interfaces, which haven't been
	completely implemented.  The dataset interface doesn't handle
	non-contiguous storage, compression, or data type and space
	conversions yet.

./src/H5F.c
./src/H5Fprivate.h
./src/H5Fpublic.h
	Removed H5Fflush() since just calls H5F_flush(), which doesn't
	do what the user would probably think it does, namely, flush
	everything.  It only flushes those things sitting in the H5AC
	cache and the boot block.

	Changed the `file_create_parms' field of H5F_low_t to just
	`create_parms' since the `file' part is obvious.

./src/H5Fistore.c
	Added some support for external files.  Mostly just in the
	file format and not supported much by the library yet. I need
	to finish some dataset functions first.

	Changed H5F_istore_new() to H5F_istore_create() to make names
	more uniform across packages.

./src/H5Flow.c
	Flushing a file causes the file to be physically extended to
	the logical eof.  This prevents H5F_open() from thinking a
	file has been truncated.  Most of the time the file will
	already be that large, and when it isn't Unix will often just
	allocate the final block anyway.

./src/H5G.c
./src/H5Gent.c
./src/H5Gnode.c
./src/H5Gpkg.h
./src/H5Gprivate.h
./src/H5Gstab.c
	Removed H5G_basename()

	Removed (temporarily) data type information from symbol table
	entries and renamed H5G_CACHED_SDATA to H5G_CACHED_SDSPACE to
	reflect that it's a simple data space and has nothing to do
	with raw data.

	Changed H5G_node_new() to H5G_node_create() and H5G_stab_new()
	to H5G_stab_create() to make names more uniform across
	packages.

	Fixed an undefined address bug that happens when H5G_node_debug()
	program doesn't pass enough info to H5G_node_load().

./src/H5H.c
./src/H5Hprivate.h
	Changed H5H_new() to H5H_create() to make the names more
	uniform across packages.

./src/H5M.c
./src/H5Mprivate.h
./src/H5Mpublic.h
	Nulled all the create functions.  Most of the other callbacks
	are to public functions.  Removed H5Mcreate().

	Changed hobjtype_t to group_t since it has to be the same
	thing anyway.

./src/H5O.c
./src/H5Oprivate.h
./src/H5Osdim.c
./src/H5Osdtyp.c
	Changed H5O_SIM_DIM to H5O_SDSPACE (simple data space) since
	`simple data space' is its official name, not `simple
	dimensions'.  Will eventually add H5O_CDSPACE for comples data
	spaces. Changed _sim_dim_ to _dspace_.

	Replaced H5O_SIM_DTYPE and the compound data type messages
	with a single H5O_DTYPE message. Changed _sim_dtype_ to _dtype_.

	Changed H5O_STD_STORE to H5O_CSTORE (contiguous storage) since
	contiguous storage is not necessarily standard.  Changed
	_std_store_ to _cstore_ in H5Ocstore.c

	Added the H5O_EFL (external file list) message.

	Changed H5O_new() to H5O_create() to make names more uniform
	across packages.

./src/H5Oefl.c		NEW
	External file list message for specifying which non-hdf5 files
	contain raw data for a dataset.

./src/H5P.c
./src/H5Pprivate.h
./src/H5Ppublic.h
	Renamed and moved data structures to make the names conform to
	our naming scheme.

./src/H5T.c
./src/H5Tprivate.h
./src/H5Tpublic.h
./src/H5Tpkg.h		NEW
	Data structures redesigned to be more flexible.  The interface
	was redesigned to make it more regular and to make some names
	more uniform across packages.

./src/H5detect.c
	Output was changed to produce a file that conforms to the hdf5
	coding standard.

./src/Makefile.in
	Generates H5Tinit.c by running H5detect.

./src/debug.c
	Moved command argument processing.

1 files changed, 44 insertions, 31 deletions

diff --git a/src/H5Flow.c b/src/H5Flow.c
index 55010bc..0dfd11c 100644
--- a/src/H5Flow.c
+++ b/src/H5Flow.c
@@ -21,6 +21,7 @@
 
 #define PABLO_MASK	H5F_low
 static hbool_t	interface_initialize_g = FALSE;
+#define INTERFACE_INIT NULL
 
 
 /*-------------------------------------------------------------------------
@@ -72,7 +73,7 @@ H5F_low_open (const H5F_low_class_t *type, const char *name, uintn flags,
 {
    H5F_low_t	*lf = NULL;
    
-   FUNC_ENTER (H5F_low_open, NULL, NULL);
+   FUNC_ENTER (H5F_low_open, NULL);
 
    assert (type && type->open);
    assert (name && *name);
@@ -117,7 +118,7 @@ H5F_low_open (const H5F_low_class_t *type, const char *name, uintn flags,
 H5F_low_t *
 H5F_low_close (H5F_low_t *lf)
 {
-   FUNC_ENTER (H5F_low_close, NULL, NULL);
+   FUNC_ENTER (H5F_low_close, NULL);
 
    if (lf) {
       if ((lf->type->close)(lf)<0) {
@@ -163,7 +164,7 @@ H5F_low_read (H5F_low_t *lf, const haddr_t *addr, size_t size,
 {
    herr_t	ret_value = FAIL;
    
-   FUNC_ENTER (H5F_low_read, NULL, FAIL);
+   FUNC_ENTER (H5F_low_read, FAIL);
 
    assert (lf && lf->type);
    assert (addr && H5F_addr_defined (addr));
@@ -213,7 +214,7 @@ H5F_low_write (H5F_low_t *lf, const haddr_t *addr, size_t size,
    herr_t	ret_value = FAIL;
    haddr_t	tmp_addr;
    
-   FUNC_ENTER (H5F_low_write, NULL, FAIL);
+   FUNC_ENTER (H5F_low_write, FAIL);
 
    assert (lf && lf->type);
    assert (addr && H5F_addr_defined (addr));
@@ -244,10 +245,14 @@ H5F_low_write (H5F_low_t *lf, const haddr_t *addr, size_t size,
  * Function:	H5F_low_flush
  *
  * Purpose:	Flushes file buffers to disk.  For instance, the stdio.h
- *		driver would call fflush().
- *
- *		If the subclass doesn't define a flush method then this
- *		function doesn't do anything.
+ *		driver would call fflush().  Flushing also insures that the
+ *		file exists to the current logical EOF (the library maintains
+ *		a notion of EOF which is independent of the physical EOF) by
+ *		reading and writing the last byte.  On some systems, this
+ *		allocates a single block at the end of the file while on
+ *		other systems it allocates all blocks up to the end of the
+ *		file.  Extending the physical file is necessary because
+ *		H5F_open() checks for truncated files.
  *
  * Return:	Success:	SUCCEED
  *
@@ -263,10 +268,24 @@ H5F_low_write (H5F_low_t *lf, const haddr_t *addr, size_t size,
 herr_t
 H5F_low_flush (H5F_low_t *lf)
 {
-   FUNC_ENTER (H5F_low_flush, NULL, FAIL);
+   haddr_t	last_byte;
+   uint8	buf[1];
+   
+   FUNC_ENTER (H5F_low_flush, FAIL);
 
    assert (lf && lf->type);
 
+   /* Make sure the last block of the file has been allocated on disk */
+   H5F_addr_reset (&last_byte);
+   if (H5F_addr_defined (&(lf->eof)) && H5F_addr_gt (&(lf->eof), &last_byte)) {
+      last_byte = lf->eof;
+      last_byte.offset -= 1;
+      if (H5F_low_read (lf, &last_byte, 1, buf)>=0) {
+	 H5F_low_write (lf, &last_byte, 1, buf);
+      }
+   }
+   
+   /* Invoke the subclass the flush method */
    if (lf->type->flush) {
       if ((lf->type->flush)(lf)<0) {
 	 /* Low level flush failed */
@@ -315,7 +334,7 @@ H5F_low_size (H5F_low_t *lf, haddr_t *eof/*out*/)
 {
    size_t	size = (size_t)(-1);	/*max possible size*/
    
-   FUNC_ENTER (H5F_low_size, NULL, 0);
+   FUNC_ENTER (H5F_low_size, 0);
 
    assert (lf && lf->type);
    assert (eof);
@@ -368,7 +387,7 @@ H5F_low_access (const H5F_low_class_t *type, const char *name, int mode,
    hbool_t	ret_value;
    struct stat	sb;
    
-   FUNC_ENTER (H5F_low_size, NULL, 0);
+   FUNC_ENTER (H5F_low_size, 0);
    assert (type);
 
    if (type->access) {
@@ -411,7 +430,7 @@ H5F_low_access (const H5F_low_class_t *type, const char *name, int mode,
 herr_t
 H5F_low_extend (H5F_low_t *lf, intn op, size_t size, haddr_t *addr/*out*/)
 {
-   FUNC_ENTER (H5F_low_alloc, NULL, FAIL);
+   FUNC_ENTER (H5F_low_alloc, FAIL);
 
    assert (lf);
    assert (size>0);
@@ -450,7 +469,7 @@ H5F_low_extend (H5F_low_t *lf, intn op, size_t size, haddr_t *addr/*out*/)
 herr_t
 H5F_low_seteof (H5F_low_t *lf, const haddr_t *addr)
 {
-   FUNC_ENTER (H5F_low_seteof, NULL, FAIL);
+   FUNC_ENTER (H5F_low_seteof, FAIL);
 
    assert (lf);
    assert (addr && H5F_addr_defined (addr));
@@ -482,7 +501,7 @@ H5F_low_seteof (H5F_low_t *lf, const haddr_t *addr)
 intn
 H5F_addr_cmp (const haddr_t *a1, const haddr_t *a2)
 {
-   FUNC_ENTER (H5F_addr_cmp, NULL, FAIL);
+   FUNC_ENTER (H5F_addr_cmp, FAIL);
    
    assert (a1 && H5F_addr_defined (a1));
    assert (a2 && H5F_addr_defined (a2));
@@ -493,7 +512,6 @@ H5F_addr_cmp (const haddr_t *a1, const haddr_t *a2)
    FUNC_LEAVE (0);
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_undef
@@ -510,8 +528,10 @@ H5F_addr_cmp (const haddr_t *a1, const haddr_t *a2)
  *-------------------------------------------------------------------------
  */
 void
-H5F_addr_undef (haddr_t *addr)
+H5F_addr_undef (haddr_t *addr/*out*/)
 {
+   assert (addr);
+   
    addr->offset = -1;
 }
 
@@ -535,11 +555,10 @@ H5F_addr_undef (haddr_t *addr)
 hbool_t
 H5F_addr_defined (const haddr_t *addr)
 {
-   FUNC_ENTER (H5F_addr_defined, NULL, FAIL);
+   FUNC_ENTER (H5F_addr_defined, FAIL);
    FUNC_LEAVE (-1!=addr->offset && addr->offset>=0);
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_reset
@@ -556,12 +575,12 @@ H5F_addr_defined (const haddr_t *addr)
  *-------------------------------------------------------------------------
  */
 void
-H5F_addr_reset (haddr_t *addr)
+H5F_addr_reset (haddr_t *addr/*out*/)
 {
+   assert (addr);
    addr->offset = 0;
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_zerop
@@ -582,11 +601,10 @@ H5F_addr_reset (haddr_t *addr)
 hbool_t
 H5F_addr_zerop (const haddr_t *addr)
 {
-   FUNC_ENTER (H5F_addr_zerop, NULL, FAIL);
+   FUNC_ENTER (H5F_addr_zerop, FAIL);
    FUNC_LEAVE (0==addr->offset);
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_encode
@@ -616,20 +634,19 @@ H5F_addr_encode (H5F_t *f, uint8 **pp, const haddr_t *addr)
 
    if (H5F_addr_defined (addr)) {
       tmp = *addr;
-      for (i=0; i<H5F_SIZEOF_OFFSET (f); i++) {
+      for (i=0; i<H5F_SIZEOF_ADDR (f); i++) {
 	 *(*pp)++ = tmp.offset & 0xff;
 	 tmp.offset >>= 8;
       }
       assert ("overflow" && 0==tmp.offset);
       
    } else {
-      for (i=0; i<H5F_SIZEOF_OFFSET (f); i++) {
+      for (i=0; i<H5F_SIZEOF_ADDR (f); i++) {
 	 *(*pp)++ = 0xff;
       }
    }
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_decode
@@ -663,7 +680,8 @@ H5F_addr_decode (H5F_t *f, const uint8 **pp, haddr_t *addr/*out*/)
    assert (addr);
 
    addr->offset = 0;
-   for (i=0; i<H5F_SIZEOF_OFFSET (f); i++) {
+   
+   for (i=0; i<H5F_SIZEOF_ADDR (f); i++) {
       c = *(*pp)++;
       if (c!=0xff) all_zero = FALSE;
       
@@ -678,7 +696,6 @@ H5F_addr_decode (H5F_t *f, const uint8 **pp, haddr_t *addr/*out*/)
    if (all_zero) H5F_addr_undef (addr);
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_print
@@ -723,7 +740,6 @@ H5F_addr_print (FILE *stream, const haddr_t *addr)
    }
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_pow2
@@ -750,7 +766,6 @@ H5F_addr_pow2 (uintn n, haddr_t *addr/*out*/)
    addr->offset <<= n;
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_inc
@@ -774,7 +789,6 @@ H5F_addr_inc (haddr_t *addr/*in,out*/, size_t inc)
    addr->offset += inc;
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_add
@@ -798,7 +812,6 @@ H5F_addr_add (haddr_t *a1/*in,out*/, const haddr_t *a2)
    a1->offset += a2->offset;
 }
 
-
 
 /*-------------------------------------------------------------------------
  * Function:	H5F_addr_hash