[PATCH weston 2/7] tests/shell: get rid of static variables

Quentin Glidic sardemff7+wayland at sardemff7.net
Tue Feb 7 10:35:30 UTC 2017


On 27/01/2017 17:30, Emilio Pozuelo Monfort wrote:
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> Stop using static variables and clean up when we're done.
> 
> [Emilio: update to latest weston_layer API]
> 
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> Signed-off-by: Emilio Pozuelo Monfort <emilio.pozuelo at collabora.co.uk>

Nice:
Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>

Thanks,


> ---
>   tests/weston-test-desktop-shell.c | 128 ++++++++++++++++++++++++--------------
>   1 file changed, 82 insertions(+), 46 deletions(-)
> 
> diff --git a/tests/weston-test-desktop-shell.c b/tests/weston-test-desktop-shell.c
> index a1ec7da7..2cf2271e 100644
> --- a/tests/weston-test-desktop-shell.c
> +++ b/tests/weston-test-desktop-shell.c
> @@ -45,58 +45,62 @@
>   #define static_assert(cond, msg)
>   #endif
>   
> -static struct weston_desktop *desktop = NULL;
> -static struct weston_layer background_layer;
> -static struct weston_surface *background_surface = NULL;
> -static struct weston_view *background_view = NULL;
> -static struct weston_layer layer;
> -static struct weston_view *view = NULL;
> -/*
> - * libweston-desktop
> - */
> +struct desktest_shell {
> +	struct wl_listener compositor_destroy_listener;
> +	struct weston_desktop *desktop;
> +	struct weston_layer background_layer;
> +	struct weston_surface *background_surface;
> +	struct weston_view *background_view;
> +	struct weston_layer layer;
> +	struct weston_view *view;
> +};
>   
>   static void
>   desktop_surface_added(struct weston_desktop_surface *desktop_surface,
>   		      void *shell)
>   {
> +	struct desktest_shell *dts = shell;
>   
> -	assert(!view);
> +	assert(!dts->view);
>   
> -	view = weston_desktop_surface_create_view(desktop_surface);
> +	dts->view = weston_desktop_surface_create_view(desktop_surface);
>   
> -	assert(view);
> +	assert(dts->view);
>   }
>   
>   static void
>   desktop_surface_removed(struct weston_desktop_surface *desktop_surface,
>   			void *shell)
>   {
> -	assert(view);
> +	struct desktest_shell *dts = shell;
>   
> -	weston_desktop_surface_unlink_view(view);
> -	weston_view_destroy(view);
> -	view = NULL;
> +	assert(dts->view);
> +
> +	weston_desktop_surface_unlink_view(dts->view);
> +	weston_view_destroy(dts->view);
> +	dts->view = NULL;
>   }
>   
>   static void
>   desktop_surface_committed(struct weston_desktop_surface *desktop_surface,
> -			  int32_t sx, int32_t sy, void *data)
> +			  int32_t sx, int32_t sy, void *shell)
>   {
> +	struct desktest_shell *dts = shell;
>   	struct weston_surface *surface =
>   		weston_desktop_surface_get_surface(desktop_surface);
>   	struct weston_geometry geometry =
>   		weston_desktop_surface_get_geometry(desktop_surface);
>   
> -	assert(view);
> +	assert(dts->view);
>   
>   	if (weston_surface_is_mapped(surface))
>   		return;
>   
>   	surface->is_mapped = true;
> -	weston_layer_entry_insert(&layer.view_list, &view->layer_link);
> -	weston_view_set_position(view, 0 - geometry.x, 0 - geometry.y);
> -	weston_view_update_transform(view);
> -	view->is_mapped = true;
> +	weston_layer_entry_insert(&dts->layer.view_list, &dts->view->layer_link);
> +	weston_view_set_position(dts->view, 0 - geometry.x, 0 - geometry.y);
> +	weston_view_update_transform(dts->view);
> +	dts->view->is_mapped = true;
>   }
>   
>   static void
> @@ -157,42 +161,74 @@ static const struct weston_desktop_api shell_desktop_api = {
>   	.pong = desktop_surface_pong,
>   };
>   
> -/* ************************ *
> - * end of libweston-desktop *
> - * ************************ */
> +static void
> +shell_destroy(struct wl_listener *listener, void *data)
> +{
> +	struct desktest_shell *dts;
> +
> +	dts = container_of(listener, struct desktest_shell,
> +			   compositor_destroy_listener);
> +
> +	weston_desktop_destroy(dts->desktop);
> +	weston_view_destroy(dts->background_view);
> +	weston_surface_destroy(dts->background_surface);
> +	free(dts);
> +}
>   
>   WL_EXPORT int
>   wet_shell_init(struct weston_compositor *ec,
>   	       int *argc, char *argv[])
>   {
> -	weston_layer_init(&layer, ec);
> -	weston_layer_init(&background_layer, ec);
> +	struct desktest_shell *dts;
> +
> +	dts = zalloc(sizeof *dts);
> +	if (!dts)
> +		return -1;
> +
> +	dts->compositor_destroy_listener.notify = shell_destroy;
> +	wl_signal_add(&ec->destroy_signal, &dts->compositor_destroy_listener);
> +
> +	weston_layer_init(&dts->layer, ec);
> +	weston_layer_init(&dts->background_layer, ec);
>   
>   	weston_layer_set_position(&dts->layer,
>   				  WESTON_LAYER_POSITION_NORMAL);
>   	weston_layer_set_position(&dts->background_layer,
>   				  WESTON_LAYER_POSITION_BACKGROUND);
>   
> -	background_surface = weston_surface_create(ec);
> -	if (background_surface == NULL)
> -		return -1;
> -	background_view = weston_view_create(background_surface);
> -	if (background_view == NULL) {
> -		weston_surface_destroy(background_surface);
> -		return -1;
> -	}
> +	dts->background_surface = weston_surface_create(ec);
> +	if (dts->background_surface == NULL)
> +		goto out_free;
> +
> +	dts->background_view = weston_view_create(dts->background_surface);
> +	if (dts->background_view == NULL)
> +		goto out_surface;
> +
> +	weston_surface_set_color(dts->background_surface, 0.0, 0.0, 0.0, 1);
> +	pixman_region32_fini(&dts->background_surface->opaque);
> +	pixman_region32_init_rect(&dts->background_surface->opaque, 0, 0, 2000, 2000);
> +	pixman_region32_fini(&dts->background_surface->input);
> +	pixman_region32_init_rect(&dts->background_surface->input, 0, 0, 2000, 2000);
> +
> +	weston_surface_set_size(dts->background_surface, 2000, 2000);
> +	weston_view_set_position(dts->background_view, 0, 0);
> +	weston_layer_entry_insert(&dts->background_layer.view_list, &dts->background_view->layer_link);
> +	weston_view_update_transform(dts->background_view);
> +
> +	dts->desktop = weston_desktop_create(ec, &shell_desktop_api, dts);
> +	if (dts->desktop == NULL)
> +		goto out_view;
> +
> +	return 0;
> +
> +out_view:
> +	weston_view_destroy(dts->background_view);
>   
> -	weston_surface_set_color(background_surface, 0.0, 0.0, 0.0, 1);
> -	pixman_region32_fini(&background_surface->opaque);
> -	pixman_region32_init_rect(&background_surface->opaque, 0, 0, 2000, 2000);
> -	pixman_region32_fini(&background_surface->input);
> -	pixman_region32_init_rect(&background_surface->input, 0, 0, 2000, 2000);
> +out_surface:
> +	weston_surface_destroy(dts->background_surface);
>   
> -	weston_surface_set_size(background_surface, 2000, 2000);
> -	weston_view_set_position(background_view, 0, 0);
> -	weston_layer_entry_insert(&background_layer.view_list, &background_view->layer_link);
> -	weston_view_update_transform(background_view);
> +out_free:
> +	free(dts);
>   
> -	desktop = weston_desktop_create(ec, &shell_desktop_api, NULL);
> -	return desktop ? 0 : -1;
> +	return -1;
>   }
> 


-- 

Quentin “Sardem FF7” Glidic


More information about the wayland-devel mailing list