[Bug 674536] tsdemux: Freeze on pts-wrap with streaming sources

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #3 from Julian Scheel <julian at jusst.de> 2012-04-24 12:59:15 UTC ---
Created an attachment (id=212690)
 View: https://bugzilla.gnome.org/attachment.cgi?id=212690
 Review: https://bugzilla.gnome.org/review?bug=674536&attachment=212690

don't reset base_pcrtime

I experimented a bit further and found out that the reset of base_pcrtime when
mpegtspacketizer detect a rolliver thwarts the rollover detection in tsdemux.c,
so that in the end the timestamp makes a jump in the size of MAX_PTS_DTS-new
base_pcrtime.
I don't see a reason why the reset of base_pcrtime would be needed at all with
the rollover code in tsdemux. The attached patch disables it (for now just
using compiler macros...), which works well for me so far. Still some comment
by a developer more familiar with the timestamp handling would be nice.

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