[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