After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 749908 - qtdemux: Set segment_seqnum on EOS events.
qtdemux: Set segment_seqnum on EOS events.
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
unspecified
Other All
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-05-26 14:24 UTC by Mathieu Duponchelle
Modified: 2017-01-15 11:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
qtdemux: Set segment_seqnum on EOS events. (838 bytes, patch)
2015-05-26 14:24 UTC, Mathieu Duponchelle
reviewed Details | Review

Description Mathieu Duponchelle 2015-05-26 14:24:37 UTC
Otherwise downstream elements relying on this seqnum will get confused.
Comment 1 Mathieu Duponchelle 2015-05-26 14:24:41 UTC
Created attachment 304013 [details] [review]
qtdemux: Set segment_seqnum on EOS events.
Comment 2 Thiago Sousa Santos 2015-05-28 06:19:02 UTC
Review of attachment 304013 [details] [review]:

Looks good, but it seems the seqnum handling is missing being reset when a soft reset is done. It would be nice to fix it too when pushing this change.
Comment 3 Tim-Philipp Müller 2017-01-15 11:17:09 UTC
Seems to have been fixed in the meantime.