[Bug 774830] uri: Add new uri API to get URI fragment as table

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Nov 22 13:08:53 UTC 2016


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

--- Comment #4 from Seungha Yang <sh.yang at lge.com> ---
(In reply to Sebastian Dröge (slomo) from comment #3)
> Review of attachment 340490 [details] [review]:

Thanks for your attention of my report :)

> Maybe something like get_media_fragment_table()? Should also mention the
> link to the spec in the docs.

get_media_fragment_table() is more correct expression. Note that, how query
"key-value" should be parsed is not defined by URI spec, though.

> @@ +2828,3 @@
> +  if (!uri->fragment)
> +    return NULL;
> +  return _gst_uri_string_to_table (uri->fragment, "&", "=", TRUE, TRUE);
> 
> Is that really enough? I didn't read the spec but that seems a bit simple :)

If only "Media Fragments URI 1.0" is concerned, it's enough. Actually, GstUri's
"URI query" is doing exactly same as this.

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