From 5077207088b08e6b21601cb19925d20e9d5af573 Mon Sep 17 00:00:00 2001 From: "arg@10ksloc.org" Date: Fri, 4 Aug 2006 10:23:36 +0200 Subject: [PATCH] rearranged several stuff --- LICENSE | 1 + config.mk | 2 +- dmenu.h | 29 ++----- draw.c | 188 +++++++++++++++++++-------------------- main.c | 256 ++++++++++++++++++++++++++---------------------------- util.c | 18 ---- 6 files changed, 222 insertions(+), 272 deletions(-) diff --git a/LICENSE b/LICENSE index aa0a3ab..67e22c3 100644 --- a/LICENSE +++ b/LICENSE @@ -1,6 +1,7 @@ MIT/X Consortium License (C)opyright MMVI Anselm R. Garbe +(C)opyright MMVI Sander van Dijk Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), diff --git a/config.mk b/config.mk index a4d21ac..2531d27 100644 --- a/config.mk +++ b/config.mk @@ -1,4 +1,4 @@ -# dwm version +# dmenu version VERSION = 0.0 # Customize below to fit your system diff --git a/dmenu.h b/dmenu.h index 8d630ba..75b6266 100644 --- a/dmenu.h +++ b/dmenu.h @@ -7,6 +7,8 @@ #include #include +#define SPACE 30 /* px */ + typedef struct Brush Brush; typedef struct DC DC; typedef struct Fnt Fnt; @@ -29,30 +31,17 @@ struct DC { /* draw context */ GC gc; }; -struct Brush { - GC gc; - Drawable drawable; - int x, y, w, h; - Fnt font; - unsigned long bg; - unsigned long fg; - unsigned long border; -}; - - +extern int screen; +extern Display *dpy; +extern DC dc; /* draw.c */ -extern void draw(Display *dpy, Brush *b, Bool border, const char *text); -extern void loadcolors(Display *dpy, int screen, Brush *b, - const char *bg, const char *fg, const char *bo); -extern void loadfont(Display *dpy, Fnt *font, const char *fontstr); -extern unsigned int textnw(Fnt *font, char *text, unsigned int len); -extern unsigned int textw(Fnt *font, char *text); -extern unsigned int texth(Fnt *font); +extern void drawtext(const char *text, Bool invert, Bool border); +extern unsigned long getcolor(const char *colstr); +extern void setfont(const char *fontstr); +extern unsigned int textw(const char *text); /* util.c */ extern void *emalloc(unsigned int size); -extern void *emallocz(unsigned int size); extern void eprint(const char *errstr, ...); extern char *estrdup(const char *str); -extern void swap(void **p1, void **p2); diff --git a/draw.c b/draw.c index d747629..e0ff001 100644 --- a/draw.c +++ b/draw.c @@ -2,45 +2,62 @@ * (C)opyright MMIV-MMVI Anselm R. Garbe * See LICENSE file for license details. */ - +#include "dmenu.h" #include #include +#include -#include "dmenu.h" +/* static */ static void -drawborder(Display *dpy, Brush *b) +drawborder(void) { XPoint points[5]; - XSetLineAttributes(dpy, b->gc, 1, LineSolid, CapButt, JoinMiter); - XSetForeground(dpy, b->gc, b->border); - points[0].x = b->x; - points[0].y = b->y; - points[1].x = b->w - 1; + + XSetLineAttributes(dpy, dc.gc, 1, LineSolid, CapButt, JoinMiter); + XSetForeground(dpy, dc.gc, dc.border); + points[0].x = dc.x; + points[0].y = dc.y; + points[1].x = dc.w - 1; points[1].y = 0; points[2].x = 0; - points[2].y = b->h - 1; - points[3].x = -(b->w - 1); + points[2].y = dc.h - 1; + points[3].x = -(dc.w - 1); points[3].y = 0; points[4].x = 0; - points[4].y = -(b->h - 1); - XDrawLines(dpy, b->drawable, b->gc, points, 5, CoordModePrevious); + points[4].y = -(dc.h - 1); + XDrawLines(dpy, dc.drawable, dc.gc, points, 5, CoordModePrevious); } -void -draw(Display *dpy, Brush *b, Bool border, const char *text) +static unsigned int +textnw(const char *text, unsigned int len) { - unsigned int x, y, w, h, len; - static char buf[256]; - XGCValues gcv; - XRectangle r = { b->x, b->y, b->w, b->h }; + XRectangle r; - XSetForeground(dpy, b->gc, b->bg); - XFillRectangles(dpy, b->drawable, b->gc, &r, 1); + if(dc.font.set) { + XmbTextExtents(dc.font.set, text, len, NULL, &r); + return r.width; + } + return XTextWidth(dc.font.xfont, text, len); +} + +/* extern */ + +void +drawtext(const char *text, Bool invert, Bool border) +{ + int x, y, w, h; + static char buf[256]; + unsigned int len; + XGCValues gcv; + XRectangle r = { dc.x, dc.y, dc.w, dc.h }; + + XSetForeground(dpy, dc.gc, invert ? dc.fg : dc.bg); + XFillRectangles(dpy, dc.drawable, dc.gc, &r, 1); w = 0; if(border) - drawborder(dpy, b); + drawborder(); if(!text) return; @@ -51,121 +68,94 @@ draw(Display *dpy, Brush *b, Bool border, const char *text) memcpy(buf, text, len); buf[len] = 0; - h = b->font.ascent + b->font.descent; - y = b->y + (b->h / 2) - (h / 2) + b->font.ascent; - x = b->x + (h / 2); + h = dc.font.ascent + dc.font.descent; + y = dc.y + (dc.h / 2) - (h / 2) + dc.font.ascent; + x = dc.x + (h / 2); /* shorten text if necessary */ - while(len && (w = textnw(&b->font, buf, len)) > b->w - h) + while(len && (w = textnw(buf, len)) > dc.w - h) buf[--len] = 0; - if(w > b->w) + if(w > dc.w) return; /* too long */ - gcv.foreground = b->fg; - gcv.background = b->bg; - if(b->font.set) { - XChangeGC(dpy, b->gc, GCForeground | GCBackground, &gcv); - XmbDrawImageString(dpy, b->drawable, b->font.set, b->gc, + gcv.foreground = invert ? dc.bg : dc.fg; + gcv.background = invert ? dc.fg : dc.bg; + if(dc.font.set) { + XChangeGC(dpy, dc.gc, GCForeground | GCBackground, &gcv); + XmbDrawImageString(dpy, dc.drawable, dc.font.set, dc.gc, x, y, buf, len); } else { - gcv.font = b->font.xfont->fid; - XChangeGC(dpy, b->gc, GCForeground | GCBackground | GCFont, &gcv); - XDrawImageString(dpy, b->drawable, b->gc, x, y, buf, len); + gcv.font = dc.font.xfont->fid; + XChangeGC(dpy, dc.gc, GCForeground | GCBackground | GCFont, &gcv); + XDrawImageString(dpy, dc.drawable, dc.gc, x, y, buf, len); } } -static unsigned long -xloadcolors(Display *dpy, Colormap cmap, const char *colstr) +unsigned long +getcolor(const char *colstr) { + Colormap cmap = DefaultColormap(dpy, screen); XColor color; + XAllocNamedColor(dpy, cmap, colstr, &color, &color); return color.pixel; } void -loadcolors(Display *dpy, int screen, Brush *b, - const char *bg, const char *fg, const char *border) -{ - Colormap cmap = DefaultColormap(dpy, screen); - b->bg = xloadcolors(dpy, cmap, bg); - b->fg = xloadcolors(dpy, cmap, fg); - b->border = xloadcolors(dpy, cmap, border); -} - -unsigned int -textnw(Fnt *font, char *text, unsigned int len) -{ - XRectangle r; - if(font->set) { - XmbTextExtents(font->set, text, len, NULL, &r); - return r.width; - } - return XTextWidth(font->xfont, text, len); -} - -unsigned int -textw(Fnt *font, char *text) -{ - return textnw(font, text, strlen(text)); -} - -unsigned int -texth(Fnt *font) -{ - return font->height + 4; -} - -void -loadfont(Display *dpy, Fnt *font, const char *fontstr) +setfont(const char *fontstr) { char **missing, *def; - int n; + int i, n; missing = NULL; - def = "?"; setlocale(LC_ALL, ""); - if(font->set) - XFreeFontSet(dpy, font->set); - font->set = XCreateFontSet(dpy, fontstr, &missing, &n, &def); + if(dc.font.set) + XFreeFontSet(dpy, dc.font.set); + dc.font.set = XCreateFontSet(dpy, fontstr, &missing, &n, &def); if(missing) { while(n--) fprintf(stderr, "missing fontset: %s\n", missing[n]); XFreeStringList(missing); - if(font->set) { - XFreeFontSet(dpy, font->set); - font->set = NULL; + if(dc.font.set) { + XFreeFontSet(dpy, dc.font.set); + dc.font.set = NULL; } } - if(font->set) { + if(dc.font.set) { XFontSetExtents *font_extents; XFontStruct **xfonts; char **font_names; - unsigned int i; - font->ascent = font->descent = 0; - font_extents = XExtentsOfFontSet(font->set); - n = XFontsOfFontSet(font->set, &xfonts, &font_names); - for(i = 0, font->ascent = 0, font->descent = 0; i < n; i++) { - if(font->ascent < (*xfonts)->ascent) - font->ascent = (*xfonts)->ascent; - if(font->descent < (*xfonts)->descent) - font->descent = (*xfonts)->descent; + dc.font.ascent = dc.font.descent = 0; + font_extents = XExtentsOfFontSet(dc.font.set); + n = XFontsOfFontSet(dc.font.set, &xfonts, &font_names); + for(i = 0, dc.font.ascent = 0, dc.font.descent = 0; i < n; i++) { + if(dc.font.ascent < (*xfonts)->ascent) + dc.font.ascent = (*xfonts)->ascent; + if(dc.font.descent < (*xfonts)->descent) + dc.font.descent = (*xfonts)->descent; xfonts++; } } else { - if(font->xfont) - XFreeFont(dpy, font->xfont); - font->xfont = NULL; - font->xfont = XLoadQueryFont(dpy, fontstr); - if (!font->xfont) - font->xfont = XLoadQueryFont(dpy, "fixed"); - if (!font->xfont) - eprint("error, cannot load 'fixed' font\n"); - font->ascent = font->xfont->ascent; - font->descent = font->xfont->descent; + if(dc.font.xfont) + XFreeFont(dpy, dc.font.xfont); + dc.font.xfont = NULL; + dc.font.xfont = XLoadQueryFont(dpy, fontstr); + if (!dc.font.xfont) + dc.font.xfont = XLoadQueryFont(dpy, "fixed"); + if (!dc.font.xfont) + eprint("error, cannot init 'fixed' font\n"); + dc.font.ascent = dc.font.xfont->ascent; + dc.font.descent = dc.font.xfont->descent; } - font->height = font->ascent + font->descent; + dc.font.height = dc.font.ascent + dc.font.descent; +} + +unsigned int +textw(const char *text) +{ + return textnw(text, strlen(text)) + dc.font.height; } diff --git a/main.c b/main.c index 18263d5..14524c9 100644 --- a/main.c +++ b/main.c @@ -16,74 +16,112 @@ #include typedef struct Item Item; - struct Item { Item *next; /* traverses all items */ Item *left, *right; /* traverses items matching current search pattern */ char *text; }; -static Display *dpy; -static Window root; -static Window win; -static Bool done = False; +/* static */ -static Item *allitem = NULL; /* first of all items */ -static Item *item = NULL; /* first of pattern matching items */ -static Item *sel = NULL; -static Item *nextoff = NULL; -static Item *prevoff = NULL; -static Item *curroff = NULL; - -static int screen, mx, my, mw, mh; -static char *title = NULL; -static char text[4096]; +static char *title, text[4096]; +static int mx, my, mw, mh; static int ret = 0; static int nitem = 0; static unsigned int cmdw = 0; static unsigned int tw = 0; static unsigned int cw = 0; -static const int seek = 30; /* 30px */ - -static Brush brush = {0}; - -static void draw_menu(); -static void kpress(XKeyEvent * e); - -static char version[] = "dmenu - " VERSION ", (C)opyright MMVI Anselm R. Garbe\n"; +static Bool done = False; +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; +static Window win; static void -update_offsets() +calcoffsets() { - unsigned int tw, w = cmdw + 2 * seek; + unsigned int tw, w; - if(!curroff) + if(!curr) return; - for(nextoff = curroff; nextoff; nextoff=nextoff->right) { - tw = textw(&brush.font, nextoff->text); + w = cmdw + 2 * SPACE; + for(next = curr; next; next=next->right) { + tw = textw(next->text); if(tw > mw / 3) tw = mw / 3; - w += tw + brush.font.height; + w += tw; if(w > mw) break; } - w = cmdw + 2 * seek; - for(prevoff = curroff; prevoff && prevoff->left; prevoff=prevoff->left) { - tw = textw(&brush.font, prevoff->left->text); + w = cmdw + 2 * SPACE; + for(prev = curr; prev && prev->left; prev=prev->left) { + tw = textw(prev->left->text); if(tw > mw / 3) tw = mw / 3; - w += tw + brush.font.height; + w += tw; if(w > mw) break; } } static void -update_items(char *pattern) +drawmenu() { - unsigned int plen = strlen(pattern); + Item *i; + + dc.x = 0; + dc.y = 0; + dc.w = mw; + dc.h = mh; + drawtext(NULL, False, False); + + /* print command */ + if(!title || text[0]) { + cmdw = cw; + if(cmdw && item) + dc.w = cmdw; + drawtext(text, False, False); + } + else { + cmdw = tw; + dc.w = cmdw; + drawtext(title, False, False); + } + dc.x += dc.w; + + if(curr) { + dc.w = SPACE; + drawtext((curr && curr->left) ? "<" : NULL, False, False); + dc.x += dc.w; + + /* determine maximum items */ + for(i = curr; i != next; i=i->right) { + dc.border = False; + dc.w = textw(i->text); + if(dc.w > mw / 3) + dc.w = mw / 3; + drawtext(i->text, sel == i, sel == i); + dc.x += dc.w; + } + + dc.x = mw - SPACE; + dc.w = SPACE; + drawtext(next ? ">" : NULL, False, False); + } + XCopyArea(dpy, dc.drawable, win, dc.gc, 0, 0, mw, mh, 0, 0); + XFlush(dpy); +} + +static void +input(char *pattern) +{ + unsigned int plen; Item *i, *j; if(!pattern) @@ -94,10 +132,11 @@ update_items(char *pattern) else cmdw = tw; + plen = strlen(pattern); item = j = NULL; nitem = 0; - for(i = allitem; i; i=i->next) + for(i = allitems; i; i=i->next) if(!plen || !strncmp(pattern, i->text, plen)) { if(!j) item = i; @@ -108,7 +147,7 @@ update_items(char *pattern) j = i; nitem++; } - for(i = allitem; i; i=i->next) + for(i = allitems; i; i=i->next) if(plen && strncmp(pattern, i->text, plen) && strstr(i->text, pattern)) { if(!j) @@ -121,75 +160,19 @@ update_items(char *pattern) nitem++; } - curroff = prevoff = nextoff = sel = item; - - update_offsets(); -} - -/* creates brush structs for brush mode drawing */ -static void -draw_menu() -{ - Item *i; - - brush.x = 0; - brush.y = 0; - brush.w = mw; - brush.h = mh; - draw(dpy, &brush, False, 0); - - /* print command */ - if(!title || text[0]) { - cmdw = cw; - if(cmdw && item) - brush.w = cmdw; - draw(dpy, &brush, False, text); - } - else { - cmdw = tw; - brush.w = cmdw; - draw(dpy, &brush, False, title); - } - brush.x += brush.w; - - if(curroff) { - brush.w = seek; - draw(dpy, &brush, False, (curroff && curroff->left) ? "<" : 0); - brush.x += brush.w; - - /* determine maximum items */ - for(i = curroff; i != nextoff; i=i->right) { - brush.border = False; - brush.w = textw(&brush.font, i->text); - if(brush.w > mw / 3) - brush.w = mw / 3; - brush.w += brush.font.height; - if(sel == i) { - swap((void **)&brush.fg, (void **)&brush.bg); - draw(dpy, &brush, True, i->text); - swap((void **)&brush.fg, (void **)&brush.bg); - } - else - draw(dpy, &brush, False, i->text); - brush.x += brush.w; - } - - brush.x = mw - seek; - brush.w = seek; - draw(dpy, &brush, False, nextoff ? ">" : 0); - } - XCopyArea(dpy, brush.drawable, win, brush.gc, 0, 0, mw, mh, 0, 0); - XFlush(dpy); + curr = prev = next = sel = item; + calcoffsets(); } static void kpress(XKeyEvent * e) { - KeySym ksym; char buf[32]; int num, prev_nitem; - unsigned int i, len = strlen(text); + unsigned int i, len; + KeySym ksym; + len = strlen(text); buf[0] = 0; num = XLookupString(e, buf, sizeof(buf), &ksym, 0); @@ -210,8 +193,8 @@ kpress(XKeyEvent * e) case XK_U: case XK_u: text[0] = 0; - update_items(text); - draw_menu(); + input(text); + drawmenu(); return; break; case XK_bracketleft: @@ -224,24 +207,24 @@ kpress(XKeyEvent * e) if(!(sel && sel->left)) return; sel=sel->left; - if(sel->right == curroff) { - curroff = prevoff; - update_offsets(); + if(sel->right == curr) { + curr = prev; + calcoffsets(); } break; case XK_Tab: if(!sel) return; strncpy(text, sel->text, sizeof(text)); - update_items(text); + input(text); break; case XK_Right: if(!(sel && sel->right)) return; sel=sel->right; - if(sel == nextoff) { - curroff = nextoff; - update_offsets(); + if(sel == next) { + curr = next; + calcoffsets(); } break; case XK_Return: @@ -265,9 +248,9 @@ kpress(XKeyEvent * e) prev_nitem = nitem; do { text[--i] = 0; - update_items(text); + input(text); } while(i && nitem && prev_nitem == nitem); - update_items(text); + input(text); } break; default: @@ -277,14 +260,14 @@ kpress(XKeyEvent * e) strncat(text, buf, sizeof(text)); else strncpy(text, buf, sizeof(text)); - update_items(text); + input(text); } } - draw_menu(); + drawmenu(); } static char * -read_allitems() +readinput() { static char *maxname = NULL; char *p, buf[1024]; @@ -306,7 +289,7 @@ read_allitems() new->next = new->left = new->right = NULL; new->text = p; if(!i) - allitem = new; + allitems = new; else i->next = new; i = new; @@ -315,21 +298,27 @@ read_allitems() return maxname; } +/* extern */ + +int screen; +Display *dpy; +DC dc = {0}; + int main(int argc, char *argv[]) { - int i; - XSetWindowAttributes wa; char *maxname; + int i; XEvent ev; + XSetWindowAttributes wa; /* command line args */ for(i = 1; i < argc; i++) { if (argv[i][0] == '-') switch (argv[i][1]) { case 'v': - fprintf(stdout, "%s", version); - exit(0); + fputs("dmenu-"VERSION", (C)opyright MMVI Anselm R. Garbe\n", stdout); + exit(EXIT_SUCCESS); break; case 't': if(++i < argc) { @@ -350,7 +339,7 @@ main(int argc, char *argv[]) screen = DefaultScreen(dpy); root = RootWindow(dpy, screen); - maxname = read_allitems(); + maxname = readinput(); /* grab as early as possible, but after reading all items!!! */ while(XGrabKeyboard(dpy, root, True, GrabModeAsync, @@ -358,8 +347,10 @@ main(int argc, char *argv[]) usleep(1000); /* style */ - loadcolors(dpy, screen, &brush, BGCOLOR, FGCOLOR, BORDERCOLOR); - loadfont(dpy, &brush.font, FONT); + dc.bg = getcolor(BGCOLOR); + dc.fg = getcolor(FGCOLOR); + dc.border = getcolor(BORDERCOLOR); + setfont(FONT); wa.override_redirect = 1; wa.background_pixmap = ParentRelative; @@ -367,28 +358,25 @@ main(int argc, char *argv[]) mx = my = 0; mw = DisplayWidth(dpy, screen); - mh = texth(&brush.font); + mh = dc.font.height + 4; win = XCreateWindow(dpy, root, mx, my, mw, mh, 0, DefaultDepth(dpy, screen), CopyFromParent, DefaultVisual(dpy, screen), CWOverrideRedirect | CWBackPixmap | CWEventMask, &wa); XDefineCursor(dpy, win, XCreateFontCursor(dpy, XC_xterm)); - XFlush(dpy); /* pixmap */ - brush.gc = XCreateGC(dpy, root, 0, 0); - brush.drawable = XCreatePixmap(dpy, win, mw, mh, - DefaultDepth(dpy, screen)); - XFlush(dpy); + dc.drawable = XCreatePixmap(dpy, root, mw, mh, DefaultDepth(dpy, screen)); + dc.gc = XCreateGC(dpy, root, 0, 0); if(maxname) - cw = textw(&brush.font, maxname) + brush.font.height; + cw = textw(maxname); if(cw > mw / 3) cw = mw / 3; if(title) { - tw = textw(&brush.font, title) + brush.font.height; + tw = textw(title); if(tw > mw / 3) tw = mw / 3; } @@ -396,10 +384,10 @@ main(int argc, char *argv[]) cmdw = title ? tw : cw; text[0] = 0; - update_items(text); + input(text); XMapRaised(dpy, win); - draw_menu(); - XFlush(dpy); + drawmenu(); + XSync(dpy, False); /* main event loop */ while(!done && !XNextEvent(dpy, &ev)) { @@ -409,7 +397,7 @@ main(int argc, char *argv[]) break; case Expose: if(ev.xexpose.count == 0) - draw_menu(); + drawmenu(); break; default: break; @@ -417,8 +405,8 @@ main(int argc, char *argv[]) } XUngrabKeyboard(dpy, CurrentTime); - XFreePixmap(dpy, brush.drawable); - XFreeGC(dpy, brush.gc); + XFreePixmap(dpy, dc.drawable); + XFreeGC(dpy, dc.gc); XDestroyWindow(dpy, win); XCloseDisplay(dpy); diff --git a/util.c b/util.c index dff7af7..1b9bc57 100644 --- a/util.c +++ b/util.c @@ -29,16 +29,6 @@ emalloc(unsigned int size) return res; } -void * -emallocz(unsigned int size) -{ - void *res = calloc(1, size); - - if(!res) - bad_malloc(size); - return res; -} - void eprint(const char *errstr, ...) { @@ -58,11 +48,3 @@ estrdup(const char *str) bad_malloc(strlen(str)); return res; } - -void -swap(void **p1, void **p2) -{ - void *tmp = *p1; - *p1 = *p2; - *p2 = tmp; -}