[mutter/wip/carlosg/unassorted-perf-fixes: 21/21] Revert "Revert "ClutterActor: Optimize away idempotent scale/position updates""



commit 517ec28dac7fe1328f3763be1c66923feceabcec
Author: Carlos Garnacho <carlosg gnome org>
Date:   Sat Jun 16 14:26:39 2018 +0200

    Revert "Revert "ClutterActor: Optimize away idempotent scale/position updates""
    
    This reverts commit 4d2647cdbc6cc2b5a41eb8fcd5183a77819283dd.
    
    A proper fix is now in GTK+, let's reintroduce this optimization.
    
    Related: gtk#844, gtk!294

 clutter/clutter/clutter-actor.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)
---
diff --git a/clutter/clutter/clutter-actor.c b/clutter/clutter/clutter-actor.c
index 521d4264c..7de72e633 100644
--- a/clutter/clutter/clutter-actor.c
+++ b/clutter/clutter/clutter-actor.c
@@ -4880,7 +4880,8 @@ clutter_actor_set_scale_factor (ClutterActor      *self,
   g_assert (pspec != NULL);
   g_assert (scale_p != NULL);
 
-  _clutter_actor_create_transition (self, pspec, *scale_p, factor);
+  if (*scale_p != factor)
+    _clutter_actor_create_transition (self, pspec, *scale_p, factor);
 }
 
 static inline void
@@ -10312,9 +10313,10 @@ clutter_actor_set_position (ClutterActor *self,
   cur_position.x = clutter_actor_get_x (self);
   cur_position.y = clutter_actor_get_y (self);
 
-  _clutter_actor_create_transition (self, obj_props[PROP_POSITION],
-                                    &cur_position,
-                                    &new_position);
+  if (!clutter_point_equals (&cur_position, &new_position))
+    _clutter_actor_create_transition (self, obj_props[PROP_POSITION],
+                                      &cur_position,
+                                      &new_position);
 }
 
 /**


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]