Re: [PATCH 3/5] socket-test: Fix indentation to follow project code style policy

2015-11-22 Thread Marek Chalupa
Hi, I think the indentation already follows project code style. From doc/Contributing: "- indent with tabs, and a tab is always 8 characters wide" current code is: ^Ichar *val = getenv("XDG_RUNTIME_DIR");$ ^Iif (!val)$ ^I^Iassert(0 && "set $XDG_RUNTIME_DIR to run this test");$ with your pat

Re: Lifetime of wl_seat objects (was Re: [PATCH wayland-protocols 1/2] Introduce wp_relative_pointer) interface

2015-11-22 Thread Jonas Ådahl
On Mon, Nov 23, 2015 at 02:42:09PM +1000, Peter Hutterer wrote: > On Fri, Nov 20, 2015 at 02:59:50PM +0800, Jonas Ådahl wrote: > [...] > > > > + > > > > + > > > > + > > > > + > > > > > > Should we add something about pairing this with wl_seat capabilities here? > > > Just looking at

Lifetime of wl_seat objects (was Re: [PATCH wayland-protocols 1/2] Introduce wp_relative_pointer) interface

2015-11-22 Thread Peter Hutterer
On Fri, Nov 20, 2015 at 02:59:50PM +0800, Jonas Ådahl wrote: [...] > > > + > > > + > > > + > > > + > > > > Should we add something about pairing this with wl_seat capabilities here? > > Just looking at this protocol doesn't make it clear what happens if the > > compositor drops the w

[PATCH wayland] protocol: fix a couple of whitespace issues

2015-11-22 Thread Peter Hutterer
8 spaces is one tab Signed-off-by: Peter Hutterer --- protocol/wayland.xml | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/protocol/wayland.xml b/protocol/wayland.xml index 525e092..f9e6d76 100644 --- a/protocol/wayland.xml +++ b/protocol/wayland.xml

Re: [PATCH libinput v9 0/2] add orientation, size and pressure of touch

2015-11-22 Thread Peter Hutterer
On Fri, Nov 13, 2015 at 08:46:42AM +0100, Andreas Pokorny wrote: > Hi, > This version contains several rewordings and documentation additions from > Peter > that I missed in the last update, within the first patch of the series, while > the second patch contains no changes compared to v8. two hi

Re: [PATCH libinput v9 1/2] libinput: add orientation and size of touch point and pressure to the API

2015-11-22 Thread Peter Hutterer
On Fri, Nov 13, 2015 at 08:46:43AM +0100, Andreas Pokorny wrote: > This change adds four new properties to touch events: > * major: diameter of the touch ellipse along the major axis > * minor: diameter perpendicular to major axis > * pressure: a pressure value mapped into the range [0, 1] > * orie

Re: [PATCH libinput v9 2/2] extend tools to print and display touch event properties

2015-11-22 Thread Peter Hutterer
On Fri, Nov 13, 2015 at 08:46:44AM +0100, Andreas Pokorny wrote: > event-gui draws the touch contact as two concentric ellipses that indicate > contact pressure through oppacity. > > Signed-off-by: Andreas Pokorny > --- > tools/event-debug.c | 29 - > tools/event-gui.

Re: [PATCH] Refine compositor grabs behavior

2015-11-22 Thread Peter Hutterer
On Sat, Nov 21, 2015 at 02:42:25PM +, Daniel Stone wrote: > Hi, > > On 20 November 2015 at 03:47, Peter Hutterer wrote: > > On Thu, Nov 12, 2015 at 01:27:05PM +0100, Carlos Garnacho wrote: > >> - There are grabbing situations where the focus surfaces don't really > >> change > >> within th

[PATCH] install pkg-config file to arch-independent location

2015-11-22 Thread Igor Gnatenko
It is arch-independent, so no need to install it to /usr/lib* Cc: Jonas Ådahl Signed-off-by: Igor Gnatenko Reviewed-by: Quentin Glidic --- Makefile.am | 3 +-- configure.ac | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/Makefile.am b/Makefile.am index a32e977..5926a41

Re: [PATCH] install pkg-config file to /usr/share

2015-11-22 Thread Quentin Glidic
On 22/11/2015 13:31, Igor Gnatenko wrote: It is arch-independent, so no need to install it to /usr/lib* Signed-off-by: Igor Gnatenko --- Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile.am b/Makefile.am index a32e977..71b8799 100644 --- a/Makefile.am +

[PATCH] install pkg-config file to /usr/share

2015-11-22 Thread Igor Gnatenko
It is arch-independent, so no need to install it to /usr/lib* Signed-off-by: Igor Gnatenko --- Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile.am b/Makefile.am index a32e977..71b8799 100644 --- a/Makefile.am +++ b/Makefile.am @@ -15,5 +15,5 @@ dist_noinst