[PATCH weston RFC] compositor: Move all views to a new primary output

2016-06-29 Thread Armin Krezović
When primary output gets disconnected and there were views created, they won't get assigned a new output when a new primary output gets plugged in. This will lead to crashes when weston tries to use an already destroyed output. This fixes the problems by force-moving all views to the newly

[PATCH] compositor-x11: Flush xcb connection from x11_output_destroy()

2016-06-29 Thread Armin Krezović
Current code flushes the connection when it receives a delete window request. This means that a destroyed window will remain available when X11 output gets removed differently (ie, from a testing module). Signed-off-by: Armin Krezović --- libweston/compositor-x11.c | 4

Re: [PATCH v4 wayland-protocols 4/4] tablet: Add pad support to the tablet protocol

2016-06-29 Thread Jonas Ådahl
On Tue, Jun 28, 2016 at 11:21:50PM +0200, Carlos Garnacho wrote: > The pad's interface is similar to the tool interface, a client is notified of > the pad after the tablet_added event. > > The pad has three functionalities: buttons, rings and strips. > Buttons are fairly straightforward, rings

Re: [PATCH weston v2 07/12] gl-renderer: Always setup gl-renderer

2016-06-29 Thread Armin Krezović
On 27.06.2016 15:08, Pekka Paalanen wrote: >> diff --git a/src/gl-renderer.c b/src/gl-renderer.c >> index 23c0cd7..28c0b50 100644 >> --- a/src/gl-renderer.c >> +++ b/src/gl-renderer.c >> @@ -2873,6 +2878,43 @@ platform_to_extension(EGLenum platform) >> } >> >> static int >>

[PATCH libinput] touchpad: unify internal/external touchpad tagging

2016-06-29 Thread Peter Hutterer
To unify this we need to move the tagging process forward so tp_init() can rely on it for config setup. This means moving it to the touchpad init code. Other than that no real functional changes, the rules stay the same: * serial/i2c/etc. are considered internal touchpads * Bluetooth is always

[PATCH weston 5/6] tests: update for manual surface/view mapping

2016-06-29 Thread Armin Krezović
This is a follow up for weston-test to manually set mapped status for views/surfaces it controls Signed-off-by: Armin Krezović --- tests/weston-test.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/weston-test.c b/tests/weston-test.c index

[PATCH weston v2 2/6] desktop-shell: update for manual surface/view mapping

2016-06-29 Thread Armin Krezović
This is a follow up for desktop-shell to manually set mapped status for views/surfaces it controls v2: - Add manual mapping to shell_fade_create_surface() and shell_ensure_fullscreen_black_view() Signed-off-by: Armin Krezović --- desktop-shell/input-panel.c | 2 ++

[PATCH weston v2 3/6] fullscreen-shell: update for manual surface/view mapping

2016-06-29 Thread Armin Krezović
This is a follow up for fullscreen-shell to manually set mapped status for views/surfaces it controls v2: - Add manual mapping to fs_output_apply_pending() Signed-off-by: Armin Krezović --- fullscreen-shell/fullscreen-shell.c | 3 +++ 1 file changed, 3 insertions(+)

[PATCH weston v2 6/6] compositor: Switch to new surface/view mapped checks

2016-06-29 Thread Armin Krezović
This patch makes use of new flags which were introduced by previous patches to check if a surface/view is mapped v2: - Rebased to apply on git master - Added comments with link to discussion about proposed changes for weston_{surface,view}_is_mapped() Signed-off-by: Armin Krezović

[PATCH weston v3 1/6] compositor: Untangle mapedness from outputs

2016-06-29 Thread Armin Krezović
Currently, weston assumes a surface/view is mapped if it has an output assigned. In a zero outputs scenario, this isn't really desirable. This patch introduces a new flag to weston_surface and weston_view, which has to be set manually to indicate that a surface/view is mapped. v2: - Remove

Re: Free software license of weston1.9

2016-06-29 Thread Jonas Ådahl
On Thu, Jun 30, 2016 at 04:43:29AM +0200, Armin Krezović wrote: > On 30.06.2016 04:19, 袁嘉伟 wrote: > > Hi, All: > > > > > > Who does know that the free software license of weston 1.9 is GPL or > > LGPL? > > Thanks. > > > > Best regards, > > Anthenony > > > > > > > >

Re: [PATCH v4 wayland-protocols 4/4] tablet: Add pad support to the tablet protocol

2016-06-29 Thread Peter Hutterer
On Tue, Jun 28, 2016 at 11:21:50PM +0200, Carlos Garnacho wrote: > The pad's interface is similar to the tool interface, a client is notified of > the pad after the tablet_added event. > > The pad has three functionalities: buttons, rings and strips. > Buttons are fairly straightforward, rings

Re: Free software license of weston1.9

2016-06-29 Thread Armin Krezović
On 30.06.2016 04:19, 袁嘉伟 wrote: > Hi, All: > > > Who does know that the free software license of weston 1.9 is GPL or LGPL? > Thanks. > > Best regards, > Anthenony > > > > ___ > wayland-devel mailing list >

[PATCH libinput] gestures: make the gesture movement threshold depending on finger count

2016-06-29 Thread Peter Hutterer
Increase the mm move threshold for 3 and 4 finger gestures to 2 and 3 mm, respectively. In multi-finger gestures it's common to have minor movement while all fingers are being put down or before the conscious movement starts. This can trigger invalid gesture detection (e.g. a pinch instead of a

Free software license of weston1.9

2016-06-29 Thread 袁嘉伟
Hi, All: Who does know that the free software license of weston 1.9 is GPL or LGPL? Thanks. Best regards, Anthenony___ wayland-devel mailing list wayland-devel@lists.freedesktop.org

Re: [PATCH wayland-web 1/1] html: Remove trailing whitespace

2016-06-29 Thread Bryce Harrington
On Tue, Jun 28, 2016 at 08:52:33AM -0500, Yong Bakos wrote: > From: Yong Bakos > > The html files contain trailing whitespace, which is an annoyance when making > a change: editors configured to remove trailing whitespace upon save end up > cluttering the diff beyond

[PATCH weston 1/2] input: Rename weston_surface_activate to weston_seat_set_keyboard_focus

2016-06-29 Thread Bryce Harrington
The name suggests that it activates surfaces, but the code says it rather just assigns keyboard focus. Rename it for clarity, and so the original function name could be used for something more appropriate later. Switch order of parameters since keyboard focus is a property of the seat. Update

[PATCH weston 2/2] input: Move weston_seat_set_keyboard_focus and document

2016-06-29 Thread Bryce Harrington
Place it with the other weston_seat functions. Signed-off-by: Bryce Harrington --- libweston/compositor.h | 7 --- libweston/input.c | 34 +++--- 2 files changed, 23 insertions(+), 18 deletions(-) diff --git a/libweston/compositor.h

Re: [PATCH weston v3 2/8] compositor: Track inhibition state in weston_surface

2016-06-29 Thread Bryce Harrington
On Tue, Jun 28, 2016 at 12:15:49PM +0300, Pekka Paalanen wrote: > On Thu, 23 Jun 2016 17:32:50 -0700 > Bryce Harrington wrote: > > > On Thu, May 26, 2016 at 06:01:27PM +0300, Pekka Paalanen wrote: > > > On Thu, 7 Apr 2016 16:44:17 -0700 > > > Bryce Harrington

Re: [PATCH weston 00/11] tests: wrap all pixel data with Pixman

2016-06-29 Thread Pekka Paalanen
On Thu, 23 Jun 2016 15:53:17 +0300 Pekka Paalanen wrote: > On Thu, 23 Jun 2016 18:18:18 +1000 > Daniel Stone wrote: > > > Hi, > > > > On 17 June 2016 at 00:13, Pekka Paalanen wrote: > > > I believe Pixman is the

[PATCH weston v2 2/2] xwayland: make the plugin usable by libweston compositors

2016-06-29 Thread Giulio Camuffo
This patch follows a similar approach taken to detach the backends from weston. But instead of passing a configuration struct when loading the plugin, we use the plugin API registry to register an API, and to get it in the compositor side. This API allows to spawn the Xwayland process in the

[PATCH weston v2 1/2] compositor: add plugin-registry

2016-06-29 Thread Giulio Camuffo
From: Pekka Paalanen Implement a simple register and lookup for function tables. This is intended for plugins to expose APIs to other plugins. It has been very hard to arrange a plugin to be able to call into another plugin without modifying Weston core to

Re: [PATCH weston 2/2] xwayland: make the plugin usable by libweston compositors

2016-06-29 Thread Giulio Camuffo
2016-06-15 16:04 GMT+02:00 Pekka Paalanen : > Hi Giulio, > > was the plugin registry easy to use? Yeah, quite straightforward. Good work on it. >> +static void >> +xserver_cleanup(struct weston_process *process, int status) >> +{ >> + struct wet_xwayland *wxw = >> +

Re: [PATCH weston 12/12] compositor: Switch to new surface/view mapped checks

2016-06-29 Thread Pekka Paalanen
On Wed, 29 Jun 2016 10:26:12 +0200 Giulio Camuffo wrote: > 2016-06-28 14:30 GMT+02:00 Pekka Paalanen : > > On Thu, 23 Jun 2016 11:59:40 +0200 > > Armin Krezović wrote: > > > >> This patch makes use of new flags which were

Re: [PATCH weston v2 2/3] ivi-shell: introduce ivi_layout_view

2016-06-29 Thread Pekka Paalanen
Hi Armin, I just merged the patch below, and I think it conflicts with your ivi-shell patch. Could you rebase on top of this, please? Thanks, pq On Tue, 14 Jun 2016 14:43:40 + "Ucan, Emre (ADITG/SW1)" wrote: > This patch introduces ivi_layout_view data struct, >

Re: [PATCH weston v2] compositor: allow to control the vt switching

2016-06-29 Thread Giulio Camuffo
2016-06-23 9:37 GMT+02:00 Daniel Stone : > Hi Guilio, > > On 5 June 2016 at 03:48, Giulio Camuffo wrote: >> @@ -97,8 +97,17 @@ switch_vt_binding(struct weston_keyboard *keyboard, >> uint32_t time, uint32_t key, void *data) >> { >>

Re: [PATCH weston 3/3] ivi-shell: implement surface_on_many_layer test

2016-06-29 Thread Pekka Paalanen
On Mon, 13 Jun 2016 14:11:44 +0300 Pekka Paalanen wrote: > On Tue, 7 Jun 2016 09:40:21 + > "Ucan, Emre (ADITG/SW1)" wrote: > > > A surface can be added to many layers. > > This test is implemented to test this use-case > > and the correct

Re: [PATCH weston v2 2/3] ivi-shell: introduce ivi_layout_view

2016-06-29 Thread Pekka Paalanen
On Thu, 23 Jun 2016 02:06:24 + "Natsume, Wataru (ADITJ/SWG)" wrote: > Hello Emre-san, > > v2 patch looks fine with me. > It passed review at my side except here. (So trivial.) > > +}; > + <--- It looks better to add blanc. > struct ivi_layout_surface {

Re: [PATCH weston] ivi-shell: add surface_created listener after launchers

2016-06-29 Thread Pekka Paalanen
On Wed, 29 Jun 2016 08:07:22 + "Ucan, Emre (ADITG/SW1)" wrote: > Hi Pekka, > > My comments are below > > Best regards > > Emre Ucan > Software Group I (ADITG/SW1) > > Tel. +49 5121 49 6937 > > > -Original Message- > > From: Pekka Paalanen

Re: [PATCH weston 12/12] compositor: Switch to new surface/view mapped checks

2016-06-29 Thread Giulio Camuffo
2016-06-28 14:30 GMT+02:00 Pekka Paalanen : > On Thu, 23 Jun 2016 11:59:40 +0200 > Armin Krezović wrote: > >> This patch makes use of new flags which were introduced >> by previous patches to check if a surface/view is mapped >> >> Signed-off-by:

RE: [PATCH weston] ivi-shell: add surface_created listener after launchers

2016-06-29 Thread Ucan, Emre (ADITG/SW1)
Hi Pekka, My comments are below Best regards Emre Ucan Software Group I (ADITG/SW1) Tel. +49 5121 49 6937 > -Original Message- > From: Pekka Paalanen [mailto:ppaala...@gmail.com] > Sent: Mittwoch, 29. Juni 2016 09:48 > To: Ucan, Emre (ADITG/SW1) > Cc:

Re: [PATCH weston] ivi-shell: add surface_created listener after launchers

2016-06-29 Thread Pekka Paalanen
On Fri, 17 Jun 2016 13:50:16 + "Ucan, Emre (ADITG/SW1)" wrote: > Add surface_created listener after the initialization of launchers. > Otherwise, surfaces of the launchers will be added to the application > layer too. > > Signed-off-by: Emre Ucan