<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">Actually, I was cleaning up the patch. The other changes were not meant to be sent.</span><div style="font-family:arial,sans-serif;font-size:13px"><br>
</div><div style="font-family:arial,sans-serif;font-size:13px">The EGL work is part of the DirectFB repository.</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">
I will send a cleaned up patch with coding style fixes.</div><div class="" style="font-family:arial,sans-serif;font-size:13px"></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Oct 7, 2013 at 9:20 AM, Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, 7 Oct 2013 02:00:15 +0200<br>
Denis Oliver Kropp <<a href="mailto:dfbdok@gmail.com">dfbdok@gmail.com</a>> wrote:<br>
<br>
> Hi Wayland!<br>
><br>
> This is our first release of the integration of Weston with DirectFB ;)<br>
><br>
> Please see <a href="http://www.directfb.org/docs/DirectFB_Foreseeing_2013-10-07.pdf" target="_blank">http://www.directfb.org/docs/DirectFB_Foreseeing_2013-10-07.pdf</a><br>
> and <a href="http://www.directfb.org/docs/DirectFB_EGL_2013-10-07.pdf" target="_blank">http://www.directfb.org/docs/DirectFB_EGL_2013-10-07.pdf</a> for details<br>
> and plans.<br>
><br>
> With this version you can run Weston on any DirectFB enabled platform<br>
> including OpenGL ES support<br>
> via EGL where available.<br>
><br>
> We're also introducing our abstract high-level EGL implementation called<br>
> "EGL United". This aims to<br>
> provide a complete EGL implementation with all extensions, e.g. WL<br>
> extensions or DIRECTFB EGLImage<br>
> support, while integrating different "low-level" (EGL) implementations in<br>
> one EGLDisplay.<br>
><br>
> The Wayland extensions in EGL United are implemented in a generic module.<br>
><br>
> No vendor support is needed for the Wayland extensions to work!<br>
><br>
> The patch is for 1.2, but I will provide a patch against master next day!<br>
<br>
</div></div>Hi,<br>
<br>
the EGL work sounds cool. :-) Where is it available?<br>
<br>
While you update your patch, could you also split it into logical<br>
commits and use git-send-email to make review more feasible, please?<br>
Right now there is not even a diffstat, not to mention commit messages,<br>
and the patch is huge.<br>
<br>
I hope the commit messages will explain why each change is done, e.g.<br>
why do you comment out asserts from simple-egl, what is the<br>
purpose of each added component, why you disable building of<br>
weston-editor, why you change the <a href="http://configure.ac" target="_blank">configure.ac</a> defaults, why add<br>
weston_surface::compositor_state, etc.<br>
<br>
Also, is there a good reason to deviate from the prevalent Weston<br>
coding style? (spaces vs. tabs, overly long lines, mixed code and<br>
declarations... not just in the C++ code)<br>
<br>
Was this patch meant for review, an RFC, or just a heads-up or what<br>
you are preparing? I see quite many commented out lines of code, too.<br>
<br>
<br>
Thanks,<br>
pq<br>
</blockquote></div><br></div>