From 4e1d114f4e4bba88e8a9728acc32ee097ee9091c Mon Sep 17 00:00:00 2001
From: Bill Kolokithas <kolokithas.b@gmail.com>
Date: Tue, 4 Feb 2014 22:28:01 +0200
Subject: [PATCH 1/2] check return value of fgets and exit main loop if EOF is
 received

---
 bar.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/bar.c b/bar.c
index 6c244fb..bea1829 100644
--- a/bar.c
+++ b/bar.c
@@ -568,7 +568,9 @@ main (int argc, char **argv)
                 else           break;               /* ...bail out */
             }
             if (pollin[0].revents & POLLIN) { /* New input, process it */
-                fgets (input, sizeof(input), stdin);
+                if (fgets (input, sizeof(input), stdin) == NULL)
+                    break; /* EOF received */
+
                 parse (input);
                 redraw = 1;
             }

From 9cdaa22e0608fe41ff463dcb9d5ddc62238716c6 Mon Sep 17 00:00:00 2001
From: Bill Kolokithas <kolokithas.b@gmail.com>
Date: Tue, 4 Feb 2014 22:35:54 +0200
Subject: [PATCH 2/2] styling change: remove uneeded parentheses around
 dereferenced pointer

---
 bar.c | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/bar.c b/bar.c
index bea1829..29676b9 100644
--- a/bar.c
+++ b/bar.c
@@ -155,41 +155,39 @@ parse (char *text)
     xcb_fill_rect (clear_gc, 0, 0, bar_width, BAR_HEIGHT);
 
     for (;;) {
-        if (*p == '\0')
-            return;
-        if (*p == '\n')
+        if (*p == '\0' || *p == '\n')
             return;
 
         if (*p == '\\' && p++ && *p != '\\' && strchr (control_characters, *p)) {
                 switch (*p++) {
                     case 'f': 
-                        xcb_set_fg (isdigit(*p) ? (*p)-'0' : 11);
+                        xcb_set_fg (isdigit(*p) ? *p-'0' : 11);
                         p++;
                         break;
                     case 'b': 
-                        xcb_set_bg (isdigit(*p) ? (*p)-'0' : 10);
+                        xcb_set_bg (isdigit(*p) ? *p-'0' : 10);
                         p++;
                         break;
                     case 'u': 
-                        xcb_set_ud (isdigit(*p) ? (*p)-'0' : 10);
+                        xcb_set_ud (isdigit(*p) ? *p-'0' : 10);
                         p++;
                         break;
 #if XINERAMA
                     case 's':
-                        if ((*p) == 'r') {
+                        if (*p == 'r') {
                             screen = &screens[num_screens - 1];
-                        } else if ((*p) == 'l') {
+                        } else if (*p == 'l') {
                             screen = &screens[0];
-                        } else if ((*p) == 'n') {
+                        } else if (*p == 'n') {
                             if (screen == &screens[num_screens - 1])
                                 break;
                             screen++;
-                        } else if ((*p) == 'p') {
+                        } else if (*p == 'p') {
                             if (screen == screens)
                                 break;
                             screen--;
                         } else if (isdigit(*p)) {
-                            int index = (*p)-'0';
+                            int index = *p-'0';
                             if (index < num_screens) {
                                 screen = &screens[index];
                             } else {