- wine-gitlab - winehq.org

Re: [PATCH v2 0/2] MR5858: odbccp32: Look at the Setup key to find the driver of ODBC config functions. - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/5] MR5903: Cmd engine rewrite (part VIII) - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/8] MR5896: win32u, server: Use the shared memory for GetClipCursor and GetAsyncKeyState. - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/2] MR5895: ntdll: Don't write partial strings with RTL_QUERY_REGISTRY_DIRECT. - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/5] MR5894: ntdll: Do not fill the IOSB or signal completion on failure. - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/5] MR5884: Use zerofill sections instead of preloader on macOS when building with Xcode 15.3. - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/2] MR5897: include: Add a couple of CRT declarations. - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/1] MR5902: kernel32: Add AppPolicyGetWindowingModel(). - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/1] MR5900: apisetschema: Add api-ms-win-appmodel-runtime-internal-l1-1-1. - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/1] MR5899: bcp47langs: Add stub dll. - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/1] MR5901: profapi: Add stub dll. - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

Re: [PATCH v2 0/6] MR5891: include: Add some interface definitions in Windows.Graphics.Capture namespace - approved
by Alexandre Julliard (＠julliard) 21 Jun '24

21 Jun '24

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

1 0

[PATCH 0/1] MR5904: Improve color mode detection
by Ilia Docin (＠IDocin) 21 Jun '24

21 Jun '24

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

3 2

[PATCH v2 0/5] MR5875: mmdevapi: Complete IAudioClient3 implementation.
by Davide Beatrici (＠davidebeatrici) 21 Jun '24

21 Jun '24

-- v2: mmdevapi: Implement IAudioClient3_GetCurrentSharedModeEnginePeriod. mmdevapi: Complete IAudioClient3_GetSharedModeEnginePeriod. mmdevapi: Complete IAudioClient3_InitializeSharedAudioStream. mmdevapi: Introduce helper stream_init(). mmdevapi: Return errors early in adjust_timing(). https://gitlab.winehq.org/wine/wine/-/merge_requests/5875

3 8

Re: [PATCH v3 0/1] MR5706: comctl32/rebar: Hide chevron if the full width of rebar is displayed
by Ilia Docin (＠IDocin) 21 Jun '24

21 Jun '24

To reproduce it, I moved gripper left and right to hide some buttons, then brought it back. On the first image chevron didn't disappear as expected. On the second one chevron went away as I applied fix. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5706#note_73935

1 0

Re: [PATCH v3 0/1] MR5706: comctl32/rebar: Hide chevron if the full width of rebar is displayed
by Ilia Docin (＠IDocin) 21 Jun '24

21 Jun '24

![21-06-2024_211736](/uploads/b2ddd5058f3f1c26004dc32f9e7e040e/21-06-2024_211736.jpg) ![21-06-2024_212110](/uploads/44a5e93157a4beceb2ba12afa10f3d1d/21-06-2024_212110.jpg) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5706#note_73934

1 0

[PATCH 0/5] MR5884: Use zerofill sections instead of preloader on macOS when building with Xcode 15.3.
by Brendan Shanks (＠bshanks) 21 Jun '24

21 Jun '24

Xcode 15.3 adds a new linker flag (`-no_huge`) which allows the loader to use zero-fill sections to reserve the areas currently being reserved by the preloader. This means the preloader is no longer needed (a good thing, since it's heavily dependent on private APIs). The preloader will still be used when Xcode <15.3 is being used, or when building for i386 (32-bit for 10.14 and earlier). I've also tested a 64-bit wineloader built with Xcode 15.3 on macOS 10.13 and found that it works correctly. The only user-visible change I've found is that `setprogname()` now correctly changes the process name, so a `ps` listing will now show (for example) `C:\windows\regedit.exe` (as it does on Linux) instead of `/Users/bshanks/wine/build64/loader/wine-preloader C:\windows\regedit.exe`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5884

4 16

[PATCH 0/8] MR5896: win32u, server: Use the shared memory for GetCursorPos and GetAsyncKeyState.
by Rémi Bernon (＠rbernon) 21 Jun '24

21 Jun '24

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

4 15

[PATCH 0/1] MR5902: kernel32: Add AppPolicyGetWindowingModel().
by Zhiyi Zhang (＠zhiyi) 21 Jun '24

21 Jun '24

Required by React Native apps. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5902

3 2

[PATCH v3 0/1] MR5658: winevulkan: Make device memory wrapper available in callbacks.
by Arek Hiler (＠ivyl) 21 Jun '24

21 Jun '24

With [`VK_EXT_device_address_binding_report`](https://registry.khronos.org/vulkan/specs/1.3-extensions/man/html/VK_EXT_device_address_binding_report.html) we can get [debug_util](https://registry.khronos.org/vulkan/specs/1.3-extensions/man/ht… callbacks used to track memory bindings. Since it's the host's implementation that starts the callback we have to be sure that we have a way of converting it to the client side's variant before it's added to the handle map - i.e. we don't know the host handle at that time yet. This is [used by vkd3d-proton](https://github.com/HansKristian-Work/vkd3d-proton/pull/1962). Requires Mesa with https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28649. before (note the missing BIND for VkDevice which actually means `VkDeviceMemory`): ``` vkd3d-proton % VKD3D_TEST_FILTER=create_placed_resource_size VKD3D_CONFIG=fault VKD3D_DEBUG=trace ~/src/wine/build/wine ./tests/d3d12.exe 2>&1 | grep vkd3d_address_binding_callback trace:vkd3d-proton:vkd3d_address_binding_callback: BIND || VkBuffer || VA ffff800100200000 || size 0000000001000000. trace:vkd3d-proton:vkd3d_address_binding_callback: BIND || VkImage || VA ffff800100200000 || size 000000000019a000. trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkImage || VA ffff800100200000 || size 000000000019a000. trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkBuffer || VA ffff800100200000 || size 0000000001000000. :trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkDevice || VA ffff800100200000 || size 0000000001000000. 232285.553:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: BIND || VkBuffer || VA ffff800100200000 || size 0000000001000000. 232285.553:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkBuffer || VA ffff800100200000 || size 0000000001000000. 232285.553:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkDevice || VA ffff800100200000 || size 0000000001000000. ``` after: ``` % VKD3D_TEST_FILTER=create_placed_resource_size VKD3D_CONFIG=fault VKD3D_DEBUG=trace ~/src/wine/build/wine ./tests/d3d12.exe 2>&1 | grep vkd3d_address_binding_callback 232338.036:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: BIND || VkDevice || VA ffff800100200000 || size 0000000001000000. 232338.036:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: BIND || VkBuffer || VA ffff800100200000 || size 0000000001000000. 232338.037:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: BIND || VkImage || VA ffff800100200000 || size 000000000019a000. 232338.037:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkImage || VA ffff800100200000 || size 000000000019a000. 232338.037:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkBuffer || VA ffff800100200000 || size 0000000001000000. 232338.037:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkDevice || VA ffff800100200000 || size 0000000001000000. 232338.037:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: BIND || VkDevice || VA ffff800100200000 || size 0000000001000000. 232338.037:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: BIND || VkBuffer || VA ffff800100200000 || size 0000000001000000. 232338.037:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkBuffer || VA ffff800100200000 || size 0000000001000000. 232338.037:0020:0024:trace:vkd3d-proton:vkd3d_address_binding_callback: UNBIND || VkDevice || VA ffff800100200000 || size 0000000001000000. ``` [The spec guarantees](https://registry.khronos.org/vulkan/specs/1.3-extensions/html/v… the following: > An application can receive multiple callbacks if multiple VkDebugUtilsMessengerEXT objects are created. A callback will always be executed in the same thread as the originating Vulkan call. As of TLS I went with a suggestion from IRC: ``` <ivyl_> There's an issue with how we handle callbacks related to VK_EXT_debug_utils and VK_EXT_debug_report. In wine_vkAllocateMemory() we call device->funcs.p_vkAllocateMemory() which may execute callback. <ivyl_> At this point we still don't have handle mapping added because we don't know what host_memory handle will be. <ivyl_> AFAIU there's a spec guarantee that callback will be executed in the same thread as the call causing it, so I was thinking about maybe keeping `memory` in a tls (if enable_wrapper_list) and using that? <ivyl_> but that's I don't see any precedent of using thread local storage on the unix side <ivyl_> tls value would be short-lived in such case - only for the duration of the call to `device->funcs.p_vkAllocateMemory()` <jacekc_> we usually use teb on unix side, see ntuser_thread_info ``` -- v3: winevulkan: Make device memory wrapper available in callbacks. https://gitlab.winehq.org/wine/wine/-/merge_requests/5658

2 1

[PATCH 0/1] MR5901: profapi: Add stub dll.
by Zhiyi Zhang (＠zhiyi) 21 Jun '24

21 Jun '24

Required by React Native apps. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5901

3 2

Re: [PATCH v5 0/7] MR5319: winemac.drv keyboard layout/dead key fixes
by Rémi Bernon (＠rbernon) 21 Jun '24

21 Jun '24

Rémi Bernon (@rbernon) commented about dlls/winemac.drv/keyboard.c: > if (!len) > break; > > + if (status == noErr && len == 1) > + { > + CFStringRef stringRef = CFStringCreateWithCharacters(kCFAllocatorDefault, buffer, len); > + CFMutableStringRef mutableStringRef = CFStringCreateMutableCopy(kCFAllocatorDefault, 0, stringRef); > + > + CFStringUppercase(mutableStringRef, CFLocaleGetSystem()); > + if (CFStringGetLength(mutableStringRef) == 1) > + CFStringGetCharacters(mutableStringRef, CFRangeMake(0, 1), buffer); > + CFRelease(stringRef); > + CFRelease(mutableStringRef); > + } Does we need to support unicode? What about: ```suggestion:-10+0 if (status == noErr && len == 1 && buffer[0] >= 'a' && buffer[0] <= 'z') buffer[0] += 'A' - 'a'; ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5319#note_73911

1 0

Re: [PATCH v5 0/7] MR5319: winemac.drv keyboard layout/dead key fixes
by Rémi Bernon (＠rbernon) 21 Jun '24

21 Jun '24

Rémi Bernon (@rbernon) commented about dlls/winemac.drv/keyboard.c: > if (!len) > break; > > + if (status == noErr && deadKeyState) > + { > + for (i = 0; i < ARRAY_SIZE(dead_key_names); i++) > + { > + if (dead_key_names[i].wchar == buffer[0]) > + { > + len = min(strlen(vkey_names[i].name) + 1, size); ```suggestion:-0+0 len = min(strlen(dead_key_names[i].name) + 1, size); ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5319#note_73912

1 0

Re: [PATCH v5 0/7] MR5319: winemac.drv keyboard layout/dead key fixes
by Rémi Bernon (＠rbernon) 21 Jun '24

21 Jun '24

Rémi Bernon (@rbernon) commented about dlls/winemac.drv/keyboard.c: > done: > /* Null-terminate the buffer, if there's room. MSDN clearly states that the > caller must not assume this is done, but some programs (e.g. Audiosurf) do. */ > - if (1 <= ret && ret < bufW_size) > - bufW[ret] = 0; > + if (0 <= len && len < bufW_size) ```suggestion:-0+0 if (len < bufW_size) ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5319#note_73910

1 0

Re: [PATCH v5 0/7] MR5319: winemac.drv keyboard layout/dead key fixes
by Rémi Bernon (＠rbernon) 21 Jun '24

21 Jun '24

Rémi Bernon (@rbernon) commented about dlls/winemac.drv/keyboard.c: > UInt32 modifierKeyState; > OptionBits options; > UInt32 deadKeyState, savedDeadKeyState; > - UniCharCount len; > + UniCharCount len = 0; Doesn't seem to be needed. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5319#note_73908

1 0

2025

2024

2023

2022

wine-gitlab