oggdemux: Drop data before new segment
authorEdward Hervey <edward@centricular.com>
Tue, 7 Nov 2017 14:09:40 +0000 (15:09 +0100)
committerEdward Hervey <bilboed@bilboed.com>
Tue, 7 Nov 2017 14:16:52 +0000 (15:16 +0100)
commitc86df789ed5f1b872c5172640e35325d93d631dc
tree080c7e2122349cbd824138c05dc6ff33567699fb
parent0a49b93ade8e2a634f5c40e0dddf60cdce098a7c
oggdemux: Drop data before new segment

When calculating duration in push-mode we seek to a certain position
and discard any data until we get data from that requested position.

The problem is that basing ourselves solely on offset to determine
whether we reached the target offset is wrong since the source might
be fast enough  to send us that target position *before* it processed
the requested seek.

This would end up in a situation where:
* We think we're done with duration estimate
* We fire a seek back to "0" in the loop thread
* We resume normal processing
* ... except that we're still getting data from too far ahead which
  we decide to process.
* And we start doing totally wrong granule/time/duration calculation
  and pushing wrong data.

Instead of this confusion, wait until we receive data from the requested
seek. We do that by using the fact that the seqnum in
seek_event_drop_til will be non-zero until the SEGMENT corresponding
to the requested SEEK has been received.

Bonus: makes startup slightly faster
ext/ogg/gstoggdemux.c