[PATCH v3 resend 1/2] ntdll/tests: Add more tests for \Device\NamedPipe\ root directory.
Jacek Caban
jacek at codeweavers.com
Tue Jan 4 16:01:25 CST 2022
Hi Jinoh,
On 1/4/22 11:07, Jinoh Kang wrote:
> + memset(&iosb, 0, sizeof(iosb));
> + iosb.Status = STATUS_PENDING;
> + status = pNtFsControlFile(handle, event, NULL, NULL, &iosb, FSCTL_PIPE_WAIT,
> + pipe_wait, pipe_wait_size, NULL, 0);
> + if (status == STATUS_PENDING && iosb.Status == STATUS_PENDING)
> + {
> + WaitForSingleObject(event, INFINITE);
> + }
> + if (SUCCEEDED(status) && iosb.Status != STATUS_PENDING)
> + {
> + status = iosb.Status;
> + }
SUCCEEDED() macro is meant for HRESULT. It's not really needed here, I
think that those two branches can be simplified to:
if (status == STATUS_SUCCESS)
{
WaitForSingleObject(event, INFINITE);
status = iosb.Status;
}
Thanks,
Jacek
More information about the wine-devel
mailing list