X-Git-Url: https://jasonwoof.com/gitweb/?a=blobdiff_plain;f=spectrwm.c;h=9286f33f48652b78019c8aab060fac868751097f;hb=38fa20eb4b70fff4b18552bb46730353f1219484;hp=d63e6e8accdd43e107241600dd4c1d12ff1eb80c;hpb=b3f747ce9668befedc5399fb8f4623e66d6163e2;p=spectrwm.git

diff --git a/spectrwm.c b/spectrwm.c
index d63e6e8..9286f33 100644
--- a/spectrwm.c
+++ b/spectrwm.c
@@ -637,6 +637,7 @@ map_window_raised(xcb_window_t win)
 		XCB_CONFIG_WINDOW_STACK_MODE, &val);
 
 	xcb_map_window(conn, win);	
+	xcb_flush(conn);
 }
 
 xcb_atom_t
@@ -1122,9 +1123,10 @@ xrandr_geteventname(XEvent *e)
 void
 dumpwins(struct swm_region *r, union arg *args)
 {
-	struct ws_win		*win;
-	uint16_t		state;
-	XWindowAttributes	wa;
+	struct ws_win				*win;
+	uint16_t				state;
+	xcb_get_window_attributes_cookie_t	c;
+	xcb_get_window_attributes_reply_t	*r;
 
 	if (r->ws == NULL) {
 		warnx("dumpwins: invalid workspace");
@@ -1132,26 +1134,33 @@ dumpwins(struct swm_region *r, union arg *args)
 	}
 
 	warnx("=== managed window list ws %02d ===", r->ws->idx);
-
 	TAILQ_FOREACH(win, &r->ws->winlist, entry) {
 		state = getstate(win->id);
-		if (!XGetWindowAttributes(display, win->id, &wa))
-			warnx("window: 0x%lx, failed XGetWindowAttributes",
+		c = xcb_get_window_attributes(conn, win->id);
+		r = xcb_get_window_attributes_reply(conn, c, NULL);
+		if (r) {
+			warnx("window: 0x%x, map_state: %d, state: %u, "
+				"transient: 0x%x", win->id, wa.map_state,
+				state, win->transient);
+			free(r);
+		} else	
+			warnx("window: 0x%x, failed xcb_get_window_attributes",
 			    win->id);
-		warnx("window: 0x%lx, map_state: %d, state: %u, "
-		    "transient: 0x%lx", win->id, wa.map_state, state,
-		    win->transient);
 	}
 
 	warnx("===== unmanaged window list =====");
 	TAILQ_FOREACH(win, &r->ws->unmanagedlist, entry) {
 		state = getstate(win->id);
-		if (!XGetWindowAttributes(display, win->id, &wa))
+		c = xcb_get_window_attributes(conn, win->id);
+		r xcb_get_window_attributes_reply(conn, c, NULL);
+		if (r) {
+			warnx("window: 0x%lx, map_state: %d, state: %u, "
+				"transient: 0x%lx", win->id, wa.map_state,
+				state, win->transient); 
+			free(r);
+		} else
 			warnx("window: 0x%lx, failed XGetWindowAttributes",
 			    win->id);
-		warnx("window: 0x%lx, map_state: %d, state: %u, "
-		    "transient: 0x%lx", win->id, wa.map_state, state,
-		    win->transient);
 	}
 
 	warnx("=================================");
@@ -1502,10 +1511,11 @@ int		urgent[SWM_WS_MAX];
 void
 bar_urgent(char *s, size_t sz)
 {
-	XWMHints		*wmh = NULL;
 	struct ws_win		*win;
 	int			i, j, num_screens;
 	char			b[8];
+	xcb_get_property_cookie_t	c;
+	xcb_wm_hints_t		hints;
 
 	for (i = 0; i < workspace_limit; i++)
 		urgent[i] = 0;
@@ -1514,13 +1524,12 @@ bar_urgent(char *s, size_t sz)
 	for (i = 0; i < num_screens; i++)
 		for (j = 0; j < workspace_limit; j++)
 			TAILQ_FOREACH(win, &screens[i].ws[j].winlist, entry) {
-				wmh = XGetWMHints(display, win->id);
-				if (wmh == NULL)
+				c = xcb_get_wm_hints(conn, win->id);
+				if (xcb_get_wm_hints_reply(conn, c,
+						&hints, NULL) == 0)
 					continue;
-
-				if (wmh->flags & XUrgencyHint)
+				if (hints.flags & XCB_WM_HINT_X_URGENCY)
 					urgent[j] = 1;
-				XFree(wmh);
 			}
 
 	for (i = 0; i < workspace_limit; i++) {
@@ -6287,7 +6296,8 @@ manage_window(xcb_window_t id)
 	const char		*errstr;
 	struct pid_e		*p;
 	struct quirk		*qp;
-
+	uint32_t		event_mask;
+	
 	if ((win = find_window(id)) != NULL)
 		return (win);	/* already being managed */
 
@@ -6506,9 +6516,12 @@ manage_window(xcb_window_t id)
 		update_window(win);
 	}
 
-	XSelectInput(display, id, EnterWindowMask | FocusChangeMask |
-	    PropertyChangeMask | StructureNotifyMask);
+	event_mask = XCB_EVENT_MASK_ENTER_WINDOW | XCB_EVENT_MASK_FOCUS_CHANGE |
+			XCB_EVENT_MASK_PROPERTY_CHANGE |
+			XCB_EVENT_MASK_STRUCTURE_NOTIFY;
 
+	xcb_change_window_attributes(conn, id, XCB_CW_EVENT_MASK, &event_mask);
+	
 	/* floaters need to be mapped if they are in the current workspace */
 	if ((win->floating || win->transient) && (ws->idx == r->ws->idx))
 		map_window_raised(win->id);
@@ -7421,10 +7434,9 @@ void
 setup_screens(void)
 {
 	int			i, j, k, num_screens;
-	int			errorbase;
 	struct workspace	*ws;
 	XGCValues		gcv;
-
+	const xcb_query_extension_reply_t *qep;
 	xcb_randr_query_version_cookie_t	c;
 	xcb_randr_query_version_reply_t		*r;
 
@@ -7435,18 +7447,16 @@ setup_screens(void)
 		    "screens");
 
 	/* initial Xrandr setup */
-	xrandr_support = XRRQueryExtension(display,
-	    &xrandr_eventbase, &errorbase);
-	if (xrandr_support) {
-		c = xcb_randr_query_version(conn, True, False);
-		r = xcb_randr_query_version_reply(conn, c, NULL);
-		if (r) {
-			if (r->major_version < 1)
-				xrandr_support = 0;
-			free(r);
-		} else
-			xrandr_support = 0;
+	xrandr_support = False;
+	c = xcb_randr_query_version(conn, True, True);
+	r = xcb_randr_query_version_reply(conn, c, NULL);
+	if (r) {
+		if (r->major_version >= 1)
+			xrandr_support = True;
+		free(r);
 	}
+	qep = xcb_get_extension_data(conn, &xcb_randr_id);
+	xrandr_eventbase = qep->first_event;
 
 	/* map physical screens */
 	for (i = 0; i < num_screens; i++) {
@@ -7551,7 +7561,6 @@ main(int argc, char *argv[])
 	int			xfd, i, num_screens;
 	fd_set			rd;
 	struct sigaction	sact;
-	xcb_generic_event_t	*evt;
 	
 	start_argv = argv;
 	warnx("Welcome to spectrwm V%s Build: %s", SPECTRWM_VERSION, buildstr);
@@ -7673,7 +7682,7 @@ noconfig:
 
 	xfd = xcb_get_file_descriptor(conn); 
 	while (running) {
-		while ((evt = xcb_poll_for_event(conn)) == 0) {
+		while (XPending(display)) {
 			XNextEvent(display, &e);
 			if (running == 0)
 				goto done;
@@ -7692,7 +7701,7 @@ noconfig:
 				    xrandr_geteventname(&e), e.type);
 
 				switch (e.type - xrandr_eventbase) {
-				case RRScreenChangeNotify:
+				case XCB_RANDR_SCREEN_CHANGE_NOTIFY:
 					screenchange(&e);
 					break;
 				default: