[Bug 757602] adaptivedemux: improved error message if availabilityStartTime is missing for a live stream
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Thu Jan 7 10:12:51 PST 2016
https://bugzilla.gnome.org/show_bug.cgi?id=757602
Tim-Philipp Müller <t.i.m at zen.co.uk> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |NEEDINFO
CC| |t.i.m at zen.co.uk
--- Comment #2 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Seems reasonable.
I'd just like to point out that these are not "error messages" really, but
indicate a programming error in the calling code somewhere, and in other
circumstances (if compiled with the certain CFLAGS) it might just crash
instead. Calling code must not rely on a g_return_if*() being there, as it
might be compiled out.
So I guess the question is if anything else needs fixing up here, or if that's
been done already.
If availabilityStartTime being NULL is a legitimate condition, then it needs to
be checked outside of a g_return_if_fail*().
--
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