[PATCH v2] Cygwin: pthread: Fix a race issue introduced by the commit 2c5433e5da82
Ken Brown
kbrown@cornell.edu
Sat Jun 1 16:08:51 GMT 2024
Hi Takashi,
On 6/1/2024 10:18 AM, Takashi Yano via Cygwin wrote:
> int
> pthread::once (pthread_once_t *once_control, void (*init_routine) (void))
> {
> /* Sign bit of once_control->state is used as done flag.
> Similary, the next significant bit is used as destroyed flag. */
Typo: Similarly
> const int done = INT_MIN; /* 0b1000000000000000 */
> const int destroyed = INT_MIN >> 1; /* 0b1100000000000000 */
Shouldn't the constants in the comments have 32 bits? Other than that,
LGTM. (But you should wait for Bruno to confirm before you commit.)
Ken
More information about the Cygwin
mailing list