February 2025 - Wine-gitlab

Re: [PATCH v6 0/4] MR7173: rtworkq: Support putting work on RTWQ_CALLBACK_QUEUE_UNDEFINED.
by Rémi Bernon 26 Feb '25

26 Feb '25

Fwiw the tests here are failing on many Windows versions, see https://test.winehq.org/data/patterns.html#rtworkq:rtworkq -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7173#note_95991

1 0

Re: [PATCH v3 0/1] MR7323: wininet: Treat Expires value "-1" the same as "0"
by Yongjie Yao (＠yaoyongjie) 26 Feb '25

26 Feb '25

In fact, some HTTP server will send "Expires: -1" in HTTP response header. You are right, HTTP spec is that invalid value, include "0". I add a test to test_http_connection() that would send "Expires: -1" in response header, I get the error log: ``` 0024:err:wininet:HTTP_ParseDateAsAsctime unexpected weekday L"-1" 0024:err:wininet:HTTP_ParseDate unexpected date format L"-1" ``` Maybe we should just treat "Expires: -1" as an invalid date format and ignore it. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7323#note_95990

1 0

[PATCH 0/1] MR6977: apisetschema: Add some Windows 10 API sets.
by Tim Clem (＠tclem) 26 Feb '25

26 Feb '25

Needed for Indiana Jones and the Great Circle. --- The naming of the shellfolders one is not a mistake; they seemed to have named it differently than all the other shell API sets. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6977

3 4

[PATCH v5 0/5] MR7370: winex11: Pass fullscreen flag to is_window_managed.
by Rémi Bernon 26 Feb '25

26 Feb '25

Instead of checking for monitor rect. Similar purpose as https://gitlab.winehq.org/wine/wine/-/merge_requests/7362 -- v5: winewayland: Pass fullscreen flag to is_window_managed. winex11: Pass fullscreen flag to is_window_managed. winex11: Check managed window changes in WindowPosChanged. winex11: Request managed/embedded in a new window_set_managed helper. winex11: Initialize window managed flag in create_whole_window. https://gitlab.winehq.org/wine/wine/-/merge_requests/7370

1 6

[PATCH v2 0/2] MR7417: mfplat: Modify MFCreateWaveFormatExFromMFMediaType to allow MF_MT_AM_FORMAT_TYPE to be missing or set to GUID_NULL.
by Brendan McGrath (＠redmcg) 26 Feb '25

26 Feb '25

This MR: - adds tests for when MF_MT_AM_FORMAT_TYPE is missing or set to GUID_NULL; and - modifies MFCreateWaveFormatExFromMFMediaType to pass these tests -- v2: mfplat: Allow MF_MT_AM_FORMAT_TYPE to be missing or set to GUID_NULL. mfplat/tests: Add additional MFCreateWaveFormatExFromMFMediaType tests. https://gitlab.winehq.org/wine/wine/-/merge_requests/7417

2 3

[PATCH v4 0/1] MR7352: winewayland: Update locked pointer position hint.
by Attila Fidan (＠atticf) 26 Feb '25

26 Feb '25

If a surface is being clipped and hides the cursor, drawing its own one, winewayland constrains using a pointer lock and enables Wayland relative motion. If the application decides to stop drawing its own cursor and make the cursor visible, winewayland will disable relative motion and pointer lock, and enable pointer confinement. The user will perceive a pointer jump from the win32/application drawn cursor to where the Wayland pointer is after being unlocked and an absolute motion event is received, because they were desynchronized due to the Wayland one being locked in place. The pointer constraints protocol says this: > If the client is drawing its own cursor, it should update the position > hint to the position of its own cursor. A compositor may use this > information to warp the pointer upon unlock in order to avoid pointer > jumps. So, right before unlocking, make a request for the compositor to warp the pointer to the win32 position on pointer unlock. -- v4: winewayland: Update locked pointer position hint. https://gitlab.winehq.org/wine/wine/-/merge_requests/7352

3 4

[PATCH v4 0/1] MR7288: winegstreamer: Avoid large buffer pushes in wg_transform.
by Yuxuan Shui (＠yshui) 26 Feb '25

26 Feb '25

## Context This is a problem discovered when running "Max: The Curse of Brotherhood" under proton. This game writes 4MB a piece into `wg_transform`, which contains about 4 seconds of compressed video. `wg_transform` calls `gst_pad_push` in `transform_ProcessOutput`, which takes \~300ms (i.e. 20 frames @ 60fps) to decode this buffer of video. So the end result is the game hitches every 4 seconds while playing cut scene videos. Proton currently has a special case for this particular game, for which it breaks up the buffer into small chunks to avoid long blocks. This MR adopts that and applies it generally. One concern raised by @redmcg is: > The only issue I can think of is if there are any decoders which don't have a parser; then they might not know how to deal with an arbitrary 4096 byte buffer (the parser is generally responsible for taking arbitrary buffers and producing something the decoder can work with, for example: a full frame). So this MR only enables this strategy when there is a parser element. -- v4: winegstreamer: Avoid large buffer pushes in wg_transform. https://gitlab.winehq.org/wine/wine/-/merge_requests/7288

2 1

[PATCH v3 0/1] MR7288: winegstreamer: Avoid large buffer pushes in wg_transform.
by Yuxuan Shui (＠yshui) 26 Feb '25

26 Feb '25

## Context This is a problem discovered when running "Max: The Curse of Brotherhood" under proton. This game writes 4MB a piece into `wg_transform`, which contains about 4 seconds of compressed video. `wg_transform` calls `gst_pad_push` in `transform_ProcessOutput`, which takes \~300ms (i.e. 20 frames @ 60fps) to decode this buffer of video. So the end result is the game hitches every 4 seconds while playing cut scene videos. Proton currently has a special case for this particular game, for which it breaks up the buffer into small chunks to avoid long blocks. This MR adopts that and applies it generally. One concern raised by @redmcg is: > The only issue I can think of is if there are any decoders which don't have a parser; then they might not know how to deal with an arbitrary 4096 byte buffer (the parser is generally responsible for taking arbitrary buffers and producing something the decoder can work with, for example: a full frame). So this MR only enables this strategy when there is a parser element. -- v3: winegstreamer: Avoid large buffer pushes in wg_transform. https://gitlab.winehq.org/wine/wine/-/merge_requests/7288

2 2

Re: [PATCH v2 0/1] MR7288: winegstreamer: Avoid large buffer pushes in wg_transform.
by Yuxuan Shui (＠yshui) 26 Feb '25

26 Feb '25

update: rebased to master and is now using the parser for buffer splitting. i found that holding onto the parser element and manually driving it is too complicated, it also causes the code paths to diverge significantly between the having-a-parser case and the not-having-a-parser case. so instead i added a proxy element which gives me control over how buffers move through the pipeline. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7288#note_95963

1 0

Re: [PATCH v3 0/2] MR6492: d2d1: Implement drawing of ID2D1GeometryGroup
by Stefan Brüns 26 Feb '25

26 Feb '25

On Tue Feb 25 20:39:47 2025 +0000, Nikolay Sivov wrote: > This isn't about percentage or how common it is. It's not clear this is > the right direction to me, for example groups could contain path > geometries with different filling modes, or other groups. So, what is the behavior you expect? It is clear the behavior is correct when the fill mode is WINDING, or when the rendering does not depend on the fill rule - that also includes overlapping geometries with correct path direction, e.g. paths generated from font outlines. You should provide a way forward, not block it because it is not perfect. It definitely *does* work properly in the majority of cases, the rendering becomes more correct. If you want changes, please be specific. Wine is full of partially incomplete and incorrect implementations, according to your line of thoughts all this should be deleted. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6492#note_95959

1 0