diff options
| author | Pedro Alves <palves@redhat.com> | 2013-09-27 13:23:32 +0000 |
|---|---|---|
| committer | Pedro Alves <palves@redhat.com> | 2013-09-27 13:23:32 +0000 |
| commit | f1844708c5c7fdf3eb6aea2f214e14f042bff0fc (patch) | |
| tree | 58161094b96612e54b8ac19ef9ac8204ee587d62 /gdb/remote.c | |
| parent | 9c4a8bfef58647615ebc96a02f8e9a468f3250d4 (diff) | |
| download | gdb-f1844708c5c7fdf3eb6aea2f214e14f042bff0fc.tar.gz | |
remote.c: Remove unnecessary fields from 'struct stop_reply'.
I noticed these fields aren't really necessary -- if the T stop reply
indicated any we have any special event, the fallthrough doesn't
really do anything.
Tested on x86_64 Fedora 17 w/ local gdbserver, and also confirmed
"catch load" against a Windows gdbserver running under Wine, which
exercises TARGET_WAITKIND_LOADED, still works as expected.
gdb/
2013-09-27 Pedro Alves <palves@redhat.com>
* remote.c (struct stop_reply) <solibs_changed, replay_event>:
Delete fields.
(remote_parse_stop_reply): Adjust, setting event->ws.kind
directly.
Diffstat (limited to 'gdb/remote.c')
| -rw-r--r-- | gdb/remote.c | 31 |
1 files changed, 11 insertions, 20 deletions
diff --git a/gdb/remote.c b/gdb/remote.c index 2e116d92efe..0fa1e2b029b 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -5284,9 +5284,6 @@ typedef struct stop_reply int stopped_by_watchpoint_p; CORE_ADDR watch_data_address; - int solibs_changed; - int replay_event; - int core; } *stop_reply_p; @@ -5546,8 +5543,6 @@ remote_parse_stop_reply (char *buf, struct stop_reply *event) event->ptid = null_ptid; event->ws.kind = TARGET_WAITKIND_IGNORE; event->ws.value.integer = 0; - event->solibs_changed = 0; - event->replay_event = 0; event->stopped_by_watchpoint_p = 0; event->regcache = NULL; event->core = -1; @@ -5611,15 +5606,14 @@ Packet: '%s'\n"), while (*p_temp && *p_temp != ';') p_temp++; - event->solibs_changed = 1; + event->ws.kind = TARGET_WAITKIND_LOADED; p = p_temp; } else if (strncmp (p, "replaylog", p1 - p) == 0) { - /* NO_HISTORY event. - p1 will indicate "begin" or "end", but - it makes no difference for now, so ignore it. */ - event->replay_event = 1; + event->ws.kind = TARGET_WAITKIND_NO_HISTORY; + /* p1 will indicate "begin" or "end", but it makes + no difference for now, so ignore it. */ p_temp = strchr (p1 + 1, ';'); if (p_temp) p = p_temp; @@ -5675,18 +5669,15 @@ Packet: '%s'\n"), buf, p); ++p; } + + if (event->ws.kind != TARGET_WAITKIND_IGNORE) + break; + /* fall through */ case 'S': /* Old style status, just signal only. */ - if (event->solibs_changed) - event->ws.kind = TARGET_WAITKIND_LOADED; - else if (event->replay_event) - event->ws.kind = TARGET_WAITKIND_NO_HISTORY; - else - { - event->ws.kind = TARGET_WAITKIND_STOPPED; - event->ws.value.sig = (enum gdb_signal) - (((fromhex (buf[1])) << 4) + (fromhex (buf[2]))); - } + event->ws.kind = TARGET_WAITKIND_STOPPED; + event->ws.value.sig = (enum gdb_signal) + (((fromhex (buf[1])) << 4) + (fromhex (buf[2]))); break; case 'W': /* Target exited. */ case 'X': |