jpeg pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=425265ca6d1d926081155d0f23a8d801b5ff56d4

commit 425265ca6d1d926081155d0f23a8d801b5ff56d4
Author: Jean-Philippe Andre <jp.an...@samsung.com>
Date:   Wed Oct 14 19:42:49 2015 +0900

    Evas GL: Fix support for the SW engines (OSMesa)
    
    Since @raster changed the behaviour of the dirty flag on images,
    damages must be added to redraw the GL surface. Evas_Image checks
    if it is an Evas GL surface by looking at its native surface.
    But in case of SW engine, there was no native surface information
    for Evas GL surfaces. Also, the OPENGL surface type was awfully
    abused for OSMesa support. Luckily EVASGL surface type lets us
    pass arbitrary pointers :)
---
 .../evas/engines/software_generic/evas_engine.c    | 17 ++++----
 .../evas/engines/software_x11/evas_engine.c        | 46 +++++++++++++++++++---
 2 files changed, 51 insertions(+), 12 deletions(-)

diff --git a/src/modules/evas/engines/software_generic/evas_engine.c 
b/src/modules/evas/engines/software_generic/evas_engine.c
index 4eaa9a4..721c4ee 100644
--- a/src/modules/evas/engines/software_generic/evas_engine.c
+++ b/src/modules/evas/engines/software_generic/evas_engine.c
@@ -1096,12 +1096,15 @@ eng_image_native_set(void *data EINA_UNUSED, void 
*image, void *native)
 
    if (!im || !ns) return im;
 
-   if ((ns->type == EVAS_NATIVE_SURFACE_OPENGL) &&
+   if ((ns->type == EVAS_NATIVE_SURFACE_EVASGL) &&
        (ns->version == EVAS_NATIVE_SURFACE_VERSION))
-     im2 = evas_cache_image_data(evas_common_image_cache_get(), 
-                                 im->w, im->h, 
-                                 ns->data.x11.visual, 1,
-                                 EVAS_COLORSPACE_ARGB8888);
+     {
+
+        im2 = evas_cache_image_data(evas_common_image_cache_get(),
+                                    im->w, im->h,
+                                    ns->data.evasgl.surface, 1,
+                                    EVAS_COLORSPACE_ARGB8888);
+     }
    else
      im2 = evas_cache_image_data(evas_common_image_cache_get(), 
                                  im->w, im->h, 
@@ -3178,9 +3181,9 @@ eng_gl_native_surface_get(void *data EINA_UNUSED, void 
*surface, void *native_su
 
    if (!sfc) return 0;
 
-   ns->type = EVAS_NATIVE_SURFACE_OPENGL;
+   ns->type = EVAS_NATIVE_SURFACE_EVASGL;
    ns->version = EVAS_NATIVE_SURFACE_VERSION;
-   ns->data.x11.visual = sfc->buffer;
+   ns->data.evasgl.surface = sfc->buffer;
    
    return 1;
 #else
diff --git a/src/modules/evas/engines/software_x11/evas_engine.c 
b/src/modules/evas/engines/software_x11/evas_engine.c
index 02e6630..5a5d1cf 100644
--- a/src/modules/evas/engines/software_x11/evas_engine.c
+++ b/src/modules/evas/engines/software_x11/evas_engine.c
@@ -632,6 +632,20 @@ eng_canvas_alpha_get(void *data, void *context EINA_UNUSED)
      (re->outbuf_alpha_get(re->generic.ob));
 }
 
+static void
+_native_evasgl_free(void *data EINA_UNUSED, void *image)
+{
+   RGBA_Image *im = image;
+   Native *n = im->native.data;
+
+   im->native.data        = NULL;
+   im->native.func.data   = NULL;
+   im->native.func.bind   = NULL;
+   im->native.func.free   = NULL;
+   //im->image.data         = NULL;
+   free(n);
+}
+
 static void *
 eng_image_native_set(void *data EINA_UNUSED, void *image, void *native)
 {
@@ -640,7 +654,13 @@ eng_image_native_set(void *data EINA_UNUSED, void *image, 
void *native)
    Image_Entry *ie = image, *ie2 = NULL;
    RGBA_Image *im = image;
 
-   if (!im || !ns) return im;
+   if (!im) return NULL;
+   if (!ns)
+     {
+        if (im->native.data && im->native.func.free)
+          im->native.func.free(im->native.func.data, im);
+        return NULL;
+     }
 
    if (ns->type == EVAS_NATIVE_SURFACE_X11)
      {
@@ -669,10 +689,10 @@ eng_image_native_set(void *data EINA_UNUSED, void *image, 
void *native)
      }
 
    // Code from software_generic
-   if ((ns->type == EVAS_NATIVE_SURFACE_OPENGL) &&
-            (ns->version == EVAS_NATIVE_SURFACE_VERSION))
+   if ((ns->type == EVAS_NATIVE_SURFACE_EVASGL) &&
+       (ns->version == EVAS_NATIVE_SURFACE_VERSION))
      ie2 = evas_cache_image_data(evas_common_image_cache_get(),
-                                 ie->w, ie->h, ns->data.x11.visual, 1,
+                                 ie->w, ie->h, ns->data.evasgl.surface, 1,
                                  EVAS_COLORSPACE_ARGB8888);
    else
      ie2 = evas_cache_image_data(evas_common_image_cache_get(),
@@ -702,10 +722,26 @@ eng_image_native_set(void *data EINA_UNUSED, void *image, 
void *native)
         return evas_xcb_image_native_set(re->generic.ob, ie, ns);
 #endif
      }
-   if (ns->type == EVAS_NATIVE_SURFACE_TBM)
+   else if (ns->type == EVAS_NATIVE_SURFACE_TBM)
      {
         return evas_native_tbm_image_set(re->generic.ob, ie, ns);
      }
+   else if (ns->type == EVAS_NATIVE_SURFACE_EVASGL)
+     {
+        /* Native contains Evas_Native_Surface. What a mess. */
+        Native *n = calloc(1, sizeof(Native));
+        if (n)
+          {
+             im = (RGBA_Image *) ie;
+             n->ns.type = EVAS_NATIVE_SURFACE_EVASGL;
+             n->ns.version = EVAS_NATIVE_SURFACE_VERSION;
+             n->ns.data.evasgl.surface = ns->data.evasgl.surface;
+             im->native.data = n;
+             im->native.func.free = _native_evasgl_free;
+             im->native.func.data = NULL;
+             im->native.func.bind = NULL;
+          }
+     }
 
    return ie;
 }

-- 


Reply via email to