diff options
-rw-r--r-- | gdb/ChangeLog | 8 | ||||
-rw-r--r-- | gdb/remote.c | 13 | ||||
-rw-r--r-- | 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 <teawater@gmail.com> + + Revert: + 2011-07-06 Hui Zhu <teawater@gmail.com> + * remote.c (remote_start_remote): Add TRY_CATCH for + remote_get_trace_status. + * tracepoint.c (disconnect_tracing): Ditto. + 2011-07-07 Andrew Burgess <aburgess@broadcom.com> * 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 |