speedup X11 display

git-svn-id: https://ssl.bulix.org/svn/lcd4linux/trunk@961 3ae390bd-cb1e-0410-b409-cd5a39f66f1f

diff --git a/drv_X11.c b/drv_X11.c
index 7b53319f5dc59aa188d0254383a77c5ab92e3a2c..767160e6738b3936262cc39a4db135198e4e532f 100644 (file)
--- a/drv_X11.c
+++ b/drv_X11.c
@@ -54,6 +54,7 @@
 #include "qprintf.h"
 #include "timer.h"
 #include "plugin.h"
+#include "rgb.h"
 #include "drv.h"
 #include "widget.h"
 #include "widget_keypad.h"
@@ -114,7 +115,6 @@ static void drv_X11_color(RGBA c, int brightness)
 static void drv_X11_blit(const int row, const int col, const int height, const int width)
 {
     int r, c;
-    int dirty = 0;
 
     for (r = row; r < row + height; r++) {
        int y = border + (r / YRES) * rgap + r * (pixel + pgap);
@@ -123,16 +123,13 @@ static void drv_X11_blit(const int row, const int col, const int height, const i
            RGBA p1 = drv_X11_FB[r * DCOLS + c];
            RGBA p2 = drv_generic_graphic_rgb(r, c);
            if (p1.R != p2.R || p1.G != p2.G || p1.B != p2.B) {
-               drv_X11_color(p2, 255); /* Fixme: this call is so slow... */
-               XFillRectangle(dp, w, gc, x, y, pixel, pixel);
+               //drv_X11_color(p2, 255);       /* Fixme: this call is so slow... */
+               //XFillRectangle(dp, w, gc, x, y, pixel, pixel);
+               XClearArea(dp, w, x, y, pixel, pixel, 1);
                drv_X11_FB[r * DCOLS + c] = p2;
-               dirty = 1;
            }
        }
     }
-    if (dirty) {
-       XSync(dp, False);
-    }
 }
 
 
@@ -229,6 +226,11 @@ static void drv_X11_expose(const int x, const int y, const int width, const int
     int yk;
     char *s;
     char unknownTxt[10];
+    XRectangle rect[DROWS * DCOLS];
+    int nrect = 0;
+    RGBA col;
+    RGBA lastCol = {0, 0, 0, 0};
+    int hasLastCol = 0;
 
     x0 = x - pixel;
     x1 = x + pixel + width;
@@ -243,10 +245,33 @@ static void drv_X11_expose(const int x, const int y, const int width, const int
            int xc = border + (c / XRES) * cgap + c * (pixel + pgap);
            if (xc < x0 || xc > x1)
                continue;
-           drv_X11_color(drv_generic_graphic_rgb(r, c), 255);
+           col = drv_generic_graphic_rgb(r, c);
+            if (hasLastCol) {
+                /* if the color of this pixel is different to the last pixels draw the old ones */
+                if (col.R != lastCol.R || col.G != lastCol.G || col.B != lastCol.B) {
+                    drv_X11_color(lastCol, 255);
+                    XFillRectangles(dp, w, gc, rect, nrect);
+                    nrect = 0;
+                    lastCol = col;
+                }
+                rect[nrect].x = xc;
+                rect[nrect].y = yc;
+                rect[nrect].width = pixel;
+                rect[nrect].height = pixel;
+                nrect++;
+            } else {
+                /* 1st shot: no old color */
+                drv_X11_color(col, 255);
+                XFillRectangle(dp, w, gc, xc, yc, pixel, pixel);
+                lastCol = col;
+                hasLastCol = 1;
+            }
            XFillRectangle(dp, w, gc, xc, yc, pixel, pixel);
        }
     }
+    /* draw the last block of rectangles */
+    drv_X11_color(lastCol, 255);
+    XFillRectangles(dp, w, gc, rect, nrect);
 
     /* Keypad on the right side */
     if (x1 >= xoffset) {
@@ -294,6 +319,7 @@ static void drv_X11_timer( __attribute__ ((unused))
                          void *notused)
 {
     XEvent ev;
+    XRectangle exp;
     int xoffset = border + (DCOLS / XRES) * cgap + DCOLS * (pixel + pgap);
     int yoffset = border + (DROWS / YRES) * rgap;
     static int btn = 0;
@@ -304,7 +330,28 @@ static void drv_X11_timer( __attribute__ ((unused))
     switch (ev.type) {
 
     case Expose:
-       drv_X11_expose(ev.xexpose.x, ev.xexpose.y, ev.xexpose.width, ev.xexpose.height);
+        /* collect all expose events in eventqueue */
+        exp.x = ev.xexpose.x;
+        exp.y = ev.xexpose.y;
+        exp.width = ev.xexpose.width;
+        exp.height = ev.xexpose.height;
+        while (XCheckWindowEvent(dp, w, ExposureMask, &ev)) {
+            if (ev.xexpose.x < exp.x) {
+                exp.width += exp.x - ev.xexpose.x;
+                exp.x -= exp.x - ev.xexpose.x;
+            }
+            if (ev.xexpose.y < exp.y) {
+                exp.height += exp.y - ev.xexpose.y;
+                exp.y -= exp.y - ev.xexpose.y;
+            }
+            if (ev.xexpose.x + ev.xexpose.width > exp.x + exp.width) {
+                exp.width += ev.xexpose.x + ev.xexpose.width - (exp.x + exp.width);
+            }
+            if (ev.xexpose.y + ev.xexpose.height > exp.y + exp.height) {
+                exp.height += ev.xexpose.y + ev.xexpose.height - (exp.y + exp.height);
+            }
+        }
+       drv_X11_expose(exp.x, exp.y, exp.width, exp.height);
        break;
 
     case ButtonPress:
@@ -545,6 +592,7 @@ int drv_X11_init(const char *section, const int quiet)
        char buffer[40];
        qprintf(buffer, sizeof(buffer), "%s %dx%d", Name, DCOLS, DROWS);
        if (drv_generic_graphic_greet(buffer, NULL)) {
+            drv_X11_expose(0, 0, dimx + 2 * border, dimy + 2 * border);
            sleep(3);
            drv_generic_graphic_clear();
        }