[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