Re: [RFA] Update UI events for tracepoints

[Andrew Cagney <ac131313 at cygnus dot com>]

Yes, ok.  Wow! This means that this is file is now more than just proof 
of concept!
BTW, I think deleting the old GDBtk only hooks should be an obvious fix :-)

	Andrew


> 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?
> 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
> 
> 
>