From: Andrew Cagney <cagney@redhat.com>
Date: Sun, 13 Jan 2002 21:58:16 +0000 (+0000)
Subject: Rewrite target_signal_to_name().
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=3cecdd5e5a8ed4d218e5f01c16e194fa92c97910;p=pf3gnuchains%2Fpf3gnuchains3x.git

Rewrite target_signal_to_name().
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index d18a67787e..9ff1de2309 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,10 @@
 2002-01-13  Andrew Cagney  <ac131313@redhat.com>
 
+	* signals.c (target_signal_to_name): Rewrite.  Only use
+	signals[].name when in bounds and non-NULL.
+	
+2002-01-13  Andrew Cagney  <ac131313@redhat.com>
+
 	From Petr Ledvina <ledvinap@kae.zcu.cz>:
 	* signals.c (target_signal_to_name): Verify that SIG is within the
 	bounds of the signals array.
diff --git a/gdb/signals.c b/gdb/signals.c
index cf0078d94f..d37ec06ee9 100644
--- a/gdb/signals.c
+++ b/gdb/signals.c
@@ -210,14 +210,13 @@ target_signal_to_string (enum target_signal sig)
 char *
 target_signal_to_name (enum target_signal sig)
 {
-  if (sig == TARGET_SIGNAL_UNKNOWN)
-    /* I think the code which prints this will always print it along with
-       the string, so no need to be verbose.  */
-    return "?";
-  else if ((sig >= TARGET_SIGNAL_FIRST) && (sig <= TARGET_SIGNAL_LAST))
+  if ((sig >= TARGET_SIGNAL_FIRST) && (sig <= TARGET_SIGNAL_LAST)
+      && signals[sig].name != NULL)
     return signals[sig].name;
   else
-    return signals[sig].name;
+    /* I think the code which prints this will always print it along
+       with the string, so no need to be verbose (very old comment).  */
+    return "?";
 }
 
 /* Given a name, return its signal.  */