- Wine-GitLab - list.winehq.org

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

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: > + > + hr = MFCreateMediaType(&media_type); > + ok(hr == S_OK, "Failed to create media type, hr %#lx.\n", hr); > + > + hr = IMFMediaType_SetGUID(media_type, &MF_MT_MAJOR_TYPE, &MFMediaType_Audio); > + ok(hr == S_OK, "Failed to set attribute, hr %#lx.\n", hr); > + hr = IMFMediaType_SetGUID(media_type, &MF_MT_SUBTYPE, &MFAudioFormat_PCM); > + ok(hr == S_OK, "Failed to set attribute, hr %#lx.\n", hr); > + > + EXPECT_REF(media_type, 1); > + hr = MFCreateSampleGrabberSinkActivate(media_type, grabber_callback, &activate); > + ok(hr == S_OK, "Failed to create grabber activate, hr %#lx.\n", hr); > + EXPECT_REF(media_type, 2); > + > + ref = IMFMediaType_Release(media_type); > + ok(ref == 1, "Release returned %ld\n", ref); Same here, it's unnecessary to check counter here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9489#note_127075

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: > + > + hr = MFStartup(MF_VERSION, MFSTARTUP_FULL); > + ok(hr == S_OK, "Failed to start up, hr %#lx.\n", hr); > + > + hr = MFCreateMediaType(&media_type); > + ok(hr == S_OK, "Failed to create media type, hr %#lx.\n", hr); > + > + hr = IMFMediaType_SetGUID(media_type, &MF_MT_MAJOR_TYPE, &MFMediaType_Audio); > + ok(hr == S_OK, "Failed to set attribute, hr %#lx.\n", hr); > + hr = IMFMediaType_SetGUID(media_type, &MF_MT_SUBTYPE, &MFAudioFormat_PCM); > + ok(hr == S_OK, "Failed to set attribute, hr %#lx.\n", hr); > + > + EXPECT_REF(media_type, 1); > + hr = MFCreateSampleGrabberSinkActivate(media_type, grabber_callback, &activate); > + ok(hr == S_OK, "Failed to create grabber activate, hr %#lx.\n", hr); > + EXPECT_REF(media_type, 2); We already have this refcount check in another test, no need to duplicate it. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9489#note_127074

1 0

[PATCH v7 0/5] MR9646: mf/tests: Scrubbing tests
by Brendan McGrath (＠redmcg) Jan. 15, 2026

Jan. 15, 2026

This MR adds tests for how scrubbing and rate changes work within the Media Session. -- v7: mf/tests: Check that Rate change in PLAY state is ignored. mf/tests: Test that there is no pre-roll during scrubbing. https://gitlab.winehq.org/wine/wine/-/merge_requests/9646

3 11

[PATCH 0/3] MR9904: d2d1/tests: Add a test for default stroke transform type.
by Nikolay Sivov (＠nsivov) Jan. 15, 2026

Jan. 15, 2026

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9904

2 3

[PATCH 0/3] MR9883: comctl32: Use BOOLEAN instead of BOOL for RegisterClassNameW().
by Zhiyi Zhang (＠zhiyi) Jan. 15, 2026

Jan. 15, 2026

The return type should be BOOLEAN instead of BOOL. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9883

3 6

[PATCH v2 0/2] MR9887: wbemprox: Add Win32_SystemEnclosure.SerialNumber.
by Paul Gofman (＠gofman) Jan. 15, 2026

Jan. 15, 2026

-- v2: wbemprox: Mark Win32_SystemEnclosure.Tag as key column. wbemprox: Add Win32_SystemEnclosure.SerialNumber. https://gitlab.winehq.org/wine/wine/-/merge_requests/9887

3 4

[PATCH 0/2] MR9887: wbemprox: Add Win32_SystemEnclosure.SerialNumber.
by Paul Gofman (＠gofman) Jan. 15, 2026

Jan. 15, 2026

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

3 6

[PATCH 0/3] MR9902: netprofm: Add IPv6 support.
by Hans Leidekker (＠hans) Jan. 15, 2026

Jan. 15, 2026

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

2 3

[PATCH 0/1] MR9899: windows.gaming.input: Enfore axis usage mapping for HID devices.
by Rémi Bernon Jan. 15, 2026

Jan. 15, 2026

Unlike for GIP devices, it seems that the HID provide does some HID usage mapping. This is the case for instance with the DualSense gamepad, which HID report descriptor enumerates axis as X/Y/Z/Rz/Rx/Ry, but when read through the WGI RawGameController interface, enumerates the axes values as X/Y/Z/Rx/Ry/Rz. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9899

1 1

[PATCH v3 0/2] MR9889: windows.ui: Stub IUISettings2::get_TextScaleFactor().
by Mohamad Al-Jaf (＠maljaf) Jan. 15, 2026

Jan. 15, 2026

Needed by HP Prime Virtual Calculator Emulator. -- v3: windows.ui: Stub IUISettings2::get_TextScaleFactor(). windows.ui/tests: Add IUISettings2::get_TextScaleFactor() tests. https://gitlab.winehq.org/wine/wine/-/merge_requests/9889

4 5

[PATCH 0/1] MR9897: mf/session: Discard end of presentation on IMFMediaSession_Stop.
by Rémi Bernon Jan. 15, 2026

Jan. 15, 2026

Some games calls IMFMediaSession_Stop while the presentation is ending, expects that command to quickly execute, interrupting the presentation end and emitting a MESessionStopped event instead of the MESessionEnded. Delaying the Stop command and emitting the MESessionEnded event breaks the game assumptions and it crashes, or waits for the MESessionStopped event forever. This is a redone of https://gitlab.winehq.org/wine/wine/-/merge_requests/1710, which still appears to be necessary for Disintegration. Haven't tested Life is Strange again, but I suspect it's probably some quirks of some Unreal Engine version. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9897

1 2