[hackers] [st] The style inquisition was here. Yes, making it a unified style. The last || Christoph Lohmann

From: <hg_AT_suckless.org>
Date: Sat, 6 Oct 2012 10:01:23 +0200 (CEST)

changeset: 351:b0ba979227b1
tag: tip
user: Christoph Lohmann <20h_AT_r-36.net>
date: Sat Oct 06 09:58:45 2012 +0200
files: st.c
description:
The style inquisition was here. Yes, making it a unified style. The last
infidels will be squashed too!


diff -r 8ce150eeab43 -r b0ba979227b1 st.c
--- a/st.c Fri Oct 05 22:59:08 2012 +0200
+++ b/st.c Sat Oct 06 09:58:45 2012 +0200
_AT_@ -180,9 +180,9 @@
 typedef struct {
         int row; /* nb row */
         int col; /* nb col */
- Line* line; /* screen */
- Line* alt; /* alternate screen */
- bool* dirty; /* dirtyness of lines */
+ Line *line; /* screen */
+ Line *alt; /* alternate screen */
+ bool *dirty; /* dirtyness of lines */
         TCursor c; /* cursor */
         int top; /* top scroll limit */
         int bot; /* bottom scroll limit */
_AT_@ -242,7 +242,7 @@
         int descent;
         short lbearing;
         short rbearing;
- XftFont* xft_set;
+ XftFont *xft_set;
 } Font;
 
 /* Drawing Context */
_AT_@ -252,7 +252,7 @@
         Font font, bfont, ifont, ibfont;
 } DC;
 
-static void die(const char*, ...);
+static void die(const char *, ...);
 static void draw(void);
 static void redraw(void);
 static void drawregion(int, int, int, int);
_AT_@ -279,7 +279,7 @@
 static void tnew(int, int);
 static void tnewline(int);
 static void tputtab(bool);
-static void tputc(char*, int);
+static void tputc(char *, int);
 static void treset(void);
 static int tresize(int, int);
 static void tscrollup(int, int);
_AT_@ -313,7 +313,7 @@
 static void expose(XEvent *);
 static void visibility(XEvent *);
 static void unmap(XEvent *);
-static char* kmap(KeySym, uint);
+static char *kmap(KeySym, uint);
 static void kpress(XEvent *);
 static void cmessage(XEvent *);
 static void resize(XEvent *);
_AT_@ -378,8 +378,10 @@
 void *
 xmalloc(size_t len) {
         void *p = malloc(len);
+
         if(!p)
                 die("Out of memory\n");
+
         return p;
 }
 
_AT_@ -387,14 +389,17 @@
 xrealloc(void *p, size_t len) {
         if((p = realloc(p, len)) == NULL)
                 die("Out of memory\n");
+
         return p;
 }
 
 void *
 xcalloc(size_t nmemb, size_t size) {
         void *p = calloc(nmemb, size);
+
         if(!p)
                 die("Out of memory\n");
+
         return p;
 }
 
_AT_@ -455,8 +460,10 @@
         } else if((c & (B7|B6|B5|B4|B3)) == (B7|B6|B5|B4)) { /* 11110xxx */
                 *u = c & (B2|B1|B0);
                 n = 3;
- } else
+ } else {
                 goto invalid;
+ }
+
         for(i = n, ++s; i > 0; --i, ++rtn, ++s) {
                 c = *s;
                 if((c & (B7|B6)) != B7) /* 10xxxxxx */
_AT_@ -464,14 +471,18 @@
                 *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))
+ (*u >= 0xD800 && *u <= 0xDFFF)) {
                 goto invalid;
+ }
+
         return rtn;
 invalid:
         *u = 0xFFFD;
+
         return rtn;
 }
 
_AT_@ -481,7 +492,7 @@
         ulong uc;
         int i, n;
 
- sp = (uchar*) s;
+ sp = (uchar *)s;
         uc = *u;
         if(uc < 0x80) {
                 *sp = uc; /* 0xxxxxxx */
_AT_@ -498,14 +509,17 @@
         } else {
                 goto invalid;
         }
+
         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:
         /* U+FFFD */
         *s++ = '\xEF';
         *s++ = '\xBF';
         *s = '\xBD';
+
         return 3;
 }
 
_AT_@ -515,38 +529,40 @@
 isfullutf8(char *s, int b) {
         uchar *c1, *c2, *c3;
 
- c1 = (uchar *) s;
- c2 = (uchar *) ++s;
- c3 = (uchar *) ++s;
- if(b < 1)
+ c1 = (uchar *)s;
+ c2 = (uchar *)++s;
+ c3 = (uchar *)++s;
+ if(b < 1) {
                 return 0;
- else if((*c1&(B7|B6|B5)) == (B7|B6) && b == 1)
+ } else if((*c1&(B7|B6|B5)) == (B7|B6) && b == 1) {
                 return 0;
- else if((*c1&(B7|B6|B5|B4)) == (B7|B6|B5) &&
+ } else if((*c1&(B7|B6|B5|B4)) == (B7|B6|B5) &&
             ((b == 1) ||
- ((b == 2) && (*c2&(B7|B6)) == B7)))
+ ((b == 2) && (*c2&(B7|B6)) == B7))) {
                 return 0;
- else if((*c1&(B7|B6|B5|B4|B3)) == (B7|B6|B5|B4) &&
+ } 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)))
+ ((b == 3) && (*c2&(B7|B6)) == B7 && (*c3&(B7|B6)) == B7))) {
                 return 0;
- else
+ } else {
                 return 1;
+ }
 }
 
 int
 utf8size(char *s) {
         uchar c = *s;
 
- if(~c&B7)
+ if(~c&B7) {
                 return 1;
- else if((c&(B7|B6|B5)) == (B7|B6))
+ } else if((c&(B7|B6|B5)) == (B7|B6)) {
                 return 2;
- else if((c&(B7|B6|B5|B4)) == (B7|B6|B5))
+ } else if((c&(B7|B6|B5|B4)) == (B7|B6|B5)) {
                 return 3;
- else
+ } else {
                 return 4;
+ }
 }
 
 void
_AT_@ -563,13 +579,18 @@
 
 static inline bool
 selected(int x, int y) {
+ int bx, ex;
+
         if(sel.ey == y && sel.by == y) {
- int bx = MIN(sel.bx, sel.ex);
- int ex = MAX(sel.bx, sel.ex);
+ bx = MIN(sel.bx, sel.ex);
+ ex = MAX(sel.bx, sel.ex);
                 return BETWEEN(x, bx, ex);
         }
- return ((sel.b.y < y&&y < sel.e.y) || (y==sel.e.y && x<=sel.e.x))
- || (y==sel.b.y && x>=sel.b.x && (x<=sel.e.x || sel.b.y!=sel.e.y));
+
+ return ((sel.b.y < y&&y < sel.e.y)
+ || (y==sel.e.y && x<=sel.e.x))
+ || (y==sel.b.y && x>=sel.b.x
+ && (x<=sel.e.x || sel.b.y!=sel.e.y));
 }
 
 void
_AT_@ -621,9 +642,9 @@
 
 void
 bpress(XEvent *e) {
- if(IS_SET(MODE_MOUSE))
+ if(IS_SET(MODE_MOUSE)) {
                 mousereport(e);
- else if(e->xbutton.button == Button1) {
+ } else if(e->xbutton.button == Button1) {
                 if(sel.bx != -1) {
                         sel.bx = -1;
                         tsetdirt(sel.b.y, sel.e.y);
_AT_@ -637,22 +658,20 @@
 
 void
 selcopy(void) {
- char *str, *ptr;
- int x, y, bufsize, is_selected = 0;
+ char *str, *ptr, *p;
+ int x, y, bufsize, is_selected = 0, size;
+ Glyph *gp;
 
- if(sel.bx == -1)
+ if(sel.bx == -1) {
                 str = NULL;
-
- else {
+ } else {
                 bufsize = (term.col+1) * (sel.e.y-sel.b.y+1) * UTF_SIZ;
                 ptr = str = xmalloc(bufsize);
 
                 /* append every set & selected glyph to the selection */
                 for(y = 0; y < term.row; y++) {
                         for(x = 0; x < term.col; x++) {
- int size;
- char *p;
- Glyph *gp = &term.line[y][x];
+ gp = &term.line[y][x];
 
                                 if(!(is_selected = selected(x, y)))
                                         continue;
_AT_@ -694,8 +713,9 @@
 }
 
 void
-selpaste() {
- XConvertSelection(xw.dpy, XA_PRIMARY, sel.xtarget, XA_PRIMARY, xw.win, CurrentTime);
+selpaste(void) {
+ XConvertSelection(xw.dpy, XA_PRIMARY, sel.xtarget, XA_PRIMARY,
+ xw.win, CurrentTime);
 }
 
 void selclear(XEvent *e) {
_AT_@ -709,7 +729,7 @@
 selrequest(XEvent *e) {
         XSelectionRequestEvent *xsre;
         XSelectionEvent xev;
- Atom xa_targets;
+ Atom xa_targets, string;
 
         xsre = (XSelectionRequestEvent *) e;
         xev.type = SelectionNotify;
_AT_@ -723,7 +743,7 @@
         xa_targets = XInternAtom(xw.dpy, "TARGETS", 0);
         if(xsre->target == xa_targets) {
                 /* respond with the supported type */
- Atom string = sel.xtarget;
+ string = sel.xtarget;
                 XChangeProperty(xsre->display, xsre->requestor, xsre->property,
                                 XA_ATOM, 32, PropModeReplace,
                                 (uchar *) &string, 1);
_AT_@ -756,18 +776,20 @@
 
 void
 brelease(XEvent *e) {
+ struct timeval now;
+
         if(IS_SET(MODE_MOUSE)) {
                 mousereport(e);
                 return;
         }
- if(e->xbutton.button == Button2)
+
+ if(e->xbutton.button == Button2) {
                 selpaste();
- else if(e->xbutton.button == Button1) {
+ } else if(e->xbutton.button == Button1) {
                 sel.mode = 0;
                 getbuttoninfo(e, NULL, &sel.ex, &sel.ey);
                 term.dirty[sel.ey] = 1;
                 if(sel.bx == sel.ex && sel.by == sel.ey) {
- struct timeval now;
                         sel.bx = -1;
                         gettimeofday(&now, NULL);
 
_AT_@ -781,34 +803,44 @@
                                 /* double click to select word */
                                 sel.bx = sel.ex;
                                 while(sel.bx > 0 && term.line[sel.ey][sel.bx-1].state & GLYPH_SET &&
- term.line[sel.ey][sel.bx-1].c[0] != ' ') sel.bx--;
+ term.line[sel.ey][sel.bx-1].c[0] != ' ') {
+ sel.bx--;
+ }
                                 sel.b.x = sel.bx;
                                 while(sel.ex < term.col-1 && term.line[sel.ey][sel.ex+1].state & GLYPH_SET &&
- term.line[sel.ey][sel.ex+1].c[0] != ' ') sel.ex++;
+ term.line[sel.ey][sel.ex+1].c[0] != ' ') {
+ sel.ex++;
+ }
                                 sel.e.x = sel.ex;
                                 sel.b.y = sel.e.y = sel.ey;
                                 selcopy();
                         }
- } else
+ } else {
                         selcopy();
+ }
         }
+
         memcpy(&sel.tclick2, &sel.tclick1, sizeof(struct timeval));
         gettimeofday(&sel.tclick1, NULL);
 }
 
 void
 bmotion(XEvent *e) {
+ int starty, endy, oldey, oldex;
+
         if(IS_SET(MODE_MOUSE)) {
                 mousereport(e);
                 return;
         }
+
         if(sel.mode) {
- int oldey = sel.ey, oldex = sel.ex;
+ oldey = sel.ey;
+ oldex = sel.ex;
                 getbuttoninfo(e, NULL, &sel.ex, &sel.ey);
 
                 if(oldey != sel.ey || oldex != sel.ex) {
- int starty = MIN(oldey, sel.ey);
- int endy = MAX(oldey, sel.ey);
+ starty = MIN(oldey, sel.ey);
+ endy = MAX(oldey, sel.ey);
                         tsetdirt(starty, endy);
                 }
         }
_AT_@ -864,9 +896,9 @@
 void
 ttynew(void) {
         int m, s;
+ struct winsize w = {term.row, term.col, 0, 0};
 
         /* seems to work fine on linux, openbsd and freebsd */
- struct winsize w = {term.row, term.col, 0, 0};
         if(openpty(&m, &s, NULL, NULL, &w) < 0)
                 die("openpty failed: %s\n", SERRNO);
 
_AT_@ -905,6 +937,7 @@
 void
 dump(char c) {
         static int col;
+
         fprintf(stderr, " %02x '%c' ", c, isprint(c)?c:'.');
         if(++col % 10 == 0)
                 fprintf(stderr, "\n");
_AT_@ -931,7 +964,7 @@
                 charsize = utf8decode(ptr, &utf8c);
                 utf8encode(&utf8c, s);
                 tputc(s, charsize);
- ptr += charsize;
+ ptr += charsize;
                 buflen -= charsize;
         }
 
_AT_@ -958,8 +991,7 @@
 }
 
 void
-tsetdirt(int top, int bot)
-{
+tsetdirt(int top, int bot) {
         int i;
 
         LIMIT(top, 0, term.row-1);
_AT_@ -970,8 +1002,7 @@
 }
 
 void
-tfulldirt(void)
-{
+tfulldirt(void) {
         tsetdirt(0, term.row-1);
 }
 
_AT_@ -979,15 +1010,18 @@
 tcursor(int mode) {
         static TCursor c;
 
- if(mode == CURSOR_SAVE)
+ if(mode == CURSOR_SAVE) {
                 c = term.c;
- else if(mode == CURSOR_LOAD)
- term.c = c, tmoveto(c.x, c.y);
+ } else if(mode == CURSOR_LOAD) {
+ term.c = c;
+ tmoveto(c.x, c.y);
+ }
 }
 
 void
 treset(void) {
         uint i;
+
         term.c = (TCursor){{
                 .mode = ATTR_NULL,
                 .fg = DefaultFG,
_AT_@ -1024,7 +1058,8 @@
 
 void
 tswapscreen(void) {
- Line* tmp = term.line;
+ Line *tmp = term.line;
+
         term.line = term.alt;
         term.alt = tmp;
         term.mode ^= MODE_ALTSCREEN;
_AT_@ -1098,10 +1133,12 @@
 void
 tnewline(int first_col) {
         int y = term.c.y;
- if(y == term.bot)
+
+ if(y == term.bot) {
                 tscrollup(term.top, 1);
- else
+ } else {
                 y++;
+ }
         tmoveto(first_col ? 0 : term.c.x, y);
 }
 
_AT_@ -1119,11 +1156,12 @@
                         csiescseq.arg[csiescseq.narg] *= 10;
                         csiescseq.arg[csiescseq.narg] += *p++ - '0'/*, noarg = 0 */;
                 }
- if(*p == ';' && csiescseq.narg+1 < ESC_ARG_SIZ)
+ if(*p == ';' && csiescseq.narg+1 < ESC_ARG_SIZ) {
                         csiescseq.narg++, p++;
- else {
+ } else {
                         csiescseq.mode = *p;
                         csiescseq.narg++;
+
                         return;
                 }
         }
_AT_@ -1140,21 +1178,21 @@
 
 void
 tsetchar(char *c) {
+ char *vt100_0[62] = { /* 0x41 - 0x7e */
+ "↑", "↓", "→", "←", "█", "▚", "☃", /* A - G */
+ 0, 0, 0, 0, 0, 0, 0, 0, /* H - O */
+ 0, 0, 0, 0, 0, 0, 0, 0, /* P - W */
+ 0, 0, 0, 0, 0, 0, 0, " ", /* X - _ */
+ "◆", "▒", "␉", "␌", "␍", "␊", "°", "±", /* ` - g */
+ "␤", "␋", "┘", "┐", "┌", "└", "┼", "⎺", /* h - o */
+ "⎻", "─", "⎼", "⎽", "├", "┤", "┴", "┬", /* p - w */
+ "│", "≤", "≥", "π", "≠", "£", "·", /* x - ~ */
+ };
+
         /*
          * The table is proudly stolen from rxvt.
          */
         if(term.c.attr.mode & ATTR_GFX) {
- char *vt100_0[62] = { /* 0x41 - 0x7e */
- "↑", "↓", "→", "←", "█", "▚", "☃", /* A - G */
- 0, 0, 0, 0, 0, 0, 0, 0, /* H - O */
- 0, 0, 0, 0, 0, 0, 0, 0, /* P - W */
- 0, 0, 0, 0, 0, 0, 0, " ", /* X - _ */
- "◆", "▒", "␉", "␌", "␍", "␊", "°", "±", /* ` - g */
- "␤", "␋", "┘", "┐", "┌", "└", "┼", "⎺", /* h - o */
- "⎻", "─", "⎼", "⎽", "├", "┤", "┴", "┬", /* p - w */
- "│", "≤", "≥", "π", "≠", "£", "·", /* x - ~ */
- };
-
                 if(c[0] >= 0x41 && c[0] <= 0x7e
                                 && vt100_0[c[0] - 0x41]) {
                         c = vt100_0[c[0] - 0x41];
_AT_@ -1200,7 +1238,9 @@
                 tclearregion(term.c.x, term.c.y, term.col-1, term.c.y);
                 return;
         }
- memmove(&term.line[term.c.y][dst], &term.line[term.c.y][src], size * sizeof(Glyph));
+
+ memmove(&term.line[term.c.y][dst], &term.line[term.c.y][src],
+ size * sizeof(Glyph));
         tclearregion(term.col-n, term.c.y, term.col-1, term.c.y);
 }
 
_AT_@ -1216,7 +1256,9 @@
                 tclearregion(term.c.x, term.c.y, term.col-1, term.c.y);
                 return;
         }
- memmove(&term.line[term.c.y][dst], &term.line[term.c.y][src], size * sizeof(Glyph));
+
+ memmove(&term.line[term.c.y][dst], &term.line[term.c.y][src],
+ size * sizeof(Glyph));
         tclearregion(src, term.c.y, dst - 1, term.c.y);
 }
 
_AT_@ -1282,13 +1324,18 @@
                 case 38:
                         if(i + 2 < l && attr[i + 1] == 5) {
                                 i += 2;
- if(BETWEEN(attr[i], 0, 255))
+ if(BETWEEN(attr[i], 0, 255)) {
                                         term.c.attr.fg = attr[i];
- else
- fprintf(stderr, "erresc: bad fgcolor %d\n", attr[i]);
+ } else {
+ fprintf(stderr,
+ "erresc: bad fgcolor %d\n",
+ attr[i]);
+ }
+ } else {
+ fprintf(stderr,
+ "erresc(38): gfx attr %d unknown\n",
+ attr[i]);
                         }
- else
- fprintf(stderr, "erresc(38): gfx attr %d unknown\n", attr[i]);
                         break;
                 case 39:
                         term.c.attr.fg = DefaultFG;
_AT_@ -1296,28 +1343,36 @@
                 case 48:
                         if(i + 2 < l && attr[i + 1] == 5) {
                                 i += 2;
- if(BETWEEN(attr[i], 0, 255))
+ if(BETWEEN(attr[i], 0, 255)) {
                                         term.c.attr.bg = attr[i];
- else
- fprintf(stderr, "erresc: bad bgcolor %d\n", attr[i]);
+ } else {
+ fprintf(stderr,
+ "erresc: bad bgcolor %d\n",
+ attr[i]);
+ }
+ } else {
+ fprintf(stderr,
+ "erresc(48): gfx attr %d unknown\n",
+ attr[i]);
                         }
- else
- fprintf(stderr, "erresc(48): gfx attr %d unknown\n", attr[i]);
                         break;
                 case 49:
                         term.c.attr.bg = DefaultBG;
                         break;
                 default:
- if(BETWEEN(attr[i], 30, 37))
+ if(BETWEEN(attr[i], 30, 37)) {
                                 term.c.attr.fg = attr[i] - 30;
- else if(BETWEEN(attr[i], 40, 47))
+ } else if(BETWEEN(attr[i], 40, 47)) {
                                 term.c.attr.bg = attr[i] - 40;
- else if(BETWEEN(attr[i], 90, 97))
+ } else if(BETWEEN(attr[i], 90, 97)) {
                                 term.c.attr.fg = attr[i] - 90 + 8;
- else if(BETWEEN(attr[i], 100, 107))
+ } else if(BETWEEN(attr[i], 100, 107)) {
                                 term.c.attr.bg = attr[i] - 100 + 8;
- else
- fprintf(stderr, "erresc(default): gfx attr %d unknown\n", attr[i]), csidump();
+ } else {
+ fprintf(stderr,
+ "erresc(default): gfx attr %d unknown\n",
+ attr[i]), csidump();
+ }
                         break;
                 }
         }
_AT_@ -1573,9 +1628,9 @@
                 tsetattr(csiescseq.arg, csiescseq.narg);
                 break;
         case 'r': /* DECSTBM -- Set Scrolling Region */
- if(csiescseq.priv)
+ if(csiescseq.priv) {
                         goto unknown;
- else {
+ } else {
                         DEFAULT(csiescseq.arg[0], 1);
                         DEFAULT(csiescseq.arg[1], term.row);
                         tsetscroll(csiescseq.arg[0]-1, csiescseq.arg[1]-1);
_AT_@ -1594,14 +1649,22 @@
 void
 csidump(void) {
         int i;
+ uint c;
+
         printf("ESC[");
         for(i = 0; i < csiescseq.len; i++) {
- uint c = csiescseq.buf[i] & 0xff;
- if(isprint(c)) putchar(c);
- else if(c == '\n') printf("(\\n)");
- else if(c == '\r') printf("(\\r)");
- else if(c == 0x1b) printf("(\\e)");
- else printf("(%02x)", c);
+ c = csiescseq.buf[i] & 0xff;
+ if(isprint(c)) {
+ putchar(c);
+ } else if(c == '\n') {
+ printf("(\\n)");
+ } else if(c == '\r') {
+ printf("(\\r)");
+ } else if(c == 0x1b) {
+ printf("(\\e)");
+ } else {
+ printf("(%02x)", c);
+ }
         }
         putchar('\n');
 }
_AT_@ -1672,14 +1735,22 @@
 void
 strdump(void) {
         int i;
+ uint c;
+
         printf("ESC%c", strescseq.type);
         for(i = 0; i < strescseq.len; i++) {
- uint c = strescseq.buf[i] & 0xff;
- if(isprint(c)) putchar(c);
- else if(c == '\n') printf("(\\n)");
- else if(c == '\r') printf("(\\r)");
- else if(c == 0x1b) printf("(\\e)");
- else printf("(%02x)", c);
+ c = strescseq.buf[i] & 0xff;
+ if(isprint(c)) {
+ putchar(c);
+ } else if(c == '\n') {
+ printf("(\\n)");
+ } else if(c == '\r') {
+ printf("(\\r)");
+ } else if(c == 0x1b) {
+ printf("(\\e)");
+ } else {
+ printf("(%02x)", c);
+ }
         }
         printf("ESC\\\n");
 }
_AT_@ -1746,7 +1817,8 @@
         if(term.esc & ESC_START) {
                 if(term.esc & ESC_CSI) {
                         csiescseq.buf[csiescseq.len++] = ascii;
- if(BETWEEN(ascii, 0x40, 0x7E) || csiescseq.len >= ESC_BUF_SIZ) {
+ if(BETWEEN(ascii, 0x40, 0x7E)
+ || csiescseq.len >= ESC_BUF_SIZ) {
                                 term.esc = 0;
                                 csiparse(), csihandle();
                         }
_AT_@ -1811,10 +1883,11 @@
                                 term.esc = 0;
                                 break;
                         case 'D': /* IND -- Linefeed */
- if(term.c.y == term.bot)
+ if(term.c.y == term.bot) {
                                         tscrollup(term.top, 1);
- else
+ } else {
                                         tmoveto(term.c.x, term.c.y+1);
+ }
                                 term.esc = 0;
                                 break;
                         case 'E': /* NEL -- Next line */
_AT_@ -1826,10 +1899,11 @@
                                 term.esc = 0;
                                 break;
                         case 'M': /* RI -- Reverse index */
- if(term.c.y == term.top)
+ if(term.c.y == term.top) {
                                         tscrolldown(term.top, 1);
- else
+ } else {
                                         tmoveto(term.c.x, term.c.y-1);
+ }
                                 term.esc = 0;
                                 break;
                         case 'c': /* RIS -- Reset to inital state */
_AT_@ -1859,7 +1933,7 @@
                                 break;
                         default:
                                 fprintf(stderr, "erresc: unknown sequence ESC 0x%02X '%c'\n",
- (uchar) ascii, isprint(ascii)?ascii:'.');
+ (uchar) ascii, isprint(ascii)? ascii:'.');
                                 term.esc = 0;
                         }
                 }
_AT_@ -1870,10 +1944,11 @@
                         if(IS_SET(MODE_WRAP) && term.c.state & CURSOR_WRAPNEXT)
                                 tnewline(1); /* always go to first col */
                         tsetchar(c);
- if(term.c.x+1 < term.col)
+ if(term.c.x+1 < term.col) {
                                 tmoveto(term.c.x+1, term.c.y);
- else
+ } else {
                                 term.c.state |= CURSOR_WRAPNEXT;
+ }
                 }
         }
 }
_AT_@ -1884,6 +1959,7 @@
         int minrow = MIN(row, term.row);
         int mincol = MIN(col, term.col);
         int slide = term.c.y - row + 1;
+ bool *bp;
 
         if(col < 1 || row < 1)
                 return 0;
_AT_@ -1930,7 +2006,7 @@
                 term.alt [i] = xcalloc(col, sizeof(Glyph));
         }
         if(col > term.col) {
- bool *bp = term.tabs + term.col;
+ bp = term.tabs + term.col;
 
                 memset(bp, 0, sizeof(*term.tabs) * (col - term.col));
                 while(--bp > term.tabs && !*bp)
_AT_@ -1987,7 +2063,8 @@
 
         for(r = 0; r < 24; r++, i++) {
                 xft_color.red = xft_color.green = xft_color.blue = 0x0808 + 0x0a0a * r;
- if(!XftColorAllocValue(xw.dpy, xw.vis, xw.cmap, &xft_color, &dc.xft_col[i])) {
+ if(!XftColorAllocValue(xw.dpy, xw.vis, xw.cmap, &xft_color,
+ &dc.xft_col[i])) {
                         die("Could not allocate color %d\n", i);
                 }
         }
_AT_@ -2263,8 +2340,7 @@
 
 void
 xdrawcursor(void) {
- static int oldx = 0;
- static int oldy = 0;
+ static int oldx = 0, oldy = 0;
         int sl;
         Glyph g = {{' '}, ATTR_NULL, DefaultBG, DefaultCS, 0};
 
_AT_@ -2277,9 +2353,11 @@
         /* remove the old cursor */
         if(term.line[oldy][oldx].state & GLYPH_SET) {
                 sl = utf8size(term.line[oldy][oldx].c);
- xdraws(term.line[oldy][oldx].c, term.line[oldy][oldx], oldx, oldy, 1, sl);
- } else
+ xdraws(term.line[oldy][oldx].c, term.line[oldy][oldx], oldx,
+ oldy, 1, sl);
+ } else {
                 xtermclear(oldx, oldy, oldx, oldy);
+ }
 
         /* draw the new one */
         if(!(term.c.state & CURSOR_HIDE)) {
_AT_@ -2334,6 +2412,7 @@
         for(y = y1; y < y2; y++) {
                 if(!term.dirty[y])
                         continue;
+
                 xtermclear(0, y, term.col, y);
                 term.dirty[y] = 0;
                 base = term.line[y][0];
_AT_@ -2342,8 +2421,9 @@
                         new = term.line[y][x];
                         if(ena_sel && *(new.c) && selected(x, y))
                                 new.mode ^= ATTR_REVERSE;
- if(ib > 0 && (!(new.state & GLYPH_SET) || ATTRCMP(base, new) ||
- ib >= DRAW_BUF_SIZ-UTF_SIZ)) {
+ if(ib > 0 && (!(new.state & GLYPH_SET)
+ || ATTRCMP(base, new)
+ || ib >= DRAW_BUF_SIZ-UTF_SIZ)) {
                                 xdraws(buf, base, ox, y, ic, ib);
                                 ic = ib = 0;
                         }
_AT_@ -2367,6 +2447,7 @@
 void
 expose(XEvent *ev) {
         XExposeEvent *e = &ev->xexpose;
+
         if(xw.state & WIN_REDRAW) {
                 if(!e->count)
                         xw.state &= ~WIN_REDRAW;
_AT_@ -2376,11 +2457,13 @@
 void
 visibility(XEvent *ev) {
         XVisibilityEvent *e = &ev->xvisibility;
- if(e->state == VisibilityFullyObscured)
+
+ if(e->state == VisibilityFullyObscured) {
                 xw.state &= ~WIN_VISIBLE;
- else if(!(xw.state & WIN_VISIBLE))
+ } else if(!(xw.state & WIN_VISIBLE)) {
                 /* need a full redraw for next Expose, not just a buf copy */
                 xw.state |= WIN_VISIBLE | WIN_REDRAW;
+ }
 }
 
 void
_AT_@ -2391,6 +2474,7 @@
 void
 xseturgency(int add) {
         XWMHints *h = XGetWMHints(xw.dpy, xw.win);
+
         h->flags = add ? (h->flags | XUrgencyHint) : (h->flags & ~XUrgencyHint);
         XSetWMHints(xw.dpy, xw.win, h);
         XFree(h);
_AT_@ -2401,18 +2485,24 @@
         if(ev->type == FocusIn) {
                 xw.state |= WIN_FOCUSED;
                 xseturgency(0);
- } else
+ } else {
                 xw.state &= ~WIN_FOCUSED;
+ }
 }
 
 char*
 kmap(KeySym k, uint state) {
         int i;
+ uint mask;
+
         state &= ~Mod2Mask;
         for(i = 0; i < LEN(key); i++) {
- uint mask = key[i].mask;
- if(key[i].k == k && ((state & mask) == mask || (mask == XK_NO_MOD && !state)))
+ mask = key[i].mask;
+
+ if(key[i].k == k && ((state & mask) == mask
+ || (mask == XK_NO_MOD && !state))) {
                         return (char*)key[i].s;
+ }
         }
         return NULL;
 }
_AT_@ -2430,22 +2520,25 @@
 
         if (IS_SET(MODE_KBDLOCK))
                 return;
+
         meta = e->state & Mod1Mask;
         shift = e->state & ShiftMask;
         len = XmbLookupString(xw.xic, e, buf, sizeof(buf), &ksym, &status);
 
         /* 1. custom keys from config.h */
- if((customkey = kmap(ksym, e->state)))
+ if((customkey = kmap(ksym, e->state))) {
                 ttywrite(customkey, strlen(customkey));
         /* 2. hardcoded (overrides X lookup) */
- else
+ } else {
                 switch(ksym) {
                 case XK_Up:
                 case XK_Down:
                 case XK_Left:
                 case XK_Right:
                         /* XXX: shift up/down doesn't work */
- sprintf(buf, "\033%c%c", IS_SET(MODE_APPKEYPAD) ? 'O' : '[', (shift ? "dacb":"DACB")[ksym - XK_Left]);
+ sprintf(buf, "\033%c%c",
+ IS_SET(MODE_APPKEYPAD) ? 'O' : '[',
+ (shift ? "dacb":"DACB")[ksym - XK_Left]);
                         ttywrite(buf, 3);
                         break;
                 case XK_Insert:
_AT_@ -2453,10 +2546,11 @@
                                 selpaste();
                         break;
                 case XK_Return:
- if(IS_SET(MODE_CRLF))
+ if(IS_SET(MODE_CRLF)) {
                                 ttywrite("\r\n", 2);
- else
+ } else {
                                 ttywrite("\r", 1);
+ }
                         break;
                         /* 3. X lookup */
                 default:
_AT_@ -2467,6 +2561,7 @@
                         }
                         break;
                 }
+ }
 }
 
 void
_AT_@ -2618,7 +2713,7 @@
                 }
         }
 
- run:
+run:
         setlocale(LC_CTYPE, "");
         tnew(80, 24);
         ttynew();
Received on Sat Oct 06 2012 - 10:01:23 CEST

This archive was generated by hypermail 2.3.0 : Sat Oct 06 2012 - 10:12:05 CEST