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
doc/svg/tablet-intuos-modes.svg | 321 ++
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 changed, 126 insertions(+), 126 de
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 P
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
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 @@ START_TEST(pad_left_hande
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 b/src/libinput.c
index 89fa594..0f676a3 10
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| 12 +
src/evdev.h
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
@@ -136,7 +136,7 @@ enum libinput_p
On Jun 5, 2016, at 12:01 PM, 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.
>
> Signed-of-by
On Fri, Jun 03, 2016 at 12:49:45PM +0200, Hans de Goede wrote:
> Hi,
>
> On 03-06-16 03:59, Peter Hutterer wrote:
> > A large part of the bugs seen right now are related to touchpads jittering
> > too
> > much. Fixing them one by one is entertaining but time consuming. Right now
> > the number of
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
>>
>> v2: Rewrap, add a couple of missing words (Pekka).
>>
>> Signed-off-by: Emil Velikov
>
> I definitely agree that updating the README, especially reg
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
> libweston source files.
>
> This is moving all the files related to the 'weston' binary. Also the
> CMS and systemd plugins are moved.
>
> xwayland plugi
Maybe it would make sense to squash this with the previous one?
Cheers,
Giulio
2016-06-03 15:41 GMT+03:00 Pekka Paalanen :
> From: Pekka Paalanen
>
> There is no need to pass the backend name string to these functions
> anymore.
>
> Signed-off-by: Pekka Paalanen
> ---
> src/main.c | 24 ++
git am complains on this one:
Applying patch #91319 using 'git am'
Description: [weston,2/5] compositor, main: use weston_compositor_load_backend()
Applying: compositor, main: use weston_compositor_load_backend()
.git/rebase-apply/patch:44: new blank line at EOF.
+
warning: 1 line adds whitespace
2016-06-03 15:41 GMT+03:00 Pekka Paalanen :
> From: Pekka Paalanen
>
> Backends do not have access to command line elements nor weston_config
> anymore. The use the backend-specific config APIs now.
Missing a y here, The -> They.
Cheers,
Giulio
>
> Signed-off-by: Pekka Paalanen
> ---
> src/co
Hi,
i have a few nitpicks on some patches, but for the whole series:
Reviewed-by: Giulio Camuffo
2016-06-03 16:12 GMT+03:00 Quentin Glidic :
> On 03/06/2016 14:41, Pekka Paalanen wrote:
>>
>> From: Pekka Paalanen
>>
>> Now that the rpi-backend is gone, we can clean this temporary stuff out,
>>
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.
Signed-of-by: Benoit Gschwind
---
v3:
- fix xcb_destroy_window arguments
v2:
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.
Signed-of-by: Benoit Gschwind
---
src/compositor-x11.c | 28 +++
Hello Yong,
Thanks for the review.
On 05/06/2016 17:47, Yong Bakos wrote:
> Hi Benoit,
> Other than the comments inline below, this patch is
> Reviewed-by: Yong Bakos
> Tested-by: Yong Bakos
>
> On Jun 4, 2016, at 9:08 AM, Benoit Gschwind wrote:
>>
>> ---
>
> I'd love to see a short message
From: Yong Bakos
Change the output make value from "waywayland" to "wayland".
References: 90bc88c710b34f46ef89e1c5765e5f63f8e02847
Signed-off-by: Yong Bakos
---
src/compositor-wayland.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/compositor-wayland.c b/src/composit
Hi Benoit,
Other than the comments inline below, this patch is
Reviewed-by: Yong Bakos
Tested-by: Yong Bakos
On Jun 4, 2016, at 9:08 AM, Benoit Gschwind wrote:
>
> ---
I'd love to see a short message for the patch here explaining the
rationale and behavior change, now that it's possible to re
21 matches
Mail list logo