From 84dd6b4af5d0ceb0ce2f87c0f97588b7d7e64333 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jonas=20Dre=C3=9Fler?= Date: Mon, 19 Oct 2020 12:48:39 +0200 Subject: [PATCH] clutter/stage: Remove redraw_pending and redraw_count The redraw_pending boolean is used to schedule the first stage update after starting Clutter. This flag is superfluous because we have the pending_finish_queue_redraws flag which does the same. While at it, also remove the redraw_count debug variable, since there should be better ways to count the number of queued redraws nowadays, for example Sysprof marks. https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1527 --- clutter/clutter/clutter-stage.c | 22 ---------------------- 1 file changed, 22 deletions(-) diff --git a/clutter/clutter/clutter-stage.c b/clutter/clutter/clutter-stage.c index e70bd1846..c3c20936d 100644 --- a/clutter/clutter/clutter-stage.c +++ b/clutter/clutter/clutter-stage.c @@ -128,10 +128,6 @@ struct _ClutterStagePrivate ClutterPickStack *pick_stack; ClutterPickMode cached_pick_mode; -#ifdef CLUTTER_ENABLE_DEBUG - gulong redraw_count; -#endif /* CLUTTER_ENABLE_DEBUG */ - ClutterStageState current_state; int update_freeze_count; @@ -139,7 +135,6 @@ struct _ClutterStagePrivate gboolean needs_update_devices; gboolean pending_finish_queue_redraws; - guint redraw_pending : 1; guint throttle_motion_events : 1; guint min_size_changed : 1; guint motion_events_enabled : 1; @@ -2813,23 +2808,6 @@ _clutter_stage_queue_actor_redraw (ClutterStage *stage, priv->pending_finish_queue_redraws = TRUE; } - if (!priv->redraw_pending) - { - CLUTTER_NOTE (PAINT, "First redraw request"); - - clutter_stage_schedule_update (stage); - priv->redraw_pending = TRUE; - } -#ifdef CLUTTER_ENABLE_DEBUG - else - { - CLUTTER_NOTE (PAINT, "Redraw request number %lu", - priv->redraw_count + 1); - - priv->redraw_count += 1; - } -#endif /* CLUTTER_ENABLE_DEBUG */ - if (entry) { /* Ignore all requests to queue a redraw for an actor if a full