E SVN: devilhorns trunk/evas/src/modules/engines/software_x11

[Enlightenment S. <no-...@en...>]

Log:
Evas: Iterating the pixmap formats from xcb_setup can be expensive, 
  so cache the found format for reuse so we don't have to keep iterating.
  
  

Author:       devilhorns
Date:         2011-08-20 03:59:53 -0700 (Sat, 20 Aug 2011)
New Revision: 62626
Trac:         http://trac.enlightenment.org/e/changeset/62626

Modified:
  trunk/evas/src/modules/engines/software_x11/evas_xcb_buffer.c 

Modified: trunk/evas/src/modules/engines/software_x11/evas_xcb_buffer.c
===================================================================
--- trunk/evas/src/modules/engines/software_x11/evas_xcb_buffer.c	2011-08-20 10:26:41 UTC (rev 62625)
+++ trunk/evas/src/modules/engines/software_x11/evas_xcb_buffer.c	2011-08-20 10:59:53 UTC (rev 62626)
@@ -418,8 +418,9 @@
 static xcb_image_t *
 _xcbob_create_native(xcb_connection_t *conn, int w, int h, xcb_image_format_t format, uint8_t depth, void *base, uint32_t bytes, uint8_t *data) 
 {
+   static uint8_t dpth = 0;
+   static xcb_format_t *fmt = NULL;
    const xcb_setup_t *setup;
-   xcb_format_t *fmt = NULL;
    xcb_image_format_t xif;
 
    /* NB: We cannot use xcb_image_create_native as it only creates images 
@@ -431,21 +432,18 @@
    if ((xif == XCB_IMAGE_FORMAT_Z_PIXMAP) && (depth == 1))
      xif = XCB_IMAGE_FORMAT_XY_PIXMAP;
 
-   fmt = _xcbob_find_format(setup, depth);
-   if (!fmt) return 0;
+   if (dpth != depth) 
+     {
+        dpth = depth;
+        fmt = _xcbob_find_format(setup, depth);
+        if (!fmt) return 0;
+     }
 
    switch (xif) 
      {
       case XCB_IMAGE_FORMAT_XY_BITMAP:
         if (depth != 1) return 0;
       case XCB_IMAGE_FORMAT_XY_PIXMAP:
-        /* return xcb_image_create(w, h, xif,  */
-        /*                         fmt->scanline_pad,  */
-        /*                         fmt->depth, fmt->bits_per_pixel,  */
-        /*                         setup->bitmap_format_scanline_unit,  */
-        /*                         setup->image_byte_order,  */
-        /*                         setup->bitmap_format_bit_order,  */
-        /*                         base, bytes, data); */
       case XCB_IMAGE_FORMAT_Z_PIXMAP:
         return xcb_image_create(w, h, xif, 
                                 fmt->scanline_pad,