[svn-r15181] Purpose: _WIN32 macro cleanup

Description:
As part of our Windows cleanup, we try to remove windows-specific tweaks in the source code.  There are many instances where Windows code is introduces via ifdef's.  We re-evaluate whether they are still required, and found that many of them are not.  Others we change to "feature"-specific code, rather than Windows-specific.

Tested:
VS2005 on WinXP
VS.NET on WinXP
h5committest (kagisopp, smirom, linew)

7 files changed, 1 insertions, 68 deletions

diff --git a/tools/h5dump/h5dumpgentest.c b/tools/h5dump/h5dumpgentest.c
index 8f48e89..0646536 100644
--- a/tools/h5dump/h5dumpgentest.c
+++ b/tools/h5dump/h5dumpgentest.c
@@ -1199,11 +1199,7 @@ static void gent_many(void)
                     for(i0 = 0; i0 < 2; i0++) {
                         dset1[j].a[i3][i2][i1][i0] = i0+j;
                         dset1[j].b[i3][i2][i1][i0] = (double)(i0+j);
-#ifdef _WIN32
-                        dset1[j].c[i3][i2][i1][i0] = (double)(i0+j+(signed __int64)sdim);
-#else
                         dset1[j].c[i3][i2][i1][i0] = (double)(i0+j+sdim);
-#endif
                     }
                 }
             }
diff --git a/tools/h5import/h5import.c b/tools/h5import/h5import.c
index f8f518b..097e298 100755
--- a/tools/h5import/h5import.c
+++ b/tools/h5import/h5import.c
@@ -372,10 +372,8 @@ readIntegerData(FILE **strm, struct Input *in)
   H5DT_INT8 *in08;
   H5DT_INT16 *in16, temp;
   H5DT_INT32 *in32;
-#ifndef _WIN32
   H5DT_INT64 *in64;
   char buffer[256];
-#endif
   hsize_t len=1;
   hsize_t i;
   int j;
@@ -488,7 +486,6 @@ readIntegerData(FILE **strm, struct Input *in)
       }
     break;
 
-#ifndef _WIN32
     case 64:
       in64 = (H5DT_INT64 *) in->data;
       switch(in->inputClass)
@@ -521,7 +518,6 @@ readIntegerData(FILE **strm, struct Input *in)
           return (-1);
       }
   	  break;
-#endif /* ifndef _WIN32 */
 
     default:
       (void) fprintf(stderr, err3);
@@ -536,10 +532,8 @@ readUIntegerData(FILE **strm, struct Input *in)
   H5DT_UINT8 *in08;
   H5DT_UINT16 *in16, temp;
   H5DT_UINT32 *in32;
-#ifndef _WIN32
   H5DT_UINT64 *in64;
   char buffer[256];
-#endif
   hsize_t len=1;
   hsize_t i;
   int j;
@@ -650,7 +644,6 @@ readUIntegerData(FILE **strm, struct Input *in)
       }
     break;
 
-#ifndef _WIN32
     case 64:
       in64 = (H5DT_UINT64 *) in->data;
       switch(in->inputClass)
@@ -683,7 +676,6 @@ readUIntegerData(FILE **strm, struct Input *in)
           return (-1);
       }
     break;
-#endif /* ifndef _WIN32 */
 
     default:
       (void) fprintf(stderr, err3);
@@ -1441,9 +1433,6 @@ validateConfigurationParameters(struct Input * in)
   const char *err4a = "OUTPUT-ARCHITECTURE cannot be STD if OUTPUT-CLASS is floating point (FP).\n";
   const char *err4b = "OUTPUT-ARCHITECTURE cannot be IEEE if OUTPUT-CLASS is integer (IN).\n";
   const char *err5 = "For OUTPUT-CLASS FP, valid values for OUTPUT-SIZE are (32, 64) .\n";
-#ifdef _WIN32
-  const char *err6 = "No support for reading 64-bit integer (INPUT-CLASS: IN, TEXTIN, UIN, TEXTUIN files\n";
-#endif
 
    /* for class STR other parameters are ignored */
   if (in->inputClass == 5) /* STR */
@@ -1502,13 +1491,6 @@ validateConfigurationParameters(struct Input * in)
       return (-1);
     }
 
-#ifdef _WIN32
-  if (in->inputSize == 64 && (in->inputClass == 0 || in->inputClass == 4 || in->inputClass == 6 || in->inputClass == 7) )
-	{
-	  (void) fprintf(stderr, err6);
-	  return -1;
-	}
-#endif
   return (0);
 }
 
diff --git a/tools/h5import/h5importtest.c b/tools/h5import/h5importtest.c
index 2a8c827..7a6bbff 100755
--- a/tools/h5import/h5importtest.c
+++ b/tools/h5import/h5importtest.c
@@ -46,12 +46,10 @@ main(void)
     int     rowo4i = (int)11 , colo4i = (int)21 , plno4i = (int)51 ;
     int     rowi4i = (int)1 , coli4i = (int)2 , plni4i = (int)5 ;
 
-#ifndef _WIN32
     long_long     b64i2[3][4], b64i3[5][3][4];
     long_long     row4i64[3], col4i64[4], pln4i64[5];
     long_long     rowo4i64 = (long_long)11 , colo4i64 = (long_long)21 , plno4i64 = (long_long)51 ;
     long_long     rowi4i64 = (long_long)1 , coli4i64 = (long_long)2 , plni4i64 = (long_long)5 ;
-#endif
 
     short     b16i3[5][3][4];
     short     row4i16[3], col4i16[4], pln4i16[5];
@@ -96,11 +94,9 @@ main(void)
     col4i[0] = colo4i;
     pln4i[0] = plno4i;
 
-#ifndef _WIN32
     row4i64[0] = rowo4i64;
     col4i64[0] = colo4i64;
     pln4i64[0] = plno4i64;
-#endif
 
     row4i16[0] = rowo4i16;
     col4i16[0] = colo4i16;
@@ -115,9 +111,7 @@ main(void)
         row4[i] = row4[i - 1] + rowi4;
         row8[i] = row8[i - 1] + rowi8;
         row4i[i] = row4i[i - 1] + rowi4i;
-#ifndef _WIN32
 	row4i64[i] = row4i64[i - 1] + rowi4i64;
-#endif
 	row4i16[i] = row4i16[i - 1] + rowi4i16;
 	row4i8[i] = row4i8[i - 1] + rowi4i8;
     }
@@ -127,9 +121,7 @@ main(void)
           col4[j] = col4[j - 1] + coli4;
           col8[j] = col8[j - 1] + coli8;
 	  col4i[j] = col4i[j - 1] + coli4i;
-#ifndef _WIN32
           col4i64[j] = col4i64[j - 1] + coli4i64;
-#endif
 	  col4i16[j] = col4i16[j - 1] + coli4i16;
 	  col4i8[j] = col4i8[j - 1] + coli4i8;
     }
@@ -138,9 +130,7 @@ main(void)
           pln4[k] = pln4[k - 1] + plni4;
           pln8[k] = pln8[k - 1] + plni8;
 	  pln4i[k] = pln4i[k - 1] + plni4i;
-#ifndef _WIN32
     	  pln4i64[k] = pln4i64[k - 1] + plni4i64;
-#endif
 	  pln4i16[k] = pln4i16[k - 1] + plni4i16;
 	  pln4i8[k] = pln4i8[k - 1] + plni4i8;
    }
@@ -149,9 +139,7 @@ main(void)
    {
      for (j = 0; j < ncol; j++)
      {
-#ifndef _WIN32
   	b64i2[i][j] = row4i64[i] + col4i64[j];
-#endif
      }
    }
 
@@ -170,9 +158,7 @@ main(void)
                       b32r3[k][i][j] = row4[i] + col4[j] + pln4[k];
                       b64r3[k][i][j] = row8[i] + col8[j] + pln8[k];
 	              b32i3[k][i][j] = row4i[i] + col4i[j] + pln4i[k];
-#ifndef _WIN32
 	     	      b64i3[k][i][j] = row4i64[i] + col4i64[j] + pln4i64[k];
-#endif
 		      b16i3[k][i][j] = row4i16[i] + col4i16[j] + pln4i16[k];
 		      b8i3[k][i][j] = row4i8[i] + col4i8[j] + pln4i8[k];
                   }
@@ -221,14 +207,12 @@ main(void)
             (void) fwrite((char *) &b32i3[k][i][j], sizeof(unsigned int), 1, sp);
     (void) fclose(sp);
 
-#ifndef _WIN32
 
     sp = fopen("bin64-2", "w");
     for (i = 0; i < nrow; i++)
         for (j = 0; j < ncol; j++)
             (void) fwrite((char *) &b64i2[i][j], sizeof(long_long), 1, sp);
     (void) fclose(sp);
-#endif
 
     sp = fopen("bfp32", "w");
     for (k = 0; k < npln; k++)
@@ -254,7 +238,6 @@ main(void)
                           sp);
     (void) fclose(sp);
 
-#ifndef _WIN32
 
         sp = fopen("bin64-3", "w");
 	for (k = 0; k < npln; k++)
@@ -263,7 +246,6 @@ main(void)
             (void) fwrite((char *) &b64i3[k][i][j], sizeof(long_long), 1,
                           sp);
     (void) fclose(sp);
-#endif
 
     sp = fopen("bin8", "w");
     for (k = 0; k < npln; k++)
diff --git a/tools/h5jam/h5jamgentest.c b/tools/h5jam/h5jamgentest.c
index 0599552..7b7e648 100644
--- a/tools/h5jam/h5jamgentest.c
+++ b/tools/h5jam/h5jamgentest.c
@@ -333,11 +333,7 @@ create_textfile(const char *name, size_t size)
     size_t i;
     char *bp;
 
-#ifdef _WIN32
-    fd = _creat(name, _S_IREAD | _S_IWRITE);
-#else /* _WIN32 */
-    fd = creat(name,(mode_t)0777);
-#endif /* _WIN32 */
+    fd = HDcreat(name,0777);
     assert(fd >= 0);
     buf = calloc(size, (size_t)1);
     assert(buf);
diff --git a/tools/h5ls/h5ls.c b/tools/h5ls/h5ls.c
index 6f07050..6545368 100644
--- a/tools/h5ls/h5ls.c
+++ b/tools/h5ls/h5ls.c
@@ -1526,11 +1526,7 @@ dataset_list2(hid_t dset, const char UNUSED *name)
                (unsigned long)total, 1==total?"":"s",
                (unsigned long)used, 1==used?"":"s");
         if (used>0) {
-#ifdef _WIN32
-            utilization = (hssize_t)total * 100.0 / (hssize_t)used;
-#else
             utilization = (total*100.0)/used;
-#endif
             printf(", %1.2f%% utilization", utilization);
         }
         putchar('\n');
diff --git a/tools/lib/h5tools_utils.c b/tools/lib/h5tools_utils.c
index 512a3fb..0be3a91 100644
--- a/tools/lib/h5tools_utils.c
+++ b/tools/lib/h5tools_utils.c
@@ -99,13 +99,8 @@ warn_msg(const char *progname, const char *fmt, ...)
 
     va_start(ap, fmt);
     HDfflush(stdout);
-#ifdef _WIN32
-    HDfprintf(stdout, "%s warning: ", progname);
-    HDvfprintf(stdout, fmt, ap);
-#else /* _WIN32 */
     HDfprintf(stderr, "%s warning: ", progname);
     HDvfprintf(stderr, fmt, ap);
-#endif /* _WIN32 */
     va_end(ap);
 }
 
diff --git a/tools/misc/h5repart.c b/tools/misc/h5repart.c
index f107e31..91df7b6 100644
--- a/tools/misc/h5repart.c
+++ b/tools/misc/h5repart.c
@@ -45,11 +45,6 @@
 #   include <sys/stat.h>
 #endif
 
-#ifdef _WIN32
-#   include <io.h>
-#	include <fcntl.h>
-#endif
-
 #ifndef FALSE
 #   define FALSE	0
 #endif
@@ -216,21 +211,12 @@ main (int argc, char *argv[])
     int		dst_is_family;		/*is dst name a family name?	*/
     int		dst_membno=0;		/*destination member number	*/
 
-#if defined(_WIN32) && ! defined (__MWERKS__)
-    __int64	left_overs=0;		/*amount of zeros left over	*/
-    __int64	src_offset=0;		/*offset in source member	*/
-    __int64	dst_offset=0;		/*offset in destination member	*/
-    __int64	src_size;		/*source logical member size	*/
-    __int64	src_act_size;		/*source actual member size	*/
-    __int64	dst_size=1 GB;		/*destination logical memb size	*/
-#else
     off_t	left_overs=0;		/*amount of zeros left over	*/
     off_t	src_offset=0;		/*offset in source member	*/
     off_t	dst_offset=0;		/*offset in destination member	*/
     off_t	src_size;		/*source logical member size	*/
     off_t	src_act_size;		/*source actual member size	*/
     off_t	dst_size=1 GB;		/*destination logical memb size	*/
-#endif
     hid_t       fapl;                   /*file access property list     */
     hid_t       file;
     hsize_t     hdsize;                 /*destination logical memb size */