[PATCH v2 libinput] pad: group the button state into a private struct

This is only set on button events so use the same approach as for rings and strips. No functional changes. Signed-off-by: Peter Hutterer --- Changes to v2: - rename button.button to button.number, brings it in line with ring/strip naming. src/libinput.c | 14

Re: [PATCH] scanner: Fix reported executable name to 'wayland-scanner'

On Mon, Jun 06, 2016 at 10:41:04AM -0700, Bryce Harrington wrote: > On Fri, Jun 03, 2016 at 07:33:09PM +1000, Peter Hutterer wrote: > > On Fri, Jun 03, 2016 at 09:16:43AM +0800, Jonas Ådahl wrote: > > > On Thu, Jun 02, 2016 at 02:42:44PM -0700, Bryce Harrington wrote: > > > > 'wayland-scanner -v'

Re: [PATCH wayland-protocols v3] Add screensaver idle inhibitor protocol

On Mon, Jun 06, 2016 at 10:08:00AM -0500, Yong Bakos wrote: > On Jun 6, 2016, at 7:18 AM, Pekka Paalanen wrote: > >> I don't believe this is a good choice. Imagine the case of a surface-less > >> 'inhibitor daemon.' > > > > An ordinary client must not be able to do that. > >

Re: [PATCH libinput 4/7] doc: add two svgs showing the modes

On Jun 6, 2016, at 1:58 AM, Peter Hutterer wrote: > > Separate patch to avoid crowding out the actual content in the patch with the > documentation. > > Signed-off-by: Peter Hutterer FWIW, Reviewed-by: Yong Bakos

Re: [PATCH v2] scanner: Fix reported executable name to 'wayland-scanner'

On Jun 6, 2016, at 12:58 PM, Bryce Harrington wrote: > > 'wayland-scanner -v' (correctly) reports the program as named > "wayland-scanner", but 'wayland-scanner -h' was inconsistent, referring > to it as './scanner'. > > Also refactor this and other references to the

[PATCH v2] scanner: Fix reported executable name to 'wayland-scanner'

'wayland-scanner -v' (correctly) reports the program as named "wayland-scanner", but 'wayland-scanner -h' was inconsistent, referring to it as './scanner'. Also refactor this and other references to the program name to use a common #define, PROGRAM_NAME. Signed-off-by: Bryce Harrington

Re: [PATCH] scanner: Fix reported executable name to 'wayland-scanner'

On Fri, Jun 03, 2016 at 07:33:09PM +1000, Peter Hutterer wrote: > On Fri, Jun 03, 2016 at 09:16:43AM +0800, Jonas Ådahl wrote: > > On Thu, Jun 02, 2016 at 02:42:44PM -0700, Bryce Harrington wrote: > > > 'wayland-scanner -v' (correctly) reports the program as named > > > "wayland-scanner", but

Re: [PATCH v3] compositor-x11: fix title overflow in x11_backend_create_output

On Sun, Jun 05, 2016 at 07:01:11PM +0200, Benoit Gschwind wrote: > sprintf can overflow the fixed length title which is char[32]. This > patch change title to dynamically allocated char array using asprintf or > strdup. If one of them fail we leave returning NULL to indicate the > failure. > >

Re: [PATCH weston] compositor-wayland: Correct output base make name

On Sun, Jun 05, 2016 at 11:01:17AM -0500, Yong Bakos wrote: > From: Yong Bakos > > Change the output make value from "waywayland" to "wayland". > > References: 90bc88c710b34f46ef89e1c5765e5f63f8e02847 > > Signed-off-by: Yong Bakos

Re: [PATCH libinput 5/7] pad: Add a new API for modes and mode groups

Hi Peter, One possible documentation error, one minor nit, and a string error (see "TABLET_PAD_MODE") inline below. Otherwise, this is, uh, Proofread-by: Yong Bakos Regards, yong > On Jun 6, 2016, at 1:50 AM, Peter Hutterer wrote: > > Move

Re: [PATCH libinput 3/7] doc: move the struct declarations to the top

> On Jun 6, 2016, at 1:50 AM, Peter Hutterer wrote: > > They don't define anything, move them to the top so we don't have ordering > requirements of the stuff that actually uses those as parameters. > > Signed-off-by: Peter Hutterer FWIW,

Re: [PATCH libinput 2/7] doc: split the tablet pad events into their own doxygen module page

> On Jun 6, 2016, at 1:50 AM, Peter Hutterer wrote: > > Signed-off-by: Peter Hutterer FWIW, Reviewed-by: Yong Bakos yong > --- > src/libinput.h | 38 +++--- > 1 file changed, 23

Re: [PATCH libinput 1/7] pad: group the button state into a private struct

Hi Peter, > On Jun 6, 2016, at 1:50 AM, Peter Hutterer wrote: > > This is only set on button events so use the same approach as for rings and > strips. No functional changes. > > Signed-off-by: Peter Hutterer > --- > src/libinput.c | 14

Re: [PATCH wayland-protocols v3] Add screensaver idle inhibitor protocol

On Jun 6, 2016, at 7:18 AM, Pekka Paalanen wrote: > > On Fri, 3 Jun 2016 10:39:24 -0500 > Yong Bakos wrote: > >> Hi, >> >> On Jun 3, 2016, at 4:04 AM, Pekka Paalanen wrote: >>> >>> On Fri, 3 Jun 2016 09:26:24 +0800 >>> Jonas

Re: [PATCH wayland-protocols v3] Add screensaver idle inhibitor protocol

On Jun 6, 2016, at 7:18 AM, Pekka Paalanen wrote: > > On Fri, 3 Jun 2016 10:39:24 -0500 > Yong Bakos wrote: > >> Hi, >> >> On Jun 3, 2016, at 4:04 AM, Pekka Paalanen wrote: >>> >>> On Fri, 3 Jun 2016 09:26:24 +0800 >>> Jonas

Re: [PATCH weston v3 6/8] ivi-shell: Support tracking active surfaces

On Fri, 3 Jun 2016 19:24:40 -0700 Bryce Harrington wrote: > On Thu, May 26, 2016 at 06:02:00PM +0300, Pekka Paalanen wrote: > > On Thu, 7 Apr 2016 16:44:21 -0700 > > Bryce Harrington wrote: > > > > > Activity for ivi-shell follows either click

Re: [PATCH weston v3 5/8] fullscreen-shell: Support tracking active surfaces

On Fri, 3 Jun 2016 18:45:59 -0700 Bryce Harrington wrote: > On Thu, May 26, 2016 at 06:01:52PM +0300, Pekka Paalanen wrote: > > On Thu, 7 Apr 2016 16:44:20 -0700 > > Bryce Harrington wrote: > > > > > Surface activity is determined by what

Re: [RFC weston 4/4] WIP: libweston: rework versioning scheme

On Fri, 3 Jun 2016 14:27:39 +0100 Emil Velikov wrote: > From: Emil Velikov > > Use libweston-$major.so.0.$minor.$patch over the current scheme. Hi, is that really a commonly used versioning pattern? Could you add a reference to the

Re: [PATCH] xdg-shell: add draw states for xdg_surface

To me, 'draw states' sounds a lot like 'mwm hints' - which would be a 180 degree reversal from semantic states back to presentation states. Do we really want to go down that road ? ___ wayland-devel mailing list wayland-devel@lists.freedesktop.org

Re: [RFC weston 3/4] XXX: README: Mention bits that we really want for libweston

On Fri, 3 Jun 2016 14:27:38 +0100 Emil Velikov wrote: > From: Emil Velikov > > --- > README | 12 > 1 file changed, 12 insertions(+) > > diff --git a/README b/README > index b8aa2e0..e411343 100644 > --- a/README > +++

Re: [PATCH weston 2/4] README: minor the libweston ABI/API documentation

On Fri, 3 Jun 2016 14:27:37 +0100 Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov > --- > README | 22 +++--- > 1 file changed, 11 insertions(+), 11 deletions(-) > >

Re: [PATCH weston 1/4] README: clarify libweston purpose/goals.

On Sun, 5 Jun 2016 23:37:57 +0100 Emil Velikov wrote: > Hi Yong, > > On 4 June 2016 at 01:58, Yong Bakos wrote: > > Hi Emil, > > > > On Jun 3, 2016, at 8:27 AM, Emil Velikov wrote: > >> > >> From: Emil Velikov

Re: [PATCH weston 1/2] Move weston source to compositor/

2016-06-06 15:52 GMT+03:00 Pekka Paalanen : > On Sun, 5 Jun 2016 21:50:33 +0300 > Giulio Camuffo wrote: > >> 2016-06-03 17:33 GMT+03:00 Pekka Paalanen : >> > From: Pekka Paalanen >> > >> > This is

Re: [PATCH weston 1/2] Move weston source to compositor/

On Sun, 5 Jun 2016 21:50:33 +0300 Giulio Camuffo wrote: > 2016-06-03 17:33 GMT+03:00 Pekka Paalanen : > > From: Pekka Paalanen > > > > This is the start of separating weston-the-compositor source files from > >

Re: [PATCH weston 0/5] Clean up after backend config conversion

On Sun, 5 Jun 2016 21:20:42 +0300 Giulio Camuffo wrote: > Hi, > > i have a few nitpicks on some patches, but for the whole series: > Reviewed-by: Giulio Camuffo Hi, I fixed the whitespace issue and the typo, but did not squash the two

Re: [PATCH weston 5/5] main: remove unused argument from backend loaders

On Sun, 5 Jun 2016 21:24:18 +0300 Giulio Camuffo wrote: > Maybe it would make sense to squash this with the previous one? Hi, I don't think so. I can't write a commit summary that is both explicit and not listing two different things at the same time, so they really

Re: [PATCH wayland-protocols v3] Add screensaver idle inhibitor protocol

On Fri, 3 Jun 2016 10:39:24 -0500 Yong Bakos wrote: > Hi, > > On Jun 3, 2016, at 4:04 AM, Pekka Paalanen wrote: > > > > On Fri, 3 Jun 2016 09:26:24 +0800 > > Jonas Ådahl wrote: > > > >> On Thu, Jun 02, 2016 at 02:24:20PM

[PATCH v2] xdg-shell: Add tiled state

When tiled, clients must obey the window geometry specified in the configure event and can choose to hide some of their decorations. Signed-off-by: Olivier Fourdan --- v2: Simplify the proposal and keep only one "tiled" state instead of one per edge. Note that

[PATCH libinput 4/7] doc: add two svgs showing the modes

Separate patch to avoid crowding out the actual content in the patch with the documentation. Signed-off-by: Peter Hutterer --- doc/Makefile.am | 2 + doc/svg/tablet-cintiq24hd-modes.svg | 460

[PATCH libinput 3/7] doc: move the struct declarations to the top

They don't define anything, move them to the top so we don't have ordering requirements of the stuff that actually uses those as parameters. Signed-off-by: Peter Hutterer --- src/libinput.h | 252 - 1 file

[PATCH/RFC libinput 0/7] pad: add a mode toggle API

Ok, I'm at the point where I can stare at this all day and don't see anything anymore, so up to you guys it is :) This implements mode switching support for tablet pads. The first two patches are just general cleanup, the meat is in 5/7 and 6/7. What is mode switching? On many tablets (Intuos

[PATCH libinput 5/7] pad: Add a new API for modes and mode groups

Move mode control to libinput. This reduces some flexibility on what we can do with modes but makes it a lot easier for anyone to implement modes correctly and have the LEDs apply appropriately, etc. Let's go with the option to make the 95% use-case easy. Note: whether the mode is actually used is

[PATCH libinput 6/7] Implement tablet pad group mode support

Signed-off-by: Peter Hutterer --- configure.ac | 2 +- src/Makefile.am | 1 + src/evdev-tablet-pad-leds.c | 621 ++ src/evdev-tablet-pad.c| 76 +- src/evdev-tablet-pad.h|

[PATCH libinput 2/7] doc: split the tablet pad events into their own doxygen module page

Signed-off-by: Peter Hutterer --- src/libinput.h | 38 +++--- 1 file changed, 23 insertions(+), 15 deletions(-) diff --git a/src/libinput.h b/src/libinput.h index ed3eb7d..dcfa09f 100644 --- a/src/libinput.h +++ b/src/libinput.h @@

[PATCH libinput 7/7] test: add pad mode group tests

Signed-off-by: Peter Hutterer --- test/pad.c | 163 + 1 file changed, 163 insertions(+) diff --git a/test/pad.c b/test/pad.c index 63bd2d7..9dea0fe 100644 --- a/test/pad.c +++ b/test/pad.c @@ -407,6 +407,164

[PATCH libinput 1/7] pad: group the button state into a private struct

This is only set on button events so use the same approach as for rings and strips. No functional changes. Signed-off-by: Peter Hutterer --- src/libinput.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/libinput.c