Re: [PATCH v4 0/6] MR2712: winewayland.drv: part 3: Handle dynamic wl_output events

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winewayland.drv/wayland_output.c:
>  /* We only update the display devices from the desktop process. */
>  if (GetCurrentProcessId() != desktop_pid) return;
>
> - wayland_queue_thread_event(desktop_hwnd, QS_SENDMESSAGE,
> -
>                           handle_init_display_devices);
> - send_message(desktop_hwnd, WM_WAYLAND_INIT_DISPLAY_DEVICES, 0, 0);

You were not previously waiting for the message reply, is this intentional that you do now?

If not, you can use `NtUserPostMessage` instead, or `send_notify_message` that you will find in other drivers. The difference is that `NtUserPostMessage` always processes the message asynchronously, while `send_notify_message` processes the message synchronously if it targets the current thread (which I expect will never be the case here anyway).