From ca64654256b1f62cbc564e8fa68fc80651835a60 Mon Sep 17 00:00:00 2001
From: Maxim Solovyov <msolovyov@protonmail.com>
Date: Tue, 10 Jan 2023 00:15:49 +0300
Subject: [PATCH] use _checked functions with xcb_request_check

---
 src/event.c | 3 ++-
 src/picom.c | 8 ++++----
 src/win.c   | 2 +-
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/event.c b/src/event.c
index 89f9624..082686b 100644
--- a/src/event.c
+++ b/src/event.c
@@ -283,7 +283,8 @@ static inline void ev_map_notify(session_t *ps, xcb_map_notify_event_t *ev) {
 	// in redirected state.
 	if (ps->overlay && ev->window == ps->overlay && !ps->redirected) {
 		log_debug("Overlay is mapped while we are not redirected");
-		auto e = xcb_request_check(ps->c, xcb_unmap_window(ps->c, ps->overlay));
+		auto e =
+		    xcb_request_check(ps->c, xcb_unmap_window_checked(ps->c, ps->overlay));
 		if (e) {
 			log_error("Failed to unmap the overlay window");
 			free(e);
diff --git a/src/picom.c b/src/picom.c
index b5ac3a3..9d97a86 100644
--- a/src/picom.c
+++ b/src/picom.c
@@ -1266,7 +1266,7 @@ static bool init_debug_window(session_t *ps) {
 		goto err_out;
 	}
 
-	err = xcb_request_check(ps->c, xcb_map_window(ps->c, ps->debug_window));
+	err = xcb_request_check(ps->c, xcb_map_window_checked(ps->c, ps->debug_window));
 	if (err) {
 		goto err_out;
 	}
@@ -2069,8 +2069,8 @@ static session_t *session_init(int argc, char **argv, Display *dpy,
 	ps->sync_fence = XCB_NONE;
 	if (ps->xsync_exists) {
 		ps->sync_fence = x_new_id(ps->c);
-		e = xcb_request_check(
-		    ps->c, xcb_sync_create_fence(ps->c, ps->root, ps->sync_fence, 0));
+		e = xcb_request_check(ps->c, xcb_sync_create_fence_checked(
+		                                 ps->c, ps->root, ps->sync_fence, 0));
 		if (e) {
 			if (ps->o.xrender_sync_fence) {
 				log_error_x_error(e, "Failed to create a XSync fence. "
@@ -2280,7 +2280,7 @@ static session_t *session_init(int argc, char **argv, Display *dpy,
 	xcb_query_tree_reply_t *query_tree_reply =
 	    xcb_query_tree_reply(ps->c, xcb_query_tree(ps->c, ps->root), NULL);
 
-	e = xcb_request_check(ps->c, xcb_ungrab_server(ps->c));
+	e = xcb_request_check(ps->c, xcb_ungrab_server_checked(ps->c));
 	if (e) {
 		log_fatal_x_error(e, "Failed to ungrab server");
 		free(e);
diff --git a/src/win.c b/src/win.c
index 502bb82..cef94fe 100644
--- a/src/win.c
+++ b/src/win.c
@@ -1299,7 +1299,7 @@ void win_mark_client(session_t *ps, struct managed_win *w, xcb_window_t client)
 	}
 
 	auto e = xcb_request_check(
-	    ps->c, xcb_change_window_attributes(
+	    ps->c, xcb_change_window_attributes_checked(
 	               ps->c, client, XCB_CW_EVENT_MASK,
 	               (const uint32_t[]){determine_evmask(ps, client, WIN_EVMODE_CLIENT)}));
 	if (e) {