This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: GDB MI Reverse Commands added [1 of 3]
- From: Vladimir Prus <vladimir at codesourcery dot com>
- To: Michael Snyder <msnyder at vmware dot com>
- Cc: Jakob Engblom <jakob at virtutech dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Wed, 16 Dec 2009 10:54:35 +0300
- Subject: Re: GDB MI Reverse Commands added [1 of 3]
- References: <00ce01ca265a$ccb66ca0$662345e0$@com> <4A95E670.9040402@vmware.com> <4B27E565.5060305@vmware.com>
On Tuesday 15 December 2009 22:37:09 Michael Snyder wrote:
> Vladimir, as far as I can tell, this seems to be the most recent
> version of part 1 of this patch, awaiting approval now that the
> copyright paperwork is completed.
>
> 2009-08-31 Tomas Holmberg >th@virtutech.com>
> * mi/mi-main.c: Added the --reverse flag to the following MI
> commands: exec-continue, exec-finish, exec-next, exec-step,
> exec-next-instruction, exec-step-instruction. This is to
> support reverse execution over the MI interface to gdb.
Michael,
thanks. This patch is OK, with the following fixes:
> Index: mi-main.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/mi/mi-main.c,v
> retrieving revision 1.156
> diff -u -p -r1.156 mi-main.c
> --- mi-main.c 2 Jul 2009 17:25:59 -0000 1.156
> +++ mi-main.c 27 Aug 2009 01:45:23 -0000
> @@ -88,8 +88,8 @@ static void mi_cmd_execute (struct mi_pa
>
> static void mi_execute_cli_command (const char *cmd, int args_p,
> const char *args);
> -static void mi_execute_async_cli_command (char *cli_command,
> - char **argv, int argc);
> +static void mi_execute_async_cli_command (char *cli_command,
> + char **argv, int argc);
This is spurious formatting change that must be undone. There's a lot of
similar changes in the patch â that must be similarly undone.
> +/* continue in reverse direction:
> + XXX: code duplicated from reverse.c */
> +
> +static void
> +exec_direction_default (void *notused)
> +{
> + /* Return execution direction to default state. */
> + execution_direction = EXEC_FORWARD;
> +}
It should be straight-forward to make the function in reverse.c globally
visible, and remove this copy-paste.
Thanks,
Volodya