* gdb.base/sigall.c (gen_*): Pass argument to handle_* to avoid
authorJim Kingdon <jkingdon@engr.sgi.com>
Thu, 12 Jan 1995 20:12:07 +0000 (20:12 +0000)
committerJim Kingdon <jkingdon@engr.sgi.com>
Thu, 12 Jan 1995 20:12:07 +0000 (20:12 +0000)
compiler warning with dec alpha compiler.

* gdb.base/commands.exp (if_while_breakpoint_command_test): Remove
extra \n from gdb_test call (this explains why the prompt test
that I removed yesterday was there).
(user_defined_command_test): Remove extra \n and prompt test.

* gdb.base/sigall.c: If SIGABRT == SIGLOST, just act as if SIGLOST
doesn't exist.  Likewise if SIGIO == SIGPOLL, pretend SIGPOLL
doesn't exist.
* gdb.base/sigall.exp: Remove special handling for SIGIO and SIGPOLL.
Also remove alpha workaround, which turns out to be because
SIGABRT == SIGLOST.

gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/sigall.exp

index 8b9e1b3934f4ce6f4fd85d8afdf71be79a6b0a9f..bd1e6f36acd672fab6f45e86bec375abb64689dc 100644 (file)
@@ -1,5 +1,20 @@
 Thu Jan 12 01:14:53 1995  Jim Kingdon  <kingdon@lioth.cygnus.com>
 
+       * gdb.base/sigall.c (gen_*): Pass argument to handle_* to avoid
+       compiler warning with dec alpha compiler.
+
+       * gdb.base/commands.exp (if_while_breakpoint_command_test): Remove
+       extra \n from gdb_test call (this explains why the prompt test
+       that I removed yesterday was there).
+       (user_defined_command_test): Remove extra \n and prompt test.
+
+       * gdb.base/sigall.c: If SIGABRT == SIGLOST, just act as if SIGLOST
+       doesn't exist.  Likewise if SIGIO == SIGPOLL, pretend SIGPOLL
+       doesn't exist.
+       * gdb.base/sigall.exp: Remove special handling for SIGIO and SIGPOLL.
+       Also remove alpha workaround, which turns out to be because
+       SIGABRT == SIGLOST.
+
        * gdb.base/sigall.exp: Add workaround for alpha weirdness.
 
        * gdb.base/help.exp: Don't test "help show".
index 3f3bab69c3ee750d3bf5efc6c75e568a4a113c2b..10de1aa31b7b7fd0000ed0a53afeecba631123d8 100644 (file)
@@ -44,32 +44,12 @@ proc test_one_sig {nextsig} {
     gdb_test "b handle_$thissig" "Breakpoint \[0-9\]"
     gdb_test "b gen_$nextsig" "Breakpoint \[0-9\]"
     if $this_sig_supported then {
-       if {"$thissig" == "POLL"} then {
-           # If SIGIO and SIGPOLL are the same signal it gets reported
-           # as SIGIO; it is a feature for GDB to always use the "preferred"
-           # name (the POSIX name if there is one, etc.); I suspect that is
-           # SIGIO in this case but I'm not sure.
-           set pattern "(POLL|IO)"
-       } else {
-           set pattern $thissig
-       }
        gdb_test "continue" \
-           "Continuing.*Program received signal SIG$pattern" \
+           "Continuing.*Program received signal SIG$thissig" \
            "get signal $thissig"
     }
-    if {"$thissig" == "IO"} then {
-       # SIGIO and SIGPOLL might be the same signal.
-       # If so we end up at handle_POLL because we
-       # establish the handler for SIGPOLL after SIGIO.
-       set pattern "Breakpoint.*handle_(POLL|IO)"
-    } else {
-       set pattern "Breakpoint.*handle_$thissig" 
-    }
 
-    # On the alpha we don't end up in handle_ABRT.  I don't know why.
-    if {"$thissig" != "ABRT" || ![istarget "*-*-osf*"]} then {
-       gdb_test "continue" $pattern "send signal $thissig"
-    }
+    gdb_test "continue" "Breakpoint.*handle_$thissig" "send signal $thissig"
 
     send "continue\n"
     expect {
@@ -107,13 +87,6 @@ test_one_sig CHLD
 test_one_sig TTIN
 test_one_sig TTOU
 test_one_sig IO
-
-# Set an extra breakpoint at handle_POLL, to deal with the fact that
-# SIGIO and SIGPOLL might be the same signal.
-
-gdb_test "b handle_POLL" "Breakpoint \[0-9\]" \
-    "set extra handle_POLL breakpoint"
-
 test_one_sig XCPU
 test_one_sig XFSZ
 test_one_sig VTALRM