removed setstatus() && simplified main()

[slstatus.git] / slstatus.c

diff --git a/slstatus.c b/slstatus.c
index 7c625d1..2e57fbb 100644 (file)
--- a/slstatus.c
+++ b/slstatus.c
@@ -27,14 +27,12 @@
 #include "strlcat.h"
 #include "strlcpy.h"
 
-typedef char *(*op_fun)();
 struct arg {
-       op_fun func;
+       char *(*func)();
        const char *format;
        const char *args;
 };
 
-static void setstatus(const char *);
 static char *smprintf(const char *, ...);
 static char *battery_perc(const char *);
 static char *cpu_perc(void);
@@ -65,14 +63,6 @@ static Display *dpy;
 
 #include "config.h"
 
-static void
-setstatus(const char *str)
-{
-       /* set WM_NAME via X11 */
-       XStoreName(dpy, DefaultRootWindow(dpy), str);
-       XSync(dpy, False);
-}
-
 static char *
 smprintf(const char *fmt, ...)
 {
@@ -263,8 +253,7 @@ entropy(void)
 static char *
 gid(void)
 {
-       gid_t gid = getgid();
-       return smprintf("%d", gid);
+       return smprintf("%d", getgid());
 }
 
 static char *
@@ -475,45 +464,31 @@ uptime(void)
 static char *
 username(void)
 {
-       register struct passwd *pw;
-       register uid_t uid;
-
-       uid = geteuid();
-       pw = getpwuid(uid);
+       uid_t uid = geteuid();
+       struct passwd *pw = getpwuid(uid);
 
-       if (pw)
+       if (pw == NULL)
                return smprintf("%s", pw->pw_name);
-       else {
-               fprintf(stderr, "Could not get username: %s\n",
-                                       strerror(errno));
-               return smprintf(UNKNOWN_STR);
-       }
 
+       fprintf(stderr, "Could not get username: %s\n",
+                                       strerror(errno));
        return smprintf(UNKNOWN_STR);
 }
 
 static char *
 uid(void)
 {
-       /* FIXME: WHY USE register modifier? */
-       register uid_t uid;
-
-       uid = geteuid();
-
-       if (uid)
-               return smprintf("%d", uid);
-       else {
-               fprintf(stderr, "Could not get uid.\n");
-               return smprintf(UNKNOWN_STR);
-       }
-
-       return smprintf(UNKNOWN_STR);
+       return smprintf("%d", geteuid());
 }
 
 
 static char *
 vol_perc(const char *soundcard)
 {
+       /*
+        * TODO: FIXME: 
+        * https://github.com/drkh5h/slstatus/issues/12
+        */
        int mute = 0;
        long vol = 0, max = 0, min = 0;
        snd_mixer_t *handle;
@@ -641,10 +616,6 @@ main(void)
        struct arg argument;
 
        dpy = XOpenDisplay(0x0);
-       if (!dpy) {
-               fprintf(stderr, "Cannot open display!\n");
-               exit(1);
-       }
 
        for (;;) {
                memset(status_string, 0, sizeof(status_string));
@@ -664,10 +635,16 @@ main(void)
                        free(element);
                }
 
-               setstatus(status_string);
+               XStoreName(dpy, DefaultRootWindow(dpy), status_string);
+               XSync(dpy, False);
                sleep(UPDATE_INTERVAL -1);
        }
 
+       /* NOT REACHED */
+       /*
+        * TODO: find out a way to exit successfully
+        * to prevent memory leaks
+        */
        XCloseDisplay(dpy);
        return 0;
 }