scroll-actor: Check for point equality in the internal setter
authorEmmanuele Bassi <ebassi@linux.intel.com>
Fri, 18 May 2012 17:00:30 +0000 (18:00 +0100)
committerEmmanuele Bassi <ebassi@linux.intel.com>
Fri, 18 May 2012 17:00:30 +0000 (18:00 +0100)
The public API should just accept any valid value; it's up to the
internal setter to check if we're getting the same value as the one
currently set.

clutter/clutter-scroll-actor.c

index 5a6662c..d6326e3 100644 (file)
@@ -187,6 +187,9 @@ clutter_scroll_actor_set_scroll_to_internal (ClutterScrollActor *self,
   ClutterScrollActorPrivate *priv = self->priv;
   ClutterActor *actor = CLUTTER_ACTOR (self);
 
+  if (clutter_point_equals (&priv->scroll_to, point))
+    return;
+
   if (point == NULL)
     clutter_point_init (&priv->scroll_to, 0.f, 0.f);
   else
@@ -424,9 +427,6 @@ clutter_scroll_actor_scroll_to_point (ClutterScrollActor *actor,
 
   priv = actor->priv;
 
-  if (clutter_point_equals (&priv->scroll_to, point))
-    return;
-
   info = _clutter_actor_get_animation_info (CLUTTER_ACTOR (actor));
 
   /* jump to the end if there is no easing state, or if the easing