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

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

16 Jun '25

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 16 Jun '25

16 Jun '25

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) 16 Jun '25

16 Jun '25

-- 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) 16 Jun '25

16 Jun '25

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) 16 Jun '25

16 Jun '25

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) 16 Jun '25

16 Jun '25

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) 16 Jun '25

16 Jun '25

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) 16 Jun '25

16 Jun '25

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

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

16 Jun '25

Zhiyi Zhang (@zhiyi) commented about dlls/kernel32/tests/file.c: > ok(GetLastError() == ERROR_SUCCESS || broken(GetLastError() == ERROR_INVALID_PARAMETER) /* some win8 machines */, > "Unexpected error %lu.\n", GetLastError()); > > + /* test long file path, the length of the long_dest is larger than MAX_PATH. */ > + if (is_enabled_long_path()) > + { > + wcscpy(long_path_1, temp_path); > + wcscat(long_path_1, a); > + 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)); `successed` is a typo. Let's use "CopyFileExW(%s, %s) failed, error %lu.\n" -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_106692

1 0

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

16 Jun '25

Zhiyi Zhang (@zhiyi) commented about dlls/kernel32/tests/file.c: > + { > + rovi.dwOSVersionInfoSize = sizeof(rovi); > + if (pRtlGetVersion && S_OK == pRtlGetVersion(&rovi)) > + trace("windows version: win %ld, build %ld\n", rovi.dwMajorVersion, rovi.dwBuildNumber); > + } > + > + return rovi.dwMajorVersion >= 10 && rovi.dwBuildNumber >= 14393; > +} > + > +static BOOL is_reg_enable_long_path(void) > +{ > + static DWORD LongPathEnabled = -1; > + > + DWORD ret; > + HKEY hkey; > + DWORD type, size = sizeof(LongPathEnabled); You can put ret, type, and size on the same line and initialise size after RegOpenKeyExA succeeds. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_106691

1 0