audiovisualizer: handle the return of the setup function

Make the class future proof by handling the gboolean return of the setup
function. So if/when a child class uses this the base class is ready.
This commit is contained in:
Luis de Bethencourt 2015-01-13 16:49:07 +00:00
parent fc56b0742c
commit ad6b8d0a3d

View file

@ -758,8 +758,8 @@ gst_audio_visualizer_src_setcaps (GstAudioVisualizer * scope, GstCaps * caps)
gst_video_frame_map (&scope->tempframe, &scope->vinfo, scope->tempbuf,
GST_MAP_READWRITE);
if (klass->setup)
res = klass->setup (scope);
if (klass->setup && !klass->setup (scope))
goto setup_failed;
GST_DEBUG_OBJECT (scope, "video: dimension %dx%d, framerate %d/%d",
GST_VIDEO_INFO_WIDTH (&info), GST_VIDEO_INFO_HEIGHT (&info),
@ -782,6 +782,12 @@ wrong_caps:
GST_DEBUG_OBJECT (scope, "error parsing caps");
return FALSE;
}
setup_failed:
{
GST_WARNING_OBJECT (scope, "failed to set up");
return FALSE;
}
}
static gboolean