[Bug 659814] basesink: take segment start into account in position reporting

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=659814
  GStreamer | gstreamer (core) | unspecified

Mark Nauwelaerts <mnauw> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mnauw at users.sourceforge.net

--- Comment #2 from Mark Nauwelaerts <mnauw at users.sourceforge.net> 2011-09-22 12:10:16 UTC ---
(In reply to comment #1)
> Created an attachment (id=197234)
 View: https://bugzilla.gnome.org/attachment.cgi?id=197234
 Review: https://bugzilla.gnome.org/review?bug=659814&attachment=197234

> basesink: take segment start into account in position reporting
> 
> This fixes position reporting on files starting at a non 0 offset.

This should not be the way to go.  The segment 'time' field is (conceptually)
in a different timeline than start/stop (buffer) timeline, and so there should
not be arithmetic (in this way).

It is probably up to upstream to send the proper stuff in 'time' field, which
actually determines the stream_time/position timeline.

-- 
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.