August 2025 - Wine-GitLab - list.winehq.org

Re: [PATCH v31 0/1] MR8573: cmd: tab completion: Treat most delimiters as literals if user specified quotes. - approved
by Alexandre Julliard (＠julliard) Aug. 26, 2025

Aug. 26, 2025

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

1 0

Re: [PATCH v4 0/6] MR8818: user32, win32u, server: Implement integral class name versioning support. - approved
by Alexandre Julliard (＠julliard) Aug. 26, 2025

Aug. 26, 2025

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

1 0

Re: [PATCH v3 0/1] MR8832: gitlab: Update CI image to debian trixie. - approved
by Alexandre Julliard (＠julliard) Aug. 26, 2025

Aug. 26, 2025

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

1 0

[PATCH 0/1] MR8840: ntdll/tests: Add check for len and avoid CommitSize with old Windows.
by Bernhard Übelacker Aug. 26, 2025

Aug. 26, 2025

Followup to 212be298db. [Test pattern page](https://test.winehq.org/data/patterns.html#ntdll:virtual) [Testbot run with this patch](https://testbot.winehq.org/JobDetails.pl?Key=159540) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8840

1 1

[PATCH 0/3] MR8839: ntdll, kernelbase: Fixes for creating Unix processes via CreateProcess/NtCreateUserProcess
by Tim Clem (＠tclem) Aug. 26, 2025

Aug. 26, 2025

An application was spawning a Unix process with CreateProcess (which returned success only when the stack was favorable; see the first patch) and then closing the handles from the PROCESS_INFORMATION, which were garbage. That obviously caused havoc elsewhere if those garbage values happened to be valid handles. It's not quite clear to me what the "right" thing to do is when a Unix process is created this way, since it's kind of a hack. But we already zero out the PS_CREATE_INFO in NtCreateUserProcess, so zeroing the handles seems in keeping with that. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8839

2 3

Re: [PATCH v17 0/1] MR8778: sane.ds: Adding a window for loading and saving the scanner configuration
by Esme Povirk (＠madewokherd) Aug. 26, 2025

Aug. 26, 2025

On Tue Aug 26 09:09:44 2025 +0000, Ivan Lyugaev wrote: > Will my MR be considered further? I think using the registry would make this significantly cleaner, enough that I don't plan to review it further while it uses files. I'd consider multiple configurations an advanced feature, and that's still possible for the UI to provide based on the registry. Transferring configurations across different machines is going to be even less common. I don't think the improvement in discoverability for that use case is worth it. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8778#note_113915

1 0

Re: [PATCH v30 0/1] MR8573: cmd: tab completion: Treat most delimiters as literals if user specified quotes.
by Joe Souza (＠JoeS209) Aug. 26, 2025

Aug. 26, 2025

On Tue Aug 26 16:38:49 2025 +0000, Alexandre Julliard wrote: > You should stop updating your MR every day. Not only is it a waste of CI > resources, but it makes it look like it's not ready since it keeps being updated. > Also you don't need additional forks, you can create a MR from a > different branch in the same fork. My apologies about the daily updating. This was a misunderstanding on my part, since the UI here states that checks have failed and that the MR is not ready for merging after the main project has taken commits. Due to that warning here, I had thought that it was necessary to keep this fork updated with main, until it has been merged with main. Since what you say here tells me that I was wrong about this, I will stop updating my fork. Thank you. This MR is ready. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8573#note_113910

1 0

[PATCH v9 0/6] MR8831: vccorlib140: Implement support for boxed values (aka IPropertyValue)
by Vibhav Pant (＠vibhavp) Aug. 26, 2025

Aug. 26, 2025

C++/CX exposes `IPropertyValue` through `Platform::Details::CreateValue`, which returns a `Platform::Object` (which just seems to be a `IPropertyValue` object + C++ RTTI) containing the boxed value. Apps don't seem to call `CreateValue` directly with a typecode. The code for boxing values gets generated when a value type gets assigned to a `Platform::Object` variable, as documented [here.](https://learn.microsoft.com/en-us/cpp/extensions/boxing-cpp-componen… The generated code will: 1. Call `__abi_make_type_id` with a `struct __abi_type_descriptor *` value, which is just a typecode + string pair. This returns a `Platform::Type` object. 2. Call `Platform::Type::GetTypeCode` on the returned `Platform::Type` object to get back the typecode. 3. Call `Platform::Details::CreateValue` with the typecode and a pointer to the actual value that needs to be boxed. Apart from the usual WinRT methods, `Platform::Type` obejcts also impement the `IEquatable` and `IPrintable` interfaces. I don't know what their vtables look like, though. The `Platform::Type` and `Platform::Object` classes also have associated RTTI (can be demonstrated by calling `__RTtypeid`), which we'll also need to emit at some point in the future. -- v9: vccorlib140: Use the correct symbol name for InitControlBlock on i386 and arm. vccorlib140: Implement Platform::Details::CreateValue. vccorlib140: Add stub for Platform::Details::CreateValue. vccorlib140: Implement Platform::Type::{Equals, GetTypeCode, ToString, FullName::get}. vccorlib140: Implement __abi_make_type_id. https://gitlab.winehq.org/wine/wine/-/merge_requests/8831

3 7

[PATCH v8 0/6] MR8831: vccorlib140: Implement support for boxed values (aka IPropertyValue)
by Vibhav Pant (＠vibhavp) Aug. 26, 2025

Aug. 26, 2025

C++/CX exposes `IPropertyValue` through `Platform::Details::CreateValue`, which returns a `Platform::Object` (which just seems to be a `IPropertyValue` object + C++ RTTI) containing the boxed value. Apps don't seem to call `CreateValue` directly with a typecode. The code for boxing values gets generated when a value type gets assigned to a `Platform::Object` variable, as documented [here.](https://learn.microsoft.com/en-us/cpp/extensions/boxing-cpp-componen… The generated code will: 1. Call `__abi_make_type_id` with a `struct __abi_type_descriptor *` value, which is just a typecode + string pair. This returns a `Platform::Type` object. 2. Call `Platform::Type::GetTypeCode` on the returned `Platform::Type` object to get back the typecode. 3. Call `Platform::Details::CreateValue` with the typecode and a pointer to the actual value that needs to be boxed. Apart from the usual WinRT methods, `Platform::Type` obejcts also impement the `IEquatable` and `IPrintable` interfaces. I don't know what their vtables look like, though. The `Platform::Type` and `Platform::Object` classes also have associated RTTI (can be demonstrated by calling `__RTtypeid`), which we'll also need to emit at some point in the future. -- v8: vccorlib140: Use the correct symbol name for InitControlBlock on i386 and arm. vccorlib140: Implement Platform::Details::CreateValue. vccorlib140: Add stub for Platform::Details::CreateValue. vccorlib140: Implement Platform::Type::{Equals, GetTypeCode, ToString, FullName::get}. vccorlib140: Implement __abi_make_type_id. https://gitlab.winehq.org/wine/wine/-/merge_requests/8831

4 12

[PATCH 0/5] MR8685: wintypes: Implement IMap::{Lookup, HasKey, Insert} for IPropertySet.
by Vibhav Pant (＠vibhavp) Aug. 26, 2025

Aug. 26, 2025

`IMap` is implemented on top of a `rb_tree` guarded with an `SRWLOCK`. We need the map to be thread safe as: * `PropertySet` is marked as `IAgileObject`. * I plan to use `PropertySet` as the backing store for [`DeviceInformation::Properties`](https://learn.microsoft.com/en-us/uwp/api/windows.devices.enumeration.deviceinformation.properties?view=winrt-26100). These properties can be updated by apps after receiving an [`Updated`](https://learn.microsoft.com/en-us/uwp/api/windows.devices.enumeration.devicewatcher.updated?view=winrt-26100#windows-devices-enumeration-devicewatcher-updated) event from the device watcher with [`DeviceInformation::Update`](https://learn.microsoft.com/en-us/uwp/api/windows.devices.enumeration.deviceinformation.update?view=winrt-26100). The current `DeviceWatcher` implementation dispatches the event delegates for `Update` from a separate thread. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8685

5 15