[Bug 749653] dashdemux: Implement binary search for stream_sidx_seek

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Mon Jun 1 23:22:23 PDT 2015


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

Edward Hervey <bilboed at bilboed.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #304302|none                        |needs-work
             status|                            |

--- Comment #14 from Edward Hervey <bilboed at bilboed.com> ---
Review of attachment 304302:
 --> (https://bugzilla.gnome.org/review?bug=749653&attachment=304302)

::: ext/dash/gstdashdemux.c
@@ +910,3 @@
+  if (entry_ts < *ts)
+    return -1;
+  else if (entry_ts > *ts)

This looks wrong. Shouldn't this second 'if' be:

"else if (entry->pts > *ts)" ?

i.e. "if the timestamp is before the beginning of the entry" as opposed to "if
the timestamp is before the end of the entry".

Another way to look at it is that the last "else" branch (the one that returns
0) will only be reached (with the current code) if "entry_ts == *ts", which is
wrong, it should be reached if "*ts >= entry->pts && *ts < entry_pts +
entry_duration"

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