JasonWoof Got questions, comments, patches, etc.? Contact Jason Woofenden
change -e behaviour and update man page.
[st.git] / st.c
diff --git a/st.c b/st.c
index b25fe89..2be6772 100644 (file)
--- a/st.c
+++ b/st.c
@@ -23,7 +23,7 @@
 
 #if   defined(__linux)
  #include <pty.h>
-#elif defined(__OpenBSD__) || defined(__NetBSD__)
+#elif defined(__OpenBSD__) || defined(__NetBSD__) || defined(__APPLE__)
  #include <util.h>
 #elif defined(__FreeBSD__) || defined(__DragonFly__)
  #include <libutil.h>
@@ -31,7 +31,7 @@
 
 #define USAGE \
        "st-" VERSION ", (c) 2010 st engineers\n" \
-       "usage: st [-t title] [-e cmd] [-v]\n"
+       "usage: st [-t title] [-c class] [-e cmd] [-v]\n"
 
 /* Arbitrary sizes */
 #define ESC_TITLE_SIZ 256
@@ -110,7 +110,7 @@ typedef struct {
 
 /* Purely graphic info */
 typedef struct {
-       Display* dis;
+       Display* dpy;
        Colormap cmap;
        Window win;
        Pixmap buf;
@@ -131,20 +131,17 @@ typedef struct {
        char s[ESC_BUF_SIZ];
 } Key;
 
-typedef struct {
-       XFontSet fs;
-       short lbearing;
-       short rbearing;
-       int ascent;
-       int descent;
-} FontInfo;
-
 /* Drawing Context */
 typedef struct {
        unsigned long col[256];
-       FontInfo font;
-       FontInfo bfont;
        GC gc;
+       struct {
+               int ascent;
+               int descent;
+               short lbearing;
+               short rbearing;
+               XFontSet set;
+       } font, bfont;
 } DC;
 
 /* TODO: use better name for vars... */
@@ -222,10 +219,10 @@ static inline int selected(int, int);
 static void selcopy(void);
 static void selpaste(void);
 
-static int stou(char *, long *);
-static int utos(long *, char *);
-static int slen(char *);
-static int canstou(char *, int);
+static int utf8decode(char *, long *);
+static int utf8encode(long *, char *);
+static int utf8size(char *);
+static int isfullutf8(char *, int);
 
 static void (*handler[LASTEvent])(XEvent *) = {
        [KeyPress] = kpress,
@@ -250,11 +247,12 @@ static CSIEscape escseq;
 static int cmdfd;
 static pid_t pid;
 static Selection sel;
-static char *opt_cmd   = NULL;
+static char **opt_cmd  = NULL;
 static char *opt_title = NULL;
+static char *opt_class = NULL;
 
-/* UTF-8 decode */
-static int stou(char *s, long *u) {
+int
+utf8decode(char *s, long *u) {
        unsigned char c;
        int i, n, rtn;
 
@@ -263,28 +261,28 @@ static int stou(char *s, long *u) {
        if(~c&B7) { /* 0xxxxxxx */
                *u = c;
                return rtn;
-       } else if ((c&(B7|B6|B5)) == (B7|B6)) { /* 110xxxxx */
+       } else if((c&(B7|B6|B5)) == (B7|B6)) { /* 110xxxxx */
                *u = c&(B4|B3|B2|B1|B0);
                n = 1;
-       } else if ((c&(B7|B6|B5|B4)) == (B7|B6|B5)) { /* 1110xxxx */
+       } else if((c&(B7|B6|B5|B4)) == (B7|B6|B5)) { /* 1110xxxx */
                *u = c&(B3|B2|B1|B0);
                n = 2;
-       } else if ((c&(B7|B6|B5|B4|B3)) == (B7|B6|B5|B4)) { /* 11110xxx */
+       } else if((c&(B7|B6|B5|B4|B3)) == (B7|B6|B5|B4)) { /* 11110xxx */
                *u = c&(B2|B1|B0);
                n = 3;
        } else
                goto invalid;
-       for (i=n,++s; i>0; --i,++rtn,++s) {
+       for(i=n,++s; i>0; --i,++rtn,++s) {
                c = *s;
-               if ((c&(B7|B6)) != B7) /* 10xxxxxx */
+               if((c&(B7|B6)) != B7) /* 10xxxxxx */
                        goto invalid;
                *u <<= 6;
                *u |= c&(B5|B4|B3|B2|B1|B0);
        }
-       if ((n == 1 && *u < 0x80) ||
-           (n == 2 && *u < 0x800) ||
-           (n == 3 && *u < 0x10000) ||
-           (*u >= 0xD800 && *u <= 0xDFFF))
+       if((n == 1 && *u < 0x80) ||
+          (n == 2 && *u < 0x800) ||
+          (n == 3 && *u < 0x10000) ||
+          (*u >= 0xD800 && *u <= 0xDFFF))
                goto invalid;
        return rtn;
 invalid:
@@ -292,30 +290,30 @@ invalid:
        return rtn;
 }
 
-/* UTF-8 encode */
-static int utos(long *u, char *s) {
+int
+utf8encode(long *u, char *s) {
        unsigned char *sp;
        unsigned long uc;
        int i, n;
 
        sp = (unsigned char*) s;
        uc = *u;
-       if (uc < 0x80) {
+       if(uc < 0x80) {
                *sp = uc; /* 0xxxxxxx */
                return 1;
-       } else if (*u < 0x800) {
+       } else if(*u < 0x800) {
                *sp = (uc >> 6) | (B7|B6); /* 110xxxxx */
                n = 1;
-       } else if (uc < 0x10000) {
+       } else if(uc < 0x10000) {
                *sp = (uc >> 12) | (B7|B6|B5); /* 1110xxxx */
                n = 2;
-       } else if (uc <= 0x10FFFF) {
+       } else if(uc <= 0x10FFFF) {
                *sp = (uc >> 18) | (B7|B6|B5|B4); /* 11110xxx */
                n = 3;
        } else {
                goto invalid;
        }
-       for (i=n,++sp; i>0; --i,++sp)
+       for(i=n,++sp; i>0; --i,++sp)
                *sp = ((uc >> 6*(i-1)) & (B5|B4|B3|B2|B1|B0)) | B7; /* 10xxxxxx */
        return n+1;
 invalid:
@@ -328,34 +326,32 @@ invalid:
 
 /* use this if your buffer is less than UTF_SIZ, it returns 1 if you can decode
    UTF-8 otherwise return 0 */
-static int canstou(char *s, int b) {
-       unsigned char c = *s;
-       int n;
+int
+isfullutf8(char *s, int b) {
+       unsigned char *c1, *c2, *c3;
 
-       if (b < 1)
+       c1 = (unsigned char *) s;
+       c2 = (unsigned char *) ++s;
+       c3 = (unsigned char *) ++s;
+       if(b < 1)
                return 0;
-       else if (~c&B7)
-               return 1;
-       else if ((c&(B7|B6|B5)) == (B7|B6))
-               n = 1;
-       else if ((c&(B7|B6|B5|B4)) == (B7|B6|B5))
-               n = 2;
-       else if ((c&(B7|B6|B5|B4|B3)) == (B7|B6|B5|B4))
-               n = 3;
-       else
-               return 1;
-       for (--b,++s; n>0&&b>0; --n,--b,++s) {
-               c = *s;
-               if ((c&(B7|B6)) != B7)
-                       break; 
-       }
-       if (n > 0 && b == 0)
+       else if((*c1&(B7|B6|B5)) == (B7|B6) && b == 1)
+               return 0;
+       else if((*c1&(B7|B6|B5|B4)) == (B7|B6|B5) &&
+           ((b == 1) || 
+           ((b == 2) && (*c2&(B7|B6)) == B7)))
+               return 0;
+       else if((*c1&(B7|B6|B5|B4|B3)) == (B7|B6|B5|B4) &&
+           ((b == 1) ||
+           ((b == 2) && (*c2&(B7|B6)) == B7) ||
+           ((b == 3) && (*c2&(B7|B6)) == B7 && (*c3&(B7|B6)) == B7)))
                return 0;
        else
                return 1;
 }
 
-static int slen(char *s) {
+int
+utf8size(char *s) {
        unsigned char c = *s;
 
        if (~c&B7)
@@ -368,13 +364,15 @@ static int slen(char *s) {
                return 4;
 }
 
-static void selinit(void) {
+void
+selinit(void) {
        sel.mode = 0;
        sel.bx = -1;
        sel.clip = NULL;
 }
 
-static inline int selected(int x, int y) {
+static inline int 
+selected(int x, int y) {
        if(sel.ey == y && sel.by == y) {
                int bx = MIN(sel.bx, sel.ex);
                int ex = MAX(sel.bx, sel.ex);
@@ -384,7 +382,8 @@ static inline int selected(int x, int y) {
                || (y==sel.b.y && x>=sel.b.x && (x<=sel.e.x || sel.b.y!=sel.e.y));
 }
 
-static void getbuttoninfo(XEvent *e, int *b, int *x, int *y) {
+void
+getbuttoninfo(XEvent *e, int *b, int *x, int *y) {
        if(b) 
                *b = e->xbutton.button;
 
@@ -396,15 +395,17 @@ static void getbuttoninfo(XEvent *e, int *b, int *x, int *y) {
        sel.e.y = MAX(sel.by, sel.ey);
 }
 
-static void bpress(XEvent *e) {
+void
+bpress(XEvent *e) {
        sel.mode = 1;
        sel.ex = sel.bx = e->xbutton.x/xw.cw;
        sel.ey = sel.by = e->xbutton.y/xw.ch;
 }
 
-static void selcopy() {
+void
+selcopy(void) {
        char *str, *ptr;
-       int ls, x, y, sz, sl;
+       int x, y, sz, sl, ls = 0;
 
        if(sel.bx == -1)
                str = NULL;
@@ -414,7 +415,7 @@ static void selcopy() {
                for(y = 0; y < term.row; y++) {
                        for(x = 0; x < term.col; x++)
                                if(term.line[y][x].state & GLYPH_SET && (ls = selected(x, y))) {
-                                       sl = slen(term.line[y][x].c);
+                                       sl = utf8size(term.line[y][x].c);
                                        memcpy(ptr, term.line[y][x].c, sl);
                                        ptr += sl;
                                }
@@ -426,7 +427,8 @@ static void selcopy() {
        xsetsel(str);
 }
 
-static void selnotify(XEvent *e) {
+void
+selnotify(XEvent *e) {
        unsigned long nitems;
        unsigned long ofs, rem;
        int format;
@@ -435,7 +437,7 @@ static void selnotify(XEvent *e) {
 
        ofs = 0;
        do {
-               if(XGetWindowProperty(xw.dis, xw.win, XA_PRIMARY, ofs, BUFSIZ/4,
+               if(XGetWindowProperty(xw.dpy, xw.win, XA_PRIMARY, ofs, BUFSIZ/4,
                                        False, AnyPropertyType, &type, &format,
                                        &nitems, &rem, &data)) {
                        fprintf(stderr, "Clipboard allocation failed\n");
@@ -448,12 +450,13 @@ static void selnotify(XEvent *e) {
        } while(rem > 0);
 }
 
-static void selpaste() {
-       XConvertSelection(xw.dis, XA_PRIMARY, XA_STRING, XA_PRIMARY, xw.win, CurrentTime);
+void
+selpaste() {
+       XConvertSelection(xw.dpy, XA_PRIMARY, XA_STRING, XA_PRIMARY, xw.win, CurrentTime);
 }
 
-static void selrequest(XEvent *e)
-{
+void
+selrequest(XEvent *e) {
        XSelectionRequestEvent *xsre;
        XSelectionEvent xev;
        Atom xa_targets;
@@ -467,7 +470,7 @@ static void selrequest(XEvent *e)
        /* reject */
        xev.property = None;
 
-       xa_targets = XInternAtom(xw.dis, "TARGETS", 0);
+       xa_targets = XInternAtom(xw.dpy, "TARGETS", 0);
        if(xsre->target == xa_targets) {
                /* respond with the supported type */
                Atom string = XA_STRING;
@@ -487,23 +490,25 @@ static void selrequest(XEvent *e)
                fprintf(stderr, "Error sending SelectionNotify event\n");
 }
 
-static void xsetsel(char *str) {
+void
+xsetsel(char *str) {
        /* register the selection for both the clipboard and the primary */
        Atom clipboard;
 
        free(sel.clip);
        sel.clip = str;
 
-       XSetSelectionOwner(xw.dis, XA_PRIMARY, xw.win, CurrentTime);
+       XSetSelectionOwner(xw.dpy, XA_PRIMARY, xw.win, CurrentTime);
 
-       clipboard = XInternAtom(xw.dis, "CLIPBOARD", 0);
-       XSetSelectionOwner(xw.dis, clipboard, xw.win, CurrentTime);
+       clipboard = XInternAtom(xw.dpy, "CLIPBOARD", 0);
+       XSetSelectionOwner(xw.dpy, clipboard, xw.win, CurrentTime);
 
-       XFlush(xw.dis);
+       XFlush(xw.dpy);
 }
 
 /* TODO: doubleclick to select word */
-static void brelease(XEvent *e) {
+void
+brelease(XEvent *e) {
        int b;
        sel.mode = 0;
        getbuttoninfo(e, &b, &sel.ex, &sel.ey);
@@ -518,33 +523,16 @@ static void brelease(XEvent *e) {
        draw(1);
 }
 
-static void bmotion(XEvent *e) {
+void
+bmotion(XEvent *e) {
        if (sel.mode) {
                getbuttoninfo(e, NULL, &sel.ex, &sel.ey);
-               //      draw(1);
+               /* XXX: draw() can't keep up, disabled for now.
+                  selection is visible on button release.
+                  draw(1); */
        }
 }
 
-#ifdef DEBUG
-void
-tdump(void) {
-       int row, col;
-       Glyph c;
-
-       for(row = 0; row < term.row; row++) {
-               for(col = 0; col < term.col; col++) {
-                       if(col == term.c.x && row == term.c.y)
-                               putchar('#');
-                       else {
-                               c = term.line[row][col];
-                               putchar(c.state & GLYPH_SET ? c.c : '.');
-                       }
-               }
-               putchar('\n');
-       }
-}
-#endif
-
 void
 die(const char *errstr, ...) {
        va_list ap;
@@ -557,13 +545,14 @@ die(const char *errstr, ...) {
 
 void
 execsh(void) {
-       char *args[] = {getenv("SHELL"), "-i", NULL};
-       if(opt_cmd)
-               args[0] = opt_cmd, args[1] = NULL;
-       else
-               DEFAULT(args[0], SHELL);
+       char **args;
+       char *envshell = getenv("SHELL");
+
+       DEFAULT(envshell, "sh");
        putenv("TERM="TNAME);
+       args = opt_cmd ? opt_cmd : (char*[]){envshell, "-i", NULL};
        execvp(args[0], args);
+       exit(EXIT_FAILURE);
 }
 
 void 
@@ -618,24 +607,31 @@ dump(char c) {
 
 void
 ttyread(void) {
-       char buf[BUFSIZ], *ptr;
+       static char buf[BUFSIZ];
+       static int buflen = 0; 
+       char *ptr;
        char s[UTF_SIZ];
-       int ret, br;
-       static int buflen = 0;
-       long u;
+       int charsize; /* size of utf8 char in bytes */
+       long utf8c;
+       int ret;
 
+       /* append read bytes to unprocessed bytes */
        if((ret = read(cmdfd, buf+buflen, LEN(buf)-buflen)) < 0)
                die("Couldn't read from shell: %s\n", SERRNO);
-       else {
-               buflen += ret;
-               for(ptr=buf; buflen>=UTF_SIZ||canstou(ptr,buflen); buflen-=br) {
-                       br = stou(ptr, &u);
-                       utos(&u, s);
-                       tputc(s);
-                       ptr += br;
-               }
-               memcpy(buf, ptr, buflen);
+
+       /* process every complete utf8 char */
+       buflen += ret;
+       ptr = buf;
+       while(buflen >= UTF_SIZ || isfullutf8(ptr,buflen)) {
+               charsize = utf8decode(ptr, &utf8c);
+               utf8encode(&utf8c, s);
+               tputc(s);
+               ptr    += charsize;
+               buflen -= charsize;
        }
+
+       /* keep any uncomplete utf8 char for the next call */
+       memmove(buf, ptr, buflen);
 }
 
 void
@@ -1197,7 +1193,7 @@ tputc(char *c) {
                        if(ascii == '\a' || term.titlelen+1 >= ESC_TITLE_SIZ) {
                                term.esc = 0;
                                term.title[term.titlelen] = '\0';
-                               XStoreName(xw.dis, xw.win, term.title);
+                               XStoreName(xw.dpy, xw.win, term.title);
                        } else {
                                term.title[term.titlelen++] = ascii;
                        }
@@ -1372,22 +1368,22 @@ xresize(int col, int row) {
        oldh = xw.bufh;
        xw.bufw = MAX(1, col * xw.cw);
        xw.bufh = MAX(1, row * xw.ch);
-       newbuf = XCreatePixmap(xw.dis, xw.win, xw.bufw, xw.bufh, XDefaultDepth(xw.dis, xw.scr));
-       XCopyArea(xw.dis, xw.buf, newbuf, dc.gc, 0, 0, xw.bufw, xw.bufh, 0, 0);
-       XFreePixmap(xw.dis, xw.buf);
-       XSetForeground(xw.dis, dc.gc, dc.col[DefaultBG]);
+       newbuf = XCreatePixmap(xw.dpy, xw.win, xw.bufw, xw.bufh, XDefaultDepth(xw.dpy, xw.scr));
+       XCopyArea(xw.dpy, xw.buf, newbuf, dc.gc, 0, 0, xw.bufw, xw.bufh, 0, 0);
+       XFreePixmap(xw.dpy, xw.buf);
+       XSetForeground(xw.dpy, dc.gc, dc.col[DefaultBG]);
        if(xw.bufw > oldw)
-               XFillRectangle(xw.dis, newbuf, dc.gc, oldw, 0,
+               XFillRectangle(xw.dpy, newbuf, dc.gc, oldw, 0,
                                xw.bufw-oldw, MIN(xw.bufh, oldh));
        else if(xw.bufw < oldw && (BORDER > 0 || xw.w > xw.bufw))
-               XClearArea(xw.dis, xw.win, BORDER+xw.bufw, BORDER,
+               XClearArea(xw.dpy, xw.win, BORDER+xw.bufw, BORDER,
                                xw.w-xw.bufh-BORDER, BORDER+MIN(xw.bufh, oldh),
                                False);
        if(xw.bufh > oldh)
-               XFillRectangle(xw.dis, newbuf, dc.gc, 0, oldh,
+               XFillRectangle(xw.dpy, newbuf, dc.gc, 0, oldh,
                                xw.bufw, xw.bufh-oldh);
        else if(xw.bufh < oldh && (BORDER > 0 || xw.h > xw.bufh))
-               XClearArea(xw.dis, xw.win, BORDER, BORDER+xw.bufh,
+               XClearArea(xw.dpy, xw.win, BORDER, BORDER+xw.bufh,
                                xw.w-2*BORDER, xw.h-xw.bufh-BORDER,
                                False);
        xw.buf = newbuf;
@@ -1397,10 +1393,10 @@ void
 xloadcols(void) {
        int i, r, g, b;
        XColor color;
-       unsigned long white = WhitePixel(xw.dis, xw.scr);
+       unsigned long white = WhitePixel(xw.dpy, xw.scr);
 
        for(i = 0; i < 16; i++) {
-               if (!XAllocNamedColor(xw.dis, xw.cmap, colorname[i], &color, &color)) {
+               if (!XAllocNamedColor(xw.dpy, xw.cmap, colorname[i], &color, &color)) {
                        dc.col[i] = white;
                        fprintf(stderr, "Could not allocate color '%s'\n", colorname[i]);
                } else
@@ -1414,7 +1410,7 @@ xloadcols(void) {
                                color.red = r == 0 ? 0 : 0x3737 + 0x2828 * r;
                                color.green = g == 0 ? 0 : 0x3737 + 0x2828 * g;
                                color.blue = b == 0 ? 0 : 0x3737 + 0x2828 * b;
-                               if (!XAllocColor(xw.dis, xw.cmap, &color)) {
+                               if (!XAllocColor(xw.dpy, xw.cmap, &color)) {
                                        dc.col[i] = white;
                                        fprintf(stderr, "Could not allocate color %d\n", i);
                                } else
@@ -1424,7 +1420,7 @@ xloadcols(void) {
 
        for(r = 0; r < 24; r++, i++) {
                color.red = color.green = color.blue = 0x0808 + 0x0a0a * r;
-               if (!XAllocColor(xw.dis, xw.cmap, &color)) {
+               if (!XAllocColor(xw.dpy, xw.cmap, &color)) {
                        dc.col[i] = white;
                        fprintf(stderr, "Could not allocate color %d\n", i);
                } else
@@ -1434,8 +1430,8 @@ xloadcols(void) {
 
 void
 xclear(int x1, int y1, int x2, int y2) {
-       XSetForeground(xw.dis, dc.gc, dc.col[DefaultBG]);
-       XFillRectangle(xw.dis, xw.buf, dc.gc,
+       XSetForeground(xw.dpy, dc.gc, dc.col[DefaultBG]);
+       XFillRectangle(xw.dpy, xw.buf, dc.gc,
                       x1 * xw.cw, y1 * xw.ch,
                       (x2-x1+1) * xw.cw, (y2-y1+1) * xw.ch);
 }
@@ -1443,7 +1439,7 @@ xclear(int x1, int y1, int x2, int y2) {
 void
 xhints(void)
 {
-       XClassHint class = {TNAME, TNAME};
+       XClassHint class = {opt_class ? opt_class : TNAME, TNAME};
        XWMHints wm = {.flags = InputHint, .input = 1};
        XSizeHints size = {
                .flags = PSize | PResizeInc | PBaseSize,
@@ -1454,59 +1450,73 @@ xhints(void)
                .base_height = 2*BORDER,
                .base_width = 2*BORDER,
        };
-       XSetWMProperties(xw.dis, xw.win, NULL, NULL, NULL, 0, &size, &wm, &class);
+       XSetWMProperties(xw.dpy, xw.win, NULL, NULL, NULL, 0, &size, &wm, &class);
+}
+
+XFontSet
+xinitfont(char *fontstr)
+{
+       XFontSet set;
+       char *def, **missing;
+       int n;
+
+       missing = NULL;
+       set = XCreateFontSet(xw.dpy, fontstr, &missing, &n, &def);
+       if(missing) {
+               while(n--)
+                       fprintf(stderr, "st: missing fontset: %s\n", missing[n]);
+               XFreeStringList(missing);
+       }
+       return set;
 }
 
 void
-xsetfontinfo(FontInfo *fi)
+xgetfontinfo(XFontSet set, int *ascent, int *descent, short *lbearing, short *rbearing)
 {
        XFontStruct **xfonts;
-       int fnum;
-       int i;
-       char **fontnames;
-
-       fi->lbearing = 0;
-       fi->rbearing = 0;
-       fi->ascent = 0;
-       fi->descent = 0;
-       fnum = XFontsOfFontSet(fi->fs, &xfonts, &fontnames);
-       for(i=0; i<fnum; i++,xfonts++,fontnames++) {
-               puts(*fontnames);
-               if(fi->ascent < (*xfonts)->ascent)
-                       fi->ascent = (*xfonts)->ascent;
-               if(fi->descent < (*xfonts)->descent)
-                       fi->descent = (*xfonts)->descent;
-               if(fi->rbearing < (*xfonts)->max_bounds.rbearing)
-                       fi->rbearing = (*xfonts)->max_bounds.rbearing;
-               if(fi->lbearing < (*xfonts)->min_bounds.lbearing)
-                       fi->lbearing = (*xfonts)->min_bounds.lbearing;
+       char **font_names;
+       int i, n;
+
+       *ascent = *descent = *lbearing = *rbearing = 0;
+       n = XFontsOfFontSet(set, &xfonts, &font_names);
+       for(i = 0; i < n; i++) {
+               *ascent = MAX(*ascent, (*xfonts)->ascent);
+               *descent = MAX(*descent, (*xfonts)->descent);
+               *lbearing = MAX(*lbearing, (*xfonts)->min_bounds.lbearing);
+               *rbearing = MAX(*rbearing, (*xfonts)->max_bounds.rbearing);
+               xfonts++;
        }
 }
 
 void
+initfonts(char *fontstr, char *bfontstr)
+{
+       if((dc.font.set = xinitfont(fontstr)) == NULL ||
+          (dc.bfont.set = xinitfont(bfontstr)) == NULL)
+               die("Can't load font %s\n", dc.font.set ? BOLDFONT : FONT);
+       xgetfontinfo(dc.font.set, &dc.font.ascent, &dc.font.descent,
+           &dc.font.lbearing, &dc.font.rbearing);
+       xgetfontinfo(dc.bfont.set, &dc.bfont.ascent, &dc.bfont.descent,
+           &dc.bfont.lbearing, &dc.bfont.rbearing);
+}
+
+void
 xinit(void) {
        XSetWindowAttributes attrs;
-       char **mc;
-       char *ds;
-       int nmc;
 
-       if(!(xw.dis = XOpenDisplay(NULL)))
+       if(!(xw.dpy = XOpenDisplay(NULL)))
                die("Can't open display\n");
-       xw.scr = XDefaultScreen(xw.dis);
+       xw.scr = XDefaultScreen(xw.dpy);
        
        /* font */
-       if ((dc.font.fs = XCreateFontSet(xw.dis, FONT, &mc, &nmc, &ds)) == NULL ||
-           (dc.bfont.fs = XCreateFontSet(xw.dis, BOLDFONT, &mc, &nmc, &ds)) == NULL)
-               die("Can't load font %s\n", dc.font.fs ? BOLDFONT : FONT); 
-       xsetfontinfo(&dc.font);
-       xsetfontinfo(&dc.bfont);
+       initfonts(FONT, BOLDFONT);
 
        /* XXX: Assuming same size for bold font */
        xw.cw = dc.font.rbearing - dc.font.lbearing;
        xw.ch = dc.font.ascent + dc.font.descent;
 
        /* colors */
-       xw.cmap = XDefaultColormap(xw.dis, xw.scr);
+       xw.cmap = XDefaultColormap(xw.dpy, xw.scr);
        xloadcols();
 
        /* window - default size */
@@ -1523,33 +1533,33 @@ xinit(void) {
                | PointerMotionMask | ButtonPressMask | ButtonReleaseMask;
        attrs.colormap = xw.cmap;
 
-       xw.win = XCreateWindow(xw.dis, XRootWindow(xw.dis, xw.scr), 0, 0,
-                       xw.w, xw.h, 0, XDefaultDepth(xw.dis, xw.scr), InputOutput,
-                       XDefaultVisual(xw.dis, xw.scr),
+       xw.win = XCreateWindow(xw.dpy, XRootWindow(xw.dpy, xw.scr), 0, 0,
+                       xw.w, xw.h, 0, XDefaultDepth(xw.dpy, xw.scr), InputOutput,
+                       XDefaultVisual(xw.dpy, xw.scr),
                        CWBackPixel | CWBorderPixel | CWBitGravity | CWEventMask
                        | CWColormap,
                        &attrs);
-       xw.buf = XCreatePixmap(xw.dis, xw.win, xw.bufw, xw.bufh, XDefaultDepth(xw.dis, xw.scr));
+       xw.buf = XCreatePixmap(xw.dpy, xw.win, xw.bufw, xw.bufh, XDefaultDepth(xw.dpy, xw.scr));
 
 
        /* input methods */
-       xw.xim = XOpenIM(xw.dis, NULL, NULL, NULL);
+       xw.xim = XOpenIM(xw.dpy, NULL, NULL, NULL);
        xw.xic = XCreateIC(xw.xim, XNInputStyle, XIMPreeditNothing 
                                           | XIMStatusNothing, XNClientWindow, xw.win, 
                                           XNFocusWindow, xw.win, NULL);
        /* gc */
-       dc.gc = XCreateGC(xw.dis, xw.win, 0, NULL);
+       dc.gc = XCreateGC(xw.dpy, xw.win, 0, NULL);
        
-       XMapWindow(xw.dis, xw.win);
+       XMapWindow(xw.dpy, xw.win);
        xhints();
-       XStoreName(xw.dis, xw.win, opt_title ? opt_title : "st");
-       XSync(xw.dis, 0);
+       XStoreName(xw.dpy, xw.win, opt_title ? opt_title : "st");
+       XSync(xw.dpy, 0);
 }
 
 void
-xdraws(char *s, Glyph base, int x, int y, int cl, int sl) {
+xdraws(char *s, Glyph base, int x, int y, int charlen, int bytelen) {
        unsigned long xfg, xbg;
-       int winx = x*xw.cw, winy = y*xw.ch + dc.font.ascent, width = cl*xw.cw;
+       int winx = x*xw.cw, winy = y*xw.ch + dc.font.ascent, width = charlen*xw.cw;
        int i;
 
        if(base.mode & ATTR_REVERSE)
@@ -1557,23 +1567,24 @@ xdraws(char *s, Glyph base, int x, int y, int cl, int sl) {
        else
                xfg = dc.col[base.fg], xbg = dc.col[base.bg];
 
-       XSetBackground(xw.dis, dc.gc, xbg);
-       XSetForeground(xw.dis, dc.gc, xfg);
+       XSetBackground(xw.dpy, dc.gc, xbg);
+       XSetForeground(xw.dpy, dc.gc, xfg);
 
-       if(base.mode & ATTR_GFX)
-               for(i = 0; i < cl; i++) {
+       if(base.mode & ATTR_GFX) {
+               for(i = 0; i < bytelen; i++) {
                        char c = gfx[(unsigned int)s[i] % 256];
                        if(c)
                                s[i] = c;
                        else if(s[i] > 0x5f)
                                s[i] -= 0x5f;
                }
+       }
 
-       XmbDrawImageString(xw.dis, xw.buf, base.mode & ATTR_BOLD ? dc.bfont.fs : dc.font.fs,
-           dc.gc, winx, winy, s, sl);
+       XmbDrawImageString(xw.dpy, xw.buf, base.mode & ATTR_BOLD ? dc.bfont.set : dc.font.set,
+           dc.gc, winx, winy, s, bytelen);
        
        if(base.mode & ATTR_UNDERLINE)
-               XDrawLine(xw.dis, xw.buf, dc.gc, winx, winy+1, winx+width-1, winy+1);
+               XDrawLine(xw.dpy, xw.buf, dc.gc, winx, winy+1, winx+width-1, winy+1);
 }
 
 void
@@ -1591,14 +1602,14 @@ xdrawcursor(void) {
 
        /* remove the old cursor */
        if(term.line[oldy][oldx].state & GLYPH_SET) {
-               sl = slen(term.line[oldy][oldx].c);
+               sl = utf8size(term.line[oldy][oldx].c);
                xdraws(term.line[oldy][oldx].c, term.line[oldy][oldx], oldx, oldy, 1, sl);
        } else
                xclear(oldx, oldy, oldx, oldy);
        
        /* draw the new one */
        if(!(term.c.state & CURSOR_HIDE) && (xw.state & WIN_FOCUSED)) {
-               sl = slen(g.c);
+               sl = utf8size(g.c);
                xdraws(g.c, g, term.c.x, term.c.y, 1, sl);
                oldx = term.c.x, oldy = term.c.y;
        }
@@ -1608,11 +1619,11 @@ xdrawcursor(void) {
 /* basic drawing routines */
 void
 xdrawc(int x, int y, Glyph g) {
-       int sl = slen(g.c);
+       int sl = utf8size(g.c);
        XRectangle r = { x * xw.cw, y * xw.ch, xw.cw, xw.ch };
-       XSetBackground(xw.dis, dc.gc, dc.col[g.bg]);
-       XSetForeground(xw.dis, dc.gc, dc.col[g.fg]);
-       XmbDrawImageString(xw.dis, xw.buf, g.mode&ATTR_BOLD?dc.bfont.fs:dc.font.fs,
+       XSetBackground(xw.dpy, dc.gc, dc.col[g.bg]);
+       XSetForeground(xw.dpy, dc.gc, dc.col[g.fg]);
+       XmbDrawImageString(xw.dpy, xw.buf, g.mode&ATTR_BOLD?dc.bfont.fs:dc.font.fs,
            dc.gc, r.x, r.y+dc.font.ascent, g.c, sl);
 }
 
@@ -1627,8 +1638,8 @@ draw(int dummy) {
                                xdrawc(x, y, term.line[y][x]);
 
        xdrawcursor();
-       XCopyArea(xw.dis, xw.buf, xw.win, dc.gc, 0, 0, xw.bufw, xw.bufh, BORDER, BORDER);
-       XFlush(xw.dis);
+       XCopyArea(xw.dpy, xw.buf, xw.win, dc.gc, 0, 0, xw.bufw, xw.bufh, BORDER, BORDER);
+       XFlush(xw.dpy);
 }
 
 #else
@@ -1660,7 +1671,7 @@ draw(int redraw_all) {
                                        ox = x;
                                        base = new;
                                }
-                               sl = slen(new.c);
+                               sl = utf8size(new.c);
                                memcpy(buf+ib, new.c, sl);
                                ib += sl;
                                ++ic;
@@ -1670,7 +1681,7 @@ draw(int redraw_all) {
                        xdraws(buf, base, ox, y, ic, ib);
        }
        xdrawcursor();
-       XCopyArea(xw.dis, xw.buf, xw.win, dc.gc, 0, 0, xw.bufw, xw.bufh, BORDER, BORDER);
+       XCopyArea(xw.dpy, xw.buf, xw.win, dc.gc, 0, 0, xw.bufw, xw.bufh, BORDER, BORDER);
 }
 
 #endif
@@ -1684,7 +1695,7 @@ expose(XEvent *ev) {
                        draw(SCREEN_REDRAW);
                }
        } else
-               XCopyArea(xw.dis, xw.buf, xw.win, dc.gc, e->x-BORDER, e->y-BORDER,
+               XCopyArea(xw.dpy, xw.buf, xw.win, dc.gc, e->x-BORDER, e->y-BORDER,
                                e->width, e->height, e->x, e->y);
 }
 
@@ -1705,9 +1716,9 @@ unmap(XEvent *ev) {
 
 void
 xseturgency(int add) {
-       XWMHints *h = XGetWMHints(xw.dis, xw.win);
+       XWMHints *h = XGetWMHints(xw.dpy, xw.win);
        h->flags = add ? (h->flags | XUrgencyHint) : (h->flags & ~XUrgencyHint);
-       XSetWMHints(xw.dis, xw.win, h);
+       XSetWMHints(xw.dpy, xw.win, h);
        XFree(h);
 }
 
@@ -1771,7 +1782,6 @@ kpress(XEvent *ev) {
                        /* 3. X lookup  */
                default:
                        if(len > 0) {
-                               buf[sizeof(buf)-1] = '\0';
                                if(meta && len == 1)
                                        ttywrite("\033", 1);
                                ttywrite(buf, len);
@@ -1804,7 +1814,7 @@ void
 run(void) {
        XEvent ev;
        fd_set rfd;
-       int xfd = XConnectionNumber(xw.dis);
+       int xfd = XConnectionNumber(xw.dpy);
 
        for(;;) {
                FD_ZERO(&rfd);
@@ -1819,8 +1829,8 @@ run(void) {
                        ttyread();
                        draw(SCREEN_UPDATE); 
                }
-               while(XPending(xw.dis)) {
-                       XNextEvent(xw.dis, &ev);
+               while(XPending(xw.dpy)) {
+                       XNextEvent(xw.dpy, &ev);
                        if (XFilterEvent(&ev, xw.win))
                                continue;
                        if(handler[ev.type])
@@ -1838,13 +1848,19 @@ main(int argc, char *argv[]) {
                case 't':
                        if(++i < argc) opt_title = argv[i];
                        break;
+               case 'c':
+                       if(++i < argc) opt_class = argv[i];
+                       break;
                case 'e':
-                       if(++i < argc) opt_cmd = argv[i];
+                       if(++i < argc) opt_cmd = &argv[i];
                        break;
                case 'v':
                default:
                        die(USAGE);
                }
+               /* -e eats every remaining arguments */
+               if(opt_cmd)
+                       break;
        }
        setlocale(LC_CTYPE, "");
        tnew(80, 24);