[Bug 776928] hlsdemux: Set fragment header uri if exists

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Mon Jan 9 13:38:45 UTC 2017


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

--- Comment #7 from Seungha Yang <sh.yang at lge.com> ---
(In reply to Jan Schmidt from comment #6)
> Review of attachment 343150 [details] [review]:
> 
> ::: ext/hls/m3u8.c
> @@ +546,3 @@
> +          } else if (strcmp (a, "BYTERANGE") == 0) {
> +            if (int64_from_string (v, &v, &size)) {
> +              if (*v == '@' && !int64_from_string (v + 1, &v, &offset)) {
> 
> Did you mean to capture into init_size and init_offset variables here?

Right. It was omitted mistakenly. Now I'm preparing new patch to allow multiple
init segments in a m3u8. I think Thiago's first suggestion (make pointer to
access init segment per media segment) is correct approach and I also prefer
it.

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