Re: [PATCH v3 0/6] MR9305: Reject pseudo-handles in NtWaitForMultipleObjects().

[Paul Gofman (＠gofman)]

Paul Gofman (@gofman) commented about dlls/kernel32/tests/sync.c:
> - {
> -
>    maxevents[0] = std_handles[i];
> -
>    SetLastError(0xdeadbeef);
> -
>    r = WaitForMultipleObjects(1, maxevents, FALSE, 100);
> -
>    ok(r != WAIT_FAILED, "expected non-fail result, got %lu (GetLastError()=%lu)\n", r, GetLastError());
>
> - timeout.QuadPart = -1000000;
> - maxevents[0] = GetCurrentThread();
> - status = pNtWaitForMultipleObjects(1, maxevents, WaitAny, FALSE, &timeout);
> - ok(status == STATUS_INVALID_HANDLE, "expected STATUS_INVALID_HANDLE, got %08lx\n", status);
> -
>    timeout.QuadPart = -1000000;
> -
>    status = pNtWaitForMultipleObjects(1, maxevents, WaitAny, FALSE, &timeout);
> -
>    ok(status == STATUS_INVALID_HANDLE, "expected STATUS_INVALID_HANDLE, got %08lx\n", status);
> - }
> -
> - for (i=1; i<MAXIMUM_WAIT_OBJECTS; i++)

Here and possibly elsewhere, spaces around '=' and '<'. If there is a bit of similar formatting in the existing test code I'd suggest to maybe change that too while there.