glutils: remove trivial helper function
authorMatthew Waters <matthew@centricular.com>
Wed, 11 Jan 2017 14:57:29 +0000 (01:57 +1100)
committerMatthew Waters <matthew@centricular.com>
Fri, 13 Jan 2017 00:20:51 +0000 (11:20 +1100)
gst_gl_caps_replace_all_caps_features() is only used in two places and can
be trivially reproduced.

docs/libs/gst-plugins-bad-libs-sections.txt
ext/gl/gstglmixer.c
gst-libs/gst/gl/gstglfilter.c
gst-libs/gst/gl/gstglutils.c
gst-libs/gst/gl/gstglutils.h

index 00c2057..d19b436 100644 (file)
@@ -1618,7 +1618,6 @@ GstGLUploadPrivate
 gst_gl_handle_set_context
 gst_gl_handle_context_query
 gst_gl_context_check_framebuffer_status
-gst_gl_caps_replace_all_caps_features
 gst_gl_ensure_element_data
 gst_gl_get_plane_data_size
 gst_gl_get_plane_start
index 3fc5aa6..9a7446a 100644 (file)
@@ -220,15 +220,21 @@ static GstCaps *
 _update_caps (GstVideoAggregator * vagg, GstCaps * caps, GstCaps * filter)
 {
   GstCaps *tmp;
+  guint i, n;
 
   if (filter) {
     tmp = gst_caps_intersect (caps, filter);
   } else {
-    tmp = caps;
+    tmp = gst_caps_copy (caps);
   }
 
-  return gst_gl_caps_replace_all_caps_features (tmp,
-      GST_CAPS_FEATURE_MEMORY_GL_MEMORY);
+  n = gst_caps_get_size (tmp);
+  for (i = 0; i < n; i++) {
+    gst_caps_set_features (tmp, i,
+        gst_caps_features_from_string (GST_CAPS_FEATURE_MEMORY_GL_MEMORY));
+  }
+
+  return tmp;
 }
 
 static GstCaps *
index f19f8c6..f0c02cf 100644 (file)
@@ -608,7 +608,15 @@ static GstCaps *
 gst_gl_filter_set_caps_features (const GstCaps * caps,
     const gchar * feature_name)
 {
-  GstCaps *ret = gst_gl_caps_replace_all_caps_features (caps, feature_name);
+  GstCaps *ret = gst_caps_copy (caps);
+  guint n = gst_caps_get_size (ret);
+  guint i = 0;
+
+  for (i = 0; i < n; i++) {
+    gst_caps_set_features (ret, i,
+        gst_caps_features_from_string (GST_CAPS_FEATURE_MEMORY_GL_MEMORY));
+  }
+
   gst_caps_set_simple (ret, "format", G_TYPE_STRING, "RGBA", NULL);
   return ret;
 }
index 4dd95c0..0b7d55d 100644 (file)
@@ -490,22 +490,6 @@ gst_gl_get_plane_start (GstVideoInfo * info, GstVideoAlignment * valign,
   return (GST_VIDEO_INFO_PLANE_OFFSET (info, plane)) - plane_start;
 }
 
-GstCaps *
-gst_gl_caps_replace_all_caps_features (const GstCaps * caps,
-    const gchar * feature_name)
-{
-  GstCaps *tmp = gst_caps_copy (caps);
-  guint n = gst_caps_get_size (tmp);
-  guint i = 0;
-
-  for (i = 0; i < n; i++) {
-    gst_caps_set_features (tmp, i,
-        gst_caps_features_from_string (feature_name));
-  }
-
-  return tmp;
-}
-
 /**
  * gst_gl_value_get_texture_target_mask:
  * @value: an initialized #GValue of type G_TYPE_STRING
index 5389c71..942371e 100644 (file)
@@ -47,9 +47,6 @@ gsize gst_gl_get_plane_data_size (GstVideoInfo * info, GstVideoAlignment * align
 GST_EXPORT
 gsize gst_gl_get_plane_start (GstVideoInfo * info, GstVideoAlignment * valign,
     guint plane);
-GST_EXPORT
-GstCaps * gst_gl_caps_replace_all_caps_features (const GstCaps * caps,
-    const gchar * feature_name);
 
 GST_EXPORT
 gboolean gst_gl_value_set_texture_target_from_mask (GValue * value,