From: Leandro Ribeiro <leandrohr@riseup.net>
Date: Mon, 30 Dec 2019 22:11:51 +0000 (-0300)
Subject: exposay: move constant value calculation out of loop
X-Git-Tag: upstream/9.0.0~11
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=fd874ff53379ccdc6584fb4412e69889f94e186f;p=platform%2Fupstream%2Fweston.git

exposay: move constant value calculation out of loop

In exposay_layout(), int pad is being calculated within the
loop. This is unnecessary, as its value is constant. Move it
out of the loop.

Signed-off-by: Leandro Ribeiro <leandrohr@riseup.net>
---

diff --git a/desktop-shell/exposay.c b/desktop-shell/exposay.c
index 9fd4383..b1a8d34 100644
--- a/desktop-shell/exposay.c
+++ b/desktop-shell/exposay.c
@@ -219,6 +219,7 @@ exposay_layout(struct desktop_shell *shell, struct shell_output *shell_output)
 	struct weston_view *view;
 	struct exposay_surface *esurface, *highlight = NULL;
 	int w, h;
+	int pad;
 	int i;
 	int last_row_removed = 0;
 
@@ -275,11 +276,10 @@ exposay_layout(struct desktop_shell *shell, struct shell_output *shell_output)
 	if (eoutput->surface_size > (output->height / 2))
 		eoutput->surface_size = output->height / 2;
 
+	pad = eoutput->surface_size + eoutput->padding_inner;
+
 	i = 0;
 	wl_list_for_each(view, &workspace->layer.view_list.link, layer_link.link) {
-		int pad;
-
-		pad = eoutput->surface_size + eoutput->padding_inner;
 
 		if (!get_shell_surface(view->surface))
 			continue;