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

[Alexandros Frantzis (＠afrantzis)]

Alexandros Frantzis (@afrantzis) commented about dlls/winewayland.drv/window.c:
> return TRUE;
>  }
>
> +static HICON get_icon_info(HICON icon, ICONINFO *ii)
> +{
> +    return icon && NtUserGetIconInfo(icon, ii, NULL, NULL, NULL, 0) ? icon : NULL;
> +}
> +
> +static HICON get_window_icon(HWND hwnd, UINT type, HICON icon, ICONINFO *ret)
> +{
> +    icon = get_icon_info(icon, ret);
> +    if (!icon)
> +    {
> +            icon = get_icon_info((HICON)send_message(hwnd, WM_GETICON, type, 0), ret);

This seems to be indented one level too deep?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6938#note_99656