Reduce refresh time

[dwmblocks.git] / dwmblocks.c

diff --git a/dwmblocks.c b/dwmblocks.c
index e3ac0348c34b0512b3415a0713ff1d2fd36c87d6..e6c20dd37d44a682a80cda47c56afd2079abfaae 100644 (file)
--- a/dwmblocks.c
+++ b/dwmblocks.c
@@ -3,7 +3,9 @@
 #include<string.h>
 #include<unistd.h>
 #include<signal.h>
+#ifndef NO_X
 #include<X11/Xlib.h>
+#endif
 #ifdef __OpenBSD__
 #define SIGPLUS                        SIGUSR1+1
 #define SIGMINUS               SIGUSR1-1
@@ -31,45 +33,52 @@ void getsigcmds(unsigned int signal);
 void setupsignals();
 void sighandler(int signum);
 int getstatus(char *str, char *last);
-void setroot();
 void statusloop();
 void termhandler();
-
-
-#include "blocks.h"
-
+void pstdout();
+#ifndef NO_X
+void setroot();
+static void (*writestatus) () = setroot;
+static int setupX();
 static Display *dpy;
 static int screen;
 static Window root;
+#else
+static void (*writestatus) () = pstdout;
+#endif
+
+
+#include "config.h"
+
 static char statusbar[LENGTH(blocks)][CMDLENGTH] = {0};
 static char statusstr[2][STATUSLENGTH];
 static int statusContinue = 1;
-static void (*writestatus) () = setroot;
+static int returnStatus = 0;
 
 //opens process *cmd and stores output in *output
 void getcmd(const Block *block, char *output)
 {
        strcpy(output, block->icon);
-       char *cmd = block->command;
-       FILE *cmdf = popen(cmd,"r");
+       FILE *cmdf = popen(block->command, "r");
        if (!cmdf)
                return;
-       char c;
        int i = strlen(block->icon);
-       fgets(output+i, CMDLENGTH-i-delimLen, cmdf);
+       fgets(output+i, CMDLENGTH-i, cmdf);
        i = strlen(output);
-       if (delim[0] != '\0' && --i)
-                strncpy(output+i, delim, delimLen); 
-       else
-               output[i++] = '\0';
+       if (i == 0) {
+               //return if block and command output are both empty
+               pclose(cmdf);
+               return;
+       } else if (output[i-1] == '\n') {
+               output[i-1] = '\0';
+       }
        pclose(cmdf);
 }
 
 void getcmds(int time)
 {
        const Block* current;
-       for(unsigned int i = 0; i < LENGTH(blocks); i++)
-       {
+       for (unsigned int i = 0; i < LENGTH(blocks); i++) {
                current = blocks + i;
                if ((current->interval != 0 && time % current->interval == 0) || time == -1)
                        getcmd(current,statusbar[i]);
@@ -79,8 +88,7 @@ void getcmds(int time)
 void getsigcmds(unsigned int signal)
 {
        const Block *current;
-       for (unsigned int i = 0; i < LENGTH(blocks); i++)
-       {
+       for (unsigned int i = 0; i < LENGTH(blocks); i++) {
                current = blocks + i;
                if (current->signal == signal)
                        getcmd(current,statusbar[i]);
@@ -91,12 +99,11 @@ void setupsignals()
 {
 #ifndef __OpenBSD__
            /* initialize all real time signals with dummy handler */
-    for(int i = SIGRTMIN; i <= SIGRTMAX; i++)
+    for (int i = SIGRTMIN; i <= SIGRTMAX; i++)
         signal(i, dummysighandler);
 #endif
 
-       for(unsigned int i = 0; i < LENGTH(blocks); i++)
-       {
+       for (unsigned int i = 0; i < LENGTH(blocks); i++) {
                if (blocks[i].signal > 0)
                        signal(SIGMINUS+blocks[i].signal, sighandler);
        }
@@ -107,25 +114,33 @@ int getstatus(char *str, char *last)
 {
        strcpy(last, str);
        str[0] = '\0';
-       for(unsigned int i = 0; i < LENGTH(blocks); i++)
+       for (unsigned int i = 0; i < LENGTH(blocks); i++)
                strcat(str, statusbar[i]);
-       str[strlen(str)-strlen(delim)] = '\0';
+       str[strlen(str)] = '\0';
        return strcmp(str, last);//0 if they are the same
 }
 
+#ifndef NO_X
 void setroot()
 {
        if (!getstatus(statusstr[0], statusstr[1]))//Only set root if text has changed.
                return;
-       Display *d = XOpenDisplay(NULL);
-       if (d) {
-               dpy = d;
+       XStoreName(dpy, root, statusstr[0]);
+       XFlush(dpy);
+}
+
+int setupX()
+{
+       dpy = XOpenDisplay(NULL);
+       if (!dpy) {
+               fprintf(stderr, "dwmblocks: Failed to open display\n");
+               return 0;
        }
        screen = DefaultScreen(dpy);
        root = RootWindow(dpy, screen);
-       XStoreName(dpy, root, statusstr[0]);
-       XCloseDisplay(dpy);
+       return 1;
 }
+#endif
 
 void pstdout()
 {
@@ -141,12 +156,12 @@ void statusloop()
        setupsignals();
        int i = 0;
        getcmds(-1);
-       while(statusContinue)
-       {
-               getcmds(i);
+       while (1) {
+               getcmds(i++);
                writestatus();
+               if (!statusContinue)
+                       break;
                sleep(1.0);
-               i++;
        }
 }
 
@@ -171,15 +186,19 @@ void termhandler()
 
 int main(int argc, char** argv)
 {
-       for(int i = 0; i < argc; i++)
-       {
-               if (!strcmp("-d",argv[i]))
-                       strncpy(delim, argv[++i], delimLen);
-               else if(!strcmp("-p",argv[i]))
+       for (int i = 0; i < argc; i++) {//Handle command line arguments
+               if (!strcmp("-p",argv[i]))
                        writestatus = pstdout;
        }
-       delim[MIN(delimLen, strlen(delim))] = '\0';
+#ifndef NO_X
+       if (!setupX())
+               return 1;
+#endif
        signal(SIGTERM, termhandler);
        signal(SIGINT, termhandler);
        statusloop();
+#ifndef NO_X
+       XCloseDisplay(dpy);
+#endif
+       return 0;
 }