Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Michael Snyder <msnyder@vmware.com>
To: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Subject: [RFA] Resubmit, reverse debugging [0/5]
Date: Wed, 08 Oct 2008 02:17:00 -0000	[thread overview]
Message-ID: <48EC1781.2030005@vmware.com> (raw)

OK, after incorporating various review suggestions,
this patch supercedes the patch of 10/01/2008.

The following change log entries are not part of the patch,
but are provided to show what has changed since last time,
mostly in response to review suggestions.


2008-10-07  Michael Snyder  <msnyder@vmware.com>

         * target.h (to_set_exec_direction, to_get_exec_direction): Remove.
         (to_can_execute_reverse): New method.
         (enum exec_direction_kind): Move to inferior.h.
         * target.c (update_current_target): Inherit to_can_execute_reverse.
         Remove to_set_exec_direction, to_get_exec_direction.
         * inferior.h (enum exec_direction_kind): Move from target.h.

         * infrun.c (set_exec_direction_func): Move here from reverse.c.
         (show_exec_direction_func): Ditto.
         (proceed): Consult global execution_direction instead of
         target method.
         (handle_inferior_event): Ditto.

         * reverse.c (set_exec_direction_func): Move to infrun.c
         (show_exec_direction_func): Ditto.
         (exec_direction_default): Set infrun global variable.
         (exec_reverse_once): Consult infrun global direction variable.

         * infcmd.c (step_1): Consult infrun global direction variable.
         (step_once): Ditto.
         (until_next_command): Ditto.
         (finish_command): Ditto.

         * record.h (record_exec_direction): Delete.
         (RECORD_IS_REPLAY): Consult infrun global direction variable.

         * record.c: (record_wait_cleanups): Use infrun state variable.
         (record_wait): Ditto.
         (record_get_exec_direction, record_set_exec_direction): Remove.
         (record_can_execute_reverse): New target method.

         * remote.c (remote_resume): Use infrun state variable.
         (remote_get_exec_direction, remote_set_exec_direction): Remove.
         (remote_can_execute_reverse): New target method.

2008-10-07  Michael Snyder  <msnyder@vmware.com>

         * infrun.c (stepped_into_function): Rename 
handle_step_into_function.
         (stepped_into_function_backward):
         Rename handle_step_into_function_backward.

         * reverse.c: Update copyright.
         (exec_reverse_once): Add function header comment.
         (_initialize_reverse): Use i18n macros for set exec-direction.
2008-10-06  Michael Snyder  <msnyder@vmware.com>

         * breakpoint.c (breakpoint_silence): Rename to 
make_breakpoint_silent.
         * breakpoint.h: Ditto.
         * infcmd.c (finish_backward): Call make_breakpoint_silent
         instead of breakpoint_silence.

         * infcmd.c (finish_command): Reject async in reverse.
         * infcmd.c (finish_command): Use i18n macros.
         * infcmd.c: Minor function reordering.
         * infrun.c (step_into_function): Rename to stepped_into_function.
         Split into two versions (normal (forward), and reverse).
         (handle_inferior_event): Call stepped_into_function or
         stepped_into_function_backward, depending on exec_direction.
         * infrun.c (handle_inferior_event): Comment rewording.
         * remote.c (remote_get_exec_direction): Use i18n macros.
         (remote_set_exec_direction): Ditto.
         * infrun.c (handle_inferior_event, step_into_function): Formatting.
         * infrun.c (handle_inferior_event): Formatting, spelling fix.

         * infrun.c (handle_inferior_event): Add special case for
         "next" in reverse.
2008-10-05  Michael Snyder  <msnyder@promb-2s-dhcp59.eng.vmware.com>

         * reverse.c (exec_reverse_once): Call do_cleanups explicitly.
         * infrun.c (handle_inferior_event): Fix typo in comment.

2008-10-04  Michael Snyder  <msnyder@vmware.com>

         * target.c, target.h: Rename execdir to exec_direction.
         * record.c, record.h: Ditto.
         * reverse.c: Ditto.
         * remote.c: Ditto.

         * reverse.c (show_exec_direction_func): Don't error, just inform.

         * reverse.c (reverse-continue): Remove a comma from docs string,
         to avoid confusing output from 'apropos'.




             reply	other threads:[~2008-10-08  2:17 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-08  2:17 Michael Snyder [this message]
2008-10-10 17:44 ` Michael Snyder
2008-10-10 17:54   ` Daniel Jacobowitz
2008-10-10 18:38     ` Michael Snyder
2008-10-11  2:43       ` Daniel Jacobowitz
2008-10-11  2:48         ` Michael Snyder
2008-10-11  8:02           ` Eli Zaretskii
2008-10-13 19:21             ` Michael Snyder
2008-10-14  7:56               ` Jakob Engblom
2008-10-14  9:07                 ` teawater
2008-10-14 12:24                   ` Daniel Jacobowitz
2008-10-14 12:27           ` Daniel Jacobowitz
2008-10-15 18:20             ` Michael Snyder
2008-10-15 18:30               ` Daniel Jacobowitz
2008-10-15 18:42                 ` Michael Snyder
2008-10-15 18:47                   ` Pedro Alves
2008-10-15 18:49                     ` Michael Snyder
2008-10-11  9:47       ` Joel Brobecker
2008-10-13 19:22         ` Michael Snyder

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=48EC1781.2030005@vmware.com \
    --to=msnyder@vmware.com \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox