- Wine-GitLab - list.winehq.org

Re: [PATCH v8 0/1] MR5373: shell32: Implement FCIDM_SHVIEW_CREATELINK
by Sven Baars (＠sbaars) Nov. 13, 2025

Nov. 13, 2025

Sven Baars (@sbaars) commented about dlls/shell32/shlview_cmenu.c: > + ITEMIDLIST *full_pidl; > + HRESULT hr; > + int length; > + int counter = 1; > + > + static const WCHAR *shortcutW = L"Shortcut"; > + > + if (FAILED(IShellLink_Constructor(NULL, &IID_IShellLinkW, (LPVOID*)&shelllink))) > + { > + ERR("couldn't create ShellLink object\n"); > + return; > + } > + full_pidl = ILCombine(This->pidl, This->apidl[0]); > + if (!full_pidl) > + { > + ERR("Out of Memory\n"); Why don't we just return an `HRESULT` here and return that from `ItemMenu_InvokeCommand`, instead of the verbose ERRs? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5373#note_121806

1 0

Re: [PATCH v8 0/1] MR5373: shell32: Implement FCIDM_SHVIEW_CREATELINK
by Sven Baars (＠sbaars) Nov. 13, 2025

Nov. 13, 2025

Sven Baars (@sbaars) commented about dlls/shell32/shlview_cmenu.c: > + return; > + } > + > + PathCombineW(lnkfile, root, link_filename); > + free(link_filename); > + counter++; > + > + if (PathFileExistsW(lnkfile)) > + { > + hr = HRESULT_FROM_WIN32(ERROR_FILE_EXISTS); > + } > + else > + { > + ERR("attempting to write link to %s\n", wine_dbgstr_w(lnkfile)); > + hr = IPersistFile_Save(persistfile, lnkfile, FALSE); > + ERR("ARIC %#lx\n",hr); Debug statements -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5373#note_121804

1 0

Re: [PATCH v8 0/1] MR5373: shell32: Implement FCIDM_SHVIEW_CREATELINK
by Sven Baars (＠sbaars) Nov. 13, 2025

Nov. 13, 2025

Sven Baars (@sbaars) commented about dlls/shell32/shlview_cmenu.c: > + full_pidl = ILCombine(This->pidl, This->apidl[0]); > + if (!full_pidl) > + { > + ERR("Out of Memory\n"); > + return; > + } > + > + hr = IShellLinkW_SetIDList(shelllink, full_pidl); > + ILFree(full_pidl); > + if (FAILED(hr)) > + { > + ERR("SetIDList failed\n"); > + return; > + } > + _ILSimpleGetTextW(This->apidl[0], (LPVOID)filename, MAX_PATH); > + length = wcslen(filename) + wcslen(shortcutW); This looks like a duplicate check. There is another one in the loop below. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5373#note_121803

1 0

[PATCH 0/4] MR9429: Avoid a few compiler warnings.
by Bernhard Übelacker Nov. 13, 2025

Nov. 13, 2025

Gitlab CI uses `--enable-werror` for its builds, but to improve backtraces with my recent address sanitizer experiments I am using these flags `-g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions` with both, gcc at the unix side and llvm-mingw at the pe side. Therefore these warnings would turn into errors. This four patches are helping gcc at the unix side. Would these patches be acceptable? Should they be separate merge requests? <details> <summary>Details</summary> ``` ccache gcc -m64 -c -o tools/winebuild/utils.o .../wine/tools/winebuild/utils.c \ -Itools/winebuild -I.../wine/tools/winebuild -Iinclude \ -I.../wine/include -D__WINESRC__ -DWINE_UNIX_LIB -Wall -pipe \ -fcf-protection=none -fvisibility=hidden -fno-stack-protector -fno-strict-aliasing \ -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Winit-self -Wlogical-op \ -Wno-packed-not-aligned -Wpointer-arith -Wshift-overflow=2 -Wstrict-prototypes -Wtype-limits \ -Wunused-but-set-parameter -Wvla -Wwrite-strings -Wno-misleading-indentation -gdwarf-4 -Werror \ -g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions .../wine/tools/winebuild/utils.c: In function 'find_tool': .../wine/tools/winebuild/utils.c:230:8: error: 'file' may be used uninitialized [-Werror=maybe-uninitialized] 230 | if (!file && cc_command.count) file = find_clang_tool( cc_command, name ); | ^ .../wine/tools/winebuild/utils.c:214:17: note: 'file' was declared here 214 | const char *file; | ^~~~ cc1: all warnings being treated as errors make: *** [Makefile:610992: tools/winebuild/utils.o] Error 1 ccache gcc -m64 -c -o dlls/ntdll/unix/file.o .../wine/dlls/ntdll/unix/file.c -Idlls/ntdll \ -I.../wine/dlls/ntdll -Iinclude \ -I.../wine/include -D__WINESRC__ -DLTC_NO_PROTOTYPES \ -DLTC_SOURCE -D_NTSYSTEM_ -D_ACRTIMP= -DWINBASEAPI= -DWINE_UNIX_LIB \ -I/usr/include/x86_64-linux-gnu -Wall -pipe -fcf-protection=none -fvisibility=hidden \ -fno-stack-protector -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body \ -Wignored-qualifiers -Winit-self -Wlogical-op -Wno-packed-not-aligned -Wpointer-arith \ -Wshift-overflow=2 -Wstrict-prototypes -Wtype-limits -Wunused-but-set-parameter -Wvla \ -Wwrite-strings -Wno-misleading-indentation -gdwarf-4 -Werror -fPIC -fasynchronous-unwind-tables \ -g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions .../wine/dlls/ntdll/unix/file.c: In function 'NtReadFile': .../wine/dlls/ntdll/unix/file.c:5758:23: error: 'avail_mode' may be used uninitialized [-Werror=maybe-uninitialized] 5758 | if (total && avail_mode) | ~~~~~~^~~~~~~~~~~~~ .../wine/dlls/ntdll/unix/file.c:5755:18: note: 'avail_mode' was declared here 5755 | BOOL avail_mode; | ^~~~~~~~~~ cc1: all warnings being treated as errors make: *** [Makefile:318598: dlls/ntdll/unix/file.o] Error 1 ccache gcc -m64 -c -o dlls/sane.ds/unixlib.o .../wine/dlls/sane.ds/unixlib.c -Idlls/sane.ds \ -I.../wine/dlls/sane.ds -Iinclude \ -I.../wine/include -D__WINESRC__ -DWINE_UNIX_LIB -Wall -pipe \ -fcf-protection=none -fvisibility=hidden -fno-stack-protector -fno-strict-aliasing \ -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Winit-self -Wlogical-op \ -Wno-packed-not-aligned -Wpointer-arith -Wshift-overflow=2 -Wstrict-prototypes -Wtype-limits \ -Wunused-but-set-parameter -Wvla -Wwrite-strings -Wno-misleading-indentation -gdwarf-4 -Werror \ -fPIC -fasynchronous-unwind-tables -g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions .../wine/dlls/sane.ds/unixlib.c: In function 'read_data': .../wine/dlls/sane.ds/unixlib.c:332:8: error: 'status' may be used uninitialized [-Werror=maybe-uninitialized] 332 | if (status == SANE_STATUS_EOF) return TWCC_SUCCESS; | ^ .../wine/dlls/sane.ds/unixlib.c:321:17: note: 'status' was declared here 321 | SANE_Status status; | ^~~~~~ cc1: all warnings being treated as errors make: *** [Makefile:366624: dlls/sane.ds/unixlib.o] Error 1 ccache gcc -m64 -c -o server/directory.o .../wine/server/directory.c -Iserver \ -I.../wine/server -Iinclude \ -I.../wine/include -D__WINESRC__ -DWINE_UNIX_LIB -Wall -pipe \ -fcf-protection=none -fvisibility=hidden -fno-stack-protector -fno-strict-aliasing \ -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Winit-self -Wlogical-op \ -Wno-packed-not-aligned -Wpointer-arith -Wshift-overflow=2 -Wstrict-prototypes -Wtype-limits \ -Wunused-but-set-parameter -Wvla -Wwrite-strings -Wno-misleading-indentation -gdwarf-4 -Werror \ -g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions .../wine/server/directory.c: In function 'create_session': .../wine/server/directory.c:327:44: error: 'snprintf' output may be truncated before the last format character [-Werror=format-truncation=] 327 | snprintf( id_strA, sizeof(id_strA), "%u", id ); | ^ .../wine/server/directory.c:327:5: note: 'snprintf' output between 2 and 11 bytes into a destination of size 10 327 | snprintf( id_strA, sizeof(id_strA), "%u", id ); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: all warnings being treated as errors make: *** [Makefile:607457: server/directory.o] Error 1 ``` </details> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9429

2 7

[PATCH v3 0/1] MR9449: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with...
by Zhiyi Zhang (＠zhiyi) Nov. 13, 2025

Nov. 13, 2025

winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. xinerama_get_fullscreen_monitors() was supposed to be called with fullscreen rects. Thus, we could assume the monitor indices are all zero when there is only one monitor as an optimization. However, after baf65867, xinerama_get_fullscreen_monitors() can be called with non-fullscreen rects. So we need to remove the optimization so that it works correctly for non-fullscreen rects as well. Fix the Guild Wars 2 game window having an offset in fullscreen mode. -- v3: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. https://gitlab.winehq.org/wine/wine/-/merge_requests/9449

3 6

[PATCH 0/2] MR9446: dmsynth: Fix instrument selection
by Anton Baskanov (＠baskanov) Nov. 13, 2025

Nov. 13, 2025

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

3 4

[PATCH v2 0/1] MR9449: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with...
by Zhiyi Zhang (＠zhiyi) Nov. 13, 2025

Nov. 13, 2025

winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. xinerama_get_fullscreen_monitors() was supposed to be called with fullscreen rects. Thus, we could assume the monitor indices are all zero when there is only one monitor as an optimization. However, after baf65867, xinerama_get_fullscreen_monitors() can be called with non-fullscreen rects. So we need to remove the optimization so that it works correctly for non-fullscreen rects as well. Fix the Guild Wars 2 game window having an offset in fullscreen mode. -- v2: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. https://gitlab.winehq.org/wine/wine/-/merge_requests/9449

3 2

Re: [PATCH v4 0/1] MR9423: ntdll: Warn when environment block is too large, instead of removing it
by Alexandre Julliard (＠julliard) Nov. 13, 2025

Nov. 13, 2025

No, I don't think it addresses the issue at all. I'll try to come up with something. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121789

1 0

[PATCH 0/1] MR9449: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with...
by Zhiyi Zhang (＠zhiyi) Nov. 13, 2025

Nov. 13, 2025

winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. xinerama_get_fullscreen_monitors() was supposed to be called with fullscreen rects. Thus, we could assume the monitor indices are all zero when there is only one monitor as an optimization. However, after baf65867, xinerama_get_fullscreen_monitors() can be called with non-fullscreen rects. So we need to remove the optimization so that it works correctly for non-fullscreen rects as well. Fix the Guild Wars 2 game window having an offset in fullscreen mode. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9449

2 2

[PATCH 0/2] MR9447: bcrypt: Complete the remaining four RSA parameters when importing the private key.
by Haoyang Chen (＠chenhaoyang) Nov. 13, 2025

Nov. 13, 2025

Some versions of libgnutls require passing all parameters when importing private keys, otherwise they will crash. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9447

3 9