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:
> - */
>
> +void wayland_surface_set_icon(struct wayland_surface *surface, UINT type, ICONINFO *ii)
> +{
> - HDC hDC;
> - struct wayland_shm_buffer *icon_buf;
> -
> - assert(ii);
> - assert(surface->xdg_toplevel && surface->role == WAYLAND_SURFACE_ROLE_TOPLEVEL);
> -
> - hDC = NtGdiCreateCompatibleDC(0);
> - icon_buf = wayland_shm_buffer_from_color_bitmaps(hDC, ii->hbmColor, ii->hbmMask);
> - NtGdiDeleteObjectApp(hDC);
> -
> - if (surface->xdg_toplevel_icon)
> - {
> -
>    xdg_toplevel_icon_manager_v1_set_icon(process_wayland.xdg_toplevel_icon_manager_v1, surface->xdg_toplevel, NULL);

Let's try to avoid very long lines if possible:
```suggestion:-0+0
        xdg_toplevel_icon_manager_v1_set_icon(process_wayland.xdg_toplevel_icon_manager_v1,
                                              surface->xdg_toplevel, NULL);
```