February 2025 - Wine-gitlab

[PATCH v12 0/1] MR7400: cmd: Allow any key to continue past DIR /P pauses.
by Joe Souza (＠JoeS209) 25 Feb '25

25 Feb '25

Current code requires entering a newline character in order to continue past DIR /P prompts. This change allows any key to be pressed instead. Code to handle this was similar to existing WCMD_pause() so we leveraged that code for this purpose as well. Key to the fix was the removal of ENABLE_LINE_INPUT from the console flags, and ENABLE_PROCESSED_INPUT was added in order to preserve the ability to abort the operation via Ctrl-C. -- v12: cmd: Allow any key to continue past DIR /P pauses. https://gitlab.winehq.org/wine/wine/-/merge_requests/7400

2 2

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

25 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. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7288

5 14

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

25 Feb '25

In iis server, use -1 to specify immediate expiration. For detailed documentation, please refer to: https://learn.microsoft.com/en-us/previous-versions/iis/6.0-sdk/ms525906(v=… -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7418

4 7

Re: [PATCH v3 0/1] MR7323: wininet: Treat Expires value "-1" the same as "0"
by Jacek Caban (＠jacek) 25 Feb '25

25 Feb '25

I can see that you quoted MSDN page in the other MR, but it's not clear to me if it's relevant. It mentions some IIS APIs, not how HTTP headers look like. I guess adding a test to `test_http_connection()` that would send "-1" "Expires" header and querying `INTERNET_OPTION_CACHE_TIMESTAMPS` on the result would tell us how it works. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7323#note_95883

1 0

[PATCH v2 0/2] MR7412: win32u: Use VK_EXT_headless_surface for nulldrv surface.
by Rémi Bernon 25 Feb '25

25 Feb '25

Allows running more tests with nulldrv, including some D3D tests with vulkan renderer. -- v2: win32u: Use VK_EXT_headless_surface for nulldrv surface. winevulkan: Enable VK_EXT_headless_surface extension. https://gitlab.winehq.org/wine/wine/-/merge_requests/7412

2 9

Re: [PATCH v3 0/1] MR7323: wininet: Treat Expires value "-1" the same as "0"
by Jacek Caban (＠jacek) 25 Feb '25

25 Feb '25

It's not clear to me what we should do in case of "-1". My understanding of HTTP spec is that it's invalid, which would suggest that failing to parse is the right thing to do. Does it break some application? If it does, maybe it expects ETag to be used and "Expires" header to be ignored? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7323#note_95881

1 0

[PATCH 0/1] MR7419: wininet: Fix crash when connection is null in NETCON_secure_connect
by Yongjie Yao (＠yaoyongjie) 25 Feb '25

25 Feb '25

There will be crashed when connect is null, so we need avoid crash by checking connect value. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7419

3 2

Re: [PATCH v2 0/7] MR7422: winebus/dinput: Support arbitrary number of axes in the virtual PID descriptor
by Tomasz Pakuła 25 Feb '25

25 Feb '25

I'm trying to change the static arrays of axis usages in `hid_device_add_physical()` into a `for` loop + a new function based on the current `hid_report_descriptor_append_usage()` but had some issues. I'll update the MR when I get it to work. Seems like a better idea if the `PID_AXIS_MAX` was ever to change. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7422#note_95872

1 0

Re: [PATCH v2 0/4] MR7272: wined3d: NV12 textures, part 1/2.
by Nikolay Sivov (＠nsivov) 25 Feb '25

25 Feb '25

On Tue Feb 25 12:14:29 2025 +0000, Rémi Bernon wrote: > Fwiw 4caa93a369b51a9bd8922a3f2cdd02a632b52543 broke the evr tests. According to bisect it was 6bc747b78c46883c82770ea876facb7d27a1cb4e. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7272#note_95871

1 0

[PATCH 0/1] MR7423: subst: Add basic implementation.
by Hans Leidekker (＠hans) 25 Feb '25

25 Feb '25

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

2 1