From 26afc0d7031bc2041ea17a4c25b04454b4e4feab Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Thu, 7 Jul 2011 13:32:55 +0000 Subject: [PATCH] 2011-07-07 Hui Zhu Revert: 2011-07-06 Hui Zhu * remote.c (remote_start_remote): Add TRY_CATCH for remote_get_trace_status. * tracepoint.c (disconnect_tracing): Ditto. --- gdb/ChangeLog | 8 ++++++++ gdb/remote.c | 13 +------------ gdb/tracepoint.c | 14 +------------- 3 files changed, 10 insertions(+), 25 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index f3ba9c7..7a76008 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2011-07-07 Hui Zhu + + Revert: + 2011-07-06 Hui Zhu + * remote.c (remote_start_remote): Add TRY_CATCH for + remote_get_trace_status. + * tracepoint.c (disconnect_tracing): Ditto. + 2011-07-07 Andrew Burgess * cli/cli-setshow.c (do_setshow_command): Display var_zinteger diff --git a/gdb/remote.c b/gdb/remote.c index b03ef59..19cc0b6 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -3146,8 +3146,6 @@ remote_start_remote (int from_tty, struct target_ops *target, int extended_p) struct remote_state *rs = get_remote_state (); struct packet_config *noack_config; char *wait_status = NULL; - int ret = 0; - volatile struct gdb_exception ex; immediate_quit++; /* Allow user to interrupt it. */ @@ -3391,16 +3389,7 @@ remote_start_remote (int from_tty, struct target_ops *target, int extended_p) /* Possibly the target has been engaged in a trace run started previously; find out where things are at. */ - TRY_CATCH (ex, RETURN_MASK_ERROR) - { - ret = remote_get_trace_status (current_trace_status ()); - } - if (ex.reason < 0) - { - warning (_("%s"), ex.message); - ret = -1; - } - if (ret != -1) + if (remote_get_trace_status (current_trace_status ()) != -1) { struct uploaded_tp *uploaded_tps = NULL; struct uploaded_tsv *uploaded_tsvs = NULL; diff --git a/gdb/tracepoint.c b/gdb/tracepoint.c index 4c54d2d..2d62f4d 100644 --- a/gdb/tracepoint.c +++ b/gdb/tracepoint.c @@ -1900,23 +1900,11 @@ trace_status_mi (int on_stop) void disconnect_tracing (int from_tty) { - int ret = 0; - volatile struct gdb_exception ex; - /* It can happen that the target that was tracing went away on its own, and we didn't notice. Get a status update, and if the current target doesn't even do tracing, then assume it's not running anymore. */ - TRY_CATCH (ex, RETURN_MASK_ERROR) - { - ret = target_get_trace_status (current_trace_status ()); - } - if (ex.reason < 0) - { - warning (_("%s"), ex.message); - ret = -1; - } - if (ret < 0) + if (target_get_trace_status (current_trace_status ()) < 0) current_trace_status ()->running = 0; /* If running interactively, give the user the option to cancel and -- 2.7.4