Re: [PATCH 1/1] gdb, breakpoint: use breakpoint_debug_printf

[Kevin Buettner <kevinb@redhat.com>]

On Wed, 09 Jul 2025 13:21:14 -0600
Tom Tromey <tom@tromey.com> wrote:

> >>>>> "Klaus" == Klaus Gerlicher <klaus.gerlicher@intel.com> writes:  
> 
> Klaus> Some debug prints in breakpoint.c are using
> Klaus> infrun_debug_printf ().  This does not seem to be the
> Klaus> right debug print macro.  
> 
> Klaus> Use breakpoint_debug_printf () exclusively in this file.  
> 
> I am not so sure about this.  It seems to me that some of these may
> mostly of interest when infrun debugging is enabled.  For example:
> 
> Klaus> -      infrun_debug_printf ("incorrect thread or task, not
> Klaus> stopping");
> Klaus> +      breakpoint_debug_printf ("incorrect thread or task, not
> Klaus> stopping");  
> 
> This kind of thing is a decision impacting infrun decisions.

Hmm, good point.  I rescind my approval.

Kevin