2002-05-09 Daniel Jacobowitz <drow@mvista.com>

        * signals.h: Update comments.
        (enum target_signal): Remove conditional compilation around
        Mach-specific signals.  Move them to after TARGET_SIGNAL_DEFAULT.

diff --git a/include/gdb/ChangeLog b/include/gdb/ChangeLog
index 5d83a64a0118187ca3e8af78cb6c733add5d3dc9..b4fee9d209f35d70d90fd1a71aca3aaa53e4df47 100644 (file)
--- a/include/gdb/ChangeLog
+++ b/include/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2002-05-09  Daniel Jacobowitz  <drow@mvista.com>
+
+       * signals.h: Update comments.
+       (enum target_signal): Remove conditional compilation around
+       Mach-specific signals.  Move them to after TARGET_SIGNAL_DEFAULT.
+
 2002-03-10  Daniel Jacobowitz  <drow@mvista.com>
 
        * signals.h: New file, from gdb/defs.h.

diff --git a/include/gdb/signals.h b/include/gdb/signals.h
index 17107504e4a42b35ad6ffea70f605617629e1bbd..b6f5d4853125e22ee31680636154b764a752ee05 100644 (file)
--- a/include/gdb/signals.h
+++ b/include/gdb/signals.h
@@ -32,7 +32,8 @@
    Since these numbers have actually made it out into other software
    (stubs, etc.), you mustn't disturb the assigned numbering.  If you
    need to add new signals here, add them to the end of the explicitly
-   numbered signals.
+   numbered signals, at the comment marker.  Add them unconditionally,
+   not within any #if or #ifdef.
 
    This is based strongly on Unix/POSIX signals for several reasons:
    (1) This set of signals represents a widely-accepted attempt to
@@ -208,15 +209,6 @@ enum target_signal
     TARGET_SIGNAL_REALTIME_126,
     TARGET_SIGNAL_REALTIME_127,
 
-#if defined(MACH) || defined(__MACH__)
-    /* Mach exceptions */
-    TARGET_EXC_BAD_ACCESS,
-    TARGET_EXC_BAD_INSTRUCTION,
-    TARGET_EXC_ARITHMETIC,
-    TARGET_EXC_EMULATION,
-    TARGET_EXC_SOFTWARE,
-    TARGET_EXC_BREAKPOINT,
-#endif
     TARGET_SIGNAL_INFO,
 
     /* Some signal we don't know about.  */
@@ -226,6 +218,18 @@ enum target_signal
        (for passing to proceed and so on).  */
     TARGET_SIGNAL_DEFAULT,
 
+    /* Mach exceptions.  In versions of GDB before 5.2, these were just before
+       TARGET_SIGNAL_INFO if you were compiling on a Mach host (and missing
+       otherwise).  */
+    TARGET_EXC_BAD_ACCESS,
+    TARGET_EXC_BAD_INSTRUCTION,
+    TARGET_EXC_ARITHMETIC,
+    TARGET_EXC_EMULATION,
+    TARGET_EXC_SOFTWARE,
+    TARGET_EXC_BREAKPOINT,
+
+    /* If you are adding a new signal, add it just above this comment.  */
+
     /* Last and unused enum value, for sizing arrays, etc.  */
     TARGET_SIGNAL_LAST
   };