[PATCH weston 1/2] compositor-x11: use _checked() with xkb_select_events to test for error

Kristian Høgsberg hoegsberg at gmail.com
Wed Oct 31 08:31:10 PDT 2012


On Wed, Oct 31, 2012 at 12:13:08AM +0200, Ran Benita wrote:
> In order to use xcb_request_check(), given a request without a reply,
> you need to use the _checked() variant of the request function.
> See xcb-requests(3).

Applied.  Did not know there was a xcb-requests man page, that would
have saved me a lot of time...

Thanks,
Kristian

> Signed-off-by: Ran Benita <ran234 at gmail.com>
> ---
>  src/compositor-x11.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/src/compositor-x11.c b/src/compositor-x11.c
> index 71287e0..c654aec 100644
> --- a/src/compositor-x11.c
> +++ b/src/compositor-x11.c
> @@ -206,14 +206,14 @@ x11_compositor_setup_xkb(struct x11_compositor *c)
>  	}
>  	c->xkb_event_base = ext->first_event;
>  
> -	select = xcb_xkb_select_events(c->conn,
> -				       XCB_XKB_ID_USE_CORE_KBD,
> -				       XCB_XKB_EVENT_TYPE_STATE_NOTIFY,
> -				       0,
> -				       XCB_XKB_EVENT_TYPE_STATE_NOTIFY,
> -				       0,
> -				       0,
> -				       NULL);
> +	select = xcb_xkb_select_events_checked(c->conn,
> +					       XCB_XKB_ID_USE_CORE_KBD,
> +					       XCB_XKB_EVENT_TYPE_STATE_NOTIFY,
> +					       0,
> +					       XCB_XKB_EVENT_TYPE_STATE_NOTIFY,
> +					       0,
> +					       0,
> +					       NULL);
>  	error = xcb_request_check(c->conn, select);
>  	if (error) {
>  		weston_log("error: failed to select for XKB state events\n");
> -- 
> 1.8.0
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list