analysis: ClutterAnimation

• Remove one unused variable.

• We ignore the result of get_timeline_internal() so we need to tell
  the compiler that - though a better solution would be to split the
  timeline implicit creation into its own function.
This commit is contained in:
Emmanuele Bassi 2010-02-11 15:09:51 +00:00
parent a287ca7a28
commit 6dc0e9632c

View File

@ -909,7 +909,6 @@ clutter_animation_update (ClutterAnimation *animation,
const gchar *property_name, const gchar *property_name,
const GValue *final) const GValue *final)
{ {
ClutterAnimationPrivate *priv;
ClutterInterval *interval; ClutterInterval *interval;
GType int_type; GType int_type;
@ -918,8 +917,6 @@ clutter_animation_update (ClutterAnimation *animation,
g_return_val_if_fail (final != NULL, NULL); g_return_val_if_fail (final != NULL, NULL);
g_return_val_if_fail (G_VALUE_TYPE (final) != G_TYPE_INVALID, NULL); g_return_val_if_fail (G_VALUE_TYPE (final) != G_TYPE_INVALID, NULL);
priv = animation->priv;
interval = clutter_animation_get_interval (animation, property_name); interval = clutter_animation_get_interval (animation, property_name);
if (interval == NULL) if (interval == NULL)
{ {
@ -1536,7 +1533,12 @@ clutter_animation_set_alpha (ClutterAnimation *animation,
animation); animation);
} }
else else
timeline = clutter_animation_get_timeline_internal (animation); {
/* FIXME - add a create_timeline_internal() because this does
* not look very good
*/
(void) clutter_animation_get_timeline_internal (animation);
}
out: out:
/* emit all relevant notifications */ /* emit all relevant notifications */