On Thu, Mar 19, 2015 at 03:07:14PM +0100, Jonny Lamb wrote: > Reviewed-by: Derek Foreman <[email protected]> > --- > clients/nested-client.c | 7 +++---- > clients/simple-egl.c | 7 ++++--- > clients/subsurfaces.c | 6 +++--- > clients/window.c | 8 ++++---- > shared/platform.h | 23 +++++++++++++++++++++++ > tests/buffer-count-test.c | 7 +++---- > 6 files changed, 40 insertions(+), 18 deletions(-) > snip > +static inline EGLSurface > +weston_platform_create_egl_window(EGLDisplay dpy, EGLConfig config, > + void *native_window, > + const EGLint *attrib_list) > +{ > +#ifdef EGL_EXT_platform_base > + if (!create_platform_window_surface_ext) > + weston_platform_get_egl_proc_addresses(); > + > + if (create_platform_window_surface_ext) > + return create_platform_window_surface_ext(dpy, config, > + native_window, > + attrib_list); > + else
You don't need this else. With that gone, you can unindent the following return: > +#endif > + return eglCreateWindowSurface(dpy, config, > + (EGLNativeWindowType) > native_window, > + attrib_list); > +} Reviewed-by: Bryce Harrington <[email protected]> _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
