This is the mail archive of the gdb-cvs@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]