On Thu Aug 29 18:00:42 2024 +0000, Alexandros Frantzis wrote:
Thanks, I have some more proposed changes/fixes at https://gitlab.winehq.org/afrantzis/wine/-/commits/wip/wayland-buffer-queue . This MR effectively makes `wp_viewport` a requirement for the Wayland driver, even for non-scaled scenarios, so I would say let's check for it at startup like we do with other protocols and remove further runtime checks (can be done in a follow-up MR).
Thanks, I've split and included them.
I made the protocol a requirement before actually using it for buffer cropping, because it would break otherwise somewhere within this MR.
Made a couple of other tweaks, kept the 0x0 check but using min(1, ...) instead. I'm not sure it's really worth it, I would say that it's not supposed to be happening.