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

[PATCH v8 0/7] MR6892: propsys: Implement IPropertyDescription for several known system properties.
by Vibhav Pant (＠vibhavp) June 16, 2025

June 16, 2025

This MR adds a very basic implementation of `IPropertyDescription` for system defined properties (i.e, the ones in \<propkey.h\>). This is needed to support the [`Properties`](https://learn.microsoft.com/en-us/uwp/api/windows.devices.enumeration.deviceinformation.properties?view=winrt-26100) method for `IDeviceInformation` in Windows.Devices.Enumeration (!6874). -- v8: propsys: Implement IPropertyDescription for several known system properties. propsys: Add IPropertyDescription stub for system defined properties. propsys/tests: Add conformance tests for PSGetNameFromPropertyKey. propsys: Add stubs for PSGetNameFromPropertyKey. propsys: Add stubs for PropertySystem. propsys/tests: Add conformance tests for PSGetPropertyKeyFromName. propsys/tests: Add conformance tests for getting PropertyDescriptions from PropertySystem. https://gitlab.winehq.org/wine/wine/-/merge_requests/6892

3 13

[PATCH v4 0/2] MR8258: comctl32/treeview: Return from TREEVIEW_LButtonDown when the treeview handle is invalid.
by Jacob Czekalla (＠JacobCzekalla) June 16, 2025

June 16, 2025

comctl32/treeview: Return from TREEVIEW_LButtonDown when the treeview handle is invalid. LButtonDown should return when the treeview handle is invalid (e.g. destroyed) after NM_CLICK to prevent further message processing. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58211 -- v4: comctl32/treeview: Return from TREEVIEW_LButtonDown when the treeview handle is invalid. comctl32/tests: Add a test for treeview deletion during NM_CLICK in LBUTTONDOWN. https://gitlab.winehq.org/wine/wine/-/merge_requests/8258

2 2

[PATCH v3 0/2] MR8258: comctl32/treeview: Return from TREEVIEW_LButtonDown when the treeview handle is invalid.
by Jacob Czekalla (＠JacobCzekalla) June 16, 2025

June 16, 2025

comctl32/treeview: Return from TREEVIEW_LButtonDown when the treeview handle is invalid. LButtonDown should return when the treeview handle is invalid (e.g. destroyed) after NM_CLICK to prevent further message processing. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58211 -- v3: comctl32/treeview: Return from TREEVIEW_LButtonDown when the treeview handle is invalid. https://gitlab.winehq.org/wine/wine/-/merge_requests/8258

3 7

[PATCH 0/2] MR8328: win32u: Better debugging for SendMessageTimeout timeouts.
by Stefan Dösinger June 16, 2025

June 16, 2025

I spent a week debugging an application only to find out that SendMessageTimeout timed out. These two patches should make it easier to catch such cases in the future. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8328

1 2

[PATCH v3 0/1] MR8306: winebuild: spec32.c: A couple changes to get a build on Linux ARM64
by Mike (＠JinShil) June 16, 2025

June 16, 2025

-- v3: winebuild: spec32.c: A couple changes to get a build on Linux ARM64 https://gitlab.winehq.org/wine/wine/-/merge_requests/8306

3 11

[PATCH 0/2] MR8327: comctl32/listview: Always set the initial background mode to TRANSPARENT.
by Zhiyi Zhang (＠zhiyi) June 16, 2025

June 16, 2025

Comctl32 v5 listview sets the initial background mix mode to TRANSPARENT. Comctl32 v6 listview sets it to OPAQUE. Use TRANSPARENT for both v5 and v6 for now. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8327

3 3

Re: [PATCH v10 0/5] MR7540: kernel32/tests: Add tests for maximum path length limitation.
by Zhiyi Zhang (＠zhiyi) June 16, 2025

June 16, 2025

Zhiyi Zhang (@zhiyi) commented about dlls/ntdll/actctx.c: > if (!RtlCreateActivationContext( &handle, &ctx )) process_actctx = check_actctx(handle); > > NtCurrentTeb()->Peb->ActivationContextData = process_actctx; > + > + if (!RtlQueryActivationContextApplicationSettings( 0, NULL, L"http://schemas.microsoft.com/SMI/2016/WindowsSettings", Please test if activating the longPathAware manifest dynamically sets IsLongPathAwareProcess, or if it is just checked at process startup. MSDN says the LongPathsEnabled registry value is cached. However, it doesn't say if the manifest is cached. This could be a local test. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_106696

1 0

Re: [PATCH v10 0/5] MR7540: kernel32/tests: Add tests for maximum path length limitation.
by Zhiyi Zhang (＠zhiyi) June 16, 2025

June 16, 2025

Zhiyi Zhang (@zhiyi) commented about dlls/ntdll/actctx.c: > return NULL; > } > > +/* Windows 10, version 1607, building number 14393, Redstone, August 2, 2016 */ > +static BOOL is_win10_1607_or_later(void) > +{ > + static RTL_OSVERSIONINFOEXW rovi = { 0 }; Add a new line. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_106695

1 0

Re: [PATCH v10 0/5] MR7540: kernel32/tests: Add tests for maximum path length limitation.
by Zhiyi Zhang (＠zhiyi) June 16, 2025

June 16, 2025

Zhiyi Zhang (@zhiyi) commented about dlls/kernel32/tests/file.c: > + > + wcscpy(long_path_1, L"\\\\?\\"); > + wcscat(long_path_1, temp_path); > + wcscat(long_path_1, a); > + SetLastError(0xdeadbeef); > + ret = DeleteFileW(long_path_1); > + todo_wine ok(ret, "DeleteFileW: error %ld\n", GetLastError()); > + wcscpy(long_path_2, L"\\\\?\\"); > + wcscat(long_path_2, temp_path); > + wcscat(long_path_2, b); > + SetLastError(0xdeadbeef); > + ret = DeleteFileW(long_path_2); > + todo_wine ok(ret, "DeleteFileW: error %ld\n", GetLastError()); > + } > + > + ret = DeleteFileW(NULL); This test doesn't belong here. Move it to test_DeleteFileW() and another patch. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_106694

1 0

Re: [PATCH v10 0/5] MR7540: kernel32/tests: Add tests for maximum path length limitation.
by Zhiyi Zhang (＠zhiyi) June 16, 2025

June 16, 2025

Zhiyi Zhang (@zhiyi) commented about dlls/kernel32/tests/file.c: > + SetLastError(0xdeadbeef); > + ret = CopyFileExW(source, long_path_1, NULL, NULL, NULL, 0); > + ok(ret, "Expected CopyFileExW successed, but got %ld, copy %s -> %s\n", GetLastError(), wine_dbgstr_w(source), wine_dbgstr_w(long_path_1)); > + > + DeleteFileW(dest); > + SetLastError(0xdeadbeef); > + ret = CopyFileExW(long_path_1, dest, NULL, NULL, NULL, 0); > + ok(ret, "Expected CopyFileExW successed, but got %ld, copy %s -> %s\n", GetLastError(), wine_dbgstr_w(long_path_1), wine_dbgstr_w(dest)); > + > + wcscpy(long_path_2, temp_path); > + wcscat(long_path_2, b); > + SetLastError(0xdeadbeef); > + ret = CopyFileExW(long_path_1, long_path_2, NULL, NULL, NULL, 0); > + ok(ret, "Expected CopyFileExW successed, but got %ld, copy %s -> %s\n", GetLastError(), wine_dbgstr_w(long_path_1), wine_dbgstr_w(long_path_2)); > + ret = DeleteFileW(long_path_1); > + ok(ret, "Unexpected DeleteFileW successed\n"); Same here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_106693

1 0