On Fri, Jan 29, 2016 at 05:02:15PM +0100, Matthias Treydte wrote:
> This allows the header to be consumed by C++ compilers, because C++ does
> away with C's implicit cast from (void*).
> ---
>  shared/platform.h | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)

Tabbing looks off maybe but otherwise just adding a cast seems fine.

Reviewed-by: Bryce Harrington <[email protected]>
 
> diff --git a/shared/platform.h b/shared/platform.h
> index cf4ecc0..dd55008 100644
> --- a/shared/platform.h
> +++ b/shared/platform.h
> @@ -75,8 +75,9 @@ weston_platform_get_egl_display(EGLenum platform, void 
> *native_display,
>       static PFNEGLGETPLATFORMDISPLAYEXTPROC get_platform_display = NULL;
>  
>       if (!get_platform_display) {
> -             get_platform_display = weston_platform_get_egl_proc_address(
> -                     "eglGetPlatformDisplayEXT");
> +             get_platform_display = (PFNEGLGETPLATFORMDISPLAYEXTPROC)
> +            weston_platform_get_egl_proc_address(
> +                "eglGetPlatformDisplayEXT");
>       }
>  
>       if (get_platform_display)
> @@ -95,8 +96,9 @@ weston_platform_create_egl_surface(EGLDisplay dpy, 
> EGLConfig config,
>               create_platform_window = NULL;
>  
>       if (!create_platform_window) {
> -             create_platform_window = weston_platform_get_egl_proc_address(
> -                             "eglCreatePlatformWindowSurfaceEXT");
> +             create_platform_window = 
> (PFNEGLCREATEPLATFORMWINDOWSURFACEEXTPROC)
> +            weston_platform_get_egl_proc_address(
> +                "eglCreatePlatformWindowSurfaceEXT");
>       }
>  
>       if (create_platform_window)
> -- 
> 2.7.0
> 
> _______________________________________________
> wayland-devel mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to