April 2023 - wine-gitlab

[PATCH v6 0/2] MR141: vkd3d-shader/ir: Normalise hull shader control point phase I/O.
by Conor McCarthy (＠cmccarthy) 12 Apr '23

12 Apr '23

To simplify SM 6 support, insert a control point id relative address where needed, and declare control point phase inputs where missing. -- v6: vkd3d-shader/ir: Insert hull shader control point input declarations if no control point phase is defined. https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/141

3 4

Re: [PATCH v5 0/2] MR2545: winmm: Remove 'dwLineID' check when MIXER_GETLINECONTROLSF_ONEBYID
by Huw Davies (＠huw) 12 Apr '23

12 Apr '23

On Wed Apr 12 09:05:59 2023 +0000, Yeshun Ye wrote: > https://gitlab.winehq.org/yeyeshun/wine/-/jobs/15769 That's the run on linux. The above failures are from runs on Windows. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2545#note_29716

1 0

Re: [PATCH v5 0/2] MR2545: winmm: Remove 'dwLineID' check when MIXER_GETLINECONTROLSF_ONEBYID
by Yeshun Ye (＠yeyeshun) 12 Apr '23

12 Apr '23

On Wed Apr 12 09:00:14 2023 +0000, Huw Davies wrote: > > I just run the test again without any change, and it passed. > I don't see that, could you post the link? https://gitlab.winehq.org/yeyeshun/wine/-/jobs/15769 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2545#note_29712

1 0

Re: [PATCH v5 0/2] MR2545: winmm: Remove 'dwLineID' check when MIXER_GETLINECONTROLSF_ONEBYID
by Huw Davies (＠huw) 12 Apr '23

12 Apr '23

> I just run the test again without any change, and it passed. I don't see that, could you post the link? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2545#note_29706

1 0

[PATCH 0/8] MR2633: mshtml: Use new Wine Gecko 2.47.4 features.
by Gabriel Ivăncescu (＠insn) 12 Apr '23

12 Apr '23

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

4 12

Re: [PATCH v6 0/1] MR2305: rpcrt4: Pass the binding handle to server call function.
by Zhiyi Zhang (＠zhiyi) 12 Apr '23

12 Apr '23

On Wed Apr 12 07:42:32 2023 +0000, Yeshun Ye wrote: > > The patch may fix an application, but that doesn't mean it's correct. > Yeah, I know that. > The problem is my test isn't just add an interface function to the > existing interface, I need to add a new interface without > 'implicit_handle'. I need to understand all the server test code of > rpcrt4 and insert my interface in a correct place, and start the server > and client at a right time. It's too hard, Visual Studio did this for me automatically. Does the following help? ``` [ explicit_handle ] interface interface-name { ... } ``` See https://learn.microsoft.com/en-us/windows/win32/midl/explicit-handle and https://learn.microsoft.com/en-us/windows/win32/rpc/explicit-binding-handles -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2305#note_29703

1 0

[PATCH 0/9] MR2637: winex11: Use messages instead of user callbacks and handle them in the default IME UI window procedure.
by Rémi Bernon (＠rbernon) 12 Apr '23

12 Apr '23

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

3 10

[PATCH 0/1] MR146: vkd3d: Append a full memory barrier after the command list batch in ExecuteCommandLists().
by Conor McCarthy (＠cmccarthy) 12 Apr '23

12 Apr '23

The D3D12 spec guarantees that lists submitted in ExecuteCommandLists() will complete execution before any subsequent commands begin execution. Based on a vkd3d-proton patch by Hans-Kristian Arntzen. -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/146

4 10

Re: [PATCH v6 0/1] MR2305: rpcrt4: Pass the binding handle to server call function.
by Yeshun Ye (＠yeyeshun) 12 Apr '23

12 Apr '23

> The patch may fix an application, but that doesn't mean it's correct. Yeah, I know that. The problem is my test isn't just add an interface function to the existing interface, I need to add a new interface without 'implicit_handle'. I need to understand all the server test code of rpcrt4 and insert my interface in a correct place, and start the server and client at a right time. It's too hard, Visual Studio did this for me automatically. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2305#note_29694

1 0

Re: [PATCH v6 0/1] MR2305: rpcrt4: Pass the binding handle to server call function.
by Zhiyi Zhang (＠zhiyi) 12 Apr '23

12 Apr '23

On Wed Apr 12 07:11:24 2023 +0000, Yeshun Ye wrote: > I tried to add an interface like this: > ``` > [ > uuid(00000000-4114-0704-2301-000000000002) > ] > > interface IServer_Expilicit > { > void need_explicit_handle([in] handle_t binding, [in] unsigned int > protseq); > } > ``` > I added the code above to the file 'server_interp.idl', the function > code to 'server.c' like this: > ``` > void __cdecl s_need_explicit_handle(RPC_BINDING_HANDLE binding, unsigned > int protseq) > { > RPC_STATUS status; > ULONG pid; > winetest_push_context("%s", client_test_name); > ok(binding != NULL, "Got unexpected binding\n"); > trace("s_need_explicit_handle\n"); > if (protseq == RPC_PROTSEQ_LRPC) /* Other protocol sequences throw > exceptions */ > { > trace("RPC_PROTSEQ_LRPC\n"); > status = I_RpcBindingInqLocalClientPID(binding, &pid); > trace("I_RpcBindingInqLocalClientPID\n"); > ok(status == RPC_S_OK, "Got unexpected %ld.\n", status); > ok(pid == client_info.dwProcessId, "Got unexpected pid.\n"); > trace("pid=%x client_info.dwProcessId=%x\n", pid, client_info.dwProcessId); > } > winetest_pop_context(); > } > ``` > Then I don't know what to do next. For a generic test case, you can see https://source.winehq.org/git/wine.git/commitdiff/056dbb04dea4d0b990e0177f6… for an example. However, in this case, you need to add a test case specifically for your changes. To be honest, I don't understand your patch, that's why I asked for a test case. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2305#note_29693

1 0

2025

2024

2023

2022

wine-gitlab April 2023