Made dwmblocks way more efficient by only setting the name of the root window when...

[dwmblocks.git] / dwmblocks.c

diff --git a/dwmblocks.c b/dwmblocks.c
index 76e7377a9a84f233e78a69b714305ffa22c7e9f3..71760aa8f9aa2b1551915f3e22ef2471a625e5ad 100644 (file)
--- a/dwmblocks.c
+++ b/dwmblocks.c
@@ -17,7 +17,7 @@ void replace(char *str, char old, char new);
 void getcmds(int time);
 void getsigcmds(int signal);
 void setupsignals();
-void getstatus(char *str);
+int getstatus(char *str, char *last);
 void setroot();
 void statusloop();
 void statusinit();
@@ -31,12 +31,8 @@ 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 statusstr[2][256];
+static int statusContinue = 1;
 
 void replace(char *str, char old, char new)
 {
@@ -56,20 +52,13 @@ void getcmd(const Block *block, char *output)
                return;
        //int N = strlen(output);
        char c;
-       int sl;
-       int i = sl = strlen(block->icon);
+       int i = strlen(block->icon);
        while((c = fgetc(cmdf)) != EOF)
                output[i++] = c;
+       if (delim != '\0' && --i)
+               output[i++] = delim;
+       output[i++] = '\0';
        pclose(cmdf);
-       if (i == sl) { //return empty string if command has no output
-               output[0] = '\0';
-               return;
-       }
-       if (delim[0] != '\0' && --i)
-               for (int j = 0; delim[j]; j++)
-                       output[i++] = delim[j];
-       else 
-               output[--i] = '\0';
 }
 
 void getcmds(int time)
@@ -104,28 +93,29 @@ void setupsignals()
 
 }
 
-void getstatus(char *str)
+int getstatus(char *str, char *last)
 {
+       strcpy(last, str);
        int j = 0;
        for(int i = 0; i < LENGTH(blocks); j+=strlen(statusbar[i++]))
        {       
                strcpy(str + j, statusbar[i]);
        }
-       for (int i = 0; delim[i]; i++)
-               str[--j] = '\0';
-
+       str[--j] = '\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);
 }
 
@@ -167,7 +157,7 @@ int main(int argc, char** argv)
        for(int i = 0; i < argc; i++)
        {       
                if (!strcmp("-d",argv[i]))
-                       delim = argv[++i];
+                       delim = argv[++i][0];
        }
        signal(SIGTERM, termhandler);
        signal(SIGINT, termhandler);