[PATCH weston v2 5/8] libinput: don't use weston_config when configuring input devices
Pekka Paalanen
ppaalanen at gmail.com
Thu Jun 2 12:08:14 UTC 2016
On Thu, 2 Jun 2016 10:44:17 +0200
Quentin Glidic <sardemff7+wayland at sardemff7.net> wrote:
> On 01/06/2016 20:43, Giulio Camuffo wrote:
> > Instead add callbacks to the drm and fbdev backends and pass that to
> > the input backens so that when a new device needs to be configured
> > that is called and the compositor can configure it.
> >
> > Signed-off-by: Giulio Camuffo <giuliocamuffo at gmail.com>
> > ---
> > Makefile.am | 3 ++-
> > src/compositor-drm.c | 3 ++-
> > src/compositor-drm.h | 11 +++++++++++
> > src/compositor-fbdev.c | 3 ++-
> > src/compositor-fbdev.h | 11 +++++++++++
> > src/compositor-rpi.c | 2 +-
> > src/libinput-device.c | 31 +------------------------------
> > src/libinput-device.h | 2 ++
> > src/libinput-seat.c | 7 ++++++-
> > src/libinput-seat.h | 9 ++++++++-
> > src/main.c | 27 +++++++++++++++++++++++++++
> > 11 files changed, 73 insertions(+), 36 deletions(-)
> >
>
> Just one comment (applying three times to this patch).
>
>
> > [snip]
> > diff --git a/src/compositor-drm.h b/src/compositor-drm.h
> > index 3f150db..1266031 100644
> > --- a/src/compositor-drm.h
> > +++ b/src/compositor-drm.h
> > @@ -36,6 +36,8 @@ extern "C" {
> >
> > #define WESTON_DRM_BACKEND_CONFIG_VERSION 1
> >
> > +struct libinput_device;
> > +
>
> I guess the idea is to avoid #include <libinput.h> here. Is that really
> a problem?
Hi,
I'm fine with the forward declaration.
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/3e6c63aa/attachment.sig>
More information about the wayland-devel
mailing list