wayland_egl: Fix color depth when not using alpha 58/182458/1
authorDerek Foreman <derekf@osg.samsung.com>
Tue, 13 Feb 2018 21:04:17 +0000 (15:04 -0600)
committerJiyoun Park <jy0703.park@samsung.com>
Mon, 25 Jun 2018 09:09:49 +0000 (18:09 +0900)
Since we don't actually set the color depth at all we can end up with
an RGB565 buffer.  We don't ask for depths because apparently the N900
had a problem with this under X.

I'm not aware of any efforts to bring wayland to the N900, so let's do
this normally.

Change-Id: I3c0651efe5623233f39c7d927cbce67a44fcd854

src/modules/evas/engines/wayland_egl/evas_wl_main.c

index 5a02467..86f18b5 100755 (executable)
@@ -130,24 +130,23 @@ eng_window_new(Evas_Engine_Info_Wayland *einfo, int w, int h, Render_Output_Swap
    config_attrs[n++] = EGL_WINDOW_BIT;
    config_attrs[n++] = EGL_RENDERABLE_TYPE;
    config_attrs[n++] = EGL_OPENGL_ES2_BIT;
-# if 0
-   /* FIXME: n900 - omap3 sgx libs break here */
+
    config_attrs[n++] = EGL_RED_SIZE;
    config_attrs[n++] = 1;
    config_attrs[n++] = EGL_GREEN_SIZE;
    config_attrs[n++] = 1;
    config_attrs[n++] = EGL_BLUE_SIZE;
    config_attrs[n++] = 1;
-   /* FIXME: end n900 breakage */
-# endif
-  config_attrs[n++] = EGL_ALPHA_SIZE;
-  config_attrs[n++] = gw->alpha;
-  config_attrs[n++] = EGL_DEPTH_SIZE;
+
+   config_attrs[n++] = EGL_ALPHA_SIZE;
+   config_attrs[n++] = gw->alpha;
+   config_attrs[n++] = EGL_DEPTH_SIZE;
 //TIZEN_ONLY(20171127): do not call ecore_wl2_window_buffer_attach
 //  config_attrs[n++] = 0;
-  config_attrs[n++] = gw->depth_bits;
+   config_attrs[n++] = gw->depth_bits;
 //
-  config_attrs[n++] = EGL_STENCIL_SIZE;
+   config_attrs[n++] = EGL_STENCIL_SIZE;
+
 //TIZEN_ONLY(20171127): do not call ecore_wl2_window_buffer_attach
 //  config_attrs[n++] = 0;
    config_attrs[n++] = gw->stencil_bits;
@@ -159,7 +158,7 @@ eng_window_new(Evas_Engine_Info_Wayland *einfo, int w, int h, Render_Output_Swap
         config_attrs[n++] = gw->msaa_bits;
      }
 //
-  config_attrs[n++] = EGL_NONE;
+   config_attrs[n++] = EGL_NONE;
 
    /* FIXME: Remove this line as soon as eglGetDisplay() autodetection
     * gets fixed. Currently it is incorrectly detecting wl_display and