glmixer: Only finalize the other context in finalize()

Otherwise we change a value of a property when going to READY state,
which is unexpected behaviour.
This commit is contained in:
Sebastian Dröge 2014-10-13 13:27:11 +02:00 committed by Tim-Philipp Müller
parent c46c642fd3
commit aed7da3d12

View file

@ -463,7 +463,13 @@ gst_gl_mixer_init (GstGLMixer * mix)
static void
gst_gl_mixer_finalize (GObject * object)
{
GstGLMixerPrivate *priv = GST_GL_MIXER (object)->priv;
GstGLMixer *mix = GST_GL_MIXER (object);
GstGLMixerPrivate *priv = mix->priv;
if (mix->other_context) {
gst_object_unref (mix->other_context);
mix->other_context = NULL;
}
g_mutex_clear (&priv->gl_resource_lock);
g_cond_clear (&priv->gl_resource_cond);
@ -1142,11 +1148,6 @@ gst_gl_mixer_stop (GstAggregator * agg)
mix->context = NULL;
}
if (mix->other_context) {
gst_object_unref (mix->other_context);
mix->other_context = NULL;
}
gst_gl_mixer_reset (mix);
return TRUE;