simplify.c (gfc_simplify_char): Use UCHAR_MAX instead of literal constant.

fortran/
2006-01-09  Feng Wang  <fengwang@nudt.edu.cn>

	* simplify.c (gfc_simplify_char): Use UCHAR_MAX instead of literal
	constant.
	(gfc_simplify_ichar): Get the result from unsinged char and in the
	range 0 to UCHAR_MAX instead of CHAR_MIN to CHAR_MAX.

Testsuite ChangeLog entry:
2006-01-09  Feng Wang  <fengwang@nudt.edu.cn>

	* gfortran.dg/intrinsic_i_char.f90: New test.

From-SVN: r109488

diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index 5cb9a7b..45fb5fe 100644 (file)
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,3 +1,10 @@
+2006-01-09  Feng Wang  <fengwang@nudt.edu.cn>
+
+       * simplify.c (gfc_simplify_char): Use UCHAR_MAX instead of literal
+       constant.
+       (gfc_simplify_ichar): Get the result from unsinged char and in the
+       range 0 to UCHAR_MAX instead of CHAR_MIN to CHAR_MAX.
+
 2005-01-08  Erik Edelmann  <eedelman@gcc.gnu.org>
 
        PR fortran/25093

diff --git a/gcc/fortran/simplify.c b/gcc/fortran/simplify.c
index 5b47e93..7c66504 100644 (file)
--- a/gcc/fortran/simplify.c
+++ b/gcc/fortran/simplify.c
@@ -662,7 +662,7 @@ gfc_simplify_char (gfc_expr * e, gfc_expr * k)
   if (e->expr_type != EXPR_CONSTANT)
     return NULL;
 
-  if (gfc_extract_int (e, &c) != NULL || c < 0 || c > 255)
+  if (gfc_extract_int (e, &c) != NULL || c < 0 || c > UCHAR_MAX)
     {
       gfc_error ("Bad character in CHAR function at %L", &e->where);
       return &gfc_bad_expr;
@@ -1370,9 +1370,9 @@ gfc_simplify_ichar (gfc_expr * e)
       return &gfc_bad_expr;
     }
 
-  index = (int) e->value.character.string[0];
+  index = (unsigned char) e->value.character.string[0];
 
-  if (index < CHAR_MIN || index > CHAR_MAX)
+  if (index < 0 || index > UCHAR_MAX)
     {
       gfc_error ("Argument of ICHAR at %L out of range of this processor",
                 &e->where);

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 2d143ce..0743d0a 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2006-01-09  Feng Wang  <fengwang@nudt.edu.cn>
+
+       * gfortran.dg/ichar_2.f90: New test.
+
 2005-01-08  Erik Edelmann  <eedelman@gcc.gnu.org>
 
        PR fortran/25093

diff --git a/gcc/testsuite/gfortran.dg/ichar_2.f90 b/gcc/testsuite/gfortran.dg/ichar_2.f90
new file mode 100644 (file)
index 0000000..27b9ffc
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/ichar_2.f90
@@ -0,0 +1,16 @@
+! { dg-do run }
+! Test char and ichar intrinsic functions
+Program test
+integer i
+
+if (ichar (char (0)) .ne. 0) call abort ()
+if (ichar (char (255)) .ne. 255) call abort ()
+if (ichar (char (127)) .ne. 127) call abort ()
+
+i = 0
+if (ichar (char (i)) .ne. i) call abort ()
+i = 255
+if (ichar (char (i)) .ne. i) call abort ()
+i = 127
+if (ichar (char (i)) .ne. i) call abort ()
+end