Clean up server-kill.exp
authorYao Qi <yao@codesourcery.com>
Wed, 8 Oct 2014 02:04:22 +0000 (10:04 +0800)
committerYao Qi <yao@codesourcery.com>
Sat, 11 Oct 2014 03:12:23 +0000 (11:12 +0800)
This patch is to remove some lines which looks unnecessary.  These
lines were added when server-kill.exp was added.  In the version 1,
https://sourceware.org/ml/gdb-patches/2013-03/msg00691.html the test
calls runto_main and delete breakpoint on main,

+if ![runto_main] {
+    return -1
+}
+
+# Otherwise the breakpoint at 'main' would not cause insert
breakpoints during
+# first step.
+delete_breakpoints

However, in the version 2
https://sourceware.org/ml/gdb-patches/2013-03/msg00854.html runto_main
is removed but delete_breakpoints is still there.  AFAICS, the line of
delete_breakpoints can be removed too.

gdb/testsuite:

2014-10-11  Yao Qi  <yao@codesourcery.com>

* gdb.server/server-kill.exp: Remove "delete_breakpoints".

gdb/testsuite/ChangeLog
gdb/testsuite/gdb.server/server-kill.exp

index e33cbd2..23eea6b 100644 (file)
@@ -1,5 +1,9 @@
 2014-10-11  Yao Qi  <yao@codesourcery.com>
 
+       * gdb.server/server-kill.exp: Remove "delete_breakpoints".
+
+2014-10-11  Yao Qi  <yao@codesourcery.com>
+
        * gdb.threads/thread-find.exp: Don't execute command
        "info threads".
        * gdb.threads/attach-into-signal.exp (corefunc): Likewise.
index 5d31103..b23c2b5 100644 (file)
@@ -33,10 +33,6 @@ gdb_test "disconnect" ".*"
 
 gdbserver_run ""
 
-# Otherwise the breakpoint at 'main' would not cause insert
-# breakpoints during first step.
-delete_breakpoints
-
 set server_pid [exp_pid -i [board_info target fileid]]
 remote_exec target "kill -9 $server_pid"