Re: [PATCH] breakpoint always inserted in record target

[Keith Seitz <keiths at redhat dot com>]

I don't normally chime in on patches, but I felt compelled to ask for 
one change in your patch (which otherwise looks okay to me):

On 04/11/2012 06:06 AM, Yao Qi wrote:
>
> +/* Invoke CALLBACK for each of bp_location.  */
> +
> +void
> +iterate_over_bp_locations (void (*callback) (struct bp_location *))
> +{
> +  struct bp_location *loc, **loc_tmp;
> +
> +  ALL_BP_LOCATIONS (loc, loc_tmp)
> +    {
> +      callback (loc);
> +    }
> +}
> +

Can you please define a typedef for this CALLBACK parameter?

Keith