[Bug 594035] Add HTTP Live Streaming playback support

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Tue Mar 29 12:55:38 PDT 2011


https://bugzilla.gnome.org/show_bug.cgi?id=594035
  GStreamer | don't know | git

--- Comment #54 from Sebastian Dröge <slomo at circular-chaos.org> 2011-03-29 19:55:33 UTC ---
Some comments about the hls element.

- Call it hlsdemux to be consistent with other elements
- The hls branch contains unrelated changes to vp8enc and the video encoder
base class
- It currently crashes for me in gsthlssrc.c:374 because hlssrc is NULL
(pipeline is souphttpsrc ! hlssrc ! fakesink)
- The query in line 323 is leaked
- The URI query should IMHO return the URI of the currently played fragment
because that's what is visible upstream
- gst_element_make_from_uri() already sets the URI, no need to do it yourself
(and if you do you should use the GstURIHandler interface)


Other than that this looks good at a first look. If you fix the name of the
element and the crash I'll merge it into gst-plugins-bad :)

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- 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