[Bug 705993] aiffparse: fix push mode / add tests
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Thu Aug 15 01:18:23 PDT 2013
https://bugzilla.gnome.org/show_bug.cgi?id=705993
GStreamer | gst-plugins-bad | git
Sebastian Dröge (slomo) <slomo> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #251614|none |needs-work
status| |
--- Comment #3 from Sebastian Dröge (slomo) <slomo at circular-chaos.org> 2013-08-15 08:18:01 UTC ---
Review of attachment 251614:
--> (https://bugzilla.gnome.org/review?bug=705993&attachment=251614)
::: gst/aiff/aiffparse.c
@@ +1713,3 @@
+ /* some debug output */
+ gst_event_copy_segment (event, &segment);
+ GST_DEBUG_OBJECT (aiff, "received newsegment %" GST_SEGMENT_FORMAT,
It's not newsegment, it's segment :) Same in a few other places in this patch
@@ +1717,3 @@
+
+ if (aiff->state != AIFF_PARSE_DATA) {
+ GST_DEBUG_OBJECT (aiff, "still starting, eating event");
The demuxer should still take into account the upstream segment when generating
its initial segment event if upstream is in TIME format at least
@@ +1776,3 @@
+ gst_event_unref (aiff->start_segment);
+ GST_DEBUG_OBJECT (aiff, "Storing newseg %" GST_SEGMENT_FORMAT,
&segment);
+ aiff->start_segment = gst_event_new_segment (&segment);
You can just push the segment event here, if you get a segment event from
upstream the demuxer is running in push mode and the event is received from the
streaming thread
--
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list