On Fri Jan 24 19:51:58 2025 +0000, Paul Gofman wrote:
On a separate note, I'd also test GetLastError() after Sleep / SleepEx (doing SetLastError(0xdeadbeef) before each of test) and SleepEx return value, to see that it is all handled correctly with new NtDelayExecution returns. If there are any apps depending on proper "no yield" status and if GetLastError() reflects that I'd expect that to be used more often than calling Nt function directly.
I had a standalone test for this actually, and the patch fixed the Sleep/SleepEx GLE in the same way. I can add tests for these too, but where should I add them? A separate test for Sleep(Ex) in `dlls/kernelbase/tests/sync.c`, or just as an addition to the existing NtDelayExecution test?