This is the mail archive of the cygwin-patches mailing list for the Cygwin project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 1/2] Add pthread_getname_np and pthread_setname_np

From: Corinna Vinschen <corinna-cygwin at cygwin dot com>
To: cygwin-patches at cygwin dot com
Date: Tue, 23 Aug 2016 10:58:05 +0200
Subject: Re: [PATCH 1/2] Add pthread_getname_np and pthread_setname_np
Authentication-results: sourceware.org; auth=none
References: <20160822180848.351616-1-jon.turney@dronecode.org.uk> <20160822180848.351616-2-jon.turney@dronecode.org.uk>
Reply-to: cygwin-patches at cygwin dot com

Hi Jon,

On Aug 22 19:08, Jon Turney wrote:
> +#define NAMELEN 16

Please use a more descriptive name here, say  THR_NAMEMAX or so, and add
a "for Linux compat" comment.  With this change the patch is ok to apply.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

Attachment: signature.asc
Description: PGP signature

References:
- [PATCH 0/2] Thread name support (v2)
  - From: Jon Turney
- [PATCH 1/2] Add pthread_getname_np and pthread_setname_np
  - From: Jon Turney

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]