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

animation: Use ::destroy with animate()

Browse Source 
ClutterAnimation uses the weak ref machinery of GObject when associated
to ClutterActor by clutter_actor_animate() and friends - all the while
taking a reference on the actor itself. In order to trigger the weak ref
callback, external code would need to unref the Actor at least twice,
which has slim chance of happening. Plus, the way to destroy an Actor is
to call destroy(), not call unref().

The destruction sequence of ClutterActor emits the ::destroy signal, which
should be used by classes to release external references the might be
holding. My oh my, this sounds *exactly* the case!

So, let's switch to using the ::destroy signal for clutter_actor_animate()
and friends, since we know that the object bound to the Animation is
an Actor, and has a ::destroy signal.

This change has the added benefit of allowing destroying an actor as the
result of the Animation::completed signal without getting a segfault or
other bad things to happen.

Obviously, the change does not affect other GObject classes, or Animation
instances created using clutter_animation_new(); for those, the current
"let's take a reference on the object to avoid it going away in-flight"
mechanism should still suffice.

Side note: it would be interesting if GObject had an interface for
"destructible" objects, so that we could do a safe type check. I guess
it's a Rainy Day Project(tm)...
This commit is contained in:
Emmanuele Bassi 2011-04-16 10:22:01 +01:00
parent b01cbbb8c3
commit 57ebce8039
1 changed files with 24 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
clutter/clutter-animation.c
View File

@ -200,19 +200,22 @@ G_DEFINE_TYPE_WITH_CODE (ClutterAnimation, clutter_animation, G_TYPE_OBJECT,
clutter_scriptable_init));
static void
on_actor_dispose (gpointer user_data,
GObject *actor_pointer)
on_actor_destroy (ClutterActor *actor,
ClutterAnimation *animation)
{
ClutterAnimation *self = user_data;
ClutterAnimationPrivate *priv = animation->priv;
GObject *obj = G_OBJECT (actor);
if (self->priv->object == actor_pointer)
if (obj == priv->object)
{
CLUTTER_NOTE (ANIMATION, "Object [%p] was unref'd", actor_pointer);
g_object_unref (self);
g_object_set_qdata (priv->object, quark_object_animation, NULL);
g_signal_handlers_disconnect_by_func (priv->object,
G_CALLBACK (on_actor_destroy),
animation);
g_object_unref (animation);
}
}
static void
clutter_animation_real_completed (ClutterAnimation *self)
{
@ -265,7 +268,9 @@ clutter_animation_real_completed (ClutterAnimation *self)
priv->object);
g_object_set_qdata (priv->object, quark_object_animation, NULL);
g_object_weak_unref (priv->object, on_actor_dispose, self);
g_signal_handlers_disconnect_by_func (priv->object,
G_CALLBACK (on_actor_destroy),
animation);
CLUTTER_NOTE (ANIMATION, "Releasing the reference Animation [%p]",
animation);
@ -1973,7 +1978,13 @@ animation_create_for_actor (ClutterActor *actor)
animation = clutter_animation_new ();
clutter_animation_set_object (animation, object);
g_object_set_qdata (object, quark_object_animation, animation);
g_object_weak_ref (object, on_actor_dispose, animation);
/* use the ::destroy signal to get a notification
* that the actor went away mid-animation
*/
g_signal_connect (object, "destroy",
G_CALLBACK (on_actor_destroy),
animation);
CLUTTER_NOTE (ANIMATION,
"Created new Animation [%p] for actor [%p]",
@ -2514,10 +2525,10 @@ clutter_actor_detach_animation (ClutterActor *actor)
clutter_timeline_stop (timeline);
}
/* set_object(NULL) does no call weak_unref() because the weak reference
* is added by animation_create_for_actor()
*/
g_object_weak_unref (priv->object, on_actor_dispose, animation);
/* disconnect the ::destroy handler added by animation_create_for_actor() */
g_signal_handlers_disconnect_by_func (actor,
G_CALLBACK (on_actor_destroy),
animation);
clutter_animation_set_object (animation, NULL);