[Cogl] [PATCH 4/4] cogl-gst: Remove the return value from attach_frame()
Neil Roberts
neil at linux.intel.com
Thu Apr 25 11:06:26 PDT 2013
Previously cogl_gst_video_sink_attach_frame returned the last layer
used by the sink. This can also be retrieved via
cogl_gst_video_sink_get_free_layer so I don't think it's necessary to
return it here and it seems kind of out of place.
---
cogl-gst/cogl-gst-video-sink.c | 4 +---
cogl-gst/cogl-gst-video-sink.h | 5 +----
2 files changed, 2 insertions(+), 7 deletions(-)
diff --git a/cogl-gst/cogl-gst-video-sink.c b/cogl-gst/cogl-gst-video-sink.c
index 8fe6aca..50c2f91 100644
--- a/cogl-gst/cogl-gst-video-sink.c
+++ b/cogl-gst/cogl-gst-video-sink.c
@@ -190,7 +190,7 @@ cogl_gst_video_sink_get_free_layer (CoglGstVideoSink *sink)
return sink->priv->free_layer;
}
-int
+void
cogl_gst_video_sink_attach_frame (CoglGstVideoSink *sink,
CoglPipeline *pln)
{
@@ -201,8 +201,6 @@ cogl_gst_video_sink_attach_frame (CoglGstVideoSink *sink,
if (priv->frame[i] != NULL)
cogl_pipeline_set_layer_texture (pln, i + priv->custom_start,
priv->frame[i]);
-
- return priv->free_layer;
}
static CoglBool
diff --git a/cogl-gst/cogl-gst-video-sink.h b/cogl-gst/cogl-gst-video-sink.h
index e76e742..a9694a6 100644
--- a/cogl-gst/cogl-gst-video-sink.h
+++ b/cogl-gst/cogl-gst-video-sink.h
@@ -262,12 +262,9 @@ cogl_gst_video_sink_get_free_layer (CoglGstVideoSink *sink);
* would then make a copy of its template pipeline and call this to
* set the textures.
*
- * Return value: the next free layer after the sink's internal private
- * layers. This is the same value that is returned by
- * cogl_gst_video_sink_get_free_layer().
* Since: 1.16
*/
-int
+void
cogl_gst_video_sink_attach_frame (CoglGstVideoSink *sink,
CoglPipeline *pln);
--
1.7.11.3.g3c3efa5
More information about the Cogl
mailing list