[Bug 788340] Dynamically reconfigure pipeline in PLAY based on transports

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Oct 17 13:07:13 UTC 2017


https://bugzilla.gnome.org/show_bug.cgi?id=788340

--- Comment #8 from Patricia Muscalu <patricia at axis.com> ---
(In reply to Sebastian Dröge (slomo) from comment #6)
> > > @@ +611,3 @@
> > >    priv = media->priv;
> > >  
> > > +  data.position = G_MAXINT64;
> > > 
> > > Why this change? Generally, -1 is returned (GST_CLOCK_TIME_NONE) if nothing
> > > else can be found
> > 
> > GST_CLOCK_TIME_NONE is not a proper initial value because we are actually
> > using MIN function now.
> 
> G_MAXINT64 is not ideal either though, for position queries in TIME format
> values above that are valid too). I'm not sure what to suggest here though,
> the position/duration query API is broken by the usage of int64 but actually
> using uint64 for TIME format.

The initial value of data.position is set to G_MAXINT64 and if the query
position fails (<=> data.ret == FALSE), the position is set to
GST_CLOCK_TIME_NONE.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list