- Wine-GitLab - list.winehq.org

Re: [PATCH v2 0/1] MR9899: windows.gaming.input: Enfore axis usage mapping for HID devices. - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9899

1 0

Re: [PATCH v4 0/2] MR9889: windows.ui: Stub IUISettings2::get_TextScaleFactor(). - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9889

1 0

[PATCH v2 0/6] MR9888: ntdll: Fix LdrLoadDll() parameters.
by Paul Gofman (＠gofman) Jan. 15, 2026

Jan. 15, 2026

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54300 Also needed for Rec Room. -- v2: kernelbase: Pass search flags instead of path to LdrLoadDll(). ntdll: Don't fill output handle on error in LdrLoadDll(). ntdll: Use LDR_DONT_RESOLVE_REFS instead of DONT_RESOLVE_DLL_REFERENCES. kernel32/tests: Add tests for LdrLoadDll(). ntdll: Handle search flags in LdrLoadDll(). ntdll: Fix LdrLoadDll() prototype. https://gitlab.winehq.org/wine/wine/-/merge_requests/9888

3 8

[PATCH 0/1] MR9905: dwrite: Zero 'current' in create_font_collection().
by Paul Gofman (＠gofman) Jan. 15, 2026

Jan. 15, 2026

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

3 3

Re: [PATCH v5 0/1] MR9288: chakra: Add stub dll. - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9288

1 0

Re: [PATCH v2 0/1] MR9890: ntdll: Avoid infinite wait during process termination. - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9890

1 0

Re: [PATCH v2 0/1] MR9893: include: Annotate VirtualAlloc functions with __WINE_ALLOC_SIZE. - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9893

1 0

Re: [PATCH v2 0/1] MR9892: winecfg: Remove unnecessary calls to strdupU2W. - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9892

1 0

Re: [PATCH v2 0/1] MR9894: cryptnet: Close leaking file handle in `find_cached_revocation_status` - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9894

1 0

Re: [PATCH v2 0/6] MR9891: msado15: Add IRowset wrapper and _Recordset::Seek implementation. - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9891

1 0

Re: [PATCH v2 0/3] MR9895: msado15: Improve adoconstruct_WrapDSOandSession implementation. - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9895

1 0

Re: [PATCH v2 0/2] MR9878: mfsrcsnk: Remain in running state on EOP. - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9878

1 0

Re: [PATCH v4 0/1] MR9764: msi/tests: Fix some typos (Coverity). - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9764

1 0

Re: [PATCH v2 0/1] MR9873: rometadata: Add additional bounds/overflow checks while accessing offsets to assembly data. - approved
by Alexandre Julliard (＠julliard) Jan. 15, 2026

Jan. 15, 2026

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9873

1 0

[PATCH 0/1] MR9629: configure: Add option for building with split debug info.
by Yuxuan Shui (＠yshui) Jan. 15, 2026

Jan. 15, 2026

* * * all the machinery for split debug info is there, and the only thing needed is just exposing a flag from `configure`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9629

4 5

[PATCH v19 0/1] MR9868: winewayland: Fix non-square icons with xdg-toplevel-icon protocol.
by Alex Schwartz (＠alexschwartz01) Jan. 15, 2026

Jan. 15, 2026

Currently, it will stop the app that is running that has a non-square icon This PR fixes that -- v19: winewayland: Fix non-square icons with xdg-toplevel-icon protocol. https://gitlab.winehq.org/wine/wine/-/merge_requests/9868

2 1

[PATCH v17 0/1] MR9868: winewayland: Fix non-square icons with xdg-toplevel-icon protocol.
by Alex Schwartz (＠alexschwartz01) Jan. 15, 2026

Jan. 15, 2026

Currently, it will stop the app that is running that has a non-square icon This PR fixes that -- v17: winewayland: Fix non-square icons with xdg-toplevel-icon protocol. https://gitlab.winehq.org/wine/wine/-/merge_requests/9868

2 1

[PATCH 0/1] MR9884: comctl32_v6: Add horizontal spacing around button text.
by Zhiyi Zhang (＠zhiyi) Jan. 15, 2026

Jan. 15, 2026

BCM_GETIDEALSIZE returns a size just enough to display the button text. Windows adds horizontal spacing around button text when it doesn't exceed the maximum button width limit. Two times DIALOG_SPACING at each side seems close to what native does. ![Screenshot_2026-01-14_16-14-04](/uploads/3e276595ecdb420122b332194108a03f/Screenshot_2026-01-14_16-14-04.png){width=858 height=226} [test_taskdialog_20251211.zip](/uploads/de1678f33a0c6b36dfc9471738175873/test_taskdialog_20251211.zip) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9884

3 2

Re: [PATCH v3 0/1] MR9719: mfmediaengine/tests: Test we receive a frame before calling Play.
by Nikolay Sivov (＠nsivov) Jan. 15, 2026

Jan. 15, 2026

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/tests/mfmediaengine.c: > > test_audio_session(FALSE); > > + /* free device context and readback texture */ It's obvious without a comment. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9719#note_127083

1 0

Re: [PATCH v3 0/1] MR9719: mfmediaengine/tests: Test we receive a frame before calling Play.
by Nikolay Sivov (＠nsivov) Jan. 15, 2026

Jan. 15, 2026

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/tests/mfmediaengine.c: > + desc.Usage = D3D11_USAGE_STAGING; > + desc.BindFlags = 0; > + desc.CPUAccessFlags = D3D11_CPU_ACCESS_READ; > + desc.MiscFlags = 0; > + hr = ID3D11Device_CreateTexture2D(device, &desc, NULL, &rb_texture); > + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); > + > + /* confirm we have a frame available before calling play */ > + pts = 0; > + hr = IMFMediaEngineEx_OnVideoStreamTick(media_engine, &pts); > + todo_wine > + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); > + todo_wine > + ok(pts == 0, "Unexpected timestamp.\n"); > + > + /* confirm we can transfer a frame before calling play */ Does it mean that setting url/media item is enough to start the pipeline internally? Any indication if it's paused immediately, or something else is going on? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9719#note_127082

1 0

Re: [PATCH v3 0/1] MR9719: mfmediaengine/tests: Test we receive a frame before calling Play.
by Nikolay Sivov (＠nsivov) Jan. 15, 2026

Jan. 15, 2026

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/tests/mfmediaengine.c: > + hr = IMFMediaEngineEx_TransferVideoFrame(notify->media_engine, (IUnknown *)texture, NULL, &dst_rect, NULL); > + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); > + > + ID3D11DeviceContext_CopySubresourceRegion(context, (ID3D11Resource *)rb_texture, > + 0, 0, 0, 0, (ID3D11Resource *)texture, 0, NULL); > + > + memset(&map_desc, 0, sizeof(map_desc)); > + hr = ID3D11DeviceContext_Map(context, (ID3D11Resource *)rb_texture, 0, D3D11_MAP_READ, 0, &map_desc); > + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); > + ok(!!map_desc.pData, "got pData %p\n", map_desc.pData); > + ok(map_desc.DepthPitch == 16384, "got DepthPitch %u\n", map_desc.DepthPitch); > + ok(map_desc.RowPitch == desc.Width * 4, "got RowPitch %u\n", map_desc.RowPitch); > + res = check_rgb32_data(L"rgb32frame.bmp", map_desc.pData, map_desc.RowPitch * desc.Height, &dst_rect); > + todo_wine > + ok(res == 0, "Unexpected %lu%% diff\n", res); > + ID3D11DeviceContext_Unmap(context, (ID3D11Resource *)rb_texture, 0); This should probably go to some helper. Are we going to use same pattern elsewhere? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9719#note_127081

1 0

Re: [PATCH v7 0/13] MR9489: mf: Fix seeking within sample grabber.
by Nikolay Sivov (＠nsivov) Jan. 15, 2026

Jan. 15, 2026

Could you move tests to a separate MR? The commit list is pretty long here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9489#note_127079

1 0

Re: [PATCH v7 0/13] MR9489: mf: Fix seeking within sample grabber.
by Nikolay Sivov (＠nsivov) Jan. 15, 2026

Jan. 15, 2026

Nikolay Sivov (@nsivov) commented about dlls/mf/tests/mf.c: > + todo_wine > + CHECK_CALLED(timer_SetTimer); > > - hr = IMFPresentationClock_Pause(clock); > + hr = IMFClockStateSink_OnClockPause(clock_sink, 0); > ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); > > - hr = IMFPresentationClock_Start(clock, 0); > + /* test over supply */ > + supply_samples(stream, 6); > + > + SET_EXPECT(timer_CancelTimer); > + hr = IMFClockStateSink_OnClockStart(clock_sink, 0, 0); > ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); > + CHECK_CALLED(timer_CancelTimer); > Normal flow is to call clock->Start(), callbacks are called internally. Why changing that? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9489#note_127078

1 0

Re: [PATCH v7 0/13] MR9489: mf: Fix seeking within sample grabber.
by Nikolay Sivov (＠nsivov) Jan. 15, 2026

Jan. 15, 2026

Nikolay Sivov (@nsivov) commented about dlls/mf/tests/mf.c: > /* Set clock. */ > - hr = MFCreatePresentationClock(&clock); > - ok(hr == S_OK, "Failed to create clock object, hr %#lx.\n", hr); > + mock_clock = create_presentation_clock(); > + clock = &mock_clock->IMFPresentationClock_iface; > > hr = IMFMediaSink_SetPresentationClock(sink, clock); > ok(hr == S_OK, "Failed to set presentation clock, hr %#lx.\n", hr); > - > - hr = MFCreateSystemTimeSource(&time_source); > - ok(hr == S_OK, "Failed to create time source, hr %#lx.\n", hr); > - > - hr = IMFPresentationClock_SetTimeSource(clock, time_source); > - ok(hr == S_OK, "Failed to set time source, hr %#lx.\n", hr); > - IMFPresentationTimeSource_Release(time_source); > + ok(!!mock_clock->sample_grabber_clock_state_sink, "AddClockStateSink not called\n"); I would remove this one. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9489#note_127077

1 0

Re: [PATCH v7 0/13] MR9489: mf: Fix seeking within sample grabber.
by Nikolay Sivov (＠nsivov) Jan. 15, 2026

Jan. 15, 2026

Nikolay Sivov (@nsivov) commented about dlls/mf/tests/mf.c: > + samples_requested = count_samples_requested(stream); > + todo_wine > + ok(samples_requested == 4, "Unexpected number of samples requested %d\n", samples_requested); > + > + /* test number of new sample requests on seek whilst paused and 3 samples provided */ > + supply_samples(stream, 3); > + > + hr = IMFPresentationClock_Pause(clock); > + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); > + > + hr = IMFPresentationClock_Start(clock, 0); > + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); > + > + samples_requested = count_samples_requested(stream); > + todo_wine > + ok(samples_requested == 3, "Unexpected number of samples requested %d\n", samples_requested); In this sequence, how could you tell that requests are coming on Pause -> Start transition (is that what we are trying to test?)? Does it rely on a fact that we pushed fewer than requested? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9489#note_127076

1 0