[PATCH wayland] shm: Make shm_pool_interface static const

Bryce Harrington bryce at osg.samsung.com
Fri Aug 12 19:41:47 UTC 2016


On Fri, Aug 12, 2016 at 12:04:41PM -0700, Yong Bakos wrote:
> From: Yong Bakos <ybakos at humanoriented.com>
> 
> Add static const modifiers to the shm_pool_interface definition,
> making it consistent with the other wl_*_interface definitions and
> mundanely safer.
> 
> Note that this does not affect the ABI, according to abi-dumper and
> abi-compliance-checker[1]; and weston and its shm-backed clients still
> run as expected.
> 
> [1]: http://lvc.github.io/abi-compliance-checker/
> 
> Signed-off-by: Yong Bakos <ybakos at humanoriented.com>
> Tested-by: Yong Bakos <ybakos at humanoriented.com>

Acked-by: Bryce Harrington <bryce at osg.samsung.com>
> ---
>  src/wayland-shm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/wayland-shm.c b/src/wayland-shm.c
> index 7fea364..8e2ef77 100644
> --- a/src/wayland-shm.c
> +++ b/src/wayland-shm.c
> @@ -247,7 +247,7 @@ shm_pool_resize(struct wl_client *client, struct wl_resource *resource,
>  		shm_pool_finish_resize(pool);
>  }
>  
> -struct wl_shm_pool_interface shm_pool_interface = {
> +static const struct wl_shm_pool_interface shm_pool_interface = {
>  	shm_pool_create_buffer,
>  	shm_pool_destroy,
>  	shm_pool_resize
> -- 
> 2.7.2
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list