Re: [PATCH 02/31] linux-nat: introduce pending_status_str

Pedro Alves
2023-03-27T18:06:17Z

On 2023-03-16 4:19 p.m., Andrew Burgess wrote:
> Pedro Alves <pedro@palves.net> writes:

>> I tweaked the comments in linux-nat.h in this new version. Let me know what
>> you think.
>>
>> From 10f88baff2e25fb87f37d1665bf283206171dd42 Mon Sep 17 00:00:00 2001
>> From: Pedro Alves <pedro@palves.net>
>> Date: Fri, 12 Nov 2021 20:50:29 +0000
>> Subject: [PATCH] linux-nat: introduce pending_status_str
>>
>> I noticed that some debug log output printing an lwp's pending status
>> wasn't considering lp->waitstatus. This fixes it, by introducing a
>> new pending_status_str function.
>>
>> Also fix the comment in gdb/linux-nat.h describing
>> lwp_info::waitstatus and details the description of lwp_info::status
>> while at it.
>>
>> Change-Id: I66e5c7a363d30a925b093b195d72925ce5b6b980
>> ---
>> gdb/linux-nat.c | 19 ++++++++++++++++---
>> gdb/linux-nat.h | 11 +++++++----
>> 2 files changed, 23 insertions(+), 7 deletions(-)
>
> LGTM.
>
> Reviewed-By: Andrew Burgess <aburgess@redhat.com>
>

Thanks, I've merged this one as well, as it's pretty much independent/standalone.