dmenu

my fork of dmenu
Index Commits Files Refs README LICENSE
commit a45f106d3c6e253109dd47b6a193f991c7bde203
parent 596033b7818f5ff126ec6c62c375afc4c1e32dda
Author: Anselm R Garbe <garbeam@gmail.com>
Date:   Fri, 13 Jun 2008 11:46:50 +0100

cosmetic fixes
Diffstat:
MLICENSE | 18+++++++++---------
Mconfig.h | 4++--
Mconfig.mk | 4+---
Mdmenu.c | 130++++++++++++++++++++++++++++++++++++++-----------------------------------------
4 files changed, 74 insertions(+), 82 deletions(-)
diff --git a/LICENSE b/LICENSE
@@ -8,16 +8,16 @@ Permission is hereby granted, free of charge, to any person obtaining a
 copy of this software and associated documentation files (the "Software"),
 to deal in the Software without restriction, including without limitation
 the rights to use, copy, modify, merge, publish, distribute, sublicense,
-and/or sell copies of the Software, and to permit persons to whom the 
+and/or sell copies of the Software, and to permit persons to whom the
 Software is furnished to do so, subject to the following conditions:
 
-The above copyright notice and this permission notice shall be included in 
-all copies or substantial portions of the Software. 
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
 
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR 
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, 
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL 
-THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER 
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING 
-FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER 
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
 DEALINGS IN THE SOFTWARE.
diff --git a/config.h b/config.h
@@ -6,5 +6,5 @@
 #define NORMFGCOLOR             "#000000"
 #define SELBGCOLOR              "#0066ff"
 #define SELFGCOLOR              "#ffffff"
-/* next macro defines the space between menu items */
-#define SPACE            30 /* px */
+
+static uint spaceitem           = 30; /* px between menu items */
diff --git a/config.mk b/config.mk
@@ -20,10 +20,8 @@ LIBS = -L/usr/lib -lc -L${X11LIB} -lX11 ${XINERAMALIBS}
 
 # flags
 CPPFLAGS = -DVERSION=\"${VERSION}\" ${XINERAMAFLAGS}
-CFLAGS = -Os ${INCS} ${CPPFLAGS}
+CFLAGS = -std=c99 -pedantic -Wall -Os ${INCS} ${CPPFLAGS}
 LDFLAGS = -s ${LIBS}
-#CFLAGS = -g -std=c99 -pedantic -Wall -O2 ${INCS} -DVERSION=\"${VERSION}\"
-#LDFLAGS = -g ${LIBS}
 
 # Solaris
 #CFLAGS = -fast ${INCS} -DVERSION=\"${VERSION}\"
diff --git a/dmenu.c b/dmenu.c
@@ -2,13 +2,14 @@
 #include <ctype.h>
 #include <locale.h>
 #include <stdarg.h>
-#include <stdlib.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
+#include <strings.h>
 #include <unistd.h>
+#include <X11/keysym.h>
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
-#include <X11/keysym.h>
 #ifdef XINERAMA
 #include <X11/extensions/Xinerama.h>
 #endif
@@ -45,55 +46,54 @@ struct Item {
 };
 
 /* forward declarations */
-void appenditem(Item *i, Item **list, Item **last);
-void calcoffsets(void);
-char *cistrstr(const char *s, const char *sub);
-void cleanup(void);
-void drawmenu(void);
-void drawtext(const char *text, ulong col[ColLast]);
-void *emalloc(uint size);
-void eprint(const char *errstr, ...);
-char *estrdup(const char *str);
-ulong getcolor(const char *colstr);
-Bool grabkeyboard(void);
-void initfont(const char *fontstr);
-void kpress(XKeyEvent * e);
-void match(char *pattern);
-void readstdin(void);
-void run(void);
-void setup(Bool topbar);
-uint textnw(const char *text, uint len);
-uint textw(const char *text);
+static void appenditem(Item *i, Item **list, Item **last);
+static void calcoffsets(void);
+static char *cistrstr(const char *s, const char *sub);
+static void cleanup(void);
+static void drawmenu(void);
+static void drawtext(const char *text, ulong col[ColLast]);
+static void *emalloc(uint size);
+static void eprint(const char *errstr, ...);
+static ulong getcolor(const char *colstr);
+static Bool grabkeyboard(void);
+static void initfont(const char *fontstr);
+static void kpress(XKeyEvent * e);
+static void match(char *pattern);
+static void readstdin(void);
+static void run(void);
+static void setup(Bool topbar);
+static int textnw(const char *text, uint len);
+static int textw(const char *text);
 
 #include "config.h"
 
 /* variables */
-char *font = FONT;
-char *maxname = NULL;
-char *normbg = NORMBGCOLOR;
-char *normfg = NORMFGCOLOR;
-char *prompt = NULL;
-char *selbg = SELBGCOLOR;
-char *selfg = SELFGCOLOR;
-char text[4096];
-int screen;
-int ret = 0;
-uint cmdw = 0;
-uint mw, mh;
-uint promptw = 0;
-uint numlockmask = 0;
-Bool running = True;
-Display *dpy;
-DC dc = {0};
-Item *allitems = NULL;    /* first of all items */
-Item *item = NULL;    /* first of pattern matching items */
-Item *sel = NULL;
-Item *next = NULL;
-Item *prev = NULL;
-Item *curr = NULL;
-Window root, win;
-int (*fstrncmp)(const char *, const char *, size_t n) = strncmp;
-char *(*fstrstr)(const char *, const char *) = strstr;
+static char *font = FONT;
+static char *maxname = NULL;
+static char *normbg = NORMBGCOLOR;
+static char *normfg = NORMFGCOLOR;
+static char *prompt = NULL;
+static char *selbg = SELBGCOLOR;
+static char *selfg = SELFGCOLOR;
+static char text[4096];
+static int cmdw = 0;
+static int promptw = 0;
+static int ret = 0;
+static int screen;
+static uint mw, mh;
+static uint numlockmask = 0;
+static Bool running = True;
+static Display *dpy;
+static DC dc = {0};
+static Item *allitems = NULL;    /* first of all items */
+static Item *item = NULL;    /* first of pattern matching items */
+static Item *sel = NULL;
+static Item *next = NULL;
+static Item *prev = NULL;
+static Item *curr = NULL;
+static Window root, win;
+static int (*fstrncmp)(const char *, const char *, size_t n) = strncmp;
+static char *(*fstrstr)(const char *, const char *) = strstr;
 
 void
 appenditem(Item *i, Item **list, Item **last) {
@@ -108,11 +108,12 @@ appenditem(Item *i, Item **list, Item **last) {
 
 void
 calcoffsets(void) {
-    uint tw, w;
+    int tw;
+    uint w;
 
     if(!curr)
         return;
-    w = promptw + cmdw + 2 * SPACE;
+    w = promptw + cmdw + 2 * spaceitem;
     for(next = curr; next; next=next->right) {
         tw = textw(next->text);
         if(tw > mw / 3)
@@ -121,7 +122,7 @@ calcoffsets(void) {
         if(w > mw)
             break;
     }
-    w = promptw + cmdw + 2 * SPACE;
+    w = promptw + cmdw + 2 * spaceitem;
     for(prev = curr; prev && prev->left; prev=prev->left) {
         tw = textw(prev->left->text);
         if(tw > mw / 3)
@@ -197,7 +198,7 @@ drawmenu(void) {
     drawtext(text[0] ? text : NULL, dc.norm);
     dc.x += cmdw;
     if(curr) {
-        dc.w = SPACE;
+        dc.w = spaceitem;
         drawtext((curr && curr->left) ? "<" : NULL, dc.norm);
         dc.x += dc.w;
         /* determine maximum items */
@@ -208,8 +209,8 @@ drawmenu(void) {
             drawtext(i->text, (sel == i) ? dc.sel : dc.norm);
             dc.x += dc.w;
         }
-        dc.x = mw - SPACE;
-        dc.w = SPACE;
+        dc.x = mw - spaceitem;
+        dc.w = spaceitem;
         drawtext(next ? ">" : NULL, dc.norm);
     }
     XCopyArea(dpy, dc.drawable, win, dc.gc, 0, 0, mw, mh, 0, 0);
@@ -275,15 +276,6 @@ eprint(const char *errstr, ...) {
     exit(EXIT_FAILURE);
 }
 
-char *
-estrdup(const char *str) {
-    void *res = strdup(str);
-
-    if(!res)
-        eprint("fatal: could not malloc() %u bytes\n", strlen(str));
-    return res;
-}
-
 ulong
 getcolor(const char *colstr) {
     Colormap cmap = DefaultColormap(dpy, screen);
@@ -358,14 +350,15 @@ kpress(XKeyEvent * e) {
     len = strlen(text);
     buf[0] = 0;
     num = XLookupString(e, buf, sizeof buf, &ksym, 0);
-    if(IsKeypadKey(ksym))
+    if(IsKeypadKey(ksym)) {
         if(ksym == XK_KP_Enter)
             ksym = XK_Return;
         else if(ksym >= XK_KP_0 && ksym <= XK_KP_9)
             ksym = (ksym - XK_KP_0) + XK_0;
+    }
     if(IsFunctionKey(ksym) || IsKeypadKey(ksym)
-    || IsMiscFunctionKey(ksym) || IsPFKey(ksym)
-    || IsPrivateKeypadKey(ksym))
+       || IsMiscFunctionKey(ksym) || IsPFKey(ksym)
+       || IsPrivateKeypadKey(ksym))
         return;
     /* first check if a control mask is omitted */
     if(e->state & ControlMask) {
@@ -569,7 +562,8 @@ readstdin(void) {
         len = strlen(buf);
         if (buf[len - 1] == '\n')
             buf[len - 1] = 0;
-        p = estrdup(buf);
+        if(!(p = strdup(buf)))
+            eprint("fatal: could not strdup() %u bytes\n", strlen(buf));
         if(max < len) {
             maxname = p;
             max = len;
@@ -677,7 +671,7 @@ setup(Bool topbar) {
     XMapRaised(dpy, win);
 }
 
-uint
+int
 textnw(const char *text, uint len) {
     XRectangle r;
 
@@ -688,7 +682,7 @@ textnw(const char *text, uint len) {
     return XTextWidth(dc.font.xfont, text, len);
 }
 
-uint
+int
 textw(const char *text) {
     return textnw(text, strlen(text)) + dc.font.height;
 }