On Fri May 15 16:51:48 2026 +0000, Elizabeth Figura wrote:
Even the test video we already use in Wine has a similar situation, so I don't think that's necessary, but thank you for the offer. Good point. Actually, quality control is something I already have in mind.
As I understand it, GStreamer expects the running time for a QoS event (ref. gst_event_new_qos). So as long as the running time is used, it works better with my changes, because the running time (on the GStreamer side) is based on the beginning of the segment. However, according to the comment added in commit 5e222064ab27266c0f44ec3986872f611772ff10, wg_parser_stream_notify_qos() is expected to receive stream time. On the other hand, the newly added gst_segment_to_running_time() in that commit converts a position within the segment into running time. I find this discrepancy confusing. I would appreciate it if you could clarify the intention behind this design. Once the intention behind these behaviors becomes clear, I plan to make the necessary changes for QC. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10864#note_140272