several performance tweaks

[dmenu.git] / dmenu.c

diff --git a/dmenu.c b/dmenu.c
index a4fca09..5191a20 100644 (file)
--- a/dmenu.c
+++ b/dmenu.c
@@ -34,15 +34,15 @@ typedef struct {
 
 typedef struct Item Item;
 struct Item {
 
 typedef struct Item Item;
 struct Item {

+       char *text;

        Item *next;             /* traverses all items */
        Item *left, *right;     /* traverses items matching current search pattern */
        Item *next;             /* traverses all items */
        Item *left, *right;     /* traverses items matching current search pattern */

-       char *text;
-       Bool matched;

 };
 
 /* forward declarations */
 };
 
 /* forward declarations */

-Item *appenditem(Item *i, Item *last);
+void appenditem(Item *i, Item **list, Item **last);

 void calcoffsets(void);
 void calcoffsets(void);

+char *cistrstr(const char *s, const char *sub);

 void cleanup(void);
 void drawmenu(void);
 void drawtext(const char *text, unsigned long col[ColLast]);
 void cleanup(void);
 void drawmenu(void);
 void drawtext(const char *text, unsigned long col[ColLast]);


@@ -56,8 +56,7 @@ void kpress(XKeyEvent * e);
 void match(char *pattern);
 void readstdin(void);
 void run(void);
 void match(char *pattern);
 void readstdin(void);
 void run(void);

-void setup(Bool bottom);
-int strcaseido(const char *text, const char *pattern);
+void setup(int x, int y, int w);

 unsigned int textnw(const char *text, unsigned int len);
 unsigned int textw(const char *text);
 
 unsigned int textnw(const char *text, unsigned int len);
 unsigned int textw(const char *text);
 


@@ -79,7 +78,6 @@ unsigned int mw, mh;
 unsigned int promptw = 0;
 unsigned int nitem = 0;
 unsigned int numlockmask = 0;
 unsigned int promptw = 0;
 unsigned int nitem = 0;
 unsigned int numlockmask = 0;

-Bool idomatch = False;

 Bool running = True;
 Display *dpy;
 DC dc = {0};
 Bool running = True;
 Display *dpy;
 DC dc = {0};


@@ -90,19 +88,18 @@ Item *next = NULL;
 Item *prev = NULL;
 Item *curr = NULL;
 Window root, win;
 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;

 
 

-Item *
-appenditem(Item *i, Item *last) {
-       if(!last)
-               item = i;
+void
+appenditem(Item *i, Item **list, Item **last) {
+       if(!(*last))
+               *list = i;

        else
        else

-               last->right = i;
-       i->matched = True;
-       i->left = last;
+               (*last)->right = i;
+       i->left = *last;

        i->right = NULL;
        i->right = NULL;

-       last = i;
-       nitem++;
-       return last;
+       *last = i;

 }
 
 void
 }
 
 void


@@ -131,6 +128,29 @@ calcoffsets(void) {
        }
 }
 
        }
 }
 

+char *
+cistrstr(const char *s, const char *sub) {
+       int c, csub;
+       unsigned int len;
+
+       if(!sub)
+               return (char *)s;
+       if((c = *sub++) != 0) {
+               c = tolower(c);
+               len = strlen(sub);
+               do {
+                       do {
+                               if((csub = *s++) == 0)
+                                       return (NULL);
+                       }
+                       while(tolower(csub) != c);
+               }
+               while(strncasecmp(s, sub, len) != 0);
+               s--;
+       }
+       return (char *)s;
+}
+

 void
 cleanup(void) {
        Item *itm;
 void
 cleanup(void) {
        Item *itm;


@@ -498,29 +518,47 @@ kpress(XKeyEvent * e) {
 void
 match(char *pattern) {
        unsigned int plen;
 void
 match(char *pattern) {
        unsigned int plen;

-       Item *i, *j;
+       Item *i, *itemend, *lexact, *lprefix, *lsubstr, *exactend, *prefixend, *substrend;

 
        if(!pattern)
                return;
        plen = strlen(pattern);
 
        if(!pattern)
                return;
        plen = strlen(pattern);

-       item = j = NULL;
+       item = lexact = lprefix = lsubstr = itemend = exactend = prefixend = substrend = NULL;

        nitem = 0;
        nitem = 0;

-       for(i = allitems; i; i=i->next)
-               i->matched = False;
-

        for(i = allitems; i; i = i->next)
        for(i = allitems; i; i = i->next)

-               if(!i->matched && !strncasecmp(pattern, i->text, plen))
-                       j = appenditem(i, j);
-
-       for (i = allitems; i; i = i->next)
-               if(!i->matched && strcasestr(i->text, pattern))
-                       j = appenditem(i, j);
-
-       if(idomatch)
-               for (i = allitems; i; i = i->next)
-                       if(!i->matched && strcaseido(i->text, pattern))
-                               j = appenditem(i, j);
-
+               if(!fstrncmp(pattern, i->text, plen + 1)) {
+                       appenditem(i, &lexact, &exactend);
+                       nitem++;
+               }
+               else if(!fstrncmp(pattern, i->text, plen)) {
+                       appenditem(i, &lprefix, &prefixend);
+                       nitem++;
+               }
+               else if(fstrstr(i->text, pattern)) {
+                       appenditem(i, &lsubstr, &substrend);
+                       nitem++;
+               }
+       if(lexact) {
+               item = lexact;
+               itemend = exactend;
+       }
+       if(lprefix) {
+               if(itemend) {
+                       itemend->right - lprefix;
+                       lprefix->left = itemend;
+               }
+               else
+                       item = lprefix;
+               itemend = prefixend;
+       }
+       if(lsubstr) {
+               if(itemend) {
+                       itemend->right = lsubstr;
+                       lsubstr->left = itemend;
+               }
+               else
+                       item = lsubstr;
+       }

        curr = prev = next = sel = item;
        calcoffsets();
 }
        curr = prev = next = sel = item;
        calcoffsets();
 }


@@ -572,7 +610,7 @@ run(void) {
 }
 
 void
 }
 
 void

-setup(Bool bottom) {
+setup(int x, int y, int w) {

        unsigned int i, j;
        XModifierKeymap *modmap;
        XSetWindowAttributes wa;
        unsigned int i, j;
        XModifierKeymap *modmap;
        XSetWindowAttributes wa;


@@ -598,10 +636,15 @@ setup(Bool bottom) {
        wa.override_redirect = 1;
        wa.background_pixmap = ParentRelative;
        wa.event_mask = ExposureMask | ButtonPressMask | KeyPressMask;
        wa.override_redirect = 1;
        wa.background_pixmap = ParentRelative;
        wa.event_mask = ExposureMask | ButtonPressMask | KeyPressMask;

-       mw = DisplayWidth(dpy, screen);
+       mw = w ? w : DisplayWidth(dpy, screen);

        mh = dc.font.height + 2;
        mh = dc.font.height + 2;

-       win = XCreateWindow(dpy, root, 0,
-                       bottom ? DisplayHeight(dpy, screen) - mh : 0, mw, mh, 0,
+       if(y < 0) {
+               if(y == (int)(unsigned int)-1)
+                       y = DisplayHeight(dpy, screen) - mh;
+               else
+                       y = (-1 * y) - mh;
+       }
+       win = XCreateWindow(dpy, root, x, y, mw, mh, 0,

                        DefaultDepth(dpy, screen), CopyFromParent,
                        DefaultVisual(dpy, screen),
                        CWOverrideRedirect | CWBackPixmap | CWEventMask, &wa);
                        DefaultDepth(dpy, screen), CopyFromParent,
                        DefaultVisual(dpy, screen),
                        CWOverrideRedirect | CWBackPixmap | CWEventMask, &wa);


@@ -625,14 +668,6 @@ setup(Bool bottom) {
        XMapRaised(dpy, win);
 }
 
        XMapRaised(dpy, win);
 }
 

-int
-strcaseido(const char *text, const char *pattern) {
-       for(; *text && *pattern; text++)
-               if (tolower(*text) == tolower(*pattern))
-                       pattern++;
-       return !*pattern;
-}                                  
-

 unsigned int
 textnw(const char *text, unsigned int len) {
        XRectangle r;
 unsigned int
 textnw(const char *text, unsigned int len) {
        XRectangle r;


@@ -651,16 +686,15 @@ textw(const char *text) {
 
 int
 main(int argc, char *argv[]) {
 
 int
 main(int argc, char *argv[]) {

-       Bool bottom = False;
+       int x = 0, y = 0, w = 0;

        unsigned int i;
 
        /* command line args */
        for(i = 1; i < argc; i++)
        unsigned int i;
 
        /* command line args */
        for(i = 1; i < argc; i++)

-               if(!strcmp(argv[i], "-b")) {
-                       bottom = True;
+               if(!strcmp(argv[i], "-i")) {
+                       fstrncmp = strncasecmp;
+                       fstrstr = cistrstr;

                }
                }

-               else if(!strcmp(argv[i], "-i"))
-                       idomatch = True;

                else if(!strcmp(argv[i], "-fn")) {
                        if(++i < argc) font = argv[i];
                }
                else if(!strcmp(argv[i], "-fn")) {
                        if(++i < argc) font = argv[i];
                }


@@ -679,11 +713,26 @@ main(int argc, char *argv[]) {
                else if(!strcmp(argv[i], "-sf")) {
                        if(++i < argc) selfg = argv[i];
                }
                else if(!strcmp(argv[i], "-sf")) {
                        if(++i < argc) selfg = argv[i];
                }

+               else if(!strcmp(argv[i], "-x")) {
+                       if(++i < argc) x = atoi(argv[i]);
+               }
+               else if(!strcmp(argv[i], "-y")) {
+                       if(++i < argc) {
+                               if(!strcmp(argv[i], "-0"))
+                                       y = (int)(unsigned int)-1;
+                               else
+                                       y = atoi(argv[i]);
+                       }
+               }
+               else if(!strcmp(argv[i], "-w")) {
+                       if(++i < argc) w = atoi(argv[i]);
+               }

                else if(!strcmp(argv[i], "-v"))
                else if(!strcmp(argv[i], "-v"))

-                       eprint("dmenu-"VERSION", © 2006-2007 Anselm R. Garbe, Sander van Dijk, Michał Janeczek\n");
+                       eprint("dmenu-"VERSION", © 2006-2008 dmenu engineers, see LICENSE for details\n");

                else
                else

-                       eprint("usage: dmenu [-b] [-i] [-fn <font>] [-nb <color>] [-nf <color>]\n"
-                       "             [-p <prompt>] [-sb <color>] [-sf <color>] [-v]\n");
+                       eprint("usage: dmenu [-i] [-fn <font>] [-nb <color>] [-nf <color>]\n"
+                              "             [-p <prompt>] [-sb <color>] [-sf <color>]\n"
+                              "             [-x <x>] [-y <y>] [-w <w>] [-v]\n");

        setlocale(LC_CTYPE, "");
        dpy = XOpenDisplay(0);
        if(!dpy)
        setlocale(LC_CTYPE, "");
        dpy = XOpenDisplay(0);
        if(!dpy)


@@ -700,7 +749,7 @@ main(int argc, char *argv[]) {
                readstdin();
        }
 
                readstdin();
        }
 

-       setup(bottom);
+       setup(x, y, w);

        drawmenu();
        XSync(dpy, False);
        run();
        drawmenu();
        XSync(dpy, False);
        run();