On Tue, Sep 18, 2012 at 8:21 PM, Carsten Haitzler <ras...@rasterman.com>wrote:
> On Mon, 17 Sep 2012 18:52:06 +0900 "Sung W. Park" <sung...@gmail.com> > said: > > you might want to check your compile warnings. :) like... > > evas_engine.c:2620:1: warning: no return statement in function returning > non-void > yeah, I caught this afterwards. :) > > :) other fun like unused parameters in evas_gl_api.c ... > > actually, I did run the compiler flag with all the warnings on and some of the variables come as unused because they're only used on target (with the #ifdef GLES_VARIETY_...). Any thoughts or suggestions on this? > hint: > > export CFLAGS="-W -Wall -Wextra" > > :) > > > and yes even tho the eglCreateWindowSurface() using root is not portable to > wayland. it is in gl_x11 which means its bound to x11. so thats ok. i > don't see > what else could have been done. if the egl/glx stuff was virtualized to be > shared then yes - it'd be a problem. :) > > evas_engine.c has lost like 2000 lines of code. that's good! :) > > i say put it in and clean up the nigglies (formatting etc.) then live. :) > will do. thanks~! > > > Hi all, > > > > I'd like to ask you devs for review on Evas_GL that I've recently > refactored > > before I push it upstream. > > > > evas_gl was introduced last year and it has gotten really messy over the > > year > > with a lot of little tweaks here and there. > > > > Also, I've noticed not too long ago that the same code was pretty much > > being > > copied over to wayland_egl engine and so I've decided that it's about > time > > to > > rewrite the ugly code. > > > > I've commonized the GL part of the code and made an interface that > > each engine has to implement to get the evas_gl running. In the overall > > scheme of things, one can argue whether this was the best design but with > > what we have currently, i thought it was reasonable. > > > > I'm attaching a patch that modifies the following files... > > > > src/modules/engines/gl_common/Makefile.am > > src/modules/engines/gl_x11/evas_engine.c > > > > and I've added 7 new files in gl_common. > > > > src/modules/engines/gl_common/evas_gl_core_private.h > > src/modules/engines/gl_common/evas_gl_core.h > > src/modules/engines/gl_common/evas_gl_core.c > > src/modules/engines/gl_common/evas_gl_api.c > > src/modules/engines/gl_common/evas_gl_api_ext_def.h > > src/modules/engines/gl_common/evas_gl_api_ext.h > > src/modules/engines/gl_common/evas_gl_api_ext.c > > > > I'm also including two samples files as well as a PPT slides I've > > made for some people here. > > > > Your comments would be greatly appreciated. > > > > cheers, > > Sung > > > -- > ------------- Codito, ergo sum - "I code, therefore I am" -------------- > The Rasterman (Carsten Haitzler) ras...@rasterman.com > > ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
