- wine-gitlab - winehq.org

[PATCH v2 0/1] MR7701: winemac.drv: Only send key down events to the window's inputContext.
by Tim Clem (＠tclem) 03 Apr '25

03 Apr '25

Sending key events through `-[NSTextInputContext handleEvent:]`, as we do in `macdrv_send_text_input_event`, results in doubled characters for certain macOS input sources. The Romaji source does this reliably, as do certain third-party sources. (The built-in dictation does it as well, but we already blacklist that - !5660 - because it remains "active" even when it's not.) I can't find this spelled out directly, but if you read between the lines, it seems clear that you're only supposed to send key *down* events to `-[NSTextInputContext handleEvent:]` and not key ups. Supporting evidence: 1. [The old documentation on the text system](https://developer.apple.com/library/archive/documentation/TextFonts… only mentions `handleEvent:` being called in the context of `-keyDown:`. 2. [The guide to implementing a custom text view/NSTextInputClient](https://developer.apple.com/library/archive/documen… only mentions overriding `-keyDown:` and sending those events to `-handleEvent:`. 3. [iTerm only sends key down events to `-handleEvent:`](https://github.com/gnachman/iTerm2/blob/6134ea0a9d9d0fee5e7d7704fc98efec1fc77c24/sources/iTermKeyboardHandler.m#L355). You have to unpack the logic a bit from there, but `-handleKeyDownEvent:...` is the only method that calls `-handleEventWithCocoa:inputContext:`, which is what (usually) calls `-handleEvent:` on the `inputContext`. 4. [The InputMethodKit method on the IMKServerInput protocol to handle NSEvents directly](https://developer.apple.com/documentation/objectivec/nsobject/138… only mentions getting key down events. So presumably input source authors sometimes do not check the type of the NSEvent they're receiving and process all key events equally, be they downs or ups, which results in the doubled input. This patch fixes the issues with Romaji and the problematic third-party method we've encountered. -- v2: winemac.drv: Only send key down events to the window's inputContext. https://gitlab.winehq.org/wine/wine/-/merge_requests/7701

3 5

[PATCH v5 0/5] MR7727: cfgmgr32, setupapi: Implement a few functions.
by Paul Gofman (＠gofman) 03 Apr '25

03 Apr '25

-- v5: cfgmgr32: Implement CM_Get_Device_Interface_PropertyW() for DEVPKEY_Device_InstanceId. cfgmgr32: Implement CM_Get_Device_Interface_List[_Size][_Ex]A(). cfgmgr32: Implement CM_Get_Device_Interface_List[_Size][_Ex]W(). setupapi/tests: Add a test for SetupDiOpenDeviceInterface(). setupapi: Implement SetupDiOpenDeviceInterface(A|W). https://gitlab.winehq.org/wine/wine/-/merge_requests/7727

4 8

[PATCH 0/1] MR7705: setupapi: Respect ERROR_SHARING_VIOLATION from open KnownDLL file handles.
by William Horvath (＠whrvt) 03 Apr '25

03 Apr '25

create_dest_file was unaware of the `FILE_SHARE_READ | FILE_SHARE_DELETE` share mode that these SessionMgr entries are opened with. This avoids noisy `(error=80)` messages during prefix creation/update. Fixes: 81ea1e7a20c9cd0698c0416282d9ad7c0109e865 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7705

3 8

[PATCH v2 0/3] MR7735: wininet: Handle HTTP status code 308 (Permanent Redirect)
by Tobias Gruetzmacher (＠TobiX) 03 Apr '25

03 Apr '25

This implements handling of HTTP status code 308 ([Permanant Redirect](https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Statu…) in wininet/winhttp. I added handling at all places I could find, but could only test the one in `dlls/wininet/http.c` with the program where it failed (Marvel Heroes). I tried to keep the implementation parallel to 307, since they serve a similar purpose (redirect while keeping the request intact). PS: `case` labels are sometimes in different order, would it be a good idea to sort them in numerical HTTP status order while I'm at it? -- v2: urlmon: Handle HTTP status code 308 (Permanent Redirect) winhttp: Handle HTTP status code 308 (Permanent Redirect) wininet: Handle HTTP status code 308 (Permanent Redirect) https://gitlab.winehq.org/wine/wine/-/merge_requests/7735

3 4

[PATCH v2 0/7] MR7698: PDB rewrite #6.
by eric pouech (＠epo) 03 Apr '25

03 Apr '25

This serie mainly moves line information for top level function (not including the inlined parts) into the new PDB reader. In details: - cleaning up current internal private structure for line information (in order to share it with PDB backend) - introduce methods to get line info from address, move to next/prev location, enumeration of line numbers and source files. Note for source files enumeration: - current implementation still needs some more fixes (no full scoping, but that's unrelated to PDB rewrite) - fixes by module scoping - it still uses the global source hash table (as there isn't one in PDB). No plan to improve that for now. -- v2: dbghelp: No longer store line information from old PDB reader. dbghelp: Add method to enumerate source files. dbghelp: Introduce method to enumerate line numbers. dbghelp: Introduce method to get next/prev line information. dbghelp: Introduce interface for line info access. dbghelp: Always copy the source file string. dbghelp: Rename struct internal_line into lineinfo. https://gitlab.winehq.org/wine/wine/-/merge_requests/7698

2 8

[PATCH 0/1] MR7735: wininet: Handle HTTP status code 308 (Permanent Redirect)
by Tobias Gruetzmacher (＠TobiX) 03 Apr '25

03 Apr '25

This implements handling of HTTP status code 308 ([Permanant Redirect](https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Statu…) in wininet/winhttp. I added handling at all places I could find, but could only test the one in `dlls/wininet/http.c` with the program where it failed (Marvel Heroes). I tried to keep the implementation parallel to 307, since they serve a similar purpose (redirect while keeping the request intact). PS: `case` labels are sometimes in different order, would it be a good idea to sort them in numerical HTTP status order while I'm at it? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7735

3 3

[PATCH 0/1] MR7701: winemac.drv: Only send key down events to the window's inputContext.
by Tim Clem (＠tclem) 03 Apr '25

03 Apr '25

Sending key events through `-[NSTextInputContext handleEvent:]`, as we do in `macdrv_send_text_input_event`, results in doubled characters for certain macOS input sources. The Romaji source does this reliably, as do certain third-party sources. (The built-in dictation does it as well, but we already blacklist that - !5660 - because it remains "active" even when it's not.) I can't find this spelled out directly, but if you read between the lines, it seems clear that you're only supposed to send key *down* events to `-[NSTextInputContext handleEvent:]`, and not key ups. Supporting evidence: 1. [The old documentation on the text system](https://developer.apple.com/library/archive/documentation/TextFonts… only mentions `handleEvent:` being called in the context of `-keyDown:`. 2. [The guide to implementing a custom text view/NSTextInputClient](https://developer.apple.com/library/archive/documen… only mentions overriding `-keyDown:` and sending those events to `-handleEvent:`. 3. [iTerm only sends key down events to `-handleEvent:`](https://github.com/gnachman/iTerm2/blob/6134ea0a9d9d0fee5e7d7704fc98efec1fc77c24/sources/iTermKeyboardHandler.m#L355). You have to unpack the logic a bit from there, but `-handleKeyDownEvent:...` is the only method that calls `-handleEventWithCocoa:inputContext:`, which is what (usually) calls `-handleEvent:` on the `inputContext`. 4. [The InputMethodKit method on the IMKServerInput protocol to handle NSEvents directly](https://developer.apple.com/documentation/objectivec/nsobject/138… only mentions getting key down events. So presumably input source authors sometimes do not check the type of the NSEvent they're receiving and process all key events equally, be they downs or ups. This change fixes doubled input with Romaji and the problematic third-party method we've encountered. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7701

3 5

[PATCH v2 0/1] MR7736: server: Fix the accumulation method when merging WM_MOUSEWHEEL message.
by Akihiro Sagawa (＠sgwaki) 03 Apr '25

03 Apr '25

Otherwise, two MK_RBUTTON messages become MK_SHIFT. -- v2: server: Fix the accumulation method when merging WM_MOUSEWHEEL message. https://gitlab.winehq.org/wine/wine/-/merge_requests/7736

2 1

[PATCH v4 0/5] MR7727: cfgmgr32, setupapi: Implement a few functions.
by Paul Gofman (＠gofman) 03 Apr '25

03 Apr '25

-- v4: cfgmgr32: Implement CM_Get_Device_Interface_PropertyW() for DEVPKEY_Device_InstanceId. cfgmgr32: Implement CM_Get_Device_Interface_List[_Size][_Ex]A(). https://gitlab.winehq.org/wine/wine/-/merge_requests/7727

4 6

[PATCH 0/1] MR7736: server: Fix the accumulation method when merging WM_MOUSEWHEEL message.
by Akihiro Sagawa (＠sgwaki) 03 Apr '25

03 Apr '25

Otherwise, two MK_RBUTTON messages become MK_SHIFT. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7736

3 3

[PATCH 0/7] MR7698: PDB rewrite #6.
by eric pouech (＠epo) 03 Apr '25

03 Apr '25

This serie mainly moves line information for top level function (not including the inlined parts) into the new PDB reader. In details: - cleaning up current internal private structure for line information (in order to share it with PDB backend) - introduce methods to get line info from address, move to next/prev location, enumeration of line numbers and source files. Note for source files enumeration: - current implementation still needs some more fixes (no full scoping, but that's unrelated to PDB rewrite) - fixes by module scoping - it still uses the global source hash table (as there isn't one in PDB). No plan to improve that for now. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7698

3 9

[PATCH v2 0/4] MR7732: bcrypt: Add support for BCRYPT_AES_WRAP_KEY_BLOB.
by Hans Leidekker (＠hans) 03 Apr '25

03 Apr '25

-- v2: bcrypt: Add support for BCRYPT_AES_WRAP_KEY_BLOB in BCryptImportKey(). bcrypt: Add support for BCRYPT_AES_WRAP_KEY_BLOB in BCryptExportKey(). https://gitlab.winehq.org/wine/wine/-/merge_requests/7732

2 5

[PATCH 0/5] MR7734: opengl32/tests: Add more tests for memory DC rendering.
by Rémi Bernon (＠rbernon) 03 Apr '25

03 Apr '25

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

2 5

[PATCH 0/2] MR7733: Fix Project CARS 3 (958400) fails to enter triple screen mode
by Zhiyi Zhang (＠zhiyi) 03 Apr '25

03 Apr '25

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

3 4

[PATCH 0/4] MR7732: bcrypt: Add support for BCRYPT_AES_WRAP_KEY_BLOB.
by Hans Leidekker (＠hans) 03 Apr '25

03 Apr '25

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

3 13

[PATCH v3 0/5] MR7618: comctl32: Updates to syslink accessibility.
by Esme Povirk (＠madewokherd) 03 Apr '25

03 Apr '25

-- v3: comctl32/tests: Test MSAA events for SysLink. comctl32/tests: Test SetWindowText and LM_GETITEM for SysLink. comctl32/tests: Test accLocation values on SysLink control. comctl32: Implement accDoDefaultAction for SysLink controls. comctl32/tests: Add test for SysLink accDoDefaultAction. https://gitlab.winehq.org/wine/wine/-/merge_requests/7618

3 7

[PATCH 0/3] MR7731: win32u: Remove unnecessary win32u_(get|set)_window_pixel_format exports.
by Rémi Bernon (＠rbernon) 03 Apr '25

03 Apr '25

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

2 3

[PATCH v2 0/2] MR7656: wined3d: Two unrelated fixes.
by Elizabeth Figura (＠zfigura) 03 Apr '25

03 Apr '25

-- v2: wined3d: Use VKD3D_SHADER_COMPILE_OPTION_TYPED_UAV_READ_FORMAT_UNKNOWN if possible. wined3d: Invert gl_FragCoord w. https://gitlab.winehq.org/wine/wine/-/merge_requests/7656

3 3

[PATCH v2 0/6] MR7561: wined3d: Vulkan H.264 decode, part 3/3.
by Elizabeth Figura (＠zfigura) 03 Apr '25

03 Apr '25

-- v2: d3d11/tests: Add tests for H.264 decoding. https://gitlab.winehq.org/wine/wine/-/merge_requests/7561

3 8

[PATCH 0/3] MR7730: shell32/tests: Use check_file_operation in test_rename.
by Ziqing Hui (＠zhui) 03 Apr '25

03 Apr '25

Also remove old windows version tests. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7730

2 3

[PATCH 0/1] MR7729: shell32/tests: Use wide character string literals.
by Alex Henrie (＠alexhenrie) 03 Apr '25

03 Apr '25

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

2 1

[PATCH v2 0/5] MR7727: cfgmgr32, setupapi: Implement a few functions.
by Paul Gofman (＠gofman) 03 Apr '25

03 Apr '25

-- v2: cfgmgr32: Implement CM_Get_Device_Interface_PropertyW() for DEVPKEY_Device_InstanceId. cfgmgr32: Implement CM_Get_Device_Interface_List[_Size][_Ex]A(). https://gitlab.winehq.org/wine/wine/-/merge_requests/7727

3 5

[PATCH v4 0/6] MR7708: Fix Imperiums: Greek Wars (1183470) not covering the entire screen in fullscreen mode.
by Zhiyi Zhang (＠zhiyi) 03 Apr '25

03 Apr '25

-- v4: win32u: Properly scale monitor work area when emulate_modeset is enabled. https://gitlab.winehq.org/wine/wine/-/merge_requests/7708

2 7

[PATCH 0/1] MR7728: cryptui: Use wide character string literal for L"".
by Alex Henrie (＠alexhenrie) 03 Apr '25

03 Apr '25

I missed this one in commit 0bc1cec34bab461d049a7968e5c3b48adc7289b1. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7728

2 1

[PATCH 0/5] MR7727: cfgmgr32, setupapi: Implement a few functions.
by Paul Gofman (＠gofman) 03 Apr '25

03 Apr '25

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

3 7

2025

2024

2023

2022

wine-gitlab