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/wayland_surface.c:
> -
>        wayland_shm_buffer_unref(surface->small_icon_buffer);
> -
>        surface->small_icon_buffer = NULL;
> -
>    }
> -
>    surface->xdg_toplevel_icon = NULL;
> - }
> -
> - if (icon_buf)
> - {
> -
>    surface->xdg_toplevel_icon = xdg_toplevel_icon_manager_v1_create_icon(process_wayland.xdg_toplevel_icon_manager_v1);
> -
> -
>    if (type == ICON_BIG) surface->big_icon_buffer = icon_buf;
> -
>    else surface->small_icon_buffer = icon_buf;
> -
> -
>    /* FIXME: what to do with scale ? */
> -
>    if (surface->big_icon_buffer) xdg_toplevel_icon_v1_add_buffer(surface->xdg_toplevel_icon, surface->big_icon_buffer->wl_buffer, 1);
> -
>    if (surface->small_icon_buffer) xdg_toplevel_icon_v1_add_buffer(surface->xdg_toplevel_icon, surface->small_icon_buffer->wl_buffer, 1);

Shorter lines:
```suggestion:-2+0
        if (surface->big_icon_buffer)
        {
            xdg_toplevel_icon_v1_add_buffer(surface->xdg_toplevel_icon,
                                            surface->big_icon_buffer->wl_buffer, 1);
        }
        if (surface->small_icon_buffer)
        {
            xdg_toplevel_icon_v1_add_buffer(surface->xdg_toplevel_icon,
                                            surface->small_icon_buffer->wl_buffer, 1);
        }
```