Re: [PATCH v4 0/1] MR6869: timeout: Windows 10 introduce the timeout command

[eric pouech (＠epo)]

eric pouech (@epo) commented about programs/timeout/main.c:
> - }
> -
> - for (int i = 1; i < argc; i++)
> - {
> -
>    if (wcscmp(argv[i], L"/?") == 0)
> -
>    {
> -
>        timeout_message(STRING_USAGE);
> -
>        ret = 0;
> -
>        goto done;
> -
>    }
> -
>    else if (wcsicmp(argv[i], L"/t") == 0)
> -
>    {
> -
>        if ((i + 1) < argc)
> -
>        {
> -
>            i++;
> -
>            wait_time = _wtoi(argv[i]);

two nitpicks here:

* native fails with "/T 10foobar"
* valid values for /T arg range from -1 to 99999