[Bug 743703] http connectivity robustness of hlsdemux (and now adaptivedemux)

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Thu Feb 5 05:12:30 PST 2015


https://bugzilla.gnome.org/show_bug.cgi?id=743703
  GStreamer | gst-plugins-bad | unspecified

--- Comment #8 from Thiago Sousa Santos <thiagossantos at gmail.com> 2015-02-05 13:12:26 UTC ---
Review of attachment 296140:
 --> (https://bugzilla.gnome.org/review?bug=743703&attachment=296140)

This is aligned with what I had in mind, but I'd prefer if we had new functions
on core to set/get those details.

Ideally I'd like to have a new GstStructure under the event's structure just to
set these element fields, perhaps under the "element-details" name. This way we
could have gst_message_set_error_element_details() and
gst_message_parse_error_element_details(). OTOH, GstStructure in GstStructure
is not so nice in 1.x because of serialization/deserialization limitations (use
of the same symbol for opening and closing -> "") so we need to think carefully
if we can do this.

Even if we don't use this approach I'd still prefer to have the set/get done
through some custom function in core, this way we can prevent that it
overwrites standard fields used and it is easier to identify code that is using
this new feature for future changes.

Mathieu and Tim, what's your take on this?

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