xdg-shell: make wlr_xdg_surface_from_resource reject NULL

Most resources must not be NULL. Make it so callers need to check for
NULL explicitly. This makes it clearer in the handlers code that the
NULL wl_resource case needs to be handled, and allows callers to make a
difference between a NULL wl_resource and an inert resource.
This commit is contained in:
Simon Ser 2019-12-29 22:42:23 +01:00 committed by Drew DeVault
parent 7e521fed97
commit 8bb2dc68ea
1 changed files with 4 additions and 5 deletions

View File

@ -237,8 +237,10 @@ static void xdg_surface_handle_get_popup(struct wl_client *client,
struct wl_resource *positioner_resource) { struct wl_resource *positioner_resource) {
struct wlr_xdg_surface *xdg_surface = struct wlr_xdg_surface *xdg_surface =
wlr_xdg_surface_from_resource(resource); wlr_xdg_surface_from_resource(resource);
struct wlr_xdg_surface *parent = struct wlr_xdg_surface *parent = NULL;
wlr_xdg_surface_from_resource(parent_resource); if (parent_resource != NULL) {
parent = wlr_xdg_surface_from_resource(parent_resource);
}
if (xdg_surface == NULL) { if (xdg_surface == NULL) {
return; // TODO: create an inert xdg_popup return; // TODO: create an inert xdg_popup
} }
@ -517,9 +519,6 @@ void destroy_xdg_surface(struct wlr_xdg_surface *surface) {
struct wlr_xdg_surface *wlr_xdg_surface_from_resource( struct wlr_xdg_surface *wlr_xdg_surface_from_resource(
struct wl_resource *resource) { struct wl_resource *resource) {
if (!resource) {
return NULL;
}
assert(wl_resource_instance_of(resource, &xdg_surface_interface, assert(wl_resource_instance_of(resource, &xdg_surface_interface,
&xdg_surface_implementation)); &xdg_surface_implementation));
return wl_resource_get_user_data(resource); return wl_resource_get_user_data(resource);