Re: [PATCH v3 0/2] MR7169: ntdll: Fix the return value of NtDelayExecution.

[Paul Gofman (＠gofman)]

Paul Gofman (@gofman) commented about dlls/ntdll/tests/sync.c:
> -
>        /* test yields */
> -
>        if (!test->timeout)
> -
>        {
> -
>            todo_wine_if(status == STATUS_TIMEOUT)
> -
>            ok( status == STATUS_SUCCESS || status == STATUS_NO_YIELD_PERFORMED ||
> -
>                (test->alertable && test->queue_apc && status == STATUS_USER_APC),
> -
>                "%s: got %#lx.\n", test->desc, status );
> -
> -
>            if (status == STATUS_NO_YIELD_PERFORMED)
> -
>                noyields[test->alertable ? 1 : 0]++;
> -
>        }
> -
>        /* test delays */
> -
>        else
> -
>        {
> -
>            todo_wine_if(status == STATUS_TIMEOUT)
> -
>            ok( status == STATUS_SUCCESS ||

checks in else look very close to if? In this probably worth removing the if and only conditionally increment noyields and coalesce the ok() condition into single expression?