Re: [PATCH 07/11] gdb: maintain per-process-target list of resumed threads with pending wait status

[Pedro Alves <pedro@palves.net>]

On 2021-07-12 11:28 p.m., Simon Marchi wrote:
>>> Arggg, we don't want that.  Since that is in a process_stratum_target
>>> method, I'll change the message to call the pid_to_str method of the
>>> current object instead:
>>>
>>>       infrun_debug_printf ("removing from resumed threads with event list: %s",
>>> 			   this->pid_to_str (thread->ptid).c_str ());
>>>
>>> Does that sound good?
>>
>> It does not.  That would mean a target higher on the stack wouldn't have a chance
>> at printing the thread.
> 
> Since this is only for debug message purposes, and I find it clearer to
> see the spelled-out ptid anyway (the pid, lwp and tid values), I added a
> to_string method to ptid_t and used that here:
> 
>     std::string
>     ptid_t::to_string () const
>     {
>       return string_printf ("%d.%d.%d", m_pid, m_lwp, m_tid);
>     }

+1.