Re: [PATCH 0/2] MR4520: Don't let GUI's app write to console.

[Alexandre Julliard (＠julliard)]

Alexandre Julliard (@julliard) commented about programs/uninstaller/main.c:
> -
>    WINE_FIXME("LoadString failed with %ld\n", GetLastError());
> -
>    return;
> -
>    va_list va_args;
> -
>    WCHAR *str;
> -
>    DWORD len;
> -
> -
>    va_start(va_args, id);
> -
>    len = FormatMessageW(FORMAT_MESSAGE_FROM_STRING|FORMAT_MESSAGE_ALLOCATE_BUFFER,
> -
>                         fmt, 0, 0, (LPWSTR)&str, 0, &va_args);
> -
>    if (len > 0 || GetLastError() == ERROR_NO_WORK_DONE)
> -
>    {
> -
>        if (silent)
> -
>            MESSAGE("%ls", str);
> -
>        else
> -
>            MessageBoxW(NULL, str, L"", MB_OK | MB_ICONSTOP);
> -
>        LocalFree(str);

You should use IDS_APPNAME instead of an empty title for the message box.