[Bug 729134] mpegtssection: Include a timestamp

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=729134
  GStreamer | gst-plugins-bad | git

--- Comment #16 from Edward Hervey <bilboed at bilboed.com> 2014-05-14 05:06:34 UTC ---
Not sure we could re-use that field, since it's when the message "was created",
whereas you want to know when "the content of this message/event applies".

So assuming we go for option 2, which would be the sanest, there's
yet-another-option which is easier to handle app-side. Depending on how
granular/reactive you want to be, this could be acceptable.

We could use (PCR or PTS converted to) stream time for those time field. And
then it would be up to your app to regularly (say 2 times per second) query the
current pipeline position (which is in stream time), and you could apply/handle
those messages/events/actions when you reach that stream time.

That being said, I'm not a great fan of calculating/setting that field on *all*
mpeg-ts sections. One reason is performance, the other one is just that ... it
makes no sense for quite a few sections (PAT, PMT, EIT, ... aren't timed
events).

Can you provide a list of specific sections you know you'll need timestamps ?

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.