On Wed, 07 Mar 2018 13:25:27 -0800 Derek Foreman <der...@osg.samsung.com> said:

this brings back memories of problems with checking for extension symbols
before a context is set up. the context may change the symbols (procaddress
returned functions) as for example the same symbol for gles1.1 might vary for
2.0 and then 3.0 ...

i know we've had these issue because that that's why the checks happen after
context init.

are you sure this is right?

> derekf pushed a commit to branch master.
> 
> http://git.enlightenment.org/core/efl.git/commit/?id=f8658d25fa604f885ee23b20e94a2892d340bceb
> 
> commit f8658d25fa604f885ee23b20e94a2892d340bceb
> Author: Derek Foreman <der...@osg.samsung.com>
> Date:   Wed Mar 7 13:11:45 2018 -0600
> 
>     gl_drm: Move the gl symbol check to immediately after display init
>     
>     We don't actually need a context first, just an initialized display.
> ---
>  src/modules/evas/engines/gl_drm/evas_outbuf.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/modules/evas/engines/gl_drm/evas_outbuf.c
> b/src/modules/evas/engines/gl_drm/evas_outbuf.c index aff5de87bf..b1235355cc
> 100644
> --- a/src/modules/evas/engines/gl_drm/evas_outbuf.c
> +++ b/src/modules/evas/engines/gl_drm/evas_outbuf.c
> @@ -226,6 +226,8 @@ _evas_outbuf_egl_setup(Outbuf *ob)
>          return EINA_FALSE;
>       }
>  
> +   eng_gl_symbols(ob->egl.disp);
> +
>     if (!eglGetConfigs(ob->egl.disp, NULL, 0, &ncfg) || (ncfg == 0))
>       {
>          ERR("eglGetConfigs() fail. code=%#x", eglGetError());
> @@ -334,8 +336,6 @@ _evas_outbuf_egl_setup(Outbuf *ob)
>          goto err;
>       }
>  
> -   eng_gl_symbols(ob->egl.disp);
> -
>     ob->gl_context = glsym_evas_gl_common_context_new();
>     if (!ob->gl_context) goto err;
>  
> 
> -- 
> 
> 


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
Carsten Haitzler - ras...@rasterman.com


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to