From c17f0870ef1620e73b18f0b0ce8303c94027d19b Mon Sep 17 00:00:00 2001 From: Ryan Dwyer Date: Fri, 3 Aug 2018 18:08:20 +1000 Subject: Fix crash when moving cursor over a fullscreen split container Calling container_at_view fails an assertion if the container isn't a view. Calling tiling_container_at works correctly, as that function checks if the container is a view and calls container_at_view if so. --- include/sway/tree/container.h | 6 +++--- sway/input/cursor.c | 2 +- sway/tree/container.c | 8 ++------ 3 files changed, 6 insertions(+), 10 deletions(-) diff --git a/include/sway/tree/container.h b/include/sway/tree/container.h index c3942e9e..44ff9f7d 100644 --- a/include/sway/tree/container.h +++ b/include/sway/tree/container.h @@ -234,9 +234,9 @@ struct sway_container *container_at(struct sway_container *workspace, double lx, double ly, struct wlr_surface **surface, double *sx, double *sy); -struct sway_container *container_at_view(struct sway_container *view, - double lx, double ly, struct wlr_surface **surface, - double *sx, double *sy); +struct sway_container *tiling_container_at( + struct sway_container *con, double lx, double ly, + struct wlr_surface **surface, double *sx, double *sy); /** * Apply the function for each descendant of the container breadth first. diff --git a/sway/input/cursor.c b/sway/input/cursor.c index c2fc4e9e..80b4f9dc 100644 --- a/sway/input/cursor.c +++ b/sway/input/cursor.c @@ -99,7 +99,7 @@ static struct sway_container *container_at_coords( return ws; } if (ws->sway_workspace->fullscreen) { - return container_at_view(ws->sway_workspace->fullscreen, lx, ly, + return tiling_container_at(ws->sway_workspace->fullscreen, lx, ly, surface, sx, sy); } if ((*surface = layer_surface_at(output, diff --git a/sway/tree/container.c b/sway/tree/container.c index 46c54e2d..4a503652 100644 --- a/sway/tree/container.c +++ b/sway/tree/container.c @@ -535,7 +535,7 @@ struct sway_container *container_parent(struct sway_container *container, return container; } -struct sway_container *container_at_view(struct sway_container *swayc, +static struct sway_container *container_at_view(struct sway_container *swayc, double lx, double ly, struct wlr_surface **surface, double *sx, double *sy) { if (!sway_assert(swayc->type == C_VIEW, "Expected a view")) { @@ -574,10 +574,6 @@ struct sway_container *container_at_view(struct sway_container *swayc, return NULL; } -static struct sway_container *tiling_container_at( - struct sway_container *con, double lx, double ly, - struct wlr_surface **surface, double *sx, double *sy); - /** * container_at for a container with layout L_TABBED. */ @@ -684,7 +680,7 @@ static struct sway_container *floating_container_at(double lx, double ly, return NULL; } -static struct sway_container *tiling_container_at( +struct sway_container *tiling_container_at( struct sway_container *con, double lx, double ly, struct wlr_surface **surface, double *sx, double *sy) { if (con->type == C_VIEW) { -- cgit v1.2.3-54-g00ecf