Revert "ClutterActor: Optimize away idempotent scale/position updates"
authorMichael Catanzaro <mcatanzaro@igalia.com>
Fri, 20 Oct 2017 00:53:13 +0000 (19:53 -0500)
committerJeremy Bicha <jbicha@debian.org>
Sat, 21 Oct 2017 23:47:22 +0000 (00:47 +0100)
This reverts commit 5cb5baa7d42712f9e0db8dfaa0485e2d833e4d83.

There's a cascade of regressions that needs to be sorted out before
relanding this:

https://bugzilla.gnome.org/show_bug.cgi?id=784314

Gbp-Pq: Name 0018-Revert-ClutterActor-Optimize-away-idempotent-scale.patch

clutter/clutter/clutter-actor.c

index dbb202f9cfaa0d753148e90b352230727afd48c9..6954f03968e6e6979d8656e5c80e8d0291b84fa2 100644 (file)
@@ -4828,8 +4828,7 @@ clutter_actor_set_scale_factor (ClutterActor      *self,
   g_assert (pspec != NULL);
   g_assert (scale_p != NULL);
 
-  if (*scale_p != factor)
-    _clutter_actor_create_transition (self, pspec, *scale_p, factor);
+  _clutter_actor_create_transition (self, pspec, *scale_p, factor);
 }
 
 static inline void
@@ -10264,10 +10263,9 @@ clutter_actor_set_position (ClutterActor *self,
   cur_position.x = clutter_actor_get_x (self);
   cur_position.y = clutter_actor_get_y (self);
 
-  if (!clutter_point_equals (&cur_position, &new_position))
-    _clutter_actor_create_transition (self, obj_props[PROP_POSITION],
-                                      &cur_position,
-                                      &new_position);
+  _clutter_actor_create_transition (self, obj_props[PROP_POSITION],
+                                    &cur_position,
+                                    &new_position);
 }
 
 /**