[PATCHv2 weston] gl-renderer: Remove gl_renderer_interface from gl-renderer.h

Kristian Høgsberg hoegsberg at gmail.com
Mon Apr 21 15:43:22 PDT 2014


On Thu, Apr 10, 2014 at 08:05:17PM +0200, John Kåre Alsaker wrote:
> The rationale here is, that this line would create an instance of
> gl_renderer_interface in every compilation unit that included
> gl-renderer.h. This is not necessary, and it can actually be harmful by
> masking the real exported gl_renderer_interface symbol, if you added
> another compilation unit to gl-renderer.so, causing a runtime failure in
> loading it.
> 
> gl-renderer.c already creates the exported symbol.

Committed, thanks.

Kristian 
> ---
>  src/gl-renderer.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/src/gl-renderer.h b/src/gl-renderer.h
> index db42f6c..6cd5f54 100644
> --- a/src/gl-renderer.h
> +++ b/src/gl-renderer.h
> @@ -101,4 +101,3 @@ struct gl_renderer_interface {
>  	void (*print_egl_error_state)(void);
>  };
>  
> -struct gl_renderer_interface gl_renderer_interface;
> -- 
> 1.9.1
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list