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

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6869#note_88285