Re: [PATCH] compositor: update the surface size when there is a size change without a buffer attach

2014-06-18 Thread Pekka Paalanen
On Wed, 18 Jun 2014 12:38:50 -0700 Jason Ekstrand wrote: > On Fri, Jun 13, 2014 at 9:14 AM, George Kiagiadakis < > george.kiagiada...@collabora.com> wrote: > > > This fixes at least the case where you want to do > > wl_viewport.set_destination > > to resize the surface but without attaching new

RE: [PATCH] build: Silence libtoolize warnings

2014-06-18 Thread Srivardhan
> -Original Message- > From: wayland-devel [mailto:wayland-devel- > boun...@lists.freedesktop.org] On Behalf Of Pekka Paalanen > Sent: Thursday, June 19, 2014 12:17 PM > To: Thierry Reding > Cc: Kristian Høgsberg; wayland-devel@lists.freedesktop.org > Subject: Re: [PATCH] build: Silence l

Re: [PATCH] build: Silence libtoolize warnings

2014-06-18 Thread Pekka Paalanen
On Wed, 18 Jun 2014 16:28:45 -0700 Kristian Høgsberg wrote: > On Tue, May 27, 2014 at 09:07:54AM +0200, Thierry Reding wrote: > > From: Thierry Reding > > > > When running the autogen.sh script, libtoolize complains thusly: > > > > libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to

[PATCH] weston-build: Fix aclocal error during autogen.sh

2014-06-18 Thread Srivardhan Hebbar
This is a fix to the following bug: https://bugs.freedesktop.org/show_bug.cgi?id=80214 Signed-off-by: Srivardhan Hebbar --- m4/.gitignore |2 ++ 1 file changed, 2 insertions(+) create mode 100644 m4/.gitignore diff --git a/m4/.gitignore b/m4/.gitignore new file mode 100644 index 000..c

Re: [PATCH libinput 1/4] evdev: Continue updating axes internally when tool leaves proximity

2014-06-18 Thread Peter Hutterer
On Thu, Jun 19, 2014 at 01:18:06AM -0400, Stephen Chandler Paul wrote: > Axis changes are now still processed by libinput regardless of whether or not > the tool is in proximity, however we refrain from reporting them unless the > tool > is in proximity. This stops bad distance events from being r

Re: [PATCH weston] desktop-shell: Don't assume there is a pointer when resizing

2014-06-18 Thread Kristian Høgsberg
On Wed, Jun 18, 2014 at 05:48:58PM +0200, Emilio Pozuelo Monfort wrote: > From: Emilio Pozuelo Monfort > > Fixes a crash on touch devices without a pointer, when touching > the window frame of a client. Thanks, applied. At some point we'll get rid of all these pointer assumptions. Kristian >

[PATCH libinput 2/4] evdev: Add tablet_mark_all_axes_changed()

2014-06-18 Thread Stephen Chandler Paul
Occasionally all the axes that are valid for a device need to be marked as updated for the caller's sake. Signed-off-by: Stephen Chandler Paul --- src/evdev-tablet.c | 26 +- 1 file changed, 17 insertions(+), 9 deletions(-) diff --git a/src/evdev-tablet.c b/src/evdev-tab

[PATCH libinput 4/4] test: Add tests for serial numbers on tools

2014-06-18 Thread Stephen Chandler Paul
These tests make sure that any tablets with the capability to report a tool's serial number do so properly, that the tool changes when another tool of the same type with a different serial number is used, and that libinput doesn't change the current tool when -1 is reported as the serial number (-1

[PATCH libinput 1/4] evdev: Continue updating axes internally when tool leaves proximity

2014-06-18 Thread Stephen Chandler Paul
Axis changes are now still processed by libinput regardless of whether or not the tool is in proximity, however we refrain from reporting them unless the tool is in proximity. This stops bad distance events from being reported without needing a huge mess of conditional statements in sanitize_axes()

[PATCH libinput 3/4] evdev: Update all axes when the tool comes back into proximity

2014-06-18 Thread Stephen Chandler Paul
Signed-off-by: Stephen Chandler Paul --- src/evdev-tablet.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/evdev-tablet.c b/src/evdev-tablet.c index 52499fd..4efb36c 100644 --- a/src/evdev-tablet.c +++ b/src/evdev-tablet.c @@ -85,6 +85,7 @@ tablet_mark_all_axes_changed

[PATCH libinput v2 3/3] Change the logging system to be per-context

2014-06-18 Thread Peter Hutterer
Rather than a single global logging function, make the logging dependent on the individual context. This way we won't stomp on each other's feet in the (admittedly unusual) case of having multiple libinput contexts. The userdata argument to the log handler was dropped. The caller has a ref to the

[PATCH libinput 4/4] Add a function to get the size of a device

2014-06-18 Thread Peter Hutterer
Signed-off-by: Peter Hutterer --- src/evdev.c | 19 +++ src/evdev.h | 5 + src/libinput.c | 10 ++ src/libinput.h | 19 +++ tools/event-debug.c | 8 +++- 5 files changed, 60 insertions(+), 1 deletion(-) diff --git a/src/

[PATCH libinput 1/4] evdev: keep the absinfo struct around instead of min/max

2014-06-18 Thread Peter Hutterer
We'll need that later for conversion to mm. Signed-off-by: Peter Hutterer --- src/evdev-mt-touchpad-buttons.c | 20 +--- src/evdev-mt-touchpad.c | 6 -- src/evdev.c | 25 + src/evdev.h | 2 +- 4 files c

[PATCH libinput 2/4] evdev: force a resolution of 1 where no resolution is set

2014-06-18 Thread Peter Hutterer
Avoids nasty surprises later when we divide by 0. This matters particularly when testing a device through uinput, which can't set the resolution. Signed-off-by: Peter Hutterer --- src/evdev.c | 25 + 1 file changed, 25 insertions(+) diff --git a/src/evdev.c b/src/evdev.c

[PATCH libinput 3/4] Change absolute and touch events to use mm as default unit

2014-06-18 Thread Peter Hutterer
Instead of device-specific coordinates that the caller can't interpret without knowing the range anyway, return mm as the default value. Signed-off-by: Peter Hutterer --- src/evdev.h | 7 +++ src/libinput.c | 20 src/libinput.h | 32 ---

Re: [PATCH 1/3] tests: add expect_protocol_error function

2014-06-18 Thread Bryce W. Harrington
On Mon, May 26, 2014 at 04:58:05PM +0200, Marek Chalupa wrote: > This function checks if a particular protocol error came in wire. > It's usefull in the cases where we hitherto used FAIL_TEST. > The problem with FAIL_TEST is that *any* assert will pass the test, > but we want only some asserts to p

Re: [PATCH] client: extend error handling

2014-06-18 Thread Bryce W. Harrington
On Mon, May 26, 2014 at 04:57:23PM +0200, Marek Chalupa wrote: > When an error occurs, wl_display_get_error() does not > provide any way of getting know if it was a local error or if it was > an error event, respectively what object caused the error and what > the error was. > > This patch introdu

Re: [PATCH libinput 00/23] Revised: Tablet support in libinput

2014-06-18 Thread Peter Hutterer
On Thu, Jun 12, 2014 at 11:28:21PM -0400, Stephen Chandler Paul wrote: > Hello! I've done a ton of revisions and changes to the patches Carlos sent > a little while ago to add tablet support to libinput. Feel free to let me > know what you think. For the archives: I've pushed those patches and a c

Re: [PATCH libinput 1/2] evdev: Safe x and y resolution for later use

2014-06-18 Thread Peter Hutterer
On Wed, Jun 18, 2014 at 02:22:23PM +0200, Hans de Goede wrote: > Signed-off-by: Hans de Goede > --- > src/evdev.c | 2 ++ > src/evdev.h | 2 +- > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/src/evdev.c b/src/evdev.c > index 51ad5e3..b72e5e1 100644 > --- a/src/evdev.c > +++ b

Re: [PATCH] connection: remove unreached code

2014-06-18 Thread Kristian Høgsberg
On Wed, Jun 04, 2014 at 11:39:08AM +0800, Boyan Ding wrote: Heh, oops, that was silly. Patch committed, thanks. Kristian > --- > src/connection.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/src/connection.c b/src/connection.c > index 47ee556..f292853 100644 > --- a/src/connectio

Re: [PATCH weston client window] Fixed the input region of the popup menu

2014-06-18 Thread Kristian Høgsberg
On Mon, Jun 02, 2014 at 01:53:38PM +0200, Tomek Obrebski wrote: > Changed the input region of the menu popup window to exclude the shadow and > border regions and set to frame's internal region only. This is a good patch but there are a few stylistic issues. First, the commit message is usually

Re: [PATCH libinput 2/2] evdev-mt-touchpad: tp_current_touch: Fix off by one error

2014-06-18 Thread Peter Hutterer
On Wed, Jun 18, 2014 at 02:22:24PM +0200, Hans de Goede wrote: > Signed-off-by: Hans de Goede > --- > src/evdev-mt-touchpad.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c > index 0294eb2..92e0651 100644 > --- a/src/evd

Re: [PATCH weston] weston/configure.ac: make lcms configurable

2014-06-18 Thread Kristian Høgsberg
On Fri, May 30, 2014 at 09:51:32AM -0700, Tim Orling wrote: > From: Tim Orling > > Make lcms support explicitly configurable. > > Signed-off-by: Tim Orling > --- > configure.ac | 22 +- > 1 file changed, 17 insertions(+), 5 deletions(-) > > diff --git a/configure.ac b/conf

Re: [PATCH] weston: Fix memleak issue in compositor.c

2014-06-18 Thread Kristian Høgsberg
On Tue, May 27, 2014 at 02:30:59PM +0530, Srivardhan Hebbar wrote: > This fix is upon "Hardening " patch. [PATCH 2/2] Handle > OOM with signal events. Thanks, applied. I edited the commit message to wrap at 72 columns. Kristian > > Signed-off-by: Srivardhan Hebbar > --- > src/compositor.c |

Re: [PATCH] compositor-wayland: Fix compiler warning

2014-06-18 Thread Kristian Høgsberg
On Tue, May 27, 2014 at 09:08:29AM +0200, Thierry Reding wrote: > From: Thierry Reding > > sizeof returns size_t, for which the correct printf specifier is %zu. > Fixes the following warning when building for ARMv7. > > src/compositor-wayland.c: In function 'wayland_output_get_shm_buffer':

Re: [PATCH] build: Silence libtoolize warnings

2014-06-18 Thread Kristian Høgsberg
On Tue, May 27, 2014 at 09:07:54AM +0200, Thierry Reding wrote: > From: Thierry Reding > > When running the autogen.sh script, libtoolize complains thusly: > > libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.ac > and > libtoolize: rerunning libtoolize, to keep t

Re: [PATCH weston 5/5] desktop-shell: make background applications less dark

2014-06-18 Thread Kristian Høgsberg
On Thu, May 22, 2014 at 10:41:34PM +0200, Jonny Lamb wrote: Patch applied. Kristian > --- > desktop-shell/shell.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/desktop-shell/shell.c b/desktop-shell/shell.c > index 51683ee..a5886d7 100644 > --- a/desktop-shell/s

Re: [PATCH weston 4/5] animation: ensure repaints are always scheduled during animations

2014-06-18 Thread Kristian Høgsberg
On Thu, May 22, 2014 at 10:41:33PM +0200, Jonny Lamb wrote: > Animations are run off the repaint cycle so if there's nothing to > repaint, an animation will stop running. This is usually not a problem > as each frame function of an animation causes something to change and > therefore a repaint to h

Re: [PATCH weston 3/5] desktop-shell: use panel location to calculate correct sizes and ranges

2014-06-18 Thread Kristian Høgsberg
On Thu, May 22, 2014 at 10:41:32PM +0200, Jonny Lamb wrote: > Now the client can let us know where the panel is using > desktop_shell.set_panel_position, we can correctly calculate where to > put new views and how big maximized views should be. > --- > desktop-shell/shell.c | 229 > ++

Re: [PATCH weston 1/5] animation: fix move scale animation

2014-06-18 Thread Kristian Høgsberg
On Thu, May 22, 2014 at 10:41:30PM +0200, Jonny Lamb wrote: > Both weston_move_scale_run() and weston_slide_run() were broken in > commit 3a869019. Commit a4a6f161 fixed and explained the problem for > weston_slide_run() but weston_move_scale_run() remained broken. > > To fix weston_move_scale_run

Re: [PATCH weston] exposay: fix crash when navigating with the keyboard

2014-06-18 Thread Kristian Høgsberg
On Sat, May 24, 2014 at 02:43:04AM +0200, Emilio Pozuelo Monfort wrote: > From: Emilio Pozuelo Monfort > > Commit a7592019 introduced an optimization that caused some > exposay struct members to not be properly initialized, particularly > cur_output, leading to crashes in some circumstances (e.g.

Re: [PATCH] protocol: remove redundant 'the' in description

2014-06-18 Thread Kristian Høgsberg
On Fri, May 23, 2014 at 07:26:56AM +0200, Silvan Jegen wrote: > Signed-off-by: Silvan Jegen Thanks, applied. Kristian > --- > protocol/wayland.xml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/protocol/wayland.xml b/protocol/wayland.xml > index 22eb6e7..113c7b7

Re: [PATCH weston] Apply output transform to the cursor while copying into a plane

2014-06-18 Thread Kristian Høgsberg
On Wed, May 21, 2014 at 07:36:21PM +0100, Neil Roberts wrote: > Previously if the output had a transform then the cursor plane would > be disabled. However as we have to copy the cursor image into a buffer > with the CPU anyway it probably won't cost much extra to also apply > the transform in the

Re: browser status on "pure wayland"

2014-06-18 Thread Andrea Giammarchi
I'll try that, and I've been at Tizen conf and know about Xwalk but building Wayland images only from a X based system is like cheating :-) I'd like to build Wayland only ozone/chromium from a Wayland only system and I'll try that flag ASAP, thanks! On Wed, Jun 18, 2014 at 11:59 AM, Tiago Vignat

Re: [PATCH] compositor: update the surface size when there is a size change without a buffer attach

2014-06-18 Thread Jason Ekstrand
On Fri, Jun 13, 2014 at 9:14 AM, George Kiagiadakis < george.kiagiada...@collabora.com> wrote: > This fixes at least the case where you want to do > wl_viewport.set_destination > to resize the surface but without attaching new content in it. > --- > src/compositor.c | 60 > +++

Re: browser status on "pure wayland"

2014-06-18 Thread Tiago Vignatti
Brazil was playing in the World Cup yesterday, so yeah I was busy :) But I've seen your messages in the chat log. I think all you need is to set -Duse_x11=0 probably. I fixed that in Chromium once and it worked like a charm (in fact Tizen has been building Wayland-only images in awhile with th

Re: browser status on "pure wayland"

2014-06-18 Thread Andrea Giammarchi
I popped it yesterday in freenode but everyone was probably busy :-) Meanwhile, I've discovered that depot_tools to build chrome and chrome on ozone including GYP and not sure about ninja requires GTK2 for some reason so I am again back to the same problem: I really would like to build in the mach

Re: browser status on "pure wayland"

2014-06-18 Thread Tiago Vignatti
It's not about Wayland per se that made VDA on Linux possible... in short, Chromium devs couldn't afford maintenance of different decoding libraries on Linux for different hw vendors so they preferred to only maintain whatever comes on ChromeOS and its devices: https://code.google.com/p/chromi

Re: browser status on "pure wayland"

2014-06-18 Thread Yomi Ogunwumi
What made adding video decoding acceleration in Chromium on Wayland possible? Would I be correct in guessing that Chromium on X11 doesn't have this functionality? Yomi On Jun 17, 2014 9:36 AM, "Tiago Vignatti" wrote: > yup, Ozone-Wayland enables a bunch of Chromium based targets including the >

[PATCH weston] desktop-shell: Don't assume there is a pointer when resizing

2014-06-18 Thread Emilio Pozuelo Monfort
From: Emilio Pozuelo Monfort Fixes a crash on touch devices without a pointer, when touching the window frame of a client. Signed-off-by: Emilio Pozuelo Monfort --- desktop-shell/shell.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/desktop-shell/shell.c b/desktop-shell

[PATCH libinput 2/2] evdev-mt-touchpad: tp_current_touch: Fix off by one error

2014-06-18 Thread Hans de Goede
Signed-off-by: Hans de Goede --- src/evdev-mt-touchpad.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c index 0294eb2..92e0651 100644 --- a/src/evdev-mt-touchpad.c +++ b/src/evdev-mt-touchpad.c @@ -139,7 +139,7 @@ tp_motion_hi

[PATCH libinput 1/2] evdev: Safe x and y resolution for later use

2014-06-18 Thread Hans de Goede
Signed-off-by: Hans de Goede --- src/evdev.c | 2 ++ src/evdev.h | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/evdev.c b/src/evdev.c index 51ad5e3..b72e5e1 100644 --- a/src/evdev.c +++ b/src/evdev.c @@ -608,11 +608,13 @@ evdev_configure_device(struct evdev_device *devi

Re: [PATCH libinput 3/3] Change the logging system to be per-context

2014-06-18 Thread Jonas Ådahl
On Wed, Jun 18, 2014 at 05:52:04PM +1000, Peter Hutterer wrote: > On Wed, Jun 18, 2014 at 09:24:03AM +0200, Jonas Ådahl wrote: > > > > The purpose of the struct was to provide an interface with the > > > > functionality that libinput would require to have to function without > > > > having to be ro

Re: [PATCH libinput 3/3] Change the logging system to be per-context

2014-06-18 Thread Peter Hutterer
On Wed, Jun 18, 2014 at 09:24:03AM +0200, Jonas Ådahl wrote: > > > The purpose of the struct was to provide an interface with the > > > functionality that libinput would require to have to function without > > > having to be root, and it doesn't feel logging function fits this > > > purpose. It was

Re: [PATCH libinput 3/3] Change the logging system to be per-context

2014-06-18 Thread Jonas Ådahl
On Wed, Jun 18, 2014 at 08:39:01AM +1000, Peter Hutterer wrote: > On Tue, Jun 17, 2014 at 11:20:00PM +0200, Jonas Ådahl wrote: > > On Fri, Jun 13, 2014 at 12:48:33PM +1000, Peter Hutterer wrote: > > > Rather than a single global logging function, make the logging dependent > > > on > > > the indiv