X-Git-Url: https://jasonwoof.com/gitweb/?a=blobdiff_plain;f=spectrwm.c;h=c81bb2ae3523f0bed7eb1c9afc7643e615220b5b;hb=1542089bc8c7a1eadb5baf5d946d0b9b860e1360;hp=af2fe15d36c8614171d1bfe7fbccddc3888db080;hpb=c2a4935a927a5bd0fcd4b9d086bf2e07c2155122;p=spectrwm.git

diff --git a/spectrwm.c b/spectrwm.c
index af2fe15..c81bb2a 100644
--- a/spectrwm.c
+++ b/spectrwm.c
@@ -279,7 +279,6 @@ volatile sig_atomic_t   restart_wm = 0;
 xcb_timestamp_t		last_event_time = 0;
 int			outputs = 0;
 int			other_wm;
-int			ss_enabled = 0;
 int			xrandr_support;
 int			xrandr_eventbase;
 unsigned int		numlockmask = 0;
@@ -351,8 +350,6 @@ int		 bar_pipe[2];
 char		 bar_ext[SWM_BAR_MAX];
 char		 bar_vertext[SWM_BAR_MAX];
 int		 bar_version = 0;
-sig_atomic_t	 bar_alarm = 0;
-int		 bar_delay = 30;
 int		 bar_enabled = 1;
 int		 bar_border_width = 1;
 int		 bar_at_bottom = 0;
@@ -834,7 +831,6 @@ void	 bar_replace_pad(char *, int *, size_t);
 char *	 bar_replace_seq(char *, char *, struct swm_region *, size_t *,
 	     size_t);
 void	 bar_setup(struct swm_region *);
-void	 bar_signal(int);
 void	 bar_title_name(char *, size_t, struct swm_region *);
 void	 bar_toggle(struct swm_region *, union arg *);
 void	 bar_update(void);
@@ -2203,10 +2199,8 @@ bar_update(void)
 	size_t			len;
 	char			*b;
 
-	if (!bar_enabled)
-		return;
-	if (bar_extra && bar_extra_running) {
-		/* ignore short reads; it'll correct itself */
+	if (bar_enabled && bar_extra && bar_extra_running) {
+		/* Ignore short reads; it'll correct itself. */
 		while ((b = fgetln(stdin, &len)) != NULL)
 			if (b && b[len - 1] == '\n') {
 				b[len - 1] = '\0';
@@ -2216,20 +2210,20 @@ bar_update(void)
 			warn("bar_update: bar_extra failed");
 			bar_extra_stop();
 		}
-	} else
-		strlcpy(bar_ext, "", sizeof bar_ext);
+	} else {
+		/*
+		 * Attempt to drain stdin, so it doesn't cause the main loop to
+		 * call us as fast as it can.
+		 */
+		fgetln(stdin, &len);
 
-	bar_fmt_print();
-	alarm(bar_delay);
-}
+		if (!bar_enabled)
+			return;
 
-void
-bar_signal(int sig)
-{
-	/* suppress unused warning since var is needed */
-	(void)sig;
+		bar_ext[0] = '\0';
+	}
 
-	bar_alarm = 1;
+	bar_fmt_print();
 }
 
 void
@@ -2249,6 +2243,8 @@ bar_toggle(struct swm_region *r, union arg *args)
 		/* Only change if master switch is enabled. */
 		if (bar_enabled)
 			r->ws->bar_enabled = !r->ws->bar_enabled;
+		else
+			bar_enabled = r->ws->bar_enabled = 1;
 		break;
 	case SWM_ARG_ID_BAR_TOGGLE:
 		bar_enabled = !bar_enabled;
@@ -2289,10 +2285,15 @@ bar_refresh(void)
 			err(1, "pipe error");
 		socket_setnonblock(bar_pipe[0]);
 		socket_setnonblock(bar_pipe[1]); /* XXX hmmm, really? */
+
+		/* Set stdin to read from the pipe. */
 		if (dup2(bar_pipe[0], 0) == -1)
 			err(1, "dup2");
+
+		/* Set stdout to write to the pipe. */
 		if (dup2(bar_pipe[1], 1) == -1)
 			err(1, "dup2");
+
 		if (signal(SIGPIPE, SIG_IGN) == SIG_ERR)
 			err(1, "could not disable SIGPIPE");
 		switch (bar_pid = fork()) {
@@ -2320,7 +2321,9 @@ bar_refresh(void)
 			xcb_change_window_attributes(conn, r->bar->id,
 			    XCB_CW_BACK_PIXEL | XCB_CW_BORDER_PIXEL, wa);
 		}
+
 	bar_update();
+	xcb_flush(conn);
 }
 
 int
@@ -2486,8 +2489,6 @@ bar_setup(struct swm_region *r)
 	    "%d x %d\n", WINID(r->bar), X(r->bar), Y(r->bar), WIDTH(r->bar),
 	    HEIGHT(r->bar));
 
-	if (signal(SIGALRM, bar_signal) == SIG_ERR)
-		err(1, "could not install bar_signal");
 	bar_refresh();
 }
 
@@ -6433,6 +6434,7 @@ enum {
 	SWM_S_BAR_BORDER_WIDTH,
 	SWM_S_BAR_DELAY,
 	SWM_S_BAR_ENABLED,
+	SWM_S_BAR_ENABLED_WS,
 	SWM_S_BAR_FONT,
 	SWM_S_BAR_FORMAT,
 	SWM_S_BAR_JUSTIFY,
@@ -6464,8 +6466,9 @@ enum {
 int
 setconfvalue(char *selector, char *value, int flags)
 {
-	int	i;
-	char	*b;
+	struct workspace	*ws;
+	int			i, ws_id, num_screens;
+	char			*b;
 
 	/* suppress unused warning since var is needed */
 	(void)selector;
@@ -6485,11 +6488,23 @@ setconfvalue(char *selector, char *value, int flags)
 			bar_border_width = 0;
 		break;
 	case SWM_S_BAR_DELAY:
-		bar_delay = atoi(value);
+		/* No longer needed; leave to not break old conf files. */
 		break;
 	case SWM_S_BAR_ENABLED:
 		bar_enabled = atoi(value);
 		break;
+	case SWM_S_BAR_ENABLED_WS:
+		ws_id = atoi(selector) - 1;
+		if (ws_id < 0 || ws_id >= workspace_limit)
+			errx(1, "setconfvalue: bar_enabled_ws: invalid "
+			    "workspace %d.", ws_id + 1);
+
+		num_screens = xcb_setup_roots_length(xcb_get_setup(conn));
+		for (i = 0; i < num_screens; i++) {
+			ws = (struct workspace *)&screens[i].ws;
+			ws[ws_id].bar_enabled = atoi(value);
+		}
+		break;
 	case SWM_S_BAR_FONT:
 		b = bar_fonts;
 		if (asprintf(&bar_fonts, "%s,%s", value, bar_fonts) == -1)
@@ -6606,9 +6621,10 @@ setconfvalue(char *selector, char *value, int flags)
 		setconfspawn("spawn_term", value, 0);
 		break;
 	case SWM_S_SS_APP:
+		/* No longer needed; leave to not break old conf files. */
 		break;
 	case SWM_S_SS_ENABLED:
-		ss_enabled = atoi(value);
+		/* No longer needed; leave to not break old conf files. */
 		break;
 	case SWM_S_STACK_ENABLED:
 		stack_enabled = atoi(value);
@@ -6855,6 +6871,7 @@ struct config_option {
 };
 struct config_option configopt[] = {
 	{ "bar_enabled",		setconfvalue,	SWM_S_BAR_ENABLED },
+	{ "bar_enabled_ws",		setconfvalue,	SWM_S_BAR_ENABLED_WS },
 	{ "bar_at_bottom",		setconfvalue,	SWM_S_BAR_AT_BOTTOM },
 	{ "bar_border",			setconfcolor,	SWM_S_COLOR_BAR_BORDER },
 	{ "bar_border_width",		setconfvalue,	SWM_S_BAR_BORDER_WIDTH },
@@ -8821,6 +8838,9 @@ main(int argc, char *argv[])
 	xcb_generic_event_t	*evt;
 	struct timeval          tv;
 	fd_set			rd;
+	int			rd_max;
+	int			do_bar_update = 0;
+	int			num_readable;
 
 	/* suppress unused warning since var is needed */
 	(void)argc;
@@ -8951,6 +8971,8 @@ noconfig:
 	xcb_ungrab_server(conn);
 	xcb_flush(conn);
 
+	rd_max = xfd > STDIN_FILENO ? xfd : STDIN_FILENO;
+
 	while (running) {
 		while ((evt = xcb_poll_for_event(conn))) {
 			if (!running)
@@ -8975,22 +8997,29 @@ noconfig:
 		}
 
 		FD_ZERO(&rd);
+		FD_SET(STDIN_FILENO, &rd);
 		FD_SET(xfd, &rd);
 		tv.tv_sec = 1;
 		tv.tv_usec = 0;
-		if (select(xfd + 1, &rd, NULL, NULL, &tv) == -1)
-			if (errno != EINTR) {
-				DNPRINTF(SWM_D_MISC, "select failed");
-			}
+		num_readable = select(rd_max + 1, &rd, NULL, NULL, &tv);
+		if (num_readable == -1 && errno != EINTR)
+			DNPRINTF(SWM_D_MISC, "select failed");
+		else if (num_readable > 0 && FD_ISSET(STDIN_FILENO, &rd))
+			do_bar_update = 1;
+
 		if (restart_wm)
 			restart(NULL, NULL);
+
 		if (search_resp)
 			search_do_resp();
+
 		if (!running)
 			goto done;
-		if (bar_alarm) {
-			bar_alarm = 0;
+
+		if (do_bar_update) {
+			do_bar_update = 0;
 			bar_update();
+			xcb_flush(conn);
 		}
 	}
 done: