WWLCDPanel has two bitmap data, and the bitmap once drawn is

Fixed not to draw.  Reduced the load on the X server.
(Improvements related to WWLCDPanel and XDisplay_DrawLCDWindow () of XDisplay.c)
(XDisplay_DrawLCDWindow () in XDisplay.c has useless malloc ()
Improvement is necessary of that)

In WWDisplay.c (WWDisplay_DrawScreen ()) of wonx-a 05 and wonx-a 06
As a result of trying both, wonx - a 05 was faster, so wonx - a 05 was better
I decided to use it.

Version 0.0.7 alpha - from wonx-a07.tar.gz

HISTORY

@@ -1,3 +1,31 @@
+2000/10/5(<28><>)
+
+wonx-a07 <20><><EFBFBD><EFBFBD>
+
+WWLCDPanel <20>˥ӥåȥޥåץǡ<D7A5><C7A1><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>褷<EFBFBD><E8A4B7><EFBFBD>ӥåȥޥåפ<C3A5>
+<EFBFBD><EFBFBD><EFBFBD>褷<EFBFBD>ʤ<EFBFBD><EFBFBD>褦<EFBFBD>˽<EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>X<EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>Ф<EFBFBD><EFBFBD><EFBFBD><EFBFBD>٤򸺤餷<EFBFBD><EFBFBD><EFBFBD><EFBFBD>
+(WWLCDPanel<65><6C>Ϣ<EFBFBD>ȡ<EFBFBD>XDisplay.c <20><> XDisplay_DrawLCDWindow() <20><><EFBFBD><EFBFBD><EFBFBD><EFBFBD>)
+(XDisplay.c <20><> XDisplay_DrawLCDWindow() <20>ˤϡ<CBA4>̵<EFBFBD>̤<EFBFBD> malloc() <20><><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>Τǡ<CEA4>
+ <20><><EFBFBD>Τ<EFBFBD><CEA4><EFBFBD><EFBFBD><EFBFBD><EFBFBD>ɤ<EFBFBD>ɬ<EFBFBD><C9AC>)
+
+wonx-a05 <20><> wonx-a06 <20><> WWDisplay.c (WWDisplay_DrawScreen()) <20><>
+ξ<EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>̡<EFBFBD>wonx-a05 <20>Τۤ<CEA4><DBA4><EFBFBD><EFBFBD><EFBFBD>®<EFBFBD><C2AE><EFBFBD>ä<EFBFBD><C3A4>Τǡ<CEA4>wonx-a05 <20>Τۤ<CEA4><DBA4><EFBFBD>
+<EFBFBD><EFBFBD><EFBFBD>Ѥ<EFBFBD><EFBFBD>뤳<EFBFBD>Ȥˤ<EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>
+
+
+
+
+2000/10/4(<28><>)
+
+wonx-a06 <20><><EFBFBD><EFBFBD>
+
+WWDisplay_DrawScreen() <20>Υ<EFBFBD><CEA5>르<EFBFBD>ꥺ<EFBFBD><EAA5BA><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>˽<EFBFBD><CBBD><EFBFBD><EFBFBD><EFBFBD>
+<EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>ƹ<EFBFBD>®<EFBFBD>ˤʤ<EFBFBD><EFBFBD>ʤ<EFBFBD><EFBFBD>ä<EFBFBD><EFBFBD><EFBFBD>X <20><><EFBFBD><EFBFBD><EFBFBD>Ф<EFBFBD><D0A4><EFBFBD><EFBFBD>褬<EFBFBD>ۥåȥ<C3A5><C8A5>ݥåȤˤʤäƤ<C3A4><C6A4><EFBFBD><EFBFBD><EFBFBD>
+<EFBFBD>פ<EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>Τǡ<EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>̤򸺤餹<EFBFBD><EFBFBD><EFBFBD>Ϥ<EFBFBD>ɬ<EFBFBD>ס<EFBFBD>
+
+
+
+
 2000/10/3(<28><>)
 
 wonx-a05 <20><><EFBFBD><EFBFBD>

Makefile

@@ -2,8 +2,8 @@ XINCLUDEDIR = /usr/X11R6/include
 INCLUDEDIR = .
 XLIBDIR = /usr/X11R6/lib
 
-VERSION = Wonx-a05
-PKGNAME = wonx-a05
+VERSION = Wonx-a07
+PKGNAME = wonx-a07
 
 OBJS = WWCharacter.o WWColorMap.o WWDisplay.o WWLCDPanel.o WWPalette.o WWScreen.o WWSprite.o WonxDisplay.o XDisplay.o bank.o comm.o disp.o text.o key.o sound.o system.o timer.o etc.o wonx.o
 

WWLCDPanel.c

@@ -13,52 +13,95 @@ int WWLCDPanel_GetHeight(WWLCDPanel p) { return (p->height); }
 int WWLCDPanel_SetWidth( WWLCDPanel p, int n) { return (p->width  = n); }
 int WWLCDPanel_SetHeight(WWLCDPanel p, int n) { return (p->height = n); }
 
-unsigned char * WWLCDPanel_GetPixelMap(WWLCDPanel p) { return (p->pixel); }
+int WWLCDPanel_ResetCurrent(WWLCDPanel p)
+{
+  return (p->current = 0);
+}
+
+int WWLCDPanel_ReverseCurrent(WWLCDPanel p)
+{
+  return (p->current = 1 - p->current);
+}
+
+int WWLCDPanel_ResetAllDraw(WWLCDPanel p) { return (p->all_draw = 0); }
+int WWLCDPanel_SetAllDraw(WWLCDPanel p)   { return (p->all_draw = 1); }
+int WWLCDPanel_IsAllDraw(WWLCDPanel p)   { return (p->all_draw); }
+
+unsigned char * WWLCDPanel_GetPixelMap(WWLCDPanel p)
+{
+  return (p->pixel[p->current]);
+}
 
 /* LCD<43>ϣ<EFBFBD><CFA3>ԥ<EFBFBD><D4A5><EFBFBD><EFBFBD><EFBFBD>16<31><36>(<28><><EFBFBD>ӥå<D3A5>ɬ<EFBFBD><C9AC>) */
-int WWLCDPanel_GetPixel(WWLCDPanel lcd_panel, int x, int y)
+static int WWLCDPanel_GetPixelByCurrent(WWLCDPanel lcd_panel, int current,
+					int x, int y)
 {
   unsigned char pixel;
 
-  if ( (x < 0) || (x > WWLCDPanel_GetWidth(lcd_panel) - 1) ||
+  if ( (x < 0) || (x > WWLCDPanel_GetWidth( lcd_panel) - 1) ||
        (y < 0) || (y > WWLCDPanel_GetHeight(lcd_panel) - 1) )
     return (-1);
 
-  pixel = lcd_panel->pixel[y * WWLCDPanel_GetWidth(lcd_panel) + x];
+  pixel = lcd_panel->pixel[current][y * WWLCDPanel_GetWidth(lcd_panel) + x];
   pixel &= 0x0f;
   return ((int)pixel);
 }
 
+static int WWLCDPanel_GetOldPixel(WWLCDPanel lcd_panel, int x, int y)
+{
+  return (WWLCDPanel_GetPixelByCurrent(lcd_panel, 1 - lcd_panel->current, x, y));
+}
+
+int WWLCDPanel_GetPixel(WWLCDPanel lcd_panel, int x, int y)
+{
+  return (WWLCDPanel_GetPixelByCurrent(lcd_panel, lcd_panel->current, x, y));
+}
+
 int WWLCDPanel_SetPixel(WWLCDPanel lcd_panel, int x, int y, int pixel)
 {
   unsigned char p;
   int n;
 
-  if ( (x < 0) || (x > WWLCDPanel_GetWidth(lcd_panel) - 1) ||
+  if ( (x < 0) || (x > WWLCDPanel_GetWidth( lcd_panel) - 1) ||
        (y < 0) || (y > WWLCDPanel_GetHeight(lcd_panel) - 1) )
     return (-1);
 
   p = ((unsigned char)pixel) & 0x0f;
   n = y * WWLCDPanel_GetWidth(lcd_panel) + x;
-  lcd_panel->pixel[n] = p;
+  lcd_panel->pixel[lcd_panel->current][n] = p;
 
   return (pixel);
 }
 
+int WWLCDPanel_IsPixelChanged(WWLCDPanel lcd_panel, int x, int y)
+{
+  int old_pixel;
+  int current_pixel;
+
+  if (WWLCDPanel_IsAllDraw(lcd_panel)) return (1);
+  old_pixel = WWLCDPanel_GetOldPixel(lcd_panel, x, y);
+  current_pixel = WWLCDPanel_GetPixel(lcd_panel, x, y);
+
+  return (!(old_pixel == current_pixel));
+}
+
 WWLCDPanel WWLCDPanel_Create(int width, int height)
 {
   WWLCDPanel lcd_panel;
-  int x, y;
+  int x, y, i;
 
   lcd_panel = (WWLCDPanel)malloc(sizeof(_WWLCDPanel));
   if (lcd_panel == NULL) Error("WWLCDPanel_Create", "Cannot allocate memory.");
 
   WWLCDPanel_SetWidth( lcd_panel, width);
   WWLCDPanel_SetHeight(lcd_panel, height);
-  lcd_panel->pixel =
-    (unsigned char *)malloc(sizeof(unsigned char) *
-			    WWLCDPanel_GetWidth(lcd_panel) *
-			    WWLCDPanel_GetHeight(lcd_panel));
+
+  for (i = 0; i < 2; i++) {
+    lcd_panel->pixel[i] =
+      (unsigned char *)malloc(sizeof(unsigned char) *
+			      WWLCDPanel_GetWidth(lcd_panel) *
+			      WWLCDPanel_GetHeight(lcd_panel));
+  }
 
   for (y = 0; y < lcd_panel->height; y++) {
     for (x = 0; x < lcd_panel->width / 2; x++) {
@@ -66,13 +109,19 @@ WWLCDPanel WWLCDPanel_Create(int width, int height)
     }
   }
 
+  WWLCDPanel_ResetCurrent(lcd_panel);
+  WWLCDPanel_SetAllDraw(lcd_panel); /* <20><><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>̤<EFBFBD><CCA4><EFBFBD><EFBFBD>褹<EFBFBD><E8A4B9> */
+
   return (lcd_panel);
 }
 
 WWLCDPanel WWLCDPanel_Destroy(WWLCDPanel lcd_panel)
 {
+  int i;
   if (lcd_panel == NULL) return (NULL);
-  if (lcd_panel->pixel) free(lcd_panel->pixel);
+  for (i = 0; i < 2; i++) {
+    if (lcd_panel->pixel[i]) free(lcd_panel->pixel[i]);
+  }
   free(lcd_panel);
   return (NULL);
 }

WWLCDPanel.h

@@ -21,9 +21,17 @@ int WWLCDPanel_GetWidth( WWLCDPanel p);
 int WWLCDPanel_GetHeight(WWLCDPanel p);
 int WWLCDPanel_SetWidth( WWLCDPanel p, int n);
 int WWLCDPanel_SetHeight(WWLCDPanel p, int n);
+
+int WWLCDPanel_ResetCurrent(WWLCDPanel p);
+int WWLCDPanel_ReverseCurrent(WWLCDPanel p);
+int WWLCDPanel_ResetAllDraw(WWLCDPanel p);
+int WWLCDPanel_SetAllDraw(WWLCDPanel p);
+int WWLCDPanel_IsAllDraw(WWLCDPanel p);
+
 unsigned char * WWLCDPanel_GetPixelMap(WWLCDPanel p);
 int WWLCDPanel_GetPixel(WWLCDPanel lcd_panel, int x, int y);
 int WWLCDPanel_SetPixel(WWLCDPanel lcd_panel, int x, int y, int pixel);
+int WWLCDPanel_IsPixelChanged(WWLCDPanel lcd_panel, int x, int y);
 WWLCDPanel WWLCDPanel_Create(int width, int height);
 WWLCDPanel WWLCDPanel_Destroy(WWLCDPanel lcd_panel);
 

WWLCDPanelP.h

@@ -14,8 +14,19 @@
 typedef struct _WWLCDPanel {
   int width;
   int height;
-  /* 16<31><36><EFBFBD>Υ<EFBFBD><CEA5>顼<EFBFBD><E9A1BC><EFBFBD>󡥣<EFBFBD><F3A1A5A3>Х<EFBFBD><D0A5>Ȥǣ<C8A4><C7A3>ԥ<EFBFBD><D4A5><EFBFBD><EFBFBD><EFBFBD>ʬ<EFBFBD>ξ<EFBFBD><CEBE><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD> */
-  unsigned char * pixel;
+
+  /* X<><58><EFBFBD><EFBFBD><EFBFBD>Ф<EFBFBD><D0A4><EFBFBD><EFBFBD>٤򸺤餹<F2B8BAA4><E9A4B9><EFBFBD>ᡤ<EFBFBD>ӥåȥޥåפ<C3A5><D7A4><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD> */
+  /* <20><><EFBFBD><EFBFBD><EFBFBD><EFBFBD>Ʊ<EFBFBD><C6B1><EFBFBD><EFBFBD>ʬ<EFBFBD>Ͻ񤭴<CFBD><F1A4ADB4><EFBFBD><EFBFBD>ʤ<EFBFBD><CAA4>褦<EFBFBD>ˤ<EFBFBD><CBA4>롥            */
+  /* pixel[current] <20><><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>Υӥåȥޥåפˤʤ롥   */
+  /* pixel[1 - current] <20><><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>Υӥåȥޥåפˤʤ롥     */
+  int current;
+
+  /* all_draw == 1 <20>ΤȤ<CEA4><C8A4>ϡ<EFBFBD><CFA1><EFBFBD><EFBFBD><EFBFBD>ʬ<EFBFBD><CAAC>̵<EFBFBD><CCB5><EFBFBD>Τǡ<CEA4>          */
+  /* <20><><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>褹<EFBFBD>뤳<EFBFBD>Ȥ򼨤<C8A4><F2BCA8A4><EFBFBD>                            */
+  int all_draw;
+
+  /* 16<31><36><EFBFBD>Υ<EFBFBD><CEA5>顼<EFBFBD><E9A1BC><EFBFBD>󡥣<EFBFBD><F3A1A5A3>Х<EFBFBD><D0A5>Ȥǣ<C8A4><C7A3>ԥ<EFBFBD><D4A5><EFBFBD><EFBFBD><EFBFBD>ʬ<EFBFBD>ξ<EFBFBD><CEBE><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD> */
+  unsigned char * pixel[2];
 } _WWLCDPanel;
 
 /*****************************************************************************/

XDisplay.c

@@ -371,6 +371,10 @@ int XDisplay_DrawLCDWindow(XDisplay x_display, WWLCDPanel ww_lcd_panel)
   num =
     WWLCDPanel_GetHeight(ww_lcd_panel) * WWLCDPanel_GetWidth(ww_lcd_panel) / 2;
 
+  /*
+   * <20><><EFBFBD><EFBFBD> malloc() <20>ϡ<EFBFBD><CFA1>ºݤˤϥ<CBA4><CFA5><EFBFBD><EFBFBD><EFBFBD><EFBFBD>ϤۤȤ<DBA4><C8A4>ɻ<EFBFBD><C9BB>Ѥ<EFBFBD><D1A4><EFBFBD><EFBFBD>Ƥ<EFBFBD><C6A4>ʤ<EFBFBD><CAA4>Τǡ<CEA4>
+   * <20><><EFBFBD>Τ<EFBFBD><CEA4><EFBFBD><EFBFBD>ʤ<EFBFBD><CAA4>Ȥ<EFBFBD><C8A4><EFBFBD><EFBFBD><EFBFBD>ɬ<EFBFBD>פ<EFBFBD><D7A4><EFBFBD><EFBFBD><EFBFBD>
+   */
   for (pixel = 0; pixel < 16; pixel++) {
     n[pixel] = 0;
     rectangles[pixel] = (XRectangle *)malloc(sizeof(XRectangle) * num);
@@ -387,6 +391,9 @@ int XDisplay_DrawLCDWindow(XDisplay x_display, WWLCDPanel ww_lcd_panel)
     py = (y * x_display->height) / ww_lcd_height;
     ph = (y+1) * x_display->height / ww_lcd_height- py;
     for (x = 0; x < ww_lcd_width; x++) {
+      if (!WWLCDPanel_IsPixelChanged(ww_lcd_panel, x, y)) {
+	continue;
+      }
       pixel = WWLCDPanel_GetPixel(ww_lcd_panel, x, y);
       px = (x * x_display->width ) / ww_lcd_width;
       rectangles[pixel][n[pixel]].x = px;
@@ -398,7 +405,8 @@ int XDisplay_DrawLCDWindow(XDisplay x_display, WWLCDPanel ww_lcd_panel)
       /* <20><><EFBFBD>ܤ<EFBFBD><DCA4>Ƥ<EFBFBD>Ʊ<EFBFBD><C6B1><EFBFBD>Υԥ<CEA5><D4A5><EFBFBD><EFBFBD><EFBFBD><EFBFBD>ϡ<EFBFBD><CFA1><EFBFBD><EFBFBD>Ϥ<EFBFBD><CFA4>ä<EFBFBD><C3A4><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>褹<EFBFBD><E8A4B9> */
       x++;
       while ( (x < ww_lcd_width) &&
-	      (pixel == WWLCDPanel_GetPixel(ww_lcd_panel, x, y)) ) {
+	      (pixel == WWLCDPanel_GetPixel(ww_lcd_panel, x, y)) &&
+	      (WWLCDPanel_IsPixelChanged(ww_lcd_panel, x, y)) ) {
 	rectangles[pixel][n[pixel]].width =
 	  (x+1) * x_display->width / ww_lcd_width - px;
 	x++;
@@ -422,6 +430,9 @@ int XDisplay_DrawLCDWindow(XDisplay x_display, WWLCDPanel ww_lcd_panel)
 	    x_display->lcd_window, x_display->copy_gc,
             0, 0, x_display->width, x_display->height, 0, 0);
 
+  WWLCDPanel_ResetAllDraw(ww_lcd_panel);
+  WWLCDPanel_ReverseCurrent(ww_lcd_panel);
+
   XDisplay_Sync(x_display);
 
   for (pixel = 0; pixel < 16; pixel++) {