Ecore_X(cb): Cache the visual and avoid extra lookups.

SVN revision: 62597

diff --git a/legacy/ecore/src/lib/ecore_x/xcb/ecore_xcb_window.c b/legacy/ecore/src/lib/ecore_x/xcb/ecore_xcb_window.c
index f917e0eb3cbaa30f536e08c2520b8e0c4c4d8255..8f0ac6b6b9730a99f58fdc4158441b6e9e74b2cf 100644 (file)
--- a/legacy/ecore/src/lib/ecore_x/xcb/ecore_xcb_window.c
+++ b/legacy/ecore/src/lib/ecore_x/xcb/ecore_xcb_window.c
@@ -1619,19 +1619,25 @@ static Ecore_X_Window
 _ecore_xcb_window_argb_internal_new(Ecore_X_Window parent, int x, int y, int w, int h, uint8_t override_redirect, uint8_t save_under) 
 {
    Ecore_X_Window win = 0;
-
-   LOGFN(__FILE__, __LINE__, __FUNCTION__);
-
 #ifdef ECORE_XCB_RENDER
    uint32_t value_list[10];
-   uint32_t value_mask, vis;
+   uint32_t value_mask;
+   static uint32_t vis = -1;
    Ecore_X_Colormap colormap;
+#endif
 
+   LOGFN(__FILE__, __LINE__, __FUNCTION__);
+
+#ifdef ECORE_XCB_RENDER
    if (parent == 0)
      parent = ((xcb_screen_t *)_ecore_xcb_screen)->root;
 
-   vis = 
-     _ecore_xcb_render_find_visual_id(XCB_RENDER_PICT_TYPE_DIRECT, EINA_TRUE);
+   if (vis < 0) 
+     {
+        vis = 
+          _ecore_xcb_render_find_visual_id(XCB_RENDER_PICT_TYPE_DIRECT, 
+                                           EINA_TRUE);
+     }
 
    colormap = xcb_generate_id(_ecore_xcb_conn);
    xcb_create_colormap(_ecore_xcb_conn, XCB_COLORMAP_ALLOC_NONE,