if stdin writer stops working, dwm consumed much IO load because it still tried to...

[dwm.git] / main.c

diff --git a/main.c b/main.c
index 3d3e896..b59ff96 100644 (file)
--- a/main.c
+++ b/main.c
@@ -3,10 +3,12 @@
  * See LICENSE file for license details.
  */
 
  * See LICENSE file for license details.
  */
 

+#include <errno.h>

 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>

+#include <unistd.h>

 
 #include <X11/cursorfont.h>
 #include <X11/Xatom.h>
 
 #include <X11/cursorfont.h>
 #include <X11/Xatom.h>


@@ -19,7 +21,6 @@
 char *tags[TLast] = {
        [Tscratch] = "scratch",
        [Tdev] = "dev",
 char *tags[TLast] = {
        [Tscratch] = "scratch",
        [Tdev] = "dev",

-       [Tirc] = "irc",

        [Twww] = "www",
        [Twork] = "work",
 };
        [Twww] = "www",
        [Twork] = "work",
 };


@@ -185,13 +186,14 @@ quit(Arg *arg)
 int
 main(int argc, char *argv[])
 {
 int
 main(int argc, char *argv[])
 {

-       int i;
+       int i, n;
+       fd_set rd;

        XSetWindowAttributes wa;
        unsigned int mask;
        XSetWindowAttributes wa;
        unsigned int mask;

+       Bool readstdin = True;

        Window w;
        XEvent ev;
 
        Window w;
        XEvent ev;
 

-       /* command line args */

        for(i = 1; (i < argc) && (argv[i][0] == '-'); i++) {
                switch (argv[i][1]) {
                case 'v':
        for(i = 1; (i < argc) && (argv[i][0] == '-'); i++) {
                switch (argv[i][1]) {
                case 'v':


@@ -263,6 +265,10 @@ main(int argc, char *argv[])
        XDefineCursor(dpy, barwin, cursor[CurNormal]);
        XMapRaised(dpy, barwin);
 
        XDefineCursor(dpy, barwin, cursor[CurNormal]);
        XMapRaised(dpy, barwin);
 

+       dc.drawable = XCreatePixmap(dpy, root, sw, bh, DefaultDepth(dpy, screen));
+       dc.gc = XCreateGC(dpy, root, 0, 0);
+       draw_bar();
+

        issel = XQueryPointer(dpy, root, &w, &w, &i, &i, &i, &i, &mask);
 
        wa.event_mask = SubstructureRedirectMask | EnterWindowMask \
        issel = XQueryPointer(dpy, root, &w, &w, &i, &i, &i, &i, &mask);
 
        wa.event_mask = SubstructureRedirectMask | EnterWindowMask \


@@ -271,17 +277,47 @@ main(int argc, char *argv[])
 
        XChangeWindowAttributes(dpy, root, CWEventMask | CWCursor, &wa);
 
 
        XChangeWindowAttributes(dpy, root, CWEventMask | CWCursor, &wa);
 

-       dc.drawable = XCreatePixmap(dpy, root, sw, bh, DefaultDepth(dpy, screen));
-       dc.gc = XCreateGC(dpy, root, 0, 0);
-

        strcpy(stext, "dwm-"VERSION);
        scan_wins();
        strcpy(stext, "dwm-"VERSION);
        scan_wins();

-       draw_bar();

 
 

+       /* main event loop, reads status text from stdin as well */
+Mainloop:

        while(running) {
        while(running) {

-               XNextEvent(dpy, &ev);
-               if(handler[ev.type])
-                       (handler[ev.type])(&ev); /* call handler */
+               FD_ZERO(&rd);
+               if(readstdin)
+                       FD_SET(STDIN_FILENO, &rd);
+               FD_SET(ConnectionNumber(dpy), &rd);
+
+               i = select(ConnectionNumber(dpy) + 1, &rd, 0, 0, 0);
+               if(i == -1 && errno == EINTR)
+                       continue;
+               if(i < 0)
+                       error("select failed\n");
+               else if(i > 0) {
+                       if(FD_ISSET(ConnectionNumber(dpy), &rd)) {
+                               while(XPending(dpy)) {
+                                       XNextEvent(dpy, &ev);
+                                       if(handler[ev.type])
+                                               (handler[ev.type])(&ev); /* call handler */
+                               }
+                       }
+                       if(readstdin && FD_ISSET(STDIN_FILENO, &rd)) {
+                               i = n = 0;
+                               for(;;) {
+                                       if((i = getchar()) == EOF) {
+                                               /* broken pipe/end of producer */
+                                               readstdin = False;
+                                               strcpy(stext, "broken pipe");
+                                               goto Mainloop;
+                                       }
+                                       if(i == '\n' || n >= sizeof(stext) - 1)
+                                               break;
+                                       stext[n++] = i;
+                               }
+                               stext[n] = 0;
+                               draw_bar();
+                       }
+               }

        }
 
        cleanup();
        }
 
        cleanup();