November 2024 - Wine-GitLab

Re: [PATCH v5 0/2] MR5633: win32u: Use the first vertical alternates table regardless of script.
by Akihiro Sagawa (＠sgwaki) Nov. 15, 2024

Nov. 15, 2024

On Fri Nov 15 14:46:06 2024 +0000, Nikolay Sivov wrote: > I think the remaining question here is if we want to remove script/lang > lookup logic entirely, or instead prioritize. @sgwaki, any idea if it's > possible to test that? Testing manually is fine. To be clear, test font > should have at least two differing vert features, with second feature > being testable with specific script/lang pair, if that is possible to > achieve with plain gdi API at all. I manually tested the vertical features using a specially crafted TrueType file. The font file displays different circled numbers as vertical substitutes for Japanese punctuation marks (U+3001, U+3002) based on its lookup table. If the implementation chooses: - the first vert feature, i.e. no script/lang lookup, the app shows ① (U+2460's glyph). - the first script/lang tag (@@@@), it shows ② (U+2461's). - the 'DFLT' script, it shows ③ (U+2462's). - the 'kana' script, it shows ④ (U+2463's). What we can see is ① with plain gdi API. ![Capture](/uploads/ce5cba3c12e0debd2bdac94fd017d842/Capture.png) See [gsub.txt](/uploads/9d5ef683f937fc0b48f9f7e833fab930/gsub.txt) file in ttx format for details, please. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5633#note_87621

1 0

[PATCH v7 0/10] MR6569: Draft: winex11: Refactor window state/config tracker to fix race conditions.
by Rémi Bernon Nov. 15, 2024

Nov. 15, 2024

This completely refactors the winex11 window state tracking, keeping track of the desired/pending/current window state and config in a fully asynchronous way, and avoiding duplicate requests. I believe this mitigates various race conditions that we're suffering from, solving most spurious event feedback loops when the X11 state is being applied while some win32 state change is being requested, and fixing the spurious d3d/ddraw/user32 test failures. -- v7: winex11: Request window state updates asynchronously. winex11: Update the window client config on window state changes. winex11: Wait for pending ConfigureNotify before updating the client state. winex11: Wait for pending _NET_WM_STATE before updating the client state. winex11: Ignore focus changes during WM_STATE transitions. winex11: Introduce a new window_update_client_config helper. winex11: Introduce a new window_update_client_state helper. winex11: Use the new window state tracker to get WM_STATE value. winex11: Use the new window state tracker to get _NET_WM_STATE value. d3d9/tests: Flush events after minimizing and restoring focus window. https://gitlab.winehq.org/wine/wine/-/merge_requests/6569

1 10

[PATCH v16 0/4] MR4551: Server-side thread priorities implementation (Part 1)
by Marc-Aurel Zent (＠mzent) Nov. 15, 2024

Nov. 15, 2024

This is the current proton thread priority implementation by @rbernon rebased for upstream with a few `#ifdef`s added since AFAIK Linux is the only operating system where threads have a unique PID which can be used to set niceness on. ~~I also ran `./tools/make_requests` on https://gitlab.winehq.org/mzent/wine/-/commit/6705d3481be0409f7e971c1d2c7a3… as well and `autoconf` on https://gitlab.winehq.org/mzent/wine/-/commit/d7bafe40c411753662b2ad97148a6… (which does blow up the line count a bit).~~ A few tiny changes ~~(with the ready variable for example)~~ are in anticipation for Part 2, which also adds Mach thread priorities and recalculates thread priorities on process priority change. Since this is a rather large MR, I hope the split here is appropriate ~~(with the second part being slightly smaller)~~, but I think logically it makes the most sense here. -- v16: server: Check wineserver privileges on init with -20 niceness. server: Use setpriority to update thread niceness when safe. ntdll: Set RLIMIT_NICE to its hard limit. server: Introduce new set_thread_priority helper. https://gitlab.winehq.org/wine/wine/-/merge_requests/4551

4 6

[PATCH 0/1] MR6822: winex11: Ignore focus changes during WM_STATE transitions.
by Rémi Bernon Nov. 15, 2024

Nov. 15, 2024

When WM_STATE is being quickly updated, and depending on the WM we might receive transient focus changes, which will disrupt the Win32 state and make us randomly lose focus. Ignore them instead, when a window is being shown, and wait for WM_STATE to be updated and stable. We will eventually receive a WM_TAKE_FOCUS / FocusIn event *after* a window has been shown. When a window is hidden or minimized, we will receive the FocusOut event during the WM_STATE transition, and can safely handle it in this case, as we should have done all the Win32 side effects and have changed the foreground window already. When there's no window state change pending, the focus change event is unexpected, coming from the user or WM, and we handle it normally. With the planned asynchronous state changes it will become more likely to receive transient focus events, and we need to ignore any focus change that is expected or known to be superseded. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6822

1 1

[PATCH v3 0/2] MR6819: Draft: include: Fix WM_IME_KEY* typo.
by Marc-Aurel Zent (＠mzent) Nov. 15, 2024

Nov. 15, 2024

As far as I can tell, ime.h is only included by the x11 keyboard driver and `WM_IMEKEYDOWN` and `WM_IMEKEYUP` are not used there, so this change should only bring the naming convention in line with what is already in winuser and MSDN. -- v3: winex11: Do not include ime.h. include: Add Japanese IME virtual key codes to winuser.rh. https://gitlab.winehq.org/wine/wine/-/merge_requests/6819

3 4

[PATCH 0/4] MR6820: winex11: Introduce some client state update helpers.
by Rémi Bernon Nov. 15, 2024

Nov. 15, 2024

Moving the logic out of the event handlers, in preparation for asynchronous state changes. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6820

1 4

[PATCH v2 0/2] MR6819: Draft: include: Fix WM_IME_KEY* typo.
by Marc-Aurel Zent (＠mzent) Nov. 15, 2024

Nov. 15, 2024

As far as I can tell, ime.h is only included by the x11 keyboard driver and `WM_IMEKEYDOWN` and `WM_IMEKEYUP` are not used there, so this change should only bring the naming convention in line with what is already in winuser and MSDN. -- v2: winex11: Do not include ime.h. include: Add Japanese IME virtual key codes to winuser.rh. https://gitlab.winehq.org/wine/wine/-/merge_requests/6819

3 3

[PATCH 0/1] MR6819: include: Fix WM_IME_KEY* typo.
by Marc-Aurel Zent (＠mzent) Nov. 15, 2024

Nov. 15, 2024

As far as I can tell, ime.h is only included by the x11 keyboard driver and `WM_IMEKEYDOWN` and `WM_IMEKEYUP` are not used there, so this change should only bring the naming convention in line with what is already in winuser and MSDN. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6819

3 3

[PATCH v8 0/6] MR6640: propsys: Support converting to BSTR for PropVariantToVariant.
by Ziqing Hui (＠zhui) Nov. 15, 2024

Nov. 15, 2024

This is based on top of !6526, will mark ready after !6526 is merged. -- v8: propsys: Support converting to BSTR for PropVariantToVariant. propsys: Use debugstr_variant for the trace in VariantToPropVariant. propsys: Implement PropVariantToBSTR. propsys/tests: Test truncating for PropVariantToString. propsys/tests: Test PropVariantToBSTR. propsys: Add PropVariantToBSTR stub. https://gitlab.winehq.org/wine/wine/-/merge_requests/6640

4 10

Re: [PATCH v4 0/7] MR6815: winex11: Fix dummy ConfigureNotify event generation for embedded windows. - approved
by Alexandre Julliard (＠julliard) Nov. 14, 2024

Nov. 14, 2024

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

1 0