Re: [PATCH 0/2] MR661: wshom: Support default style argument in IWshShell3::Run.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/wshom.ocx/shell.c:
> if (!style || !wait || !exit_code)
>          return E_POINTER;
>
> -    VariantInit(&s);
> -    hr = VariantChangeType(&s, style, 0, VT_I4);
> -    if (FAILED(hr))
> -    {
> -        ERR("failed to convert style argument, %#lx\n", hr);
> -        return hr;
> +    if (is_optional_argument(style))
> +        show = 1;
> +    else {

Please use existing constant name for this instead of just 1.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/661#note_6393