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?
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/7169#note_92812