April 2025 - Wine-GitLab - list.winehq.org

[PATCH 0/1] MR7556: wmp: Check the return value of IOleClientSite_QueryInterface().
by Yongjie Yao (＠yaoyongjie) April 3, 2025

April 3, 2025

It should be assigned to hres before checking hres as "if (FAILED(hres)) xxx". -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7556

3 2

[PATCH 0/6] MR7686: winebth.sys part 8: Add support for replying to authentication/pairing requests.
by Vibhav Pant (＠vibhavp) April 3, 2025

April 3, 2025

Implement `BluetoothSendAuthenticationResponseEx` over the `IOCTL_WINEBTH_RADIO_SEND_AUTH_RESPONSE` command in winebth.sys. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7686

3 7

[PATCH v6 0/5] MR7727: cfgmgr32, setupapi: Implement a few functions.
by Paul Gofman (＠gofman) April 3, 2025

April 3, 2025

-- v6: 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 7

[PATCH 0/3] MR7737: mshtml/jscript: Regression + leak fixes.
by Gabriel Ivăncescu April 3, 2025

April 3, 2025

For some reason I thought it wasn't needed anymore (I had it initially), sorry for the mess up. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7737

2 5

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

April 3, 2025

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) April 3, 2025

April 3, 2025

-- 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) April 3, 2025

April 3, 2025

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) April 3, 2025

April 3, 2025

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) April 3, 2025

April 3, 2025

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) April 3, 2025

April 3, 2025

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