June 2025 - wine-gitlab

[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

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(ret, "DeleteFileA: error %ld\n", GetLastError()); > } > > +/* 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 newline after variable declaration. Same at other places. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_106690

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: > static void (WINAPI *pRtlFreeUnicodeString)(PUNICODE_STRING); > static BOOL (WINAPI *pSetFileCompletionNotificationModes)(HANDLE, UCHAR); > static HANDLE (WINAPI *pFindFirstStreamW)(LPCWSTR filename, STREAM_INFO_LEVELS infolevel, void *data, DWORD flags); > +static NTSTATUS(WINAPI* pRtlGetVersion)(PRTL_OSVERSIONINFOEXW); Change it to "static NTSTATUS (WINAPI *pRtlGetVersion)(PRTL_OSVERSIONINFOEXW);" for style consistency. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_106689

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/kernelbase/file.c: > SetLastError( ERROR_INVALID_PARAMETER ); > return FALSE; > } > + > + if (!RtlGetCurrentPeb()->IsLongPathAwareProcess && wcsncmp(source, L"\\\\?\\", 4) && wcslen(source) >= MAX_PATH ) Let reorder the conditions to `wcslen(source) >= MAX_PATH && wcsncmp(source, L"\\\\?\\", 4) && !RtlGetCurrentPeb()->IsLongPathAwareProcess)`. I find it clearer this way. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7540#note_106688

1 0

2025

2024

2023

2022

wine-gitlab June 2025