From: Doug Evans <dje@google.com>
Date: Sun, 27 Jun 2010 16:40:14 +0000 (+0000)
Subject: 	* value.c (value_static_field): Use `switch' instead of `if'.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=1596cb5d604ef0aae6d6a655e568816804c5582d;p=binutils-gdb.git

	* value.c (value_static_field): Use `switch' instead of `if'.
	Assert-fail if passed invalid TYPE_FIELD_LOC_KIND.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 2fb33bb1271..6e700a4d84d 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,8 @@
 2010-06-27  Doug Evans  <dje@google.com>
 
+	* value.c (value_static_field): Use `switch' instead of `if'.
+	Assert-fail if passed invalid TYPE_FIELD_LOC_KIND.
+
 	* valops.c (search_struct_field): Fix typo in error message.
 
 2010-06-26  Ulrich Weigand  <uweigand@de.ibm.com>
diff --git a/gdb/value.c b/gdb/value.c
index 5e0e8d8268b..1c0bc57792c 100644
--- a/gdb/value.c
+++ b/gdb/value.c
@@ -1850,7 +1850,7 @@ unpack_pointer (struct type *type, const gdb_byte *valaddr)
 }
 
 
-/* Get the value of the FIELDN'th field (which must be static) of
+/* Get the value of the FIELDNO'th field (which must be static) of
    TYPE.  Return NULL if the field doesn't exist or has been
    optimized out. */
 
@@ -1859,12 +1859,13 @@ value_static_field (struct type *type, int fieldno)
 {
   struct value *retval;
 
-  if (TYPE_FIELD_LOC_KIND (type, fieldno) == FIELD_LOC_KIND_PHYSADDR)
+  switch (TYPE_FIELD_LOC_KIND (type, fieldno))
     {
+    case FIELD_LOC_KIND_PHYSADDR:
       retval = value_at_lazy (TYPE_FIELD_TYPE (type, fieldno),
 			      TYPE_FIELD_STATIC_PHYSADDR (type, fieldno));
-    }
-  else
+      break;
+    case FIELD_LOC_KIND_PHYSNAME:
     {
       char *phys_name = TYPE_FIELD_STATIC_PHYSNAME (type, fieldno);
       /*TYPE_FIELD_NAME (type, fieldno);*/
@@ -1897,7 +1898,12 @@ value_static_field (struct type *type, int fieldno)
       if (retval && VALUE_LVAL (retval) == lval_memory)
 	SET_FIELD_PHYSADDR (TYPE_FIELD (type, fieldno),
 			    value_address (retval));
+      break;
     }
+    default:
+      gdb_assert (0);
+    }
+
   return retval;
 }