[PATCH weston 00/10] platform_base cleanups and review reply

[Pekka Paalanen]

On Tue, 24 Mar 2015 15:33:59 +0200
Pekka Paalanen <pekka.paalanen at collabora.co.uk> wrote:

> On Tue, 24 Mar 2015 13:11:59 +0100
> Jonny Lamb <jonny.lamb at collabora.co.uk> wrote:
> 
> > In reply to Pekka's late review, I've written some more patches to
> > address his comments:
> > 
> > Jonny Lamb (10):
> >   compositor-{drm,x11,wayland}: stop including EGL headers here
> >   gl-renderer: fix build with --disable-egl
> >   gl-renderer: move all EGL_PLATFORM_* defines here
> >   compositor-{drm,wayland,x11}: remove useless tests for ->supports
> >   gl-renderer: call supports() automatically in create()
> >   gl-renderer: provide platform_base fallbacks and remove ifdefs
> >   nested-client: include platform.h
> >   platform: provide platform_base fallbacks and remove ifdefs
> >   platform: remove global variables
> >   platform: rename create_egl_window to create_egl_surface
> > 
> >  clients/nested-client.c   |   6 ++-
> >  clients/simple-egl.c      |   6 +--
> >  clients/subsurfaces.c     |   6 +--
> >  clients/window.c          |   6 +--
> >  shared/platform.h         | 106 ++++++++++++++++++++++++++++------------------
> >  src/compositor-drm.c      |  14 +-----
> >  src/compositor-fbdev.c    |   3 +-
> >  src/compositor-wayland.c  |  14 +-----
> >  src/compositor-x11.c      |   9 ----
> >  src/gl-renderer.c         |  66 +++++++++++++++++++----------
> >  src/gl-renderer.h         |  25 +++++++++--
> >  tests/buffer-count-test.c |   6 +--
> >  12 files changed, 150 insertions(+), 117 deletions(-)
> > 
> 
> Yup, it's all landable stuff on my behalf.

Hi,

no-one objected/bikeshedded, so I landed this.
   41d2ccc..abff883  master -> master


Thanks,
pq