From 80bd44cba0467ad16b40d0d29579fca340b6b7b9 Mon Sep 17 00:00:00 2001 From: Georges Basile Stavracas Neto Date: Fri, 9 Oct 2020 19:23:41 -0300 Subject: [PATCH] clutter/actor: Use switch instead of if/else To improve legibility of the code. https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1489 --- clutter/clutter/clutter-actor.c | 19 +++++++++++++------ 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/clutter/clutter/clutter-actor.c b/clutter/clutter/clutter-actor.c index 94a5e4cee..c314c49a8 100644 --- a/clutter/clutter/clutter-actor.c +++ b/clutter/clutter/clutter-actor.c @@ -3364,12 +3364,19 @@ _clutter_actor_paint_cull_result (ClutterActor *self, if (success) { - if (result == CLUTTER_CULL_RESULT_IN) - clutter_color_init (&color, 0, 255, 0, 255); - else if (result == CLUTTER_CULL_RESULT_OUT) - clutter_color_init (&color, 0, 0, 255, 255); - else - clutter_color_init (&color, 0, 255, 255, 255); + switch (result) + { + case CLUTTER_CULL_RESULT_IN: + clutter_color_init (&color, 0, 255, 0, 255); + break; + case CLUTTER_CULL_RESULT_OUT: + clutter_color_init (&color, 0, 0, 255, 255); + break; + case CLUTTER_CULL_RESULT_PARTIAL: + default: + clutter_color_init (&color, 0, 255, 255, 255); + break; + } } else clutter_color_init (&color, 255, 255, 255, 255);