brl/mutter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

cogl-gst: protect public API using type checks

Browse Source 
(cherry picked from commit 9a8c6536088368060a85f74d082c3dd8487b877c)
This commit is contained in:
Lionel Landwerlin 2013-07-05 17:27:34 +01:00
parent 7b645d51cc
commit 86cd26b67c
1 changed files with 32 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
cogl-gst/cogl-gst-video-sink.c
View File

@ -246,6 +246,8 @@ void
cogl_gst_video_sink_set_first_layer (CoglGstVideoSink *sink, cogl_gst_video_sink_set_first_layer (CoglGstVideoSink *sink,
int first_layer) int first_layer)
{ {
g_return_if_fail (COGL_GST_IS_VIDEO_SINK (sink));
if (first_layer != sink->priv->custom_start) if (first_layer != sink->priv->custom_start)
{ {
sink->priv->custom_start = first_layer; sink->priv->custom_start = first_layer;
@ -261,6 +263,8 @@ void
cogl_gst_video_sink_set_default_sample (CoglGstVideoSink *sink, cogl_gst_video_sink_set_default_sample (CoglGstVideoSink *sink,
CoglBool default_sample) CoglBool default_sample)
{ {
g_return_if_fail (COGL_GST_IS_VIDEO_SINK (sink));
if (default_sample != sink->priv->default_sample) if (default_sample != sink->priv->default_sample)
{ {
sink->priv->default_sample = default_sample; sink->priv->default_sample = default_sample;
@ -272,7 +276,10 @@ void
cogl_gst_video_sink_setup_pipeline (CoglGstVideoSink *sink, cogl_gst_video_sink_setup_pipeline (CoglGstVideoSink *sink,
CoglPipeline *pipeline) CoglPipeline *pipeline)
{ {
sink->priv->renderer->setup_pipeline (sink, pipeline); g_return_if_fail (COGL_GST_IS_VIDEO_SINK (sink));
if (sink->priv->renderer)
sink->priv->renderer->setup_pipeline (sink, pipeline);
} }
static SnippetCacheEntry * static SnippetCacheEntry *
@ -368,7 +375,11 @@ setup_pipeline_from_cache_entry (CoglGstVideoSink *sink,
CoglPipeline * CoglPipeline *
cogl_gst_video_sink_get_pipeline (CoglGstVideoSink *vt) cogl_gst_video_sink_get_pipeline (CoglGstVideoSink *vt)
{ {
CoglGstVideoSinkPrivate *priv = vt->priv; CoglGstVideoSinkPrivate *priv;
g_return_val_if_fail (COGL_GST_IS_VIDEO_SINK (vt), NULL);
priv = vt->priv;
if (priv->pipeline == NULL) if (priv->pipeline == NULL)
{ {
@ -1272,7 +1283,11 @@ cogl_gst_video_sink_new (CoglContext *ctx)
float float
cogl_gst_video_sink_get_aspect (CoglGstVideoSink *vt) cogl_gst_video_sink_get_aspect (CoglGstVideoSink *vt)
{ {
GstVideoInfo *info = &vt->priv->info; GstVideoInfo *info;
g_return_val_if_fail (COGL_GST_IS_VIDEO_SINK (vt), 0.);
info = &vt->priv->info;
return ((float)info->width * (float)info->par_n) / return ((float)info->width * (float)info->par_n) /
((float)info->height * (float)info->par_d); ((float)info->height * (float)info->par_d);
} }
@ -1281,7 +1296,11 @@ float
cogl_gst_video_sink_get_width_for_height (CoglGstVideoSink *vt, cogl_gst_video_sink_get_width_for_height (CoglGstVideoSink *vt,
float height) float height)
{ {
float aspect = cogl_gst_video_sink_get_aspect (vt); float aspect;
g_return_val_if_fail (COGL_GST_IS_VIDEO_SINK (vt), 0.);
aspect = cogl_gst_video_sink_get_aspect (vt);
return height * aspect; return height * aspect;
} }
@ -1289,7 +1308,11 @@ float
cogl_gst_video_sink_get_height_for_width (CoglGstVideoSink *vt, cogl_gst_video_sink_get_height_for_width (CoglGstVideoSink *vt,
float width) float width)
{ {
float aspect = cogl_gst_video_sink_get_aspect (vt); float aspect;
g_return_val_if_fail (COGL_GST_IS_VIDEO_SINK (vt), 0.);
aspect = cogl_gst_video_sink_get_aspect (vt);
return width / aspect; return width / aspect;
} }
@ -1298,6 +1321,10 @@ cogl_gst_video_sink_fit_size (CoglGstVideoSink *vt,
const CoglGstRectangle *available, const CoglGstRectangle *available,
CoglGstRectangle *output) CoglGstRectangle *output)
{ {
g_return_if_fail (COGL_GST_IS_VIDEO_SINK (vt));
g_return_if_fail (available != NULL);
g_return_if_fail (output != NULL);
if (available->height == 0.0f) if (available->height == 0.0f)
{ {
output->x = available->x; output->x = available->x;