From: Mark Kettenis <kettenis@gnu.org>
Date: Sun, 16 Jan 2005 20:06:54 +0000 (+0000)
Subject: * sparc-tdep.c (sparc_integral_or_pointer_p): Simplify.
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=80ad1639f798a968c3a537be68f885f1ed055dec;p=deliverable%2Fbinutils-gdb.git

* sparc-tdep.c (sparc_integral_or_pointer_p): Simplify.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 57feab0825..9caec1f6b5 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2005-01-16  Mark Kettenis  <kettenis@gnu.org>
+
+	* sparc-tdep.c (sparc_integral_or_pointer_p): Simplify.
+
 2005-01-16  Andrew Cagney  <cagney@gnu.org>
 
 	* exceptions.c (struct catcher): Delete saved_error_pre_print and
diff --git a/gdb/sparc-tdep.c b/gdb/sparc-tdep.c
index b4051c9ba6..e17e0a716e 100644
--- a/gdb/sparc-tdep.c
+++ b/gdb/sparc-tdep.c
@@ -188,6 +188,8 @@ sparc_address_from_register (int regnum)
 static int
 sparc_integral_or_pointer_p (const struct type *type)
 {
+  int len = TYPE_LENGTH (type);
+
   switch (TYPE_CODE (type))
     {
     case TYPE_CODE_INT:
@@ -195,22 +197,14 @@ sparc_integral_or_pointer_p (const struct type *type)
     case TYPE_CODE_CHAR:
     case TYPE_CODE_ENUM:
     case TYPE_CODE_RANGE:
-      {
-	/* We have byte, half-word, word and extended-word/doubleword
-           integral types.  The doubleword is an extension to the
-           original 32-bit ABI by the SCD 2.4.x.  */
-	int len = TYPE_LENGTH (type);
-	return (len == 1 || len == 2 || len == 4 || len == 8);
-      }
-      return 1;
+      /* We have byte, half-word, word and extended-word/doubleword
+	 integral types.  The doubleword is an extension to the
+	 original 32-bit ABI by the SCD 2.4.x.  */
+      return (len == 1 || len == 2 || len == 4 || len == 8);
     case TYPE_CODE_PTR:
     case TYPE_CODE_REF:
-      {
-	/* Allow either 32-bit or 64-bit pointers.  */
-	int len = TYPE_LENGTH (type);
-	return (len == 4 || len == 8);
-      }
-      return 1;
+      /* Allow either 32-bit or 64-bit pointers.  */
+      return (len == 4 || len == 8);
     default:
       break;
     }