Re: [RFA] Update UI events for tracepoints

Andrew Cagney
2001-06-06T06:06:00Z

> Hi,
>
> Currently, the only ui events implemented in gdb-events.sh is breakpoint
> stuff. Tracepoints would be a logical runner-up for someone just beginning
> to tinker with this. :-)
>
> Changes to tracepoint.c to follow after this.
>
> Comments/approval?

Yes! At last someone is looking at this! Just do it as two separate
commits. The s/free/xfree/ change is pretty obvious and can separatly
go straight in.

Andrew

PS: I've wondered about renaming gdb-events.* to gdb-hooks.* but I don't
care.

> Keith
>
>
> ChangeLog:
> 2001-05-29 Keith Seitz <keiths@redhat.com>
>
> * gdb-events.sh: Update copyright.
> Use xfree () instead of free ().
> (function_list): Add tracepoint events.
> * gdb-events.h: Regenerated.
> * gdb-events.c: Regenerated.
>
> Patch:
> Index: gdb-events.sh
> ===================================================================
> RCS file: /cvs/src/src/gdb/gdb-events.sh,v
> retrieving revision 1.5
> diff -u -p -r1.5 gdb-events.sh
> --- gdb-events.sh 2001/03/06 08:21:07 1.5
> +++ gdb-events.sh 2001/05/29 19:14:55
> @@ -1,7 +1,7 @@
> #!/bin/sh
>
> # User Interface Events.
> -# Copyright 1999, 2000 Free Software Foundation, Inc.
> +# Copyright 1999, 2000, 2001 Free Software Foundation, Inc.
> #
> # Contributed by Cygnus Solutions.
> #
> @@ -61,6 +61,9 @@ function_list ()
> f:void:breakpoint_create:int b:b
> f:void:breakpoint_delete:int b:b
> f:void:breakpoint_modify:int b:b
> +f:void:tracepoint_create:int number:number
> +f:void:tracepoint_delete:int number:number
> +f:void:tracepoint_modify:int number:number
> #*:void:annotate_starting_hook:void
> #*:void:annotate_stopped_hook:void
> #*:void:annotate_signalled_hook:void
> @@ -109,7 +112,7 @@ copyright ()
> {
> cat <<EOF
> /* User Interface Events.
> - Copyright 1999 Free Software Foundation, Inc.
> + Copyright 1999, 2000, 2001 Free Software Foundation, Inc.
>
> Contributed by Cygnus Solutions.
>
> @@ -488,7 +491,7 @@ gdb_events_deliver (struct gdb_events *v
> {
> struct event *event = delivering_events;
> delivering_events = event->next;
> - free (event);
> + xfree (event);
> }
> /* Process any pending events. Because one of the deliveries could
> bail out we move everything off of the pending queue onto an
> @@ -523,7 +526,7 @@ done
> cat <<EOF
> }
> delivering_events = event->next;
> - free (event);
> + xfree (event);
> }
> }
> EOF
>
>
>