[PATCH weston v2 1/8] Rename weston_compositor_xkb_init to weston_compositor_set_xkb_rule_names

Pekka Paalanen ppaalanen at gmail.com
Thu Jun 2 11:04:16 UTC 2016


On Thu, 2 Jun 2016 10:07:10 +0200
Quentin Glidic <sardemff7+wayland at sardemff7.net> wrote:

> On 01/06/2016 20:43, Giulio Camuffo wrote:
> > weston_compositor_xkb_destroy() is called automatically so having only
> > weston_compositor_xkb_init() to be called by the user was a bit weird.
> > So rename it so that it makes more sense.
> > Also export it, since libweston compositors need to call it.
> > ---
> >
> > v2: renamed xkb_init to set_xkb_rule_names
> >
> >  src/compositor.h |  4 ++--
> >  src/input.c      | 12 ++++++------
> >  src/main.c       |  2 +-
> >  3 files changed, 9 insertions(+), 9 deletions(-)
> >  
> 
> That rename sounds like a good idea, the fact that init is done there is 
> then a mere implementation detail, so:
> 
> Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>

Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>


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


More information about the wayland-devel mailing list