Re: [PATCH v2 0/6] MR2275: winewayland.drv: part 1: Introduce the Wayland driver

[Link Mauve (＠linkmauve)]

Link Mauve (@linkmauve) commented about dlls/winewayland.drv/wayland.c:
> struct wl_display *process_wl_display = NULL;
>
> +static pthread_mutex_t thread_wayland_mutex = PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP;
> +static struct wl_list thread_wayland_list = {&thread_wayland_list, &thread_wayland_list};
> -
>
> +/**********************************************************************
> -
> -
>      Registry handling
> - */
> -
>
> +static void registry_handle_global(void *data, struct wl_registry *registry,
> -
>                               uint32_t id, const char *interface,
> -
>                               uint32_t version)
>
> +{
> - struct wayland *wayland = data;
> -
> - TRACE("interface=%s version=%d\n id=%u\n", interface, version, id);

You have one `\n` too many here, also `version` is also unsigned:
```suggestion:-0+0
    TRACE("interface=%s version=%u id=%u\n", interface, version, id);
```