benkyd/picom
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

use _checked functions with xcb_request_check

Browse Source
This commit is contained in:
Maxim Solovyov
2023-01-10 00:15:49 +03:00
committed by Arda Atci
parent a667886959
commit 7846b17c54
3 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/event.c
View File

@@ -283,7 +283,8 @@ static inline void ev_map_notify(session_t *ps, xcb_map_notify_event_t *ev) {
// in redirected state. // in redirected state.
if (ps->overlay && ev->window == ps->overlay && !ps->redirected) { if (ps->overlay && ev->window == ps->overlay && !ps->redirected) {
log_debug("Overlay is mapped while we are not 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) { if (e) {
log_error("Failed to unmap the overlay window"); log_error("Failed to unmap the overlay window");
free(e); free(e);

8
src/picom.c
View File

@@ -1438,7 +1438,7 @@ static bool init_debug_window(session_t *ps) {
goto err_out; 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) { if (err) {
goto err_out; goto err_out;
} }
@@ -2257,8 +2257,8 @@ static session_t *session_init(int argc, char **argv, Display *dpy,
ps->sync_fence = XCB_NONE; ps->sync_fence = XCB_NONE;
if (ps->xsync_exists) { if (ps->xsync_exists) {
ps->sync_fence = x_new_id(ps->c); ps->sync_fence = x_new_id(ps->c);
e = xcb_request_check( e = xcb_request_check(ps->c, xcb_sync_create_fence_checked(
ps->c, xcb_sync_create_fence(ps->c, ps->root, ps->sync_fence, 0)); ps->c, ps->root, ps->sync_fence, 0));
if (e) { if (e) {
if (ps->o.xrender_sync_fence) { if (ps->o.xrender_sync_fence) {
log_error_x_error(e, "Failed to create a XSync fence. " log_error_x_error(e, "Failed to create a XSync fence. "
@@ -2469,7 +2469,7 @@ static session_t *session_init(int argc, char **argv, Display *dpy,
xcb_query_tree_reply_t *query_tree_reply = xcb_query_tree_reply_t *query_tree_reply =
xcb_query_tree_reply(ps->c, xcb_query_tree(ps->c, ps->root), NULL); 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) { if (e) {
log_fatal_x_error(e, "Failed to ungrab server"); log_fatal_x_error(e, "Failed to ungrab server");
free(e); free(e);

2
src/win.c
View File

@@ -1565,7 +1565,7 @@ void win_mark_client(session_t *ps, struct managed_win *w, xcb_window_t client)
} }
auto e = xcb_request_check( 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, ps->c, client, XCB_CW_EVENT_MASK,
(const uint32_t[]){determine_evmask(ps, client, WIN_EVMODE_CLIENT)})); (const uint32_t[]){determine_evmask(ps, client, WIN_EVMODE_CLIENT)}));
if (e) { if (e) {