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 e33cbd27b724afd1e03e4416e0af71de1569f414..23eea6bf81f0b3d089009daafa752af31852c5ff 100644 (file)
@@ -1,3 +1,7 @@
+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
index 5d31103ef661832e88b171cdd9db92e7af7d32cf..b23c2b54d7f79c3c44a9b55688ffb155b7f66a4c 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"