Added ifndef for openbsd compatibility. SIGRTMIN is not defined on

[dwmblocks.git] / dwmblocks.c

diff --git a/dwmblocks.c b/dwmblocks.c
index 2db3489144bb57c5bccb6e96d53113ac6c3e5db4..88bdfb0b08f23616f70b6e2b08c643824eeea3a3 100644 (file)
--- a/dwmblocks.c
+++ b/dwmblocks.c
@@ -5,6 +5,7 @@
 #include<signal.h>
 #include<X11/Xlib.h>
 #define LENGTH(X)               (sizeof(X) / sizeof (X[0]))
+#define CMDLENGTH              50
 
 typedef struct {
        char* icon;
@@ -15,13 +16,14 @@ typedef struct {
 void sighandler(int num);
 void replace(char *str, char old, char new);
 void getcmds(int time);
+#ifndef __OpenBSD__
 void getsigcmds(int signal);
 void setupsignals();
-void getstatus(char *str);
+void sighandler(int signum);
+#endif
+int getstatus(char *str, char *last);
 void setroot();
 void statusloop();
-void statusinit();
-void sighandler(int signum);
 void termhandler(int signum);
 
 
@@ -30,13 +32,10 @@ void termhandler(int signum);
 static Display *dpy;
 static int screen;
 static Window root;
-static char statusbar[LENGTH(blocks)][50] = {0};
-static char statusstr[256];
-static char *statuscat;
-static const char *volupcmd[]  = { "volup", NULL };
-static const char *voldowncmd[]  = { "voldown", NULL };
-static const char *volmutecmd[]  = { "volmute", NULL };
-static int statusContinue = 1,volmuted = 0;
+static char statusbar[LENGTH(blocks)][CMDLENGTH] = {0};
+static char statusstr[2][256];
+static int statusContinue = 1;
+static void (*writestatus) () = setroot;
 
 void replace(char *str, char old, char new)
 {
@@ -54,11 +53,10 @@ void getcmd(const Block *block, char *output)
        FILE *cmdf = popen(cmd,"r");
        if (!cmdf)
                return;
-       //int N = strlen(output);
        char c;
        int i = strlen(block->icon);
-       while((c = fgetc(cmdf)) != EOF)
-               output[i++] = c;
+       fgets(output+i, CMDLENGTH-i, cmdf);
+       i = strlen(output);
        if (delim != '\0' && --i)
                output[i++] = delim;
        output[i++] = '\0';
@@ -76,6 +74,7 @@ void getcmds(int time)
        }
 }
 
+#ifndef __OpenBSD__
 void getsigcmds(int signal)
 {
        const Block *current;
@@ -96,57 +95,64 @@ void setupsignals()
        }
 
 }
+#endif
 
-void getstatus(char *str)
+int getstatus(char *str, char *last)
 {
-       int j = 0;
-       for(int i = 0; i < LENGTH(blocks); j+=strlen(statusbar[i++]))
-       {       
-               strcpy(str + j, statusbar[i]);
-       }
-       str[--j] = '\0';
-
+       strcpy(last, str);
+       str[0] = '\0';
+       for(int i = 0; i < LENGTH(blocks); i++)
+               strcat(str, statusbar[i]);
+       str[strlen(str)-1] = '\0';
+       return strcmp(str, last);//0 if they are the same
 }
 
 void setroot()
 {
+       if (!getstatus(statusstr[0], statusstr[1]))//Only set root if text has changed.
+               return;
        Display *d = XOpenDisplay(NULL);
        if (d) {
                dpy = d;
        }
        screen = DefaultScreen(dpy);
        root = RootWindow(dpy, screen);
-       getstatus(statusstr);
-       XStoreName(dpy, root, statusstr);
+       XStoreName(dpy, root, statusstr[0]);
        XCloseDisplay(dpy);
 }
 
+void pstdout()
+{
+       if (!getstatus(statusstr[0], statusstr[1]))//Only write out if text has changed.
+               return;
+       printf("%s\n",statusstr[0]);
+       fflush(stdout);
+}
+
 
 void statusloop()
 {
+#ifndef __OpenBSD__
        setupsignals();
+#endif
        int i = 0;
        getcmds(-1);
        while(statusContinue)
        {
                getcmds(i);
-               setroot();
+               writestatus();
                sleep(1.0);
                i++;
        }
 }
 
-void statusinit()
-{
-       statusloop();
-}
-
-
+#ifndef __OpenBSD__
 void sighandler(int signum)
 {
        getsigcmds(signum-SIGRTMIN);
-       setroot();
+       writestatus();
 }
+#endif
 
 void termhandler(int signum)
 {
@@ -160,8 +166,10 @@ int main(int argc, char** argv)
        {       
                if (!strcmp("-d",argv[i]))
                        delim = argv[++i][0];
+               else if(!strcmp("-p",argv[i]))
+                       writestatus = pstdout;
        }
        signal(SIGTERM, termhandler);
        signal(SIGINT, termhandler);
-       statusinit();
+       statusloop();
 }