December 2024 - wine-gitlab

Re: [PATCH v10 0/6] MR6569: winex11: Refactor window state/config tracker to fix race conditions.
by Rémi Bernon (＠rbernon) 02 Dec '24

02 Dec '24

On Mon Dec 2 22:29:18 2024 +0000, Robbert van der Helm wrote: > Ah I completely missed the assignment there at the end when I was > grepping for usages of `wm_state_serial`. I still can't reliably get > Wine to accept configuration events I sent to it though. If I set the > `WM_STATE` property it does process the next event, but then it gets > stuck again after that because now `net_wm_state_serial` will be stuck > on a nonzero value. I could probably figure out a way to get Wine to > cooperate, but that's almost certainly going to break in a future Wine release. > To reiterate, my goal isn't necessarily to send `ConfigureNotify` events > to Wine. I just want a Wine window that's reparented to another window > to handle mouse events correctly, as otherwise Wine seems to interpret > pointer events as being relative to the root window. So for example, if > I have a regular top level X11 window at position 50x50 relative to the > top left of the root window, and I reparent a Wine window into that > window, clicking at coordinates 20x20 from the top left of the > reparented Wine window will cause Wine to register a click at coordinate > 70x70. Up until now it was possible to work around this by telling Wine > about its location on screen through `ConfigureNotify` events, but that > is of course a hack. > Since you've been working on Wine's X11 driver, what do you think is the > best way to approach this in a way that isn't likely to break again in > the future? Would extending Wine's existing tracking of > reparented/embedded windows so any events related to pointer coordinates > get translated accordingly make sense? There has been various changes in the way we handle window positioning, and it's now probably more flexible to arbitrary X11 parent windows, but in any case we still always consider the X root window coordinates as the origin for the Win32 desktop. If you want to embed all Wine windows into another window and use its position as the Win32 desktop origin, it's basically what the "virtual desktop" mode is doing already. It's an option in winecfg, where we create a dedicated window for the Win32 desktop and every Wine window will be a child of it. You could probably then reparent the desktop window to the window you want? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6569#note_89502

1 0

Re: [PATCH v10 0/6] MR6569: winex11: Refactor window state/config tracker to fix race conditions.
by Robbert van der Helm (＠robbert-vdh) 02 Dec '24

02 Dec '24

On Mon Dec 2 10:04:22 2024 +0000, Rémi Bernon wrote: > `wm_state_serial` should be cleared when the window state actually > changes and the `WM_STATE` PropertyNotify event is received, calling > `window_wm_state_notify` which clears it. Ah I completely missed the assignment there at the end when I was grepping for usages of `wm_state_serial`. I still can't reliably get Wine to accept configuration events I sent to it though. If I set the `WM_STATE` property it does process the next event, but then it gets stuck again after that because now `net_wm_state_serial` will be stuck on a nonzero value. I could probably figure out a way to get Wine to cooperate, but that's almost certainly going to break in a future Wine release. To reiterate, my goal isn't necessarily to send `ConfigureNotify` events to Wine. I just want a Wine window that's reparented to another window to handle mouse events correctly, as otherwise Wine seems to interpret pointer events as being relative to the root window. So for example, if I have a regular top level X11 window at position 50x50 relative to the top left of the root window, and I reparent a Wine window into that window, clicking at coordinates 20x20 from the top left of the reparented Wine window will cause Wine to register a click at coordinate 70x70. Up until now it was possible to work around this by telling Wine about its location on screen through `ConfigureNotify` events, but that is of course a hack. Since you've been working on Wine's X11 driver, what do you think is the best way to approach this in a way that isn't likely to break again in the future? Would extending Wine's existing tracking of reparented/embedded windows so any events related to pointer coordinates get translated accordingly make sense? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6569#note_89501

1 0

[PATCH 0/1] MR6941: winevulkan: Update to VK spec version 1.4.303.
by Georg Lehmann (＠DadSchoorse) 02 Dec '24

02 Dec '24

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/6941

3 2

Re: [PATCH v2 0/2] MR6921: ntdll: Fix reported exception code for some brk immediates. - approved
by Alexandre Julliard (＠julliard) 02 Dec '24

02 Dec '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6921

1 0

Re: [PATCH v3 0/1] MR6926: winex11: Don't tell win32 about offscreen window position. - approved
by Alexandre Julliard (＠julliard) 02 Dec '24

02 Dec '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6926

1 0

Re: [PATCH v2 0/1] MR6930: d2d1/effect: Improve handling of blob properties. - approved
by Alexandre Julliard (＠julliard) 02 Dec '24

02 Dec '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6930

1 0

Re: [PATCH v3 0/2] MR6929: kernelbase: Properly return 0 from EnumSystemFirmwareTable on error. - approved
by Alexandre Julliard (＠julliard) 02 Dec '24

02 Dec '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6929

1 0

Re: [PATCH v3 0/1] MR6934: kernel32: Fix HeapFree entry in spec file. - approved
by Alexandre Julliard (＠julliard) 02 Dec '24

02 Dec '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6934

1 0

Re: [PATCH v4 0/8] MR6931: win32u: Move surface and swapchain wrappers from winevulkan. - approved
by Alexandre Julliard (＠julliard) 02 Dec '24

02 Dec '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6931

1 0

Re: [PATCH v3 0/3] MR6939: include: Enable format attributes for debug traces in Clang MSVC mode. - approved
by Alexandre Julliard (＠julliard) 02 Dec '24

02 Dec '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6939

1 0

2025

2024

2023

2022

wine-gitlab December 2024