Check the right proc name
Commit Message
In lib/gdbserver-support.exp, we rename gdb_exit to
gdbserver_orig_gdb_exit, but we check the existence gdbserver_gdb_exit.
We should check gdbserver_orig_gdb_exit instead. Looks it is a typo
or an oversight.
gdb/testsuite:
2016-09-21 Yao Qi <yao.qi@linaro.org>
* lib/gdbserver-support.exp: Check the existence of
gdbserver_orig_gdb_exit rather than gdbserver_gdb_exit.
---
gdb/testsuite/lib/gdbserver-support.exp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On 09/21/2016 04:40 PM, Yao Qi wrote:
> In lib/gdbserver-support.exp, we rename gdb_exit to
> gdbserver_orig_gdb_exit, but we check the existence gdbserver_gdb_exit.
> We should check gdbserver_orig_gdb_exit instead. Looks it is a typo
> or an oversight.
Definitely. OK.
Thanks,
Pedro Alves
On Wed, Sep 21, 2016 at 4:52 PM, Pedro Alves <palves@redhat.com> wrote:
>
> Definitely. OK.
>
Patch is pushed in.
@@ -358,7 +358,7 @@ proc close_gdbserver {} {
# Hook into GDB exit, and close GDBserver.
-if { [info procs gdbserver_gdb_exit] == "" } {
+if { [info procs gdbserver_orig_gdb_exit] == "" } {
rename gdb_exit gdbserver_orig_gdb_exit
}
proc gdb_exit {} {