November 2024 - wine-gitlab

[PATCH v4 0/8] MR6427: winevulkan: Introduce a vulkan object structure.
by Rémi Bernon (＠rbernon) 26 Nov '24

26 Nov '24

I'm starting to consider wrapping more vulkan objects in win32u, I think it'll be useful to implement compositing (and incidentally shared resources with interop with the D3DKMT/NtGdi API), starting with nulldrv/offscreen swapchains. Alternatively we could export more functions from win32u and keep all the wrappers in winevulkan, but having access to win32u internals with a generic vulkan object wrapping mechanism seems to be more interesting. Right now this is not very convenient because the vulkan object hierarchy is only available in winevulkan. This introduces shared structures and API between win32u and winevulkan so they can manipulate the wrapped objects from both side. The interface would only expose what is necessary, ie: the object hierarchy and wrapped handles, the instance and device functions, and the objects rbtree for debugging, so win32u can insert its wrapper there as well. After moving the surface and swapchain wrappers to win32u, the interface would look like this: https://gitlab.winehq.org/rbernon/wine/-/blob/65ca1280d46a2dff792e298c22242… (I pushed a complete WIP branch to https://gitlab.winehq.org/rbernon/wine/-/commits/wip/vulkan-win32u-driver?r…). We can also consider moving all the winevulkan wrappers to win32u but it's not clear whether it's necessary or desirable, and we can also do that later using this interface. -- v4: winevulkan: Introduce a new vulkan_physical_device base structure. https://gitlab.winehq.org/wine/wine/-/merge_requests/6427

3 11

Re: [PATCH v24 0/2] MR6869: timeout: Windows 10 introduce the timeout command
by eric pouech (＠epo) 26 Nov '24

26 Nov '24

On Tue Nov 26 09:45:58 2024 +0000, Michele Dionisio wrote: > I remove the check as you request. but in my opinion now the test fails > on linux and windows too they fail on linux when run from a unix console, I don't see failure on windows you can add the attached patch to work around this [patch-timeout](/uploads/e311af5c8318f780aad5ff66a714ba26/patch-timeout) I let you fix the remaining failure Note: since MR!6888 has been merged, you don't need to take care of returning STATUS_CONTROL_C_EXIT, it's done by the default handlers now Note2: the SetStdHandle() call between FreeConsole() and AllocConsole() in this patch looks suspicious : it's not needed under Windows, so likely another Wine bug (but it's needed for now, esp. when running the tests under Wine from a unix console) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6869#note_88940

1 0

Re: [PATCH v2 0/2] MR6736: Do not modify audio client buffer size when changing sample rate
by Arek Hiler (＠ivyl) 26 Nov '24

26 Nov '24

just a minor nitpick with indent, thanks for improving the implementation! -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6736#note_88937

1 0

Re: [PATCH v2 0/2] MR6736: Do not modify audio client buffer size when changing sample rate - approved
by Arek Hiler (＠ivyl) 26 Nov '24

26 Nov '24

This merge request was approved by Arek Hiler. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6736

1 0

Re: [PATCH v2 0/2] MR6736: Do not modify audio client buffer size when changing sample rate
by Arek Hiler (＠ivyl) 26 Nov '24

26 Nov '24

Arek Hiler (@ivyl) commented about dlls/mmdevapi/tests/render.c: > + ok(hr == S_OK, "SetSampleRate failed: %08lx\n", hr); > + > + /* Wait for frame processing */ > + WaitForSingleObject(event, 1000); > + > + hr = IAudioClient_GetBufferSize(ac, &bufsize); > + ok(bufsize == expected_bufsize, "unexpected bufsize %d expected %d\n", bufsize, expected_bufsize); > + > + hr = IAudioClockAdjustment_SetSampleRate(aca, 22050.00f); > + ok(hr == S_OK, "SetSampleRate failed: %08lx\n", hr); > + > + /* Wait for frame processing */ > + WaitForSingleObject(event, 1000); > + > + hr = IAudioClient_GetBufferSize(ac, &bufsize); > + todo_wine ok(bufsize == expected_bufsize, "unexpected bufsize %d expected %d\n", bufsize, expected_bufsize); bad indent -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6736#note_88936

1 0

[PATCH v10 0/4] MR5759: comctl32 & user32: Mark radio buttons at selected when focused
by Fabian Maurer (＠DarkShadow44) 26 Nov '24

26 Nov '24

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56109 -- v10: user32: Send parent BN_CLICKED notification when a radio button get focused. user32/tests: Add tests for radio button WM_SETFOCUS. comctl32: Send parent BN_CLICKED notification when a radio button get focused. comctl32/tests: Add tests for radio button WM_SETFOCUS. https://gitlab.winehq.org/wine/wine/-/merge_requests/5759

4 7

[PATCH v11 0/2] MR419: win32u/window: Ignore changing WS_EX_TOPMOST for message-only windows.
by Giovanni Mascellani (＠giomasce) 26 Nov '24

26 Nov '24

-- v11: win32u/window: Ignore changing WS_EX_TOPMOST for message-only windows. user32/tests: Check that message-only windows ignore WS_EX_TOPMOST. https://gitlab.winehq.org/wine/wine/-/merge_requests/419

4 5

[PATCH v10 0/2] MR419: win32u/window: Ignore changing WS_EX_TOPMOST for message-only windows.
by Giovanni Mascellani (＠giomasce) 26 Nov '24

26 Nov '24

-- v10: win32u/window: Ignore changing WS_EX_TOPMOST for message-only windows. user32/tests: Check that message-only windows ignore WS_EX_TOPMOST. https://gitlab.winehq.org/wine/wine/-/merge_requests/419

3 3

[PATCH v2 0/2] MR6898: comctl32: Update combobox dropdown size.
by Orin Varley (＠ovarley1) 26 Nov '24

26 Nov '24

When the combobox height is to be set by the application (as the CBS_NOINTEGRALHEIGHT style is on), and when a large size is specified but only a small number of items in the list, the height of the combobox should be set by the number of items rather than the size specified to avoid empty lines. The first commit fixes this and the second is a test for this behavior. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57360 Closes #7 -- v2: comctl32: Make CBS_NOINTEGRALHEIGHT only set minimum combobox height. comctl32/tests: Add tests for a small number of items but big size to the combobox dropdown size tests. https://gitlab.winehq.org/wine/wine/-/merge_requests/6898

2 2

[PATCH v4 0/4] MR6289: dsound: Some 3D fixes.
by Aida Jonikienė (＠DodoGTA) 26 Nov '24

26 Nov '24

-- v4: dsound: Add non-NaN value tests for SetOrientation(). dsound/tests: Add NaN tests for floating-point 3D functions. dsound: Add an angle check for SetOrientation(). dsound: Handle NaN values in the 3D code. https://gitlab.winehq.org/wine/wine/-/merge_requests/6289

3 9

2025

2024

2023

2022

wine-gitlab November 2024