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

[PATCH 0/1] MR6183: nsiproxy.sys: Fix the build on non-Apple, non-Linux systems
by Gerald Pfeifer (＠gerald) Aug. 2, 2024

Aug. 2, 2024

commit 5261811d471 revamped the code around pcblist structures, alas assumes a variable mib is always present on non-Apple, non-Linux systems when it is not. And in any case, ARRAY_SIZE should be used for the actual structure it applies to, not a similar one. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6183

4 3

[PATCH 0/1] MR6182: atl: Correct comment above AtlModuleRegisterTypeLib function.
by Alex Henrie (＠alexhenrie) Aug. 2, 2024

Aug. 2, 2024

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

3 2

[PATCH 0/1] MR6181: atl: Only warn in AtlModuleGetClassObject if the class was not found.
by Alex Henrie (＠alexhenrie) Aug. 2, 2024

Aug. 2, 2024

A warning should only be printed if something went wrong, not every time the function is run. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6181

3 2

[PATCH 0/2] MR6180: gdiplus: Fixes when compiling against wine headers.
by Alistair Leslie-Hughes (＠alesliehughes) Aug. 2, 2024

Aug. 2, 2024

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

3 3

[PATCH 0/1] MR6179: include: Ensure that x86_64 syscall thunks have a consistent length when built with Clang.
by Brendan Shanks (＠bshanks) Aug. 2, 2024

Aug. 2, 2024

GCC always assembles `jmp 1f` to `eb 01`, as does Clang when using -O1 or higher optimization. But with -O0, Clang outputs `e9 01 00 00 00`. The `subq $0xb,0x70(%rcx)` line in __wine_syscall_dispatcher relies on `jmp 1f` being 2 bytes. Since Wine defaults to `-g -O2` for `CFLAGS`, this wouldn't show up with a default Clang build. But the FreeBSD `wine-devel` port must override `CFLAGS`, it uses Clang for PE and was crashing on launch. See <https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=280000>. Here are compiler explorer links to see the difference: [PE](https://godbolt.org/z/35TxW8dK9) and [ELF](https://godbolt.org/z/1h3j4c4ja). Adding -O1 to the clang side will make them equivalent. I may also report this as an LLVM bug, it doesn't seem like there's any reason why Clang would prefer the long form at `-O0`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6179

3 3

[PATCH 0/3] MR6178: ntdll: Use environ/_NSGetEnviron() directly rather than caching it in main_envp.
by Brendan Shanks (＠bshanks) Aug. 2, 2024

Aug. 2, 2024

Fixes an unpredictable use-after-free crash on launch on macOS. Also, use `_NSGetEnviron()` instead of `environ` in `msv1_0` and `ntdll`. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57005 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6178

3 4

[PATCH v2 0/1] MR6174: comctl32: Handle WM_GETOBJECT in tab control.
by Esme Povirk (＠madewokherd) Aug. 2, 2024

Aug. 2, 2024

-- v2: comctl32: Handle WM_GETOBJECT in tab control. https://gitlab.winehq.org/wine/wine/-/merge_requests/6174

4 4

[PATCH v5 0/2] MR6160: comctl32: Fix sorting for listview.
by Jacob Czekalla (＠JacobCzekalla) Aug. 2, 2024

Aug. 2, 2024

The listview bug is caused by passing the incorrect index to the LISTVIEW_SortItems callers custom compare function. If we pass the copied array indices, which are different because of the sorting, the caller is going to use those indices for their unsorted array which don't match. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56140 -- v5: comctl32: Fix sorting for listview. comctl32/tests: Add test for listview sorting order. https://gitlab.winehq.org/wine/wine/-/merge_requests/6160

4 10

[PATCH 0/8] MR6177: mshtml: Use JS bindings for more objects.
by Jacek Caban (＠jacek) Aug. 2, 2024

Aug. 2, 2024

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

3 9

Re: [PATCH v13 0/1] MR6150: win32u: fix a regression bug that virtual desktop not effective.
by Fan WenJie (＠fanwenjie) Aug. 2, 2024

Aug. 2, 2024

On Wed Jul 31 13:55:13 2024 +0000, Zhiyi Zhang wrote: > I don't think you're going in the right direction. Why did you remove > all this code? You should be able to parse the resolution from the > command line and set it as the resolution for the virtual desktop window > and resolve the regression. There is no need for parsing, because parse resolution is doing by explorer.exe, explorer.exe parse resolution by arguments and setting by NtUserChangeDisplaySetting -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6150#note_77582

1 0