This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
gdb and binutils branch master updated. cdaa5b7326cf76c778aa304e925ad908688f022c
- From: palves at sourceware dot org
- To: gdb-cvs at sourceware dot org
- Date: 28 Oct 2013 16:56:15 -0000
- Subject: gdb and binutils branch master updated. cdaa5b7326cf76c778aa304e925ad908688f022c
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "gdb and binutils".
The branch, master has been updated
via cdaa5b7326cf76c778aa304e925ad908688f022c (commit)
via 94c57d6a62c6564e9a8ecbdbe3ee9ade20e5d269 (commit)
via fcf3daefe6e48a4f3b802f0adad2a16639cef126 (commit)
via c447ac0bfb51568d4c239dd11e016dc0281ee358 (commit)
via f05e4c1115dec06d408394e2d9a529d1475f96ee (commit)
from ca20d46296a700f493edaa05401b75b1759e78c8 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=cdaa5b7326cf76c778aa304e925ad908688f022c
commit cdaa5b7326cf76c778aa304e925ad908688f022c
Author: Pedro Alves <palves@redhat.com>
Date: Mon Oct 28 16:39:06 2013 +0000
infrun.c:process_event_stop_test: Reindent.
gdb/
2013-10-28 Pedro Alves <palves@redhat.com>
* infrun.c (process_event_stop_test): Remove unnecessary scoping
level and reindent.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=94c57d6a62c6564e9a8ecbdbe3ee9ade20e5d269
commit 94c57d6a62c6564e9a8ecbdbe3ee9ade20e5d269
Author: Pedro Alves <palves@redhat.com>
Date: Mon Oct 28 16:39:06 2013 +0000
infrun.c:handle_inferior_event: Make process_event_stop_test label a function.
Now that all ecs->random_signal handing is always done before the
'process_event_stop_test' label, we can easily make that a real
function and actually give it a describing comment that somewhat makes
sense.
Reindenting the new function will be handled in a follow up patch.
2013-10-28 Pedro Alves <palves@redhat.com>
* infrun.c (process_event_stop_test): New function, factored out
from handle_inferior_event.
(handle_inferior_event): 'process_event_stop_test' is now a
function instead of a goto label -- adjust.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=fcf3daefe6e48a4f3b802f0adad2a16639cef126
commit fcf3daefe6e48a4f3b802f0adad2a16639cef126
Author: Pedro Alves <palves@redhat.com>
Date: Mon Oct 28 16:39:06 2013 +0000
infrun.c:handle_inferior_event: Move process_event_stop_test goto label.
We only ever call "goto process_event_stop_test;" right after checking
that ecs->random_signal is clear. The code at the
process_event_stop_test label looks like:
/* For the program's own signals, act according to
the signal handling tables. */
if (ecs->random_signal)
{
... random signal handling ...
return;
}
else
{
... the stop tests that actually matter for the goto callers.
}
So this moves the label into the else branch. It'll make converting
process_event_stop_test into a function a bit clearer.
gdb/
2013-10-28 Pedro Alves <palves@redhat.com>
* infrun.c (handle_inferior_event): Move process_event_stop_test
goto label to the else branch of the ecs->random_signal check,
along with FRAME and GDBARCH re-fetching.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=c447ac0bfb51568d4c239dd11e016dc0281ee358
commit c447ac0bfb51568d4c239dd11e016dc0281ee358
Author: Pedro Alves <palves@redhat.com>
Date: Mon Oct 28 16:39:05 2013 +0000
infrun.c:handle_inferior_event: Put all ecs->random_signal tests together.
I recently added a new ecs->random_signal test after the "switch back to
stepped thread" code, and before the stepping tests. Looking at
making process_event_stop_test a proper function, I realized it'd be
better to keep ecs->random_signal related code together. To do that,
I needed to factor out the "switch back to stepped thread" code to a new
function, and call it in both the "random signal" and "not random
signal" paths.
gdb/
2013-10-28 Pedro Alves <palves@redhat.com>
* infrun.c (switch_back_to_stepped_thread): New function, factored
out from handle_inferior_event.
(handle_inferior_event): Adjust to call
switch_back_to_stepped_thread. Call it also at the tail of the
random signal handling, and return, instead of also handling
random signals just before the stepping tests.
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=f05e4c1115dec06d408394e2d9a529d1475f96ee
commit f05e4c1115dec06d408394e2d9a529d1475f96ee
Author: Pedro Alves <palves@redhat.com>
Date: Mon Oct 28 16:39:05 2013 +0000
infrun.c:handle_inferior_event: Remove some more dead code.
'ecs' is always memset before being passed to handle_inferior_event.
The stop func is only filled in later in the flow. And since "Remove
dead sets/clears of ecs->random signal", nothing ever sets
ecs->random_signal before this part is reached either.
(Also tested with some added assertions in place.)
gdb/
2013-10-28 Pedro Alves <palves@redhat.com>
* infrun.c (clear_stop_func): Delete.
(handle_inferior_event): Don't call clear_stop_func and don't
clear 'ecs->random_signal'.
-----------------------------------------------------------------------
Summary of changes:
gdb/ChangeLog | 33 ++++
gdb/infrun.c | 585 +++++++++++++++++++++++++++++----------------------------
2 files changed, 330 insertions(+), 288 deletions(-)
hooks/post-receive
--
gdb and binutils