[Bug win32/14529] Make gdb capable of JIT-debugging on W32

Tom Tromey tromey@redhat.com
Mon Sep 10 18:15:00 GMT 2012


>>>>> "LRN" == LRN  <lrn1986@gmail.com> writes:

LRN> diff --git a/gdb/infcmd.c b/gdb/infcmd.c
[...]

LRN> +#if W32_JITDBG
LRN> +void
LRN> +signal_event_command (char *args, int from_tty)
LRN> +{

Two notes here.

First, does it make sense to put this in windows-nat.c instead?
(I don't know anything about the Windows port...)
It doesn't matter hugely.

Second, it is odd to call this a _command and give it the above
prototype.  Normally this convention is used for things which are really
commands -- visible to the user from the gdb CLI.

I suppose this is so you can use catch_command_errors; but it seems like
you could equally well use catch_errors.

LRN> +  struct cleanup *back_to = make_cleanup (null_cleanup, NULL);

You don't need a null cleanup unless you plan to make other cleanups.
But AFAICT there aren't any.

LRN> +  dont_repeat ();		/* Not for the faint of heart */

You only need this for a real command.

Tom



More information about the Gdb-patches mailing list