July 2025 - Wine-gitlab - list.winehq.org

Re: [PATCH v15 0/1] MR8573: cmd: tab completion: Treat most delimiters as literals if user specified quotes.
by Joe Souza (＠JoeS209) 31 Jul '25

31 Jul '25

You guys need to take this fix. Current code is broken for reasons indicated in MR description and in comments here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8573#note_111726

1 0

Re: [PATCH v2 0/1] MR8683: msado15/tests: Fixup error return values for the ConnectionPoint tests.
by Piotr Caban (＠piotr) 31 Jul '25

31 Jul '25

On Thu Jul 31 16:27:31 2025 +0000, Piotr Caban wrote: > I think it would be best to move these tests before other tests instead > (and add comment mentioning that this should be the first test). The > function returns error returned by last function that failed (or E_FAIL > if there was none). The tests can be easily modified so ConnectionPoint > tests return different error. I can see in your patch that it's probably more complicated (the error is always the same if I move test_ADORecordsetConstruction before test_ConnectionPoint). Anyway, it's probably better to run ConnectionPoint tests first. Another option is to try to investigate what's happening there but I'm not sure if it's worth the effort. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8683#note_111715

1 0

Re: [PATCH v2 0/1] MR8683: msado15/tests: Fixup error return values for the ConnectionPoint tests.
by Piotr Caban (＠piotr) 31 Jul '25

31 Jul '25

I think it would be best to move these tests before other tests instead (and add comment mentioning that this should be the first test). The function returns error returned by last function that failed (or E_FAIL if there was none). The tests can be easily modified so ConnectionPoint tests return different error. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8683#note_111714

1 0

[PATCH v3 0/1] MR8605: winebus: Set FFB autcenter to 0 instead of 100 on device reset
by Tomasz Pakuła 31 Jul '25

31 Jul '25

FFB Autocenter introduced in https://gitlab.winehq.org/wine/wine/-/merge_requests/4911 had one major misunderstanding. The USB PID standard doesn't actually define any explicit way to autocenter a device. One could of course use the spring effect with a deadzone of 0 and dead band of 0. This is what I'm actually working on for the Linux PID driver (spring + friction/damper). Some devices implement autocenter in firmware when they receive the DC Disable Actuators command. Very few, if not just one, implement this weird autocenter effect on slot 1. This is, from what I can gather, only implemented on the MS SideWinder joystick(s) and the Windows' USB PID driver is created around these devices. Windows PID driver is a bit out of spec, is quite permissive when it comes to fields missing in the descriptor (basically, only effect types and their effect type blocks are optional). Another thing it does is handling of this out-of-spec autocentering for their joysticks. Funnliy enough, the creator of the Linux PID driver based the initial code on testing with MS Sidewinder so it's autocentering is supported. This is where the autocentering mentioned in the MR comes from. It's not the directinput api that does it but the Windows PID driver. As such, autocentering on reset should be left to the drivers, not handeled by Wine. SDL lacks full reset support and Linux is even more barebones, whre it's not even possible to query the device state, effects etc (something I'm working on slowly). As such, when games send out RESET to prepare the device, the device starts autocentering for no good reason and the effect is not removed once other effect are uploaded and played which would be the case for MS sidewinder. In any case, even Windows in inconsistent and I think the directinput documentation has some errors. directinput ffb api is largely based on USB PID but: ![image](/uploads/6548386550b874d2e5548f35f7fe76b0/image.png) ![image](/uploads/851484a555c744c6791d48b3fd194e91/image.png) Testing with my Moza Racing R9 wheelbase, the DC Reset PID command is sent, but there's no DC Disable Actuators in sight. The games still call reset then enable actuators though. tl;dr Set autocentering to 0 instead of max value when DISFFC_RESET is received to remove the unwanted autocenter behavior. Signed-off-by: Tomasz Pakuła <tomasz.pakula.oficjalny(a)gmail.com> -- v3: winebus: Do not touch autocenter on device init and device reset https://gitlab.winehq.org/wine/wine/-/merge_requests/8605

2 6

[PATCH 0/3] MR8686: Draft: winebus: Add some per-device/vendor hidrawn and autocenter options.
by Rémi Bernon 31 Jul '25

31 Jul '25

The per-device/vendor option list is allocated on the PE side and shared with the unix, it should be alright to access linked list from both side as winebus should not need and doesn't implement WOW64. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8686

1 3

[PATCH v2 0/1] MR8605: winebus: Set FFB autcenter to 0 instead of 100 on device reset
by Tomasz Pakuła 31 Jul '25

31 Jul '25

FFB Autocenter introduced in https://gitlab.winehq.org/wine/wine/-/merge_requests/4911 had one major misunderstanding. The USB PID standard doesn't actually define any explicit way to autocenter a device. One could of course use the spring effect with a deadzone of 0 and dead band of 0. This is what I'm actually working on for the Linux PID driver (spring + friction/damper). Some devices implement autocenter in firmware when they receive the DC Disable Actuators command. Very few, if not just one, implement this weird autocenter effect on slot 1. This is, from what I can gather, only implemented on the MS SideWinder joystick(s) and the Windows' USB PID driver is created around these devices. Windows PID driver is a bit out of spec, is quite permissive when it comes to fields missing in the descriptor (basically, only effect types and their effect type blocks are optional). Another thing it does is handling of this out-of-spec autocentering for their joysticks. Funnliy enough, the creator of the Linux PID driver based the initial code on testing with MS Sidewinder so it's autocentering is supported. This is where the autocentering mentioned in the MR comes from. It's not the directinput api that does it but the Windows PID driver. As such, autocentering on reset should be left to the drivers, not handeled by Wine. SDL lacks full reset support and Linux is even more barebones, whre it's not even possible to query the device state, effects etc (something I'm working on slowly). As such, when games send out RESET to prepare the device, the device starts autocentering for no good reason and the effect is not removed once other effect are uploaded and played which would be the case for MS sidewinder. In any case, even Windows in inconsistent and I think the directinput documentation has some errors. directinput ffb api is largely based on USB PID but: ![image](/uploads/6548386550b874d2e5548f35f7fe76b0/image.png) ![image](/uploads/851484a555c744c6791d48b3fd194e91/image.png) Testing with my Moza Racing R9 wheelbase, the DC Reset PID command is sent, but there's no DC Disable Actuators in sight. The games still call reset then enable actuators though. tl;dr Set autocentering to 0 instead of max value when DISFFC_RESET is reveived to remove the unwanted autocenter behavior. Signed-off-by: Tomasz Pakuła <tomasz.pakula.oficjalny(a)gmail.com> -- v2: winebus: Set FFB autcenter to 0 instead of 100 on device reset. https://gitlab.winehq.org/wine/wine/-/merge_requests/8605

3 16

[PATCH 0/1] MR8684: joy.cpl: Initialize size before calling RegGetValueW.
by Rémi Bernon 31 Jul '25

31 Jul '25

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

1 1

[PATCH v4 0/7] MR8490: d3dx{9,10}: Beginnings of d3dx code sharing.
by Connor McAdams (＠cmcadams) 31 Jul '25

31 Jul '25

These are the first steps towards code sharing, finally :) Patch 4 is a bit big, but it's just copying code from various source files into a single source file. I can attempt to split if if that's preferable for review purposes. -- v4: d3dx10: Use shared d3dx code in get_image_info when possible. d3dx9: Don't include d3dx9 header in d3dx_helpers. d3dx9: Get rid of ID3DXBuffer usage in d3dx_helpers. d3dx9: Move functions intended for code sharing into a separate source file. d3dx9: Introduce d3dx_image_file_format enumeration. d3dx9: Introduce d3dx_resource_type enumeration. d3dx9: Replace D3DFORMAT constants with enum d3dx_pixel_format_id constants. https://gitlab.winehq.org/wine/wine/-/merge_requests/8490

3 10

Re: [PATCH v47 0/6] MR6766: wintypes: Add stubs for Windows.Foundation.Collections.PropertySet. - approved
by Alexandre Julliard (＠julliard) 31 Jul '25

31 Jul '25

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

1 0

Re: [PATCH v5 0/4] MR8554: windows.media: Implement IActivationFactory::ActivateInstance() for IMediaTranscoder. - approved
by Alexandre Julliard (＠julliard) 31 Jul '25

31 Jul '25

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

1 0