[Bug 757602] adaptivedemux: improved error message if availabilityStartTime is missing for a live stream

[GStreamer (GNOME Bugzilla)]

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.