subtitleoverlay: Use gst_caps_merge() instead of gst_caps_union()
authorSebastian Dröge <sebastian.droege@collabora.co.uk>
Fri, 4 Nov 2011 09:37:12 +0000 (10:37 +0100)
committerSebastian Dröge <sebastian.droege@collabora.co.uk>
Fri, 4 Nov 2011 09:38:39 +0000 (10:38 +0100)
This keeps the caps order and is more efficient.

gst/playback/gstsubtitleoverlay.c

index b3b53c7e1e5dbaa217e9abd21f3ac0ebf96cc52b..49d790bf4d483752d867b7f0548fad56a5aaecd6 100644 (file)
@@ -298,26 +298,16 @@ _factory_filter (GstPluginFeature * feature, GstCaps ** subcaps)
   templ_caps = _get_sub_caps (factory);
 
   if (is_renderer && have_video_sink && templ_caps) {
-    GstCaps *tmp;
-
     GST_DEBUG ("Found renderer element %s (%s) with caps %" GST_PTR_FORMAT,
         gst_element_factory_get_longname (factory),
         gst_plugin_feature_get_name (feature), templ_caps);
-    tmp = gst_caps_union (*subcaps, templ_caps);
-    gst_caps_unref (templ_caps);
-    gst_caps_replace (subcaps, tmp);
-    gst_caps_unref (tmp);
+    gst_caps_merge (*subcaps, templ_caps);
     return TRUE;
   } else if (!is_renderer && !have_video_sink && templ_caps) {
-    GstCaps *tmp;
-
     GST_DEBUG ("Found parser element %s (%s) with caps %" GST_PTR_FORMAT,
         gst_element_factory_get_longname (factory),
         gst_plugin_feature_get_name (feature), templ_caps);
-    tmp = gst_caps_union (*subcaps, templ_caps);
-    gst_caps_unref (templ_caps);
-    gst_caps_replace (subcaps, tmp);
-    gst_caps_unref (tmp);
+    gst_caps_merge (*subcaps, templ_caps);
     return TRUE;
   } else {
     if (templ_caps)