all repos — slstatus @ 57f69e900bf7e3e7c60a720bf09f2134b724c5cc

my build of slstatus (tools.suckless.org/slstatus/) 

Remove o-flag as it is redundant

Even if we extend the flag to be an n-flag with a numerical input, we
already have other Unix-tools to take care of us much more flexibly,
e.g. with head(1) you can easily get the first 10 outputs with

	slstatus -s | head -n 10,

but also e.g. discard the first one and then get the consecutive 10
outputs.

For the X11-root-window-name, you can limit the runtime with timeout(1)
or a comparable solution.

This reverts commit fa8b939990ed0648f99b0cbeba4efed807160db3.
Laslo Hunhold dev@frign.de
Mon, 21 May 2018 21:31:53 +0200
commit

57f69e900bf7e3e7c60a720bf09f2134b724c5cc

parent

9c9ce8a125f35bab61c829bdd58bdf77b8352d5e

2 files changed, 4 insertions(+), 13 deletions(-)

jump to
M slstatus.1 → slstatus.1 
@@ -6,7 +6,7 @@ .Nm slstatus
 .Nd suckless status monitor
 .Sh SYNOPSIS
 .Nm
-.Op Fl so
+.Op Fl s
 .Sh DESCRIPTION
 .Nm
 is a suckless status monitor for window managers that use WM_NAME (e.g. dwm) or

@@ -18,8 +18,6 @@ .Sh OPTIONS
 .Bl -tag -width Ds
 .It Fl s
 Write to stdout instead of WM_NAME.
-.It Fl o
-Write only once and exit.
 .El
 .Sh CUSTOMIZATION
 .Nm
M slstatus.c → slstatus.c 
@@ -42,7 +42,7 @@ 
 static void
 usage(void)
 {
-	die("usage: %s [-so]", argv0);
+	die("usage: %s [-s]", argv0);
 }
 
 int

@@ -51,17 +51,14 @@ {
 	struct sigaction act;
 	struct timespec start, current, diff, intspec, wait;
 	size_t i, len;
-	int sflag, oflag, ret;
+	int sflag, ret;
 	char status[MAXLEN];
 	const char *res;
 
-	sflag = oflag = 0;
+	sflag = 0;
 	ARGBEGIN {
 		case 's':
 			sflag = 1;
-			break;
-		case 'o':
-			oflag = 1;
 			break;
 		default:
 			usage();

@@ -110,10 +107,6 @@ if (XStoreName(dpy, DefaultRootWindow(dpy), status) < 0) {
 				die("XStoreName: Allocation failed");
 			}
 			XFlush(dpy);
-		}
-
-		if (oflag) {
-			done = 1;
 		}
 
 		if (!done) {