October 2023 - Wine-gitlab - list.winehq.org

Re: [PATCH v13 0/7] MR3938: winegstreamer: Implement CLSID_CMpegVideoCodec
by Zebediah Figura (＠zfigura) 16 Oct '23

16 Oct '23

Zebediah Figura (@zfigura) commented about dlls/winegstreamer/quartz_parser.c: > if (seeking->llStop) > stop_flags = AM_SEEKING_AbsolutePositioning; > wg_parser_stream_seek(filter->sources[0]->wg_stream, seeking->dRate, > - seeking->llCurrent, seeking->llStop, AM_SEEKING_AbsolutePositioning, stop_flags); > + seeking->llCurrent, seeking->llStop == seeking->llDuration ? -1 : seeking->llStop, > + AM_SEEKING_AbsolutePositioning, stop_flags); Isn't this redundant with the same logic done in wg_parser_stream_seek()? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3938#note_48898

1 0

Re: [PATCH v13 0/7] MR3938: winegstreamer: Implement CLSID_CMpegVideoCodec
by Zebediah Figura (＠zfigura) 16 Oct '23

16 Oct '23

Zebediah Figura (@zfigura) commented about dlls/winegstreamer/wg_parser.c: > stop_type = GST_SEEK_TYPE_NONE; > > if (!push_event(get_stream(params->stream)->my_sink, gst_event_new_seek(params->rate, GST_FORMAT_TIME, > - flags, start_type, params->start_pos * 100, stop_type, params->stop_pos * 100))) > + flags, start_type, params->start_pos * 100, stop_type, > + params->stop_pos == -1 ? -1 : params->stop_pos * 100))) I think we want this, but this isn't relevant to this patch and should be split. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3938#note_48899

1 0

Re: [PATCH v13 0/7] MR3938: winegstreamer: Implement CLSID_CMpegVideoCodec
by Zebediah Figura (＠zfigura) 16 Oct '23

16 Oct '23

Zebediah Figura (@zfigura) commented about dlls/winegstreamer/quartz_parser.c: > * but as long as every sample fits into our allocator, we're fine. */ > return format->u.video_cinepak.width * format->u.video_cinepak.height * 3; > > + case WG_MAJOR_TYPE_VIDEO_MPEG1: > + return wg_format_get_bytes_for_uncompressed(WG_VIDEO_FORMAT_YV12, > + format->u.video_mpeg1.width, format->u.video_mpeg1.height); > + This is overestimation because we can't do any better, right? In that case it probably deserves a comment just to clarify that's what's happening. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3938#note_48896

1 0

Re: [PATCH v13 0/7] MR3938: winegstreamer: Implement CLSID_CMpegVideoCodec
by Zebediah Figura (＠zfigura) 16 Oct '23

16 Oct '23

Zebediah Figura (@zfigura) commented about dlls/winegstreamer/wg_format.c: > if (gst_video_info_from_caps(&info, caps)) > wg_format_from_video_info(format, &info); > } > - else if (!strcmp(name, "audio/mpeg")) > + else if (!strcmp(name, "audio/mpeg") && gst_structure_get_boolean(structure, "parsed", &parsed) && parsed) This is correct, I think, but also orthogonal to the purpose of this patch. Can you please split this out into a separate patch? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3938#note_48897

1 0

Re: [PATCH v6 0/1] MR3970: quartz/tests: Improve VMR9 ChangeD3DDevice test
by Zebediah Figura (＠zfigura) 16 Oct '23

16 Oct '23

On Mon Oct 16 21:55:32 2023 +0000, Alfred Agrell wrote: > Unfortunately, we do. I had sleeps everywhere during development, then > removed every one I could, but that specific one is load-bearing. > https://testbot.winehq.org/JobDetails.pl?Key=138782&f101=exe64.report > https://testbot.winehq.org/JobDetails.pl?Key=138783&f101=exe64.report > I tried swapping it for IBaseFilter_GetState(), but that just returned > immediately then gave the same segfault. https://testbot.winehq.org/JobDetails.pl?Key=138786&f101=exe64.report > Calling GetState twice worked, but I'm pretty sure that's just because > those calls take long enough to satisfy its timing demands. A sleep is > cleaner. (Slightly. Still ugly.) > I could also move the sleep, but I don't think that would accomplish anything. Hrm, that's suspicious, IMemAllocator::GetBuffer() returns VFW_E_NOT_COMMITTED. Does the VMR's IMemAllocator change by any chance when we reconnect? I.e. is IMemInputPin::GetAllocator() actually returning the same allocator that it was returning before? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3970#note_48893

1 0

Re: [PATCH v6 0/1] MR3970: quartz/tests: Improve VMR9 ChangeD3DDevice test
by Alfred Agrell (＠Alcaro) 16 Oct '23

16 Oct '23

On Mon Oct 16 21:25:57 2023 +0000, Zebediah Figura wrote: > Do we still need this? Unfortunately, we do. I had sleeps everywhere during development, then removed every one I could, but that specific one is load-bearing. https://testbot.winehq.org/JobDetails.pl?Key=138782&f101=exe64.report https://testbot.winehq.org/JobDetails.pl?Key=138783&f101=exe64.report I tried swapping it for IBaseFilter_GetState(), but that just returned immediately then gave the same segfault. https://testbot.winehq.org/JobDetails.pl?Key=138786&f101=exe64.report Calling GetState twice worked, but I'm pretty sure that's just because those calls take long enough to satisfy its timing demands. A sleep is cleaner. (Slightly. Still ugly.) I could also move the sleep, but I don't think that would accomplish anything. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3970#note_48883

1 0

[PATCH 0/1] MR4111: winegstreamer: Implement IWMResamplerProps
by Fabian Maurer (＠DarkShadow44) 16 Oct '23

16 Oct '23

Some programs just assume this interface exists, this fixes the resulting crash. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55789 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4111

3 3

[PATCH 0/8] MR412: tests: Fix a number of tests so they pass on Windows.
by Giovanni Mascellani (＠giomasce) 16 Oct '23

16 Oct '23

This is a continuation to !406. With this MR all the tests pass on Windows. [The next step is to allow the CI to run crosstests on Windows](https://gitlab.winehq.org/giomasce/vkd3d/-/commits/ci3). -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/412

3 9

[PATCH 0/7] MR4110: dmime: Simplify message thread and use latency time to schedule messages.
by Rémi Bernon 16 Oct '23

16 Oct '23

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

2 10

Re: [PATCH v6 0/1] MR3970: quartz/tests: Improve VMR9 ChangeD3DDevice test
by Zebediah Figura (＠zfigura) 16 Oct '23

16 Oct '23

Zebediah Figura (@zfigura) commented about dlls/quartz/tests/vmr9.c: > + hr = IMemAllocator_SetProperties(allocator, &req_props, &ret_props); > + ok(hr == S_OK, "Got hr %#lx.\n", hr); > + hr = IMemAllocator_Commit(allocator); > + ok(hr == S_OK, "Got hr %#lx.\n", hr); > + > device2 = create_device(window); > ok(device2 != NULL, "Couldn't create device\n"); > > - ok(presenter.got_TerminateDevice == 0, "got %d\n", presenter.got_TerminateDevice); > - > hr = IVMRSurfaceAllocatorNotify9_ChangeD3DDevice(notify, device2, MonitorFromWindow(window, MONITOR_DEFAULTTOPRIMARY)); > ok(hr == S_OK, "Got hr %#lx.\n", hr); > - ok(presenter.got_TerminateDevice == 1, "got %d\n", presenter.got_TerminateDevice); > + presenter.device = device2; > + presenter.got_PresentImage = 0; > + Sleep(200); /* ChangeD3DDevice is partially asynchronous */ Do we still need this? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3970#note_48875

1 0