Re: [PATCH v5 0/2] MR4049: ntdll: Implement thread-ID alerts using kqueue()/kevent()

[Brendan Shanks (＠bshanks)]

Brendan Shanks (@bshanks) commented about dlls/ntdll/unix/sync.c:
> -        case KERN_ABORTED: continue;
> -        case KERN_OPERATION_TIMED_OUT: return STATUS_TIMEOUT;
> -        default: return STATUS_INVALID_HANDLE;
> -        }
> +        ret = kevent( entry->kq, NULL, 0, &wait_event, 1, timeout ? ×pec : NULL );
> +
> +    } while (ret == -1 && errno == EINTR);
> +
> +    switch (ret)
> +    {
> +    case 1:
> +        return STATUS_ALERTED;
> +    case 0:
> +        return STATUS_TIMEOUT;
> +    default:
> +        assert( ret == -1 );
> I don't think we want to `assert()` for this case.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4049#note_49188