From 4058828b54c6cf46a3e6d1bf90efc98c5ba8d7fa Mon Sep 17 00:00:00 2001 From: Pekka Paalanen Date: Tue, 5 Nov 2019 15:40:25 +0200 Subject: [PATCH] tests: remove static data from viewporter This caching is actually never hit. I tested by making the early return abort() instead and all works just fine. Remove the caching. The static variable will cause problems when we stop fork()'ing for each test case. Signed-off-by: Pekka Paalanen --- tests/viewporter-test.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tests/viewporter-test.c b/tests/viewporter-test.c index 95fb9b46..f9b19d9e 100644 --- a/tests/viewporter-test.c +++ b/tests/viewporter-test.c @@ -41,10 +41,7 @@ get_viewporter(struct client *client) { struct global *g; struct global *global_wpr = NULL; - static struct wp_viewporter *wpr; - - if (wpr) - return wpr; + struct wp_viewporter *wpr; wl_list_for_each(g, &client->global_list, link) { if (strcmp(g->interface, wp_viewporter_interface.name))