gst/asfdemux/gstasfdemux.c: Send newsegment event only once per pad, fixes #336550.

Original commit message from CVS:
Patch by: Fabrizio Gennari  <fabrizio dot ge at tiscali dot it>
* gst/asfdemux/gstasfdemux.c: (gst_asf_demux_sink_event),
(gst_asf_demux_push_buffer):
Send newsegment event only once per pad, fixes #336550.
This commit is contained in:
Fabrizio Gennari 2006-04-07 09:14:26 +00:00 committed by Tim-Philipp Müller
parent 62712a628c
commit 7a8e48e17c
3 changed files with 18 additions and 4 deletions

View file

@ -1,3 +1,11 @@
2006-04-07 Tim-Philipp Müller <tim at centricular dot net>
Patch by: Fabrizio Gennari <fabrizio dot ge at tiscali dot it>
* gst/asfdemux/gstasfdemux.c: (gst_asf_demux_sink_event),
(gst_asf_demux_push_buffer):
Send newsegment event only once per pad, fixes #336550.
2006-04-01 Thomas Vander Stichele <thomas at apestaart dot org> 2006-04-01 Thomas Vander Stichele <thomas at apestaart dot org>
* docs/plugins/gst-plugins-ugly-plugins-docs.sgml: * docs/plugins/gst-plugins-ugly-plugins-docs.sgml:

2
common

@ -1 +1 @@
Subproject commit 623fe1c2cce45bc30d5823c05716349874ae994e Subproject commit 1783855e983a5294434673694e8a57e44980b6f1

View file

@ -135,6 +135,7 @@ gst_asf_demux_sink_event (GstPad * pad, GstEvent * event)
demux = GST_ASF_DEMUX (gst_pad_get_parent (pad)); demux = GST_ASF_DEMUX (gst_pad_get_parent (pad));
GST_LOG_OBJECT (demux, "handling %s event", GST_EVENT_TYPE_NAME (event));
switch (GST_EVENT_TYPE (event)) { switch (GST_EVENT_TYPE (event)) {
case GST_EVENT_NEWSEGMENT:{ case GST_EVENT_NEWSEGMENT:{
GstFormat newsegment_format; GstFormat newsegment_format;
@ -1749,13 +1750,18 @@ gst_asf_demux_push_buffer (GstASFDemux * demux, asf_stream_context * stream,
/* do we need to send a newsegment event? */ /* do we need to send a newsegment event? */
if (stream->need_newsegment) { if (stream->need_newsegment) {
GST_DEBUG ("sending new-segment event on pad %s", GST_DEBUG ("sending new-segment event on pad %s: %" GST_TIME_FORMAT " - %"
GST_PAD_NAME (stream->pad)); GST_TIME_FORMAT, GST_PAD_NAME (stream->pad),
GST_TIME_ARGS (demux->segment.start),
GST_TIME_ARGS (demux->segment.stop));
/* FIXME: if we need to send a newsegment event on this pad and /* FIXME: if we need to send a newsegment event on this pad and
* the buffer doesn't have a timestamp, should we just drop the buffer * the buffer doesn't have a timestamp, should we just drop the buffer
* and wait for one with a timestamp before sending it? */ * and wait for one with a timestamp before sending it? */
gst_asf_demux_send_event_unlocked (demux, gst_event_new_new_segment (FALSE, demux->segment.rate, GST_FORMAT_TIME, demux->segment.start, demux->segment.stop, demux->segment.start)); /* last parameter isn't right */ /* FIXME: last parameter in newsegment isn't right, is it?! */
gst_pad_push_event (stream->pad,
gst_event_new_new_segment (FALSE, demux->segment.rate, GST_FORMAT_TIME,
demux->segment.start, demux->segment.stop, demux->segment.start));
stream->need_newsegment = FALSE; stream->need_newsegment = FALSE;
} }