[svn-r9445] Purpose: Bug fix

Description:  "char" was considered as always "signed char" in data type conversion.  However, ISO C leaves
the definition of "char" to individual implementation.  i.e. for IBM AIX C compiler, it's treated as "unsigned
char".

Solution: Changed all "char" to "signed char".  Don't even do "char" anymore because its definition is up
to each vendor.

Platforms tested:  h5committest

4 files changed, 78 insertions, 78 deletions

diff --git a/src/H5T.c b/src/H5T.c
index 602da96..0c85c3f 100644
--- a/src/H5T.c
+++ b/src/H5T.c
@@ -1098,8 +1098,8 @@ H5T_init_interface(void)
     status |= H5T_register(H5T_PERS_HARD, "uchar_schar", native_uchar, native_schar, H5T_conv_uchar_schar, H5AC_dxpl_id);
 
     /* From char to floats */
-    status |= H5T_register(H5T_PERS_HARD, "char_flt", native_schar, native_float, H5T_conv_char_float, H5AC_dxpl_id);
-    status |= H5T_register(H5T_PERS_HARD, "char_dbl", native_schar, native_double, H5T_conv_char_double, H5AC_dxpl_id);
+    status |= H5T_register(H5T_PERS_HARD, "schar_flt", native_schar, native_float, H5T_conv_schar_float, H5AC_dxpl_id);
+    status |= H5T_register(H5T_PERS_HARD, "schar_dbl", native_schar, native_double, H5T_conv_schar_double, H5AC_dxpl_id);
 
     /* From unsigned char to floats */
     status |= H5T_register(H5T_PERS_HARD, "uchar_flt", native_uchar, native_float, H5T_conv_uchar_float, H5AC_dxpl_id);
@@ -1140,8 +1140,8 @@ H5T_init_interface(void)
 #endif /* H5_ULLONG_TO_FP_CAST_WORKS */
 
     /* From floats to char */
-    status |= H5T_register(H5T_PERS_HARD, "flt_char", native_float, native_schar, H5T_conv_float_char, H5AC_dxpl_id);
-    status |= H5T_register(H5T_PERS_HARD, "dbl_char", native_double, native_schar, H5T_conv_double_char, H5AC_dxpl_id);
+    status |= H5T_register(H5T_PERS_HARD, "flt_schar", native_float, native_schar, H5T_conv_float_schar, H5AC_dxpl_id);
+    status |= H5T_register(H5T_PERS_HARD, "dbl_schar", native_double, native_schar, H5T_conv_double_schar, H5AC_dxpl_id);
 
     /* From floats to unsigned char */
     status |= H5T_register(H5T_PERS_HARD, "flt_uchar", native_float, native_uchar, H5T_conv_float_uchar, H5AC_dxpl_id);
diff --git a/src/H5Tconv.c b/src/H5Tconv.c
index 30f41fe..871e8e1 100644
--- a/src/H5Tconv.c
+++ b/src/H5Tconv.c
@@ -6976,9 +6976,9 @@ done:
 
 
 /*-------------------------------------------------------------------------
- * Function:	H5T_conv_char_float
+ * Function:	H5T_conv_schar_float
  *
- * Purpose:	Convert native char to native float using hardware.
+ * Purpose:	Convert native signed char to native float using hardware.
  *		This is a fast special case.
  *
  * Return:	Non-negative on success/Negative on failure
@@ -6991,16 +6991,16 @@ done:
  *-------------------------------------------------------------------------
  */
 herr_t
-H5T_conv_char_float (hid_t src_id, hid_t dst_id, H5T_cdata_t *cdata,
+H5T_conv_schar_float (hid_t src_id, hid_t dst_id, H5T_cdata_t *cdata,
 		       size_t nelmts, size_t buf_stride,
                        size_t UNUSED bkg_stride, void *buf, void UNUSED *bkg,
                        hid_t UNUSED dxpl_id)
 {
     herr_t      ret_value=SUCCEED;      /* Return value         */
     
-    FUNC_ENTER_NOAPI(H5T_conv_char_float, FAIL);
+    FUNC_ENTER_NOAPI(H5T_conv_schar_float, FAIL);
 
-    H5T_CONV_xF(SCHAR, FLOAT, char, float, -, -);
+    H5T_CONV_xF(SCHAR, FLOAT, signed char, float, -, -);
 
 done:
     FUNC_LEAVE_NOAPI(ret_value);
@@ -7008,9 +7008,9 @@ done:
 
 
 /*-------------------------------------------------------------------------
- * Function:	H5T_conv_char_double
+ * Function:	H5T_conv_schar_double
  *
- * Purpose:	Convert native char to native double using hardware.
+ * Purpose:	Convert native signed char to native double using hardware.
  *		This is a fast special case.
  *
  * Return:	Non-negative on success/Negative on failure
@@ -7023,16 +7023,16 @@ done:
  *-------------------------------------------------------------------------
  */
 herr_t
-H5T_conv_char_double (hid_t src_id, hid_t dst_id, H5T_cdata_t *cdata,
+H5T_conv_schar_double (hid_t src_id, hid_t dst_id, H5T_cdata_t *cdata,
 		       size_t nelmts, size_t buf_stride,
                        size_t UNUSED bkg_stride, void *buf, void UNUSED *bkg,
                        hid_t UNUSED dxpl_id)
 {
     herr_t      ret_value=SUCCEED;      /* Return value         */
     
-    FUNC_ENTER_NOAPI(H5T_conv_char_double, FAIL);
+    FUNC_ENTER_NOAPI(H5T_conv_schar_double, FAIL);
 
-    H5T_CONV_xF(SCHAR, DOUBLE, char, double, -, -);
+    H5T_CONV_xF(SCHAR, DOUBLE, signed char, double, -, -);
 
 done:
     FUNC_LEAVE_NOAPI(ret_value);
@@ -7619,9 +7619,9 @@ done:
 
 
 /*-------------------------------------------------------------------------
- * Function:	H5T_conv_float_char
+ * Function:	H5T_conv_float_schar
  *
- * Purpose:	Convert native float to native char using 
+ * Purpose:	Convert native float to native signed char using 
  *              hardware.  This is a fast special case.
  *
  * Return:	Non-negative on success/Negative on failure
@@ -7634,16 +7634,16 @@ done:
  *-------------------------------------------------------------------------
  */
 herr_t
-H5T_conv_float_char (hid_t src_id, hid_t dst_id, H5T_cdata_t *cdata,
+H5T_conv_float_schar (hid_t src_id, hid_t dst_id, H5T_cdata_t *cdata,
 		       size_t nelmts, size_t buf_stride,
                        size_t UNUSED bkg_stride, void *buf, void UNUSED *bkg,
                        hid_t UNUSED dxpl_id)
 {
     herr_t      ret_value=SUCCEED;      /* Return value         */
     
-    FUNC_ENTER_NOAPI(H5T_conv_float_char, FAIL);
+    FUNC_ENTER_NOAPI(H5T_conv_float_schar, FAIL);
 
-    H5T_CONV_Fx(FLOAT, SCHAR, float, char, CHAR_MIN, CHAR_MAX);
+    H5T_CONV_Fx(FLOAT, SCHAR, float, signed char, SCHAR_MIN, SCHAR_MAX);
 
 done:
     FUNC_LEAVE_NOAPI(ret_value);
@@ -7683,9 +7683,9 @@ done:
 
 
 /*-------------------------------------------------------------------------
- * Function:	H5T_conv_double_char
+ * Function:	H5T_conv_double_schar
  *
- * Purpose:	Convert native float to native char using 
+ * Purpose:	Convert native float to native signed char using 
  *              hardware.  This is a fast special case.
  *
  * Return:	Non-negative on success/Negative on failure
@@ -7698,16 +7698,16 @@ done:
  *-------------------------------------------------------------------------
  */
 herr_t
-H5T_conv_double_char (hid_t src_id, hid_t dst_id, H5T_cdata_t *cdata,
+H5T_conv_double_schar (hid_t src_id, hid_t dst_id, H5T_cdata_t *cdata,
 		       size_t nelmts, size_t buf_stride,
                        size_t UNUSED bkg_stride, void *buf, void UNUSED *bkg,
                        hid_t UNUSED dxpl_id)
 {
     herr_t      ret_value=SUCCEED;      /* Return value         */
     
-    FUNC_ENTER_NOAPI(H5T_conv_double_char, FAIL);
+    FUNC_ENTER_NOAPI(H5T_conv_double_schar, FAIL);
 
-    H5T_CONV_Fx(DOUBLE, SCHAR, double, char, CHAR_MIN, CHAR_MAX);
+    H5T_CONV_Fx(DOUBLE, SCHAR, double, signed char, SCHAR_MIN, SCHAR_MAX);
 
 done:
     FUNC_LEAVE_NOAPI(ret_value);
diff --git a/src/H5Tpkg.h b/src/H5Tpkg.h
index 2993f2f..f1c71c9 100644
--- a/src/H5Tpkg.h
+++ b/src/H5Tpkg.h
@@ -857,12 +857,12 @@ H5_DLL herr_t H5T_conv_double_float(hid_t src_id, hid_t dst_id,
 				     size_t buf_stride, size_t bkg_stride,
                                      void *buf, void *bkg,
                                      hid_t dset_xfer_plist);
-H5_DLL herr_t H5T_conv_char_float(hid_t src_id, hid_t dst_id,
+H5_DLL herr_t H5T_conv_schar_float(hid_t src_id, hid_t dst_id,
 				     H5T_cdata_t *cdata, size_t nelmts,
 				     size_t buf_stride, size_t bkg_stride,
                                      void *buf, void *bkg,
                                      hid_t dset_xfer_plist);
-H5_DLL herr_t H5T_conv_char_double(hid_t src_id, hid_t dst_id,
+H5_DLL herr_t H5T_conv_schar_double(hid_t src_id, hid_t dst_id,
 				     H5T_cdata_t *cdata, size_t nelmts,
 				     size_t buf_stride, size_t bkg_stride,
                                      void *buf, void *bkg,
@@ -959,7 +959,7 @@ H5_DLL herr_t H5T_conv_ullong_double(hid_t src_id, hid_t dst_id,
                                      void *buf, void *bkg,
                                      hid_t dset_xfer_plist);
 #endif /* H5_ULLONG_TO_FP_CAST_WORKS */
-H5_DLL herr_t H5T_conv_float_char(hid_t src_id, hid_t dst_id,
+H5_DLL herr_t H5T_conv_float_schar(hid_t src_id, hid_t dst_id,
 				     H5T_cdata_t *cdata, size_t nelmts,
 				     size_t buf_stride, size_t bkg_stride,
                                      void *buf, void *bkg,
@@ -1009,7 +1009,7 @@ H5_DLL herr_t H5T_conv_float_ullong(hid_t src_id, hid_t dst_id,
 				     size_t buf_stride, size_t bkg_stride,
                                      void *buf, void *bkg,
                                      hid_t dset_xfer_plist);
-H5_DLL herr_t H5T_conv_double_char(hid_t src_id, hid_t dst_id,
+H5_DLL herr_t H5T_conv_double_schar(hid_t src_id, hid_t dst_id,
 				     H5T_cdata_t *cdata, size_t nelmts,
 				     size_t buf_stride, size_t bkg_stride,
                                      void *buf, void *bkg,
diff --git a/test/dtypes.c b/test/dtypes.c
index 3dfb53b..b27de1d 100644
--- a/test/dtypes.c
+++ b/test/dtypes.c
@@ -82,7 +82,7 @@ typedef struct complex_t {
 #define ENDIAN(Z,I)	(H5T_ORDER_BE==endian?(I):(Z)-((I)+1))
 
 typedef enum dtype_t {
-    INT_CHAR, INT_UCHAR, INT_SHORT, INT_USHORT, INT_INT, INT_UINT,
+    INT_SCHAR, INT_UCHAR, INT_SHORT, INT_USHORT, INT_INT, INT_UINT,
     INT_LONG, INT_ULONG, INT_LLONG, INT_ULLONG, FLT_FLOAT, FLT_DOUBLE, 
     FLT_LDOUBLE, OTHER
 } dtype_t;
@@ -3784,7 +3784,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
     /* What are the names of the source and destination types */
     if (H5Tequal(src, H5T_NATIVE_SCHAR)) {
 	src_type_name = "signed char";
-	src_type = INT_CHAR;
+	src_type = INT_SCHAR;
     } else if (H5Tequal(src, H5T_NATIVE_UCHAR)) {
 	src_type_name = "unsigned char";
 	src_type = INT_UCHAR;
@@ -3819,7 +3819,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
     
     if (H5Tequal(dst, H5T_NATIVE_SCHAR)) {
 	dst_type_name = "signed char";
-	dst_type = INT_CHAR;
+	dst_type = INT_SCHAR;
     } else if (H5Tequal(dst, H5T_NATIVE_UCHAR)) {
 	dst_type_name = "unsigned char";
 	dst_type = INT_UCHAR;
@@ -3904,10 +3904,10 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 
 	/* Check the results from the library against hardware */
 	for (j=0; j<nelmts; j++) {
-	    if (INT_CHAR==dst_type) {
+	    if (INT_SCHAR==dst_type) {
 		hw = (unsigned char*)&hw_char;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_char = (char)(*((char*)aligned));
 		    break;
@@ -3953,7 +3953,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 	    } else if (INT_UCHAR==dst_type) {
 		hw = (unsigned char*)&hw_uchar;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_uchar = (unsigned char)(*((char*)aligned));
 		    break;
@@ -3999,7 +3999,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 	    } else if (INT_SHORT==dst_type) {
 		hw = (unsigned char*)&hw_short;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_short = (short)(*((char*)aligned));
 		    break;
@@ -4046,7 +4046,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 	    } else if (INT_USHORT==dst_type) {
 		hw = (unsigned char*)&hw_ushort;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_ushort = (unsigned short)(*((char*)aligned));
 		    break;
@@ -4092,7 +4092,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 	    } else if (INT_INT==dst_type) {
 		hw = (unsigned char*)&hw_int;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_int = (int)(*((char*)aligned));
 		    break;
@@ -4138,7 +4138,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 	    } else if (INT_UINT==dst_type) {
 		hw = (unsigned char*)&hw_uint;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_uint = (unsigned int)(*((char*)aligned));
 		    break;
@@ -4184,7 +4184,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 	    } else if (INT_LONG==dst_type) {
 		hw = (unsigned char*)&hw_long;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_long = (long int)(*((char*)aligned));
 		    break;
@@ -4230,7 +4230,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 	    } else if (INT_ULONG==dst_type) {
 		hw = (unsigned char*)&hw_ulong;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_ulong = (unsigned long)(*((char*)aligned));
 		    break;
@@ -4276,7 +4276,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 	    } else if (INT_LLONG==dst_type) {
 		hw = (unsigned char*)&hw_llong;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_llong = (long_long)(*((char*)aligned));
 		    break;
@@ -4322,7 +4322,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 	    } else if (INT_ULLONG==dst_type) {
 		hw = (unsigned char*)&hw_ullong;
 		switch (src_type) {
-		case INT_CHAR:
+		case INT_SCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
 		    hw_ullong = (unsigned long_long)(*((char*)aligned));
 		    break;
@@ -4496,7 +4496,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 		printf(" %02x", saved[j*src_size+ENDIAN(src_size, k)]);
 	    printf("%*s", (int)(3*MAX(0, (ssize_t)dst_size-(ssize_t)src_size)), "");
 	    switch (src_type) {
-                case INT_CHAR:
+                case INT_SCHAR:
                     HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
                     printf(" %29d\n", (int)*((char*)aligned));
                     break;
@@ -4545,7 +4545,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 		printf(" %02x", buf[j*dst_size+ENDIAN(dst_size, k)]);
 	    printf("%*s", (int)(3*MAX(0, (ssize_t)src_size-(ssize_t)dst_size)), "");
 	    switch (dst_type) {
-                case INT_CHAR:
+                case INT_SCHAR:
                     HDmemcpy(aligned, buf+j*sizeof(char), sizeof(char));
                     printf(" %29d\n", (int)*((char*)aligned));
                     break;
@@ -4594,7 +4594,7 @@ test_conv_int_1(const char *name, hid_t src, hid_t dst)
 		printf(" %02x", hw[ENDIAN(dst_size, k)]);
 	    printf("%*s", (int)(3*MAX(0, (ssize_t)src_size-(ssize_t)dst_size)), "");
 	    switch (dst_type) {
-                case INT_CHAR:
+                case INT_SCHAR:
                     printf(" %29d\n", (int)*((signed char*)hw));
                     break;
                 case INT_UCHAR:
@@ -4768,7 +4768,7 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
     float		hw_float;
     double	        hw_double;
     long double		hw_ldouble;
-    signed char		hw_char;
+    signed char		hw_schar;
     unsigned char	hw_uchar;
     short		hw_short;
     unsigned short	hw_ushort;
@@ -4782,7 +4782,7 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
     /* What is the name of the source type */
     if (H5Tequal(src, H5T_NATIVE_SCHAR)) {
 	src_type_name = "signed char";
-	src_type = INT_CHAR;
+	src_type = INT_SCHAR;
     } else if (H5Tequal(src, H5T_NATIVE_UCHAR)) {
 	src_type_name = "unsigned char";
 	src_type = INT_UCHAR;
@@ -4829,7 +4829,7 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
     /* What is the name of the destination type */
     if (H5Tequal(dst, H5T_NATIVE_SCHAR)) {
 	dst_type_name = "signed char";
-	dst_type = INT_CHAR;
+	dst_type = INT_SCHAR;
     } else if (H5Tequal(dst, H5T_NATIVE_UCHAR)) {
 	dst_type_name = "unsigned char";
 	dst_type = INT_UCHAR;
@@ -4883,7 +4883,7 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
 	goto error;
     }
 
-    if ((INT_CHAR==src_type || INT_UCHAR==src_type || INT_SHORT==src_type ||
+    if ((INT_SCHAR==src_type || INT_UCHAR==src_type || INT_SHORT==src_type ||
         INT_USHORT==src_type || INT_INT==src_type || INT_UINT==src_type ||
         INT_LONG==src_type || INT_ULONG==src_type || INT_LLONG==src_type ||
         INT_ULLONG==src_type) && 
@@ -4897,7 +4897,7 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
     }
 
     if ((FLT_FLOAT==src_type || FLT_DOUBLE==src_type || FLT_LDOUBLE==src_type)
-        && (INT_CHAR!=dst_type && INT_UCHAR!=dst_type && INT_SHORT!=dst_type
+        && (INT_SCHAR!=dst_type && INT_UCHAR!=dst_type && INT_SHORT!=dst_type
         && INT_USHORT!=dst_type && INT_INT!=dst_type && INT_UINT!=dst_type
         && INT_LONG!=dst_type && INT_ULONG!=dst_type && INT_LLONG!=dst_type 
         && INT_ULLONG!=dst_type)) {
@@ -4974,16 +4974,16 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
 
 	/* Check the results from the library against hardware */
 	for (j=0; j<nelmts; j++) {
-             if(FLT_FLOAT==src_type || FLT_DOUBLE==src_type || FLT_LDOUBLE==src_type)
+            if(FLT_FLOAT==src_type || FLT_DOUBLE==src_type || FLT_LDOUBLE==src_type)
                 if(my_isnan(src_type, saved+j*src_size))
                     continue;
 
 	    if (FLT_FLOAT==dst_type) {
 		hw = (unsigned char*)&hw_float;
 		switch (src_type) {
-		case INT_CHAR:
-		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
-		    hw_float = (float)(*((char*)aligned));
+		case INT_SCHAR:
+		    HDmemcpy(aligned, saved+j*sizeof(signed char), sizeof(signed char));
+		    hw_float = (float)(*((signed char*)aligned));
 		    break;
 		case INT_UCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(unsigned char), sizeof(unsigned char));
@@ -5029,9 +5029,9 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
 	    } else if (FLT_DOUBLE==dst_type) {
 		hw = (unsigned char*)&hw_double;
 		switch (src_type) {
-		case INT_CHAR:
-		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
-		    hw_double = (double)(*((char*)aligned));
+		case INT_SCHAR:
+		    HDmemcpy(aligned, saved+j*sizeof(signed char), sizeof(signed char));
+		    hw_double = (double)(*((signed char*)aligned));
 		    break;
 		case INT_UCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(unsigned char), sizeof(unsigned char));
@@ -5077,9 +5077,9 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
 	    } else if (FLT_LDOUBLE==dst_type) {
 		hw = (unsigned char*)&hw_ldouble;
 		switch (src_type) {
-		case INT_CHAR:
-		    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
-		    hw_ldouble = (long double)(*((char*)aligned));
+		case INT_SCHAR:
+		    HDmemcpy(aligned, saved+j*sizeof(signed char), sizeof(signed char));
+		    hw_ldouble = (long double)(*((signed char*)aligned));
 		    break;
 		case INT_UCHAR:
 		    HDmemcpy(aligned, saved+j*sizeof(unsigned char), sizeof(unsigned char));
@@ -5122,20 +5122,20 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
 		default:
 		    break;
 		}
-	    } else if (INT_CHAR==dst_type) {
-		hw = (unsigned char*)&hw_char;
+	    } else if (INT_SCHAR==dst_type) {
+		hw = (unsigned char*)&hw_schar;
 		switch (src_type) {
 		case FLT_FLOAT:
 		    HDmemcpy(aligned, saved+j*sizeof(float), sizeof(float));
-		    hw_char = (char)(*((float*)aligned));
+		    hw_schar = (signed char)(*((float*)aligned));
 		    break;
 		case FLT_DOUBLE:
 		    HDmemcpy(aligned, saved+j*sizeof(double), sizeof(double));
-		    hw_char = (char)(*((double*)aligned));
+		    hw_schar = (signed char)(*((double*)aligned));
 		    break;
 		case FLT_LDOUBLE:
 		    HDmemcpy(aligned, saved+j*sizeof(long double), sizeof(long double));
-		    hw_char = (char)(*((long double*)aligned));
+		    hw_schar = (signed char)(*((long double*)aligned));
 		    break;
 		default:
 		    break;
@@ -5336,7 +5336,7 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
              * desired value was set.
 	     */
             if ((FLT_FLOAT==src_type || FLT_DOUBLE==src_type || FLT_LDOUBLE==src_type)
-                    && (INT_CHAR==dst_type || INT_SHORT==dst_type || INT_INT==dst_type 
+                    && (INT_SCHAR==dst_type || INT_SHORT==dst_type || INT_INT==dst_type 
                     || INT_LONG==dst_type || INT_LLONG==dst_type)) {
                 if(0==H5T_bit_get_d(src_bits, src_nbits-1, 1) &&
                         overflows(src_bits, src_type, src_size, dst_nbits-1)) {
@@ -5457,9 +5457,9 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
 		printf(" %02x", saved[j*src_size+ENDIAN(src_size, k)]);
 	    printf("%*s", (int)(3*MAX(0, (ssize_t)dst_size-(ssize_t)src_size)), "");
 	    switch (src_type) {
-                case INT_CHAR:
-                    HDmemcpy(aligned, saved+j*sizeof(char), sizeof(char));
-                    printf(" %29d\n", (int)*((char*)aligned));
+                case INT_SCHAR:
+                    HDmemcpy(aligned, saved+j*sizeof(signed char), sizeof(signed char));
+                    printf(" %29d\n", (int)*((signed char*)aligned));
                     break;
                 case INT_UCHAR:
                     HDmemcpy(aligned, saved+j*sizeof(unsigned char), sizeof(unsigned char));
@@ -5518,9 +5518,9 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
 		printf(" %02x", buf[j*dst_size+ENDIAN(dst_size, k)]);
 	    printf("%*s", (int)(3*MAX(0, (ssize_t)src_size-(ssize_t)dst_size)), "");
 	    switch (dst_type) {
-                case INT_CHAR:
-                    HDmemcpy(aligned, buf+j*sizeof(char), sizeof(char));
-                    printf(" %29d\n", (int)*((char*)aligned));
+                case INT_SCHAR:
+                    HDmemcpy(aligned, buf+j*sizeof(signed char), sizeof(signed char));
+                    printf(" %29d\n", (int)*((signed char*)aligned));
                     break;
                 case INT_UCHAR:
                     HDmemcpy(aligned, buf+j*sizeof(unsigned char), sizeof(unsigned char));
@@ -5579,8 +5579,8 @@ test_conv_int_float(const char *name, hid_t src, hid_t dst)
 		printf(" %02x", hw[ENDIAN(dst_size, k)]);
 	    printf("%*s", (int)(3*MAX(0, (ssize_t)src_size-(ssize_t)dst_size)), "");
 	    switch (dst_type) {
-                case INT_CHAR:
-                    printf(" %29d\n", (int)*((char*)hw));
+                case INT_SCHAR:
+                    printf(" %29d\n", (int)*((signed char*)hw));
                     break;
                 case INT_UCHAR:
                     printf(" %29u\n", (unsigned)*((unsigned char*)hw));
@@ -6476,8 +6476,8 @@ run_int_float_conv(const char *name)
 {
     int		nerrors = 0;
 
-    nerrors += test_conv_int_float(name, H5T_NATIVE_CHAR, H5T_NATIVE_FLOAT);
-    nerrors += test_conv_int_float(name, H5T_NATIVE_CHAR, H5T_NATIVE_DOUBLE);
+    nerrors += test_conv_int_float(name, H5T_NATIVE_SCHAR, H5T_NATIVE_FLOAT);
+    nerrors += test_conv_int_float(name, H5T_NATIVE_SCHAR, H5T_NATIVE_DOUBLE);
 
     nerrors += test_conv_int_float(name, H5T_NATIVE_UCHAR, H5T_NATIVE_FLOAT);
     nerrors += test_conv_int_float(name, H5T_NATIVE_UCHAR, H5T_NATIVE_DOUBLE);
@@ -6551,8 +6551,8 @@ run_float_int_conv(const char *name)
 {
     int		nerrors = 0;
 
-    nerrors += test_conv_int_float(name, H5T_NATIVE_FLOAT, H5T_NATIVE_CHAR);
-    nerrors += test_conv_int_float(name, H5T_NATIVE_DOUBLE, H5T_NATIVE_CHAR);
+    nerrors += test_conv_int_float(name, H5T_NATIVE_FLOAT, H5T_NATIVE_SCHAR);
+    nerrors += test_conv_int_float(name, H5T_NATIVE_DOUBLE, H5T_NATIVE_SCHAR);
     
     nerrors += test_conv_int_float(name, H5T_NATIVE_FLOAT, H5T_NATIVE_UCHAR);
     nerrors += test_conv_int_float(name, H5T_NATIVE_DOUBLE, H5T_NATIVE_UCHAR);