[svn-r8987] Purpose:

    Code cleanup

Description:
    Fix another batch of minor differences between the development and release
branches.

Platforms tested:
    FreeBSD 4.10 (sleipnir) w/parallel
    Too minor to require h5committest

7 files changed, 19 insertions, 23 deletions

diff --git a/src/H5D.c b/src/H5D.c
index 177e3e1..be85340 100644
--- a/src/H5D.c
+++ b/src/H5D.c
@@ -31,7 +31,6 @@
 #include "H5MMprivate.h"	/* Memory management			*/
 #include "H5Sprivate.h"		/* Dataspaces 				*/
 #include "H5Vprivate.h"		/* Vectors and arrays 			*/
-#include "H5Zprivate.h"
 
 /*#define H5D_DEBUG*/
 
diff --git a/src/H5Dio.c b/src/H5Dio.c
index 9fb860e..b7d9580 100644
--- a/src/H5Dio.c
+++ b/src/H5Dio.c
@@ -757,7 +757,7 @@ H5D_read(H5D_t *dataset, hid_t mem_type_id, const H5S_t *mem_space,
     } /* end switch */
 
     /* Get dataspace functions */
-    if (NULL==(sconv=H5S_find(dataset->ent.file,mem_space, file_space, sconv_flags, &use_par_opt_io,&dataset->layout)))
+    if (NULL==(sconv=H5S_find(dataset->ent.file, mem_space, file_space, sconv_flags, &use_par_opt_io, &dataset->layout)))
         HGOTO_ERROR (H5E_DATASET, H5E_UNSUPPORTED, FAIL, "unable to convert from file to memory data space")
 
 #ifdef H5_HAVE_PARALLEL
@@ -1004,7 +1004,7 @@ H5D_write(H5D_t *dataset, hid_t mem_type_id, const H5S_t *mem_space,
     } /* end switch */
 
     /* Get dataspace functions */
-    if (NULL==(sconv=H5S_find(dataset->ent.file,mem_space, file_space, sconv_flags, &use_par_opt_io,&dataset->layout)))
+    if (NULL==(sconv=H5S_find(dataset->ent.file, mem_space, file_space, sconv_flags, &use_par_opt_io, &dataset->layout)))
 	HGOTO_ERROR (H5E_DATASET, H5E_UNSUPPORTED, FAIL, "unable to convert from memory to file data space")
         
 #ifdef H5_HAVE_PARALLEL
@@ -1031,7 +1031,6 @@ H5D_write(H5D_t *dataset, hid_t mem_type_id, const H5S_t *mem_space,
      }
 #endif /* H5_HAVE_INSTRUMENTED_LIBRARY */
 
-     /* end Test for collective chunk IO */
     /* Don't reset the transfer mode if we can't or won't use it */
     if(!use_par_opt_io || !H5T_path_noop(tpath))
         H5D_io_assist_mpio(dxpl_id, dxpl_cache, &xfer_mode_changed);
diff --git a/src/H5Dmpio.c b/src/H5Dmpio.c
index ea15076..581a20a 100644
--- a/src/H5Dmpio.c
+++ b/src/H5Dmpio.c
@@ -138,7 +138,7 @@ H5D_mpio_spaces_xfer(H5F_t *f, const H5D_t *dset, size_t elmt_size,
 			       &mft_is_derived )<0)
     	HGOTO_ERROR(H5E_DATASPACE, H5E_BADTYPE, FAIL,"couldn't create MPI file type");
 
-
+    /* Get the base address of the contiguous dataset or the chunk */
     if(dset->layout.type == H5D_CONTIGUOUS)
        addr = H5D_contig_get_addr(dset) + mpi_file_offset;
     else {
@@ -222,7 +222,7 @@ H5D_mpio_spaces_read(H5F_t *f, const H5D_dxpl_cache_t UNUSED *dxpl_cache, hid_t
     FUNC_ENTER_NOAPI_NOFUNC(H5D_mpio_spaces_read);
 
     ret_value = H5D_mpio_spaces_xfer(f, dset, elmt_size, file_space,
-        mem_space, dxpl_id, buf,store, 0/*read*/);
+        mem_space, dxpl_id, buf, store, 0/*read*/);
 
     FUNC_LEAVE_NOAPI(ret_value);
 } /* end H5D_mpio_spaces_read() */
@@ -261,9 +261,8 @@ H5D_mpio_spaces_write(H5F_t *f, const H5D_dxpl_cache_t UNUSED *dxpl_cache, hid_t
 
     /*OKAY: CAST DISCARDS CONST QUALIFIER*/
     ret_value = H5D_mpio_spaces_xfer(f, dset, elmt_size, file_space,
-        mem_space, dxpl_id, (void*)buf, store,1/*write*/);
+        mem_space, dxpl_id, (void*)buf, store, 1/*write*/);
 
     FUNC_LEAVE_NOAPI(ret_value);
 } /* end H5D_mpio_spaces_write() */
-
 #endif  /* H5_HAVE_PARALLEL */
diff --git a/src/H5S.c b/src/H5S.c
index 70e8b55..3a8a799 100644
--- a/src/H5S.c
+++ b/src/H5S.c
@@ -23,14 +23,14 @@
 #define PABLO_MASK	H5S_mask
 
 #define _H5S_IN_H5S_C
-#include "H5private.h"		/* Generic Functions			  */
-#include "H5Eprivate.h"		/* Error handling		  */
-#include "H5Fpkg.h"		/* Dataspace functions			  */
-#include "H5FLprivate.h"	/* Free Lists	  */
-#include "H5Iprivate.h"		/* ID Functions		  */
-#include "H5MMprivate.h"	/* Memory Management functions		  */
-#include "H5Oprivate.h"		/* object headers		  */
-#include "H5Spkg.h"		/* Dataspace functions			  */
+#include "H5private.h"		/* Generic Functions			*/
+#include "H5Eprivate.h"		/* Error handling		  	*/
+#include "H5Fpkg.h"             /* File access				*/
+#include "H5FLprivate.h"	/* Free lists                           */
+#include "H5Iprivate.h"		/* IDs			  		*/
+#include "H5MMprivate.h"	/* Memory management			*/
+#include "H5Oprivate.h"		/* Object headers		  	*/
+#include "H5Spkg.h"		/* Dataspaces 				*/
 
 /* Local macro definitions */
 #define H5S_ENCODE_VERSION      0
@@ -1375,7 +1375,7 @@ H5S_set_extent_simple (H5S_t *space, unsigned rank, const hsize_t *dims,
             HDmemcpy(space->extent.max, max, sizeof(hsize_t) * rank);
         } /* end if */
         else {
-           space->extent.max = NULL;
+            space->extent.max = NULL;
         }
     }
 
@@ -1807,7 +1807,7 @@ done:
  *
  *-------------------------------------------------------------------------
  */
-herr_t
+static herr_t
 H5S_encode(H5S_t *obj, unsigned char *buf, size_t *nalloc)
 {
     size_t      extent_size;
@@ -1928,7 +1928,7 @@ done:
  *
  *-------------------------------------------------------------------------
  */
-H5S_t*
+static H5S_t*
 H5S_decode(const unsigned char *buf)
 {
     H5S_t       *ds;
@@ -2198,7 +2198,7 @@ done:
  *          by H5Sset_extent_simple() ).  Helps avoid write errors.
  *
  * Return: TRUE if dataspace has extent set
- *          FALSE if dataspace's extent is uninitialized
+ *         FALSE if dataspace's extent is uninitialized
  *
  * Programmer: James Laird
  *
diff --git a/src/H5Shyper.c b/src/H5Shyper.c
index 2ee7356..2bc8dcc 100644
--- a/src/H5Shyper.c
+++ b/src/H5Shyper.c
@@ -729,7 +729,7 @@ H5S_hyper_iter_next(H5S_sel_iter_t *iter, size_t nelem)
                 /* Increment absolute position */
                 if(curr_dim==fast_dim) {
                     size_t actual_elem;     /* Actual # of elements advanced on each iteration through loop */
-                    hsize_t span_elem;       /* Number of elements left in a span */
+                    hsize_t span_elem;      /* Number of elements left in a span */
 
                     /* Compute the number of elements left in block */
                     span_elem=(curr_span->high-abs_arr[curr_dim])+1;
diff --git a/src/H5Smpio.c b/src/H5Smpio.c
index 6c7ddd1..b04faeb 100644
--- a/src/H5Smpio.c
+++ b/src/H5Smpio.c
@@ -743,5 +743,4 @@ H5S_mpio_opt_possible( const H5F_t *file, const H5S_t *mem_space, const H5S_t *f
 done:
     FUNC_LEAVE_NOAPI(ret_value);
 } /* H5S_mpio_opt_possible() */
-
 #endif  /* H5_HAVE_PARALLEL */
diff --git a/src/H5Sprivate.h b/src/H5Sprivate.h
index 732a4d2..efa9b30 100644
--- a/src/H5Sprivate.h
+++ b/src/H5Sprivate.h
@@ -211,10 +211,10 @@ H5_DLL H5S_conv_t *H5S_find(const H5F_t *file,const H5S_t *mem_space, const H5S_
 H5_DLL H5S_class_t H5S_get_simple_extent_type(const H5S_t *ds);
 H5_DLL hssize_t H5S_get_simple_extent_npoints(const H5S_t *ds);
 H5_DLL hsize_t H5S_get_npoints_max(const H5S_t *ds);
+H5_DLL hbool_t H5S_has_extent(const H5S_t *ds);
 H5_DLL int H5S_get_simple_extent_ndims(const H5S_t *ds);
 H5_DLL int H5S_get_simple_extent_dims(const H5S_t *ds, hsize_t dims[]/*out*/,
 					hsize_t max_dims[]/*out*/);
-H5_DLL hbool_t H5S_has_extent(const H5S_t *ds);
 H5_DLL herr_t H5S_modify(struct H5G_entry_t *ent, const H5S_t *space,
         hbool_t update_time, hid_t dxpl_id);
 H5_DLL herr_t H5S_append(H5F_t *f, hid_t dxpl_id, struct H5O_t *oh, const H5S_t *ds);