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?