Pull term references out of xdrawcursor
[st.git] / x.c
diff --git a/x.c b/x.c
index c5826cf..d205ca7 100644 (file)
--- a/x.c
+++ b/x.c
@@ -129,7 +129,6 @@ static int xmakeglyphfontspecs(XftGlyphFontSpec *, const Glyph *, int, int, int)
 static void xdrawglyphfontspecs(const XftGlyphFontSpec *, Glyph, int, int, int);
 static void xdrawglyph(Glyph, int, int);
 static void xclear(int, int, int, int);
-static void xdrawcursor(void);
 static int xgeommasktogravity(int);
 static void xinit(void);
 static void cresize(int, int);
@@ -1388,41 +1387,26 @@ xdrawglyph(Glyph g, int x, int y)
 }
 
 void
-xdrawcursor(void)
+xdrawcursor(int cx, int cy, Glyph g, int ox, int oy, Glyph og)
 {
-       static int oldx = 0, oldy = 0;
-       int curx;
-       Glyph g = {' ', ATTR_NULL, defaultbg, defaultcs}, og;
        Color drawcol;
 
-       LIMIT(oldx, 0, term.col-1);
-       LIMIT(oldy, 0, term.row-1);
-
-       curx = term.c.x;
-
-       /* adjust position if in dummy */
-       if (term.line[oldy][oldx].mode & ATTR_WDUMMY)
-               oldx--;
-       if (term.line[term.c.y][curx].mode & ATTR_WDUMMY)
-               curx--;
-
        /* remove the old cursor */
-       og = term.line[oldy][oldx];
-       if (selected(oldx, oldy))
+       if (selected(ox, oy))
                og.mode ^= ATTR_REVERSE;
-       xdrawglyph(og, oldx, oldy);
-
-       g.u = term.line[term.c.y][term.c.x].u;
-       g.mode |= term.line[term.c.y][term.c.x].mode &
-                 (ATTR_BOLD | ATTR_ITALIC | ATTR_UNDERLINE | ATTR_STRUCK);
+       xdrawglyph(og, ox, oy);
 
        /*
         * Select the right color for the right mode.
         */
+       g.mode &= ATTR_BOLD|ATTR_ITALIC|ATTR_UNDERLINE|ATTR_STRUCK|ATTR_WIDE;
+       g.fg = defaultbg;
+       g.bg = defaultcs;
+
        if (IS_SET(MODE_REVERSE)) {
                g.mode |= ATTR_REVERSE;
                g.bg = defaultfg;
-               if (selected(term.c.x, term.c.y)) {
+               if (selected(cx, cy)) {
                        drawcol = dc.col[defaultcs];
                        g.fg = defaultrcs;
                } else {
@@ -1430,7 +1414,7 @@ xdrawcursor(void)
                        g.fg = defaultcs;
                }
        } else {
-               if (selected(term.c.x, term.c.y)) {
+               if (selected(cx, cy)) {
                        drawcol = dc.col[defaultrcs];
                        g.fg = defaultfg;
                        g.bg = defaultrcs;
@@ -1450,44 +1434,42 @@ xdrawcursor(void)
                case 0: /* Blinking Block */
                case 1: /* Blinking Block (Default) */
                case 2: /* Steady Block */
-                       g.mode |= term.line[term.c.y][curx].mode & ATTR_WIDE;
-                       xdrawglyph(g, term.c.x, term.c.y);
+                       xdrawglyph(g, cx, cy);
                        break;
                case 3: /* Blinking Underline */
                case 4: /* Steady Underline */
                        XftDrawRect(xw.draw, &drawcol,
-                                       borderpx + curx * win.cw,
-                                       borderpx + (term.c.y + 1) * win.ch - \
+                                       borderpx + cx * win.cw,
+                                       borderpx + (cy + 1) * win.ch - \
                                                cursorthickness,
                                        win.cw, cursorthickness);
                        break;
                case 5: /* Blinking bar */
                case 6: /* Steady bar */
                        XftDrawRect(xw.draw, &drawcol,
-                                       borderpx + curx * win.cw,
-                                       borderpx + term.c.y * win.ch,
+                                       borderpx + cx * win.cw,
+                                       borderpx + cy * win.ch,
                                        cursorthickness, win.ch);
                        break;
                }
        } else {
                XftDrawRect(xw.draw, &drawcol,
-                               borderpx + curx * win.cw,
-                               borderpx + term.c.y * win.ch,
+                               borderpx + cx * win.cw,
+                               borderpx + cy * win.ch,
                                win.cw - 1, 1);
                XftDrawRect(xw.draw, &drawcol,
-                               borderpx + curx * win.cw,
-                               borderpx + term.c.y * win.ch,
+                               borderpx + cx * win.cw,
+                               borderpx + cy * win.ch,
                                1, win.ch - 1);
                XftDrawRect(xw.draw, &drawcol,
-                               borderpx + (curx + 1) * win.cw - 1,
-                               borderpx + term.c.y * win.ch,
+                               borderpx + (cx + 1) * win.cw - 1,
+                               borderpx + cy * win.ch,
                                1, win.ch - 1);
                XftDrawRect(xw.draw, &drawcol,
-                               borderpx + curx * win.cw,
-                               borderpx + (term.c.y + 1) * win.ch - 1,
+                               borderpx + cx * win.cw,
+                               borderpx + (cy + 1) * win.ch - 1,
                                win.cw, 1);
        }
-       oldx = curx, oldy = term.c.y;
 }
 
 void
@@ -1512,59 +1494,51 @@ xsettitle(char *p)
        XFree(prop.value);
 }
 
-void
-draw(void)
+int
+xstartdraw(void)
 {
-       drawregion(0, 0, term.col, term.row);
-       XCopyArea(xw.dpy, xw.buf, xw.win, dc.gc, 0, 0, win.w,
-                       win.h, 0, 0);
-       XSetForeground(xw.dpy, dc.gc,
-                       dc.col[IS_SET(MODE_REVERSE)?
-                               defaultfg : defaultbg].pixel);
+       return IS_SET(MODE_VISIBLE);
 }
 
 void
-drawregion(int x1, int y1, int x2, int y2)
+xdrawline(Line line, int x1, int y1, int x2)
 {
-       int i, x, y, ox, numspecs;
+       int i, x, ox, numspecs;
        Glyph base, new;
-       XftGlyphFontSpec *specs;
+       XftGlyphFontSpec *specs = xw.specbuf;
 
-       if (!(IS_SET(MODE_VISIBLE)))
-               return;
-
-       for (y = y1; y < y2; y++) {
-               if (!term.dirty[y])
+       numspecs = xmakeglyphfontspecs(specs, &line[x1], x2 - x1, x1, y1);
+       i = ox = 0;
+       for (x = x1; x < x2 && i < numspecs; x++) {
+               new = line[x];
+               if (new.mode == ATTR_WDUMMY)
                        continue;
-
-               term.dirty[y] = 0;
-
-               specs = xw.specbuf;
-               numspecs = xmakeglyphfontspecs(specs, &term.line[y][x1], x2 - x1, x1, y);
-
-               i = ox = 0;
-               for (x = x1; x < x2 && i < numspecs; x++) {
-                       new = term.line[y][x];
-                       if (new.mode == ATTR_WDUMMY)
-                               continue;
-                       if (selected(x, y))
-                               new.mode ^= ATTR_REVERSE;
-                       if (i > 0 && ATTRCMP(base, new)) {
-                               xdrawglyphfontspecs(specs, base, i, ox, y);
-                               specs += i;
-                               numspecs -= i;
-                               i = 0;
-                       }
-                       if (i == 0) {
-                               ox = x;
-                               base = new;
-                       }
-                       i++;
+               if (selected(x, y1))
+                       new.mode ^= ATTR_REVERSE;
+               if (i > 0 && ATTRCMP(base, new)) {
+                       xdrawglyphfontspecs(specs, base, i, ox, y1);
+                       specs += i;
+                       numspecs -= i;
+                       i = 0;
                }
-               if (i > 0)
-                       xdrawglyphfontspecs(specs, base, i, ox, y);
+               if (i == 0) {
+                       ox = x;
+                       base = new;
+               }
+               i++;
        }
-       xdrawcursor();
+       if (i > 0)
+               xdrawglyphfontspecs(specs, base, i, ox, y1);
+}
+
+void
+xfinishdraw(void)
+{
+       XCopyArea(xw.dpy, xw.buf, xw.win, dc.gc, 0, 0, win.w,
+                       win.h, 0, 0);
+       XSetForeground(xw.dpy, dc.gc,
+                       dc.col[IS_SET(MODE_REVERSE)?
+                               defaultfg : defaultbg].pixel);
 }
 
 void