* s390-tdep.c (s390_register_raw_size): Replace unnecessarily

obscure range comparison with legible equivalent.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 9091a608948fbd6df68d618065ab568eaf4a6db9..6d55fa7a19c3748647738b9124715694927605ee 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2001-12-19  Jim Blandy  <jimb@redhat.com>
+
+       * s390-tdep.c (s390_register_raw_size): Replace unnecessarily
+       obscure range comparison with legible equivalent.
+
 Wed Dec 19 12:18:57 2001  Jeffrey A Law  (law@redhat.com)
 
        * config/pa/tm-hppa.h (STORE_RETURN_VALUE): Use hppa_store_return_value.

diff --git a/gdb/s390-tdep.c b/gdb/s390-tdep.c
index ddeacadd7fe51af0b8016cb941de00e8720af398..ba8c9a3e31457ec41c7cbccff359f2d63542aa9f 100644 (file)
--- a/gdb/s390-tdep.c
+++ b/gdb/s390-tdep.c
@@ -42,14 +42,15 @@
 
 
 /* Number of bytes of storage in the actual machine representation
-   for register N. 
-   Note that the unsigned cast here forces the result of the
-   subtraction to very high positive values if N < S390_FP0_REGNUM */
+   for register N.  */
 int
 s390_register_raw_size (int reg_nr)
 {
-  return ((unsigned) reg_nr - S390_FP0_REGNUM) <
-    S390_NUM_FPRS ? S390_FPR_SIZE : 4;
+  if (S390_FP0_REGNUM <= reg_nr
+      && reg_nr < S390_FP0_REGNUM + S390_NUM_FPRS)
+    return S390_FPR_SIZE;
+  else
+    return 4;
 }
 
 int
@@ -1759,6 +1760,7 @@ s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
   gdbarch = gdbarch_alloc (&info, NULL);
 
   set_gdbarch_believe_pcc_promotion (gdbarch, 0);
+  set_gdbarch_char_signed (gdbarch, 0);
 
   set_gdbarch_frame_args_skip (gdbarch, 0);
   set_gdbarch_frame_args_address (gdbarch, s390_frame_args_address);