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

[Paul Gofman (＠gofman)]

On Mon Jan 27 18:03:22 2025 +0000, William Horvath wrote:
> Tests 3 and 4 are alertable without APCs queued, tests 5 and 6 are
> alertable with APCs queued. But adding the extra `SleepEx(0, TRUE)` if
> we queued an APC is harmless.
>         if (tests[i].queue_apc)
>             SleepEx( 0, TRUE );
>
> after the alertable NtDelay and SleepEx.

Sorry, yes, we indeed don't have, but it is hard to follow that it never happens in the table. Maybe just leave SleepEx( 0, TRUE ); outside of any conditions and without any checks after that, to make it clear that  we make sure there are no leftover APCs whatever the test configuration is.

UPDATED: edited.