Re: [PATCH v21 0/2] MR6869: timeout: Windows 10 introduce the timeout command

[eric pouech (＠epo)]

eric pouech (@epo) commented about programs/timeout/tests/timeout.c:
> - GetExitCodeProcess(process_info.hProcess, &exitcode);
> - CloseHandle(process_info.hProcess);
> - CloseHandle(process_info.hThread);
> - ok_(file, line)(exitcode == exitcode_expected, "Expected exitcode %d, got %ld\n",
> -
>                exitcode_expected, exitcode);
>
> +}
> -
>
> +static void test_basic(void)
> +{
> - /* No options */
> - run_timeout_stdin("", 1);
> -
> - /* /? */
> - run_timeout_stdin("/?", 0);
> -
> - if (VerifyConsoleIoHandle(GetStdHandle(STD_INPUT_HANDLE)))

The VerifyConsoleIoHandle (is previous comments) is meant for the main program, not the tests