[PATCH wayland] shm: Make shm_pool_interface static const

Pekka Paalanen ppaalanen at gmail.com
Thu Nov 10 11:08:58 UTC 2016


On Tue, 16 Aug 2016 11:12:35 +0300
Pekka Paalanen <ppaalanen at gmail.com> wrote:

> On Fri, 12 Aug 2016 12:41:47 -0700
> Bryce Harrington <bryce at osg.samsung.com> wrote:
> 
> > 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>  
> 
> Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> I don't see the symbol exported, so this should be completely safe
> indeed.

Pushed:
   ae139d8..da58e07  master -> master

Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 801 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20161110/6c28566f/attachment-0001.sig>


More information about the wayland-devel mailing list