[Bug 639055] discoverer: add support for subtitle streams

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Wed Aug 24 23:35:10 PDT 2011


https://bugzilla.gnome.org/show_bug.cgi?id=639055
  GStreamer | gst-plugins-base | git

Sebastian Dröge <slomo> changed:

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

--- Comment #11 from Sebastian Dröge <slomo at circular-chaos.org> 2011-08-25 06:35:06 UTC ---
Review of attachment 194600:
 --> (https://bugzilla.gnome.org/review?bug=639055&attachment=194600)

::: gst-libs/gst/pbutils/gstdiscoverer.c
@@ +754,3 @@
+      if (gst_tag_list_get_string (((GstDiscovererStreamInfo *) info)->tags,
+              GST_TAG_LANGUAGE_CODE, &language)) {
+        info->language = g_strdup (language);

Return value of gst_tag_list_get_string() is already owned by the caller, no
need to copy

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