Re: [PATCH v12 0/1] MR6938: winewayland.drv: Implement support for xdg-toplevel-icon.

[Alexandros Frantzis (＠afrantzis)]

Alexandros Frantzis (@afrantzis) commented about dlls/winewayland.drv/window.c:
> - ICONINFO ii;
> -
> - TRACE("hwnd=%p text=%u icon=%p\n", hwnd, type, icon);
> -
> - if (process_wayland.xdg_toplevel_icon_manager_v1)
> - {
> -
>    icon = get_window_icon(hwnd, type, icon, &ii);
> -
>    if (icon && (data = wayland_win_data_get(hwnd)))
> -
>    {
> -
>        wayland_surface_set_icon(data->wayland_surface, type, &ii);
> -
>        wayland_win_data_release(data);
> -
>    }
> - }
> - else if (!once)
> - {
> -
>    FIXME("missing xdg_toplevel_icon_manager_v1, returning\n");

Applications don't always use `SetWindowIcon` explicitly to set their icon, so it's better to put the message along with the other optional protocol error messages in `wayland_process_init()` in wayland.c