[PATCH 1/2] kernelbase: Use +threadname channel for SetThreadDescription().

[Zebediah Figura]

On 12/7/21 00:23, Brendan Shanks wrote:
> Signed-off-by: Brendan Shanks <bshanks at codeweavers.com>
> ---
>   dlls/kernelbase/thread.c | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/dlls/kernelbase/thread.c b/dlls/kernelbase/thread.c
> index e9c12d85771..3dd83befe10 100644
> --- a/dlls/kernelbase/thread.c
> +++ b/dlls/kernelbase/thread.c
> @@ -37,6 +37,7 @@
>   #include "wine/heap.h"
>   
>   WINE_DEFAULT_DEBUG_CHANNEL(thread);
> +WINE_DECLARE_DEBUG_CHANNEL(threadname);
>   
>   
>   /***********************************************************************
> @@ -420,7 +421,10 @@ HRESULT WINAPI DECLSPEC_HOTPATCH SetThreadDescription( HANDLE thread, PCWSTR des
>       THREAD_NAME_INFORMATION info;
>       int length;
>   
> -    TRACE( "(%p, %s)\n", thread, debugstr_w( description ));
> +    if (thread == GetCurrentThread())
> +        WARN_(threadname)( "Thread renamed to %s\n", debugstr_w( description ));
> +    else
> +        WARN_(threadname)( "Thread %p renamed to %s\n", thread, debugstr_w( description ));
>   
>       length = description ? lstrlenW( description ) * sizeof(WCHAR) : 0;
>   
> 

It really seems non-ideal that this is a thread handle, but the other, 
nearly identical, message prints a thread ID...