September 2023 - wine-gitlab

Re: [PATCH v5 0/4] MR3672: appxdeploymentclient: Add IPackageManager2 stub interface.
by Rémi Bernon (＠rbernon) 08 Sep '23

08 Sep '23

On Fri Sep 8 06:44:32 2023 +0000, Mohamad Al-Jaf wrote: > I've removed it, seems like it's not really necessary to test for it. > Though, I don't understand why Office queries for `IAgileObject`. It's a > Microsoft program so they should know it's not supported. Minecraft also > queries for an interface that returns `E_NOINTERFACE`, it seems bizarre. You should still remove the `broken(...)` condition, it will otherwise let every test succeed silently on Windows. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3672#note_44665

1 0

Re: [PATCH v3 0/27] MR332: include: Fill in missing enum/struct/interfaces in vkd3d_d3d12.idl
by Martin Storsjö (＠mstorsjo) 08 Sep '23

08 Sep '23

On Fri Sep 8 06:50:30 2023 +0000, Henri Verbeet wrote: > At first sight this looks fine, thanks. Unfortunately I probably won't > be able to properly review this this week, but I should be able to get > to it early next week. > Note that this currently fails the CI though. Probably because it's > intended to be applied on top of !325, so hopefully that will get > resolved once those patches actually get committed. It still fails CI after !325 was merged (and even before - as it was based on top of !325, those commits were part of this branch so testing would just have iterated over those too) - not quite sure what is wrong here... -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/332#note_44664

1 0

Re: [PATCH v5 0/4] MR3672: appxdeploymentclient: Add IPackageManager2 stub interface.
by Mohamad Al-Jaf (＠maljaf) 08 Sep '23

08 Sep '23

On Thu Sep 7 13:49:19 2023 +0000, Rémi Bernon wrote: > I missed this before, but I think the check_interface check is a bit > pointless as it will always succeed on Windows. > Something like that would be better (using `check_interface_broken` only > where it is needed): > ```suggestion:-14+0 > #define check_interface_broken( obj, iid, supported ) check_interface_( > __LINE__, obj, iid, supported, TRUE ) > #define check_interface( obj, iid, supported ) check_interface_( > __LINE__, obj, iid, supported, FALSE ) > static void check_interface_( unsigned int line, void *obj, const IID > *iid, BOOL supported, BOOL is_broken ) > { > HRESULT hr, expected_hr, broken_hr; > IUnknown *iface = obj; > IUnknown *unk; > expected_hr = supported ? S_OK : E_NOINTERFACE; > broken_hr = supported ? E_NOINTERFACE : S_OK; > hr = IUnknown_QueryInterface( iface, iid, (void **)&unk ); > ok_(__FILE__, line)( hr == expected_hr || broken( is_broken && hr == > broken_hr ), "got hr %#lx.\n", hr ); > if (SUCCEEDED(hr)) IUnknown_Release( unk ); > } > ``` > Or, maybe even better, use the basic `check_interface`, without the > broken result and remove the broken check for IAgileObject interface > which probably doesn't really matter on the factory. I've removed it, seems like it's not really necessary to test for it. Though, I don't understand why Office queries for `IAgileObject`. It's a Microsoft program so they should know it's not supported. Minecraft also queries for an interface that returns `E_NOINTERFACE`, it seems bizarre. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3672#note_44663

1 0

[PATCH v4 0/4] MR3672: appxdeploymentclient: Add IPackageManager2 stub interface.
by Mohamad Al-Jaf (＠maljaf) 08 Sep '23

08 Sep '23

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55403 -- v4: appxdeploymentclient: Add IPackageManager2 stub interface. include: Add Windows.Management.Deployment.IPackageManager2 interface definition. appxdeploymentclient: Implement IActivationFactory::ActivateInstance(). appxdeploymentclient: Add stub DLL. https://gitlab.winehq.org/wine/wine/-/merge_requests/3672

2 4

[PATCH 0/3] MR3737: winegstreamer: Implement wg_muxer.
by Ziqing Hui (＠zhui) 08 Sep '23

08 Sep '23

This patch set is part of !3303 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3737

4 15

[PATCH 0/1] MR3765: winemenubuilder: Use CRT allocation functions.
by Alex Henrie (＠alexhenrie) 08 Sep '23

08 Sep '23

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

2 1

[PATCH 0/1] MR3764: wineboot: Use CRT allocation functions.
by Alex Henrie (＠alexhenrie) 08 Sep '23

08 Sep '23

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

2 1

[PATCH 0/1] MR3763: oleview: Use CRT allocation functions.
by Alex Henrie (＠alexhenrie) 08 Sep '23

08 Sep '23

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

2 1

[PATCH 0/1] MR3762: extrac32: Use CRT allocation functions.
by Alex Henrie (＠alexhenrie) 08 Sep '23

08 Sep '23

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

2 1

[PATCH 0/1] MR3761: wintrust: Use CRT allocation functions.
by Alex Henrie (＠alexhenrie) 08 Sep '23

08 Sep '23

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

2 1

2025

2024

2023

2022

wine-gitlab September 2023