November 2022 - Wine-gitlab

[PATCH v2 0/5] MR1560: d3d10/effect: Preshader instructions support improvements.
by Nikolay Sivov (＠nsivov) 30 Nov '22

30 Nov '22

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- v2: d3d10/effect: Add sin/cos instruction support for expressions. d3d10/effect: Add 'mul' instruction support for expressions. d3d10/effect: Add 'neg' instruction support for expressions. d3d10/effect: Add min/max instructions support for expressions. d3d10/effect: Remove unused field from property description data. https://gitlab.winehq.org/wine/wine/-/merge_requests/1560

3 7

[PATCH v14 0/4] MR1148: ntdll: Mark created files beginning with a dot as not hidden.
by Torge Matthies (＠tmatthies) 30 Nov '22

30 Nov '22

Fixes bug [53826](https://bugs.winehq.org/show_bug.cgi?id=53826). -- v14: ntdll: Set xattr in NtCreateFile if inferred and requested attributes don't match. ntdll: Only infer hidden attribute from file name if xattr is not present. ntdll: Handle hidden file names inside get_file_info instead of after it. https://gitlab.winehq.org/wine/wine/-/merge_requests/1148

3 7

[PATCH 0/3] MR1608: localspl: Use providor for print jobs implementation
by Piotr Caban (＠piotr) 30 Nov '22

30 Nov '22

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

4 5

[PATCH v13 0/4] MR1148: ntdll: Mark created files beginning with a dot as not hidden.
by Torge Matthies (＠tmatthies) 30 Nov '22

30 Nov '22

Fixes bug [53826](https://bugs.winehq.org/show_bug.cgi?id=53826). -- v13: ntdll: Set xattr in NtCreateFile if inferred and requested attributes don't match. ntdll: Only infer hidden attribute from file name if xattr is not present. ntdll: Handle hidden file names inside get_file_info instead of after it. ntdll/tests: Add test for file attributes of files with names beginning with a dot. https://gitlab.winehq.org/wine/wine/-/merge_requests/1148

3 7

[PATCH 0/3] MR1616: loader: In macOS preloader, manually set 'environ' to work around a bug in macOS 12 and later.
by Brendan Shanks (＠bshanks) 30 Nov '22

30 Nov '22

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54009 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1616

2 3

[PATCH 0/1] MR1615: wineusb.sys: Do not close the device handle more than once.
by Zebediah Figura (＠zfigura) 30 Nov '22

30 Nov '22

Since multiple interfaces can hold onto it, we need to refcount it. In order to do this just refcount the parent device, and close the handle only when the parent device refcount reaches 0. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1615

2 1

Re: [PATCH v5 0/1] MR1503: wineusb.sys: Move event handling to a single thread. - approved
by Zebediah Figura (＠zfigura) 30 Nov '22

30 Nov '22

This merge request was approved by Zebediah Figura. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1503

1 0

[PATCH v5 0/1] MR1554: Draft: win32u: Update focused item when removing menu item
by Vladislav Timonin (＠vt) 30 Nov '22

30 Nov '22

Fixes an out of bounds access in `show_popup`. ```c if (menu->FocusedItem != NO_SELECTED_ITEM) { menu->items[menu->FocusedItem].fState &= ~(MF_HILITE|MF_MOUSESELECT); // <- can crash here menu->FocusedItem = NO_SELECTED_ITEM; } ``` Menu resets focused item on show, not on close. If items were later deleted, next time user opens the menu, menu can crash on out of bounds access and won't show up again, as menu thinks it's still on screen. In other words: - open split button dropdown - click on any item - clear dropdown items - open dropdown again Menu borked and won't open again. Source for the testing app for reproduction. [test.c](/uploads/2f703f63891b33e1a0eb0fcd27c912b7/test.c) I guess, the alternative is to reset focused item when menu is closed, but not sure where's the best place to do that, haven't dug deep enough. -- v5: win32u: Reset focused item if it was removed when removing a menu item https://gitlab.winehq.org/wine/wine/-/merge_requests/1554

2 2

Re: [PATCH v4 0/1] MR1418: msvcm: add stubs to avoid Athenas ASS and IL-DSP crash.
by Piotr Caban (＠piotr) 30 Nov '22

30 Nov '22

On Wed Nov 30 17:41:13 2022 +0000, Bartosz Kosiorek wrote: > Thanks Piotr for detailed analysis. > I have found following information about `__clrcall`: > https://learn.microsoft.com/en-us/cpp/cpp/clrcall?view=msvc-140 > Unfortunately I don't know how `__clrcall` could be implemented It should be OK to use __fastcall instead (__fastcall is already used in wine). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1418#note_18100

1 0

Re: [PATCH v2 0/1] MR1552: winevulkan: Use direct calls for vkGetDescriptorEXT.
by Jacek Caban (＠jacek) 30 Nov '22

30 Nov '22

On Wed Nov 30 10:54:56 2022 +0000, Georg Lehmann wrote: > Yes, we can remove it when we stop using direct calls for descriptor set > updates. But at the moment I really don't want to have worse performance > with descriptor buffers compared to descriptor sets caused by wine. > Especially since it's unlikely that Rémi's work will be in 8.0. > This is not just some theoretical problem. With the current non direct > calls, vkd3d-proton's descriptor buffer path regresses performance for > radv users. I'm more optimistic about 8.0. Anyway, I wouldn't mind committing it if we need to. At the same time, since there is an active work on the final solution, we may as well wait a few days and see how it goes. Let me defer that to Alexandre (I think he took over that optimization work). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1552#note_18096

1 0