Simplify logic in match().

[st.git] / st.c

diff --git a/st.c b/st.c
index 331509f..da2ce3f 100644 (file)
--- a/st.c
+++ b/st.c
@@ -256,15 +256,15 @@ typedef struct {
 } XWindow;
 
 typedef struct {
 } XWindow;
 
 typedef struct {

-       int b;
+       uint b;

        uint mask;
        uint mask;

-       char s[ESC_BUF_SIZ];
+       char *s;

 } Mousekey;
 
 typedef struct {
        KeySym k;
        uint mask;
 } Mousekey;
 
 typedef struct {
        KeySym k;
        uint mask;

-       char s[ESC_BUF_SIZ];
+       char *s;

        /* three valued logic variables: 0 indifferent, 1 on, -1 off */
        signed char appkey;    /* application keypad */
        signed char appcursor; /* application cursor */
        /* three valued logic variables: 0 indifferent, 1 on, -1 off */
        signed char appkey;    /* application keypad */
        signed char appcursor; /* application cursor */


@@ -386,6 +386,7 @@ static inline bool match(uint, uint);
 static void ttynew(void);
 static void ttyread(void);
 static void ttyresize(void);
 static void ttynew(void);
 static void ttyread(void);
 static void ttyresize(void);

+static void ttysend(char *, size_t);

 static void ttywrite(const char *, size_t);
 
 static void xdraws(char *, Glyph, int, int, int, int);
 static void ttywrite(const char *, size_t);
 
 static void xdraws(char *, Glyph, int, int, int, int);


@@ -893,9 +894,7 @@ bpress(XEvent *e) {
        for(mk = mshortcuts; mk < mshortcuts + LEN(mshortcuts); mk++) {
                if(e->xbutton.button == mk->b
                                && match(mk->mask, e->xbutton.state)) {
        for(mk = mshortcuts; mk < mshortcuts + LEN(mshortcuts); mk++) {
                if(e->xbutton.button == mk->b
                                && match(mk->mask, e->xbutton.state)) {

-                       ttywrite(mk->s, strlen(mk->s));
-                       if(IS_SET(MODE_ECHO))
-                               techo(mk->s, strlen(mk->s));
+                       ttysend(mk->s, strlen(mk->s));

                        return;
                }
        }
                        return;
                }
        }


@@ -1031,7 +1030,7 @@ selnotify(XEvent *e) {
 
                if(IS_SET(MODE_BRCKTPASTE))
                        ttywrite("\033[200~", 6);
 
                if(IS_SET(MODE_BRCKTPASTE))
                        ttywrite("\033[200~", 6);

-               ttywrite((const char *)data, nitems * format / 8);
+               ttysend((char *)data, nitems * format / 8);

                if(IS_SET(MODE_BRCKTPASTE))
                        ttywrite("\033[201~", 6);
                XFree(data);
                if(IS_SET(MODE_BRCKTPASTE))
                        ttywrite("\033[201~", 6);
                XFree(data);


@@ -1299,6 +1298,13 @@ ttywrite(const char *s, size_t n) {
                die("write error on tty: %s\n", SERRNO);
 }
 
                die("write error on tty: %s\n", SERRNO);
 }
 

+void
+ttysend(char *s, size_t n) {
+       ttywrite(s, n);
+       if(IS_SET(MODE_ECHO))
+               techo(s, n);
+}
+

 void
 ttyresize(void) {
        struct winsize w;
 void
 ttyresize(void) {
        struct winsize w;


@@ -3495,15 +3501,7 @@ focus(XEvent *ev) {
 
 static inline bool
 match(uint mask, uint state) {
 
 static inline bool
 match(uint mask, uint state) {

-       state &= ~ignoremod;
-
-       if(mask == XK_NO_MOD && state)
-               return false;
-       if(mask != XK_ANY_MOD && mask != XK_NO_MOD && !state)
-               return false;
-       if(mask == XK_ANY_MOD)
-               return true;
-       return state == mask;
+       return mask == XK_ANY_MOD || mask == (state & ~ignoremod);

 }
 
 void
 }
 
 void


@@ -3573,7 +3571,6 @@ kpress(XEvent *ev) {
                return;
 
        len = XmbLookupString(xw.xic, e, buf, sizeof buf, &ksym, &status);
                return;
 
        len = XmbLookupString(xw.xic, e, buf, sizeof buf, &ksym, &status);

-       e->state &= ~Mod2Mask;

        /* 1. shortcuts */
        for(bp = shortcuts; bp < shortcuts + LEN(shortcuts); bp++) {
                if(ksym == bp->keysym && match(bp->mod, e->state)) {
        /* 1. shortcuts */
        for(bp = shortcuts; bp < shortcuts + LEN(shortcuts); bp++) {
                if(ksym == bp->keysym && match(bp->mod, e->state)) {


@@ -3584,30 +3581,26 @@ kpress(XEvent *ev) {
 
        /* 2. custom keys from config.h */
        if((customkey = kmap(ksym, e->state))) {
 
        /* 2. custom keys from config.h */
        if((customkey = kmap(ksym, e->state))) {

-               len = strlen(customkey);
-               memcpy(buf, customkey, len);
-       /* 3. composed string from input method */
-       } else {
-               if(len == 0)
-                       return;
+               ttysend(customkey, strlen(customkey));
+               return;
+       }

 
 

-               if(len == 1 && e->state & Mod1Mask) {
-                       if(IS_SET(MODE_8BIT)) {
-                               if(*buf < 0177) {
-                                       c = *buf | 0x80;
-                                       len = utf8encode(&c, buf);
-                               }
-                       } else {
-                               buf[1] = buf[0];
-                               buf[0] = '\033';
-                               len = 2;
+       /* 3. composed string from input method */
+       if(len == 0)
+               return;
+       if(len == 1 && e->state & Mod1Mask) {
+               if(IS_SET(MODE_8BIT)) {
+                       if(*buf < 0177) {
+                               c = *buf | 0x80;
+                               len = utf8encode(&c, buf);

                        }
                        }

+               } else {
+                       buf[1] = buf[0];
+                       buf[0] = '\033';
+                       len = 2;

                }
        }
                }
        }

-
-       ttywrite(buf, len);
-       if(IS_SET(MODE_ECHO))
-               techo(buf, len);
+       ttysend(buf, len);

 }
 
 
 }
 
 


@@ -3685,6 +3678,8 @@ run(void) {
        gettimeofday(&last, NULL);
 
        for(xev = actionfps;;) {
        gettimeofday(&last, NULL);
 
        for(xev = actionfps;;) {

+               long deltatime;
+

                FD_ZERO(&rfd);
                FD_SET(cmdfd, &rfd);
                FD_SET(xfd, &rfd);
                FD_ZERO(&rfd);
                FD_SET(cmdfd, &rfd);
                FD_SET(xfd, &rfd);


@@ -3718,8 +3713,9 @@ run(void) {
                        gettimeofday(&lastblink, NULL);
                        dodraw = 1;
                }
                        gettimeofday(&lastblink, NULL);
                        dodraw = 1;
                }

-               if(TIMEDIFF(now, last) \
-                               > (xev? (1000/xfps) : (1000/actionfps))) {
+               deltatime = TIMEDIFF(now, last);
+               if(deltatime > (xev? (1000/xfps) : (1000/actionfps))
+                               || deltatime < 0) {

                        dodraw = 1;
                        last = now;
                }
                        dodraw = 1;
                        last = now;
                }