Re: [PATCH v4 0/2] MR5178: winewayland.drv: set wayland app-id from the process name

[Alexandros Frantzis (＠afrantzis)]

Alexandros Frantzis (@afrantzis) commented about dlls/winewayland.drv/window.c:
>  }
>
> }
>
> +/*****************************************************************
> -
> -
> WAYLAND_SetWindowText
> - */
>
> +void WAYLAND_SetWindowText(HWND hwnd, LPCWSTR text)
> +{
> - struct wayland_surface *wsurface = wayland_surface_lock_hwnd(hwnd);
> -
> - TRACE("hwnd=%p text=%s\n", hwnd, wine_dbgstr_w(text));
> -
> - if (wsurface && wsurface->xdg_toplevel)

We need to protect against NULL wayland_surface * (the experimental code handled this differently).

Also please use `surface` instead of `wsurface` as the variable name (for consistency with how we ended up naming such variables in the upstream version).

```suggestion:-0+3
    if (surface)
    {
        if (surface->xdg_surface) wayland_surface_set_title(surface, text);
        pthread_mutex_unlock(&surface->mutex);
    }
```