Re: Expected behaviour of eglSwapBuffers and the event queue

2013-02-28 Thread Sam Spilsbury
Hey, thanks for the followup On Fri, Mar 1, 2013 at 12:50 PM, Daniel Stone wrote: > Hi, > >> >> I wanted to know what the expected behaviour around this was meant to >> be. If eglSwapBuffers is meant to read () the fd, then that means that >> we need to rely on it being called in order to popula

Re: Expected behaviour of eglSwapBuffers and the event queue

2013-02-28 Thread Daniel Stone
Hi, On 28 February 2013 20:02, Sam Spilsbury wrote: > Because all of the data has been read from the fd, there is a > condition where we have read every single event that needed to be > delivered to us. Then when we return to poll (), it will block > indefinitely because there is nothing left in

Re: [PATCH weston v2] toytoolkit: avoid redrawing a window between maximize and configure

2013-02-28 Thread Scott Moreau
Hi, I tested this patch briefly and it fixes the issue. A couple comments below. On Thu, Feb 28, 2013 at 5:24 PM, MoD wrote: > Resolve a bad frame visible when maximizing toytoolkit programs with the > the > maximize button in decorations. Windows now use wl_display.sync requests to > wait for

Expected behaviour of eglSwapBuffers and the event queue

2013-02-28 Thread Sam Spilsbury
Hi, I'm a little confused on the behaviour of eglSwapBuffers operating on the event queue. At the moment, it calls wl_connection_read () on the display fd, which will read events into the local event buffer. I imagine it needs to do this in order to fetch some information about the next backbuffer

Re: Patch queue update

2013-02-28 Thread Scott Moreau
> > * surface_data and surface panel list (Scott Moreau) > > - This one was in good shape at the last resend, but I think there > was a few more issues we still we discussion. I forget (sorry). > Hi Kristian, you reviewed the main patch and commented here: http://lists.freedesktop.org/archi

[PATCH weston v2] toytoolkit: avoid redrawing a window between maximize and configure

2013-02-28 Thread MoD
Resolve a bad frame visible when maximizing toytoolkit programs with the the maximize button in decorations. Windows now use wl_display.sync requests to wait for a maximize to finish before drawing again, following suggestions from http://lists.freedesktop.org/archives/wayland-devel/2013-February/0

[PATCH v2] shell: grab the parent popup when a sub popup window is deleted or hidden

2013-02-28 Thread Giulio Camuffo
If the client opens a popup menu and submenu, when it closes or hides the submenu the pointer grab should return to the parent menu. Furthermore, when clicking outside the client area the popup_done event is sent to the active popup surface and recursively to its parent. --- src/shell.c | 57 +

Re: [PATCH 1/5] server: Create socket path prior the socket

2013-02-28 Thread Benjamin Franzke
2013/2/28 Kristian Høgsberg : > On Wed, Feb 27, 2013 at 12:10:16PM +0100, Benjamin Franzke wrote: >> This is just in preparation for systemd socket activation >> since we need to know the path before creating/acquiring the socket. > > Hey Benjamin, > > The patches look good and are fairly non-intru

Re: Patch queue update

2013-02-28 Thread Giulio Camuffo
I think you've forgotten http://lists.freedesktop.org/archives/wayland-devel/2013-February/007523.html :) Giulio 2013/2/28 Kristian Høgsberg : > Here's the patch queue as of Feb 28. > > Kristian > > > * Per-wl_object dispatchers (Jason Ekstrand, Feb 28) > > - Looks good. Just need to fix the "

Patch queue update

2013-02-28 Thread Kristian Høgsberg
Here's the patch queue as of Feb 28. Kristian * Per-wl_object dispatchers (Jason Ekstrand, Feb 28) - Looks good. Just need to fix the "creative" wl_resource use in weston. * systemd socket activation patches (Benjamin Franzke, Feb 27) - Patches look fine, but not sure when we'd be

Re: [PATCH 1/5] server: Create socket path prior the socket

2013-02-28 Thread Kristian Høgsberg
On Wed, Feb 27, 2013 at 12:10:16PM +0100, Benjamin Franzke wrote: > This is just in preparation for systemd socket activation > since we need to know the path before creating/acquiring the socket. Hey Benjamin, The patches look good and are fairly non-intrusive, but I'm not sure when weston would

Re: [RFC wayland 0/4] Custom dispatchers via wl_object instead of

2013-02-28 Thread Kristian Høgsberg
On Wed, Feb 27, 2013 at 09:29:54PM -0600, Jason Ekstrand wrote: > This sequence demonstrates how we could implement custom dispatchers via > wl_object instead of wl_interface. This has the added benefit of not breaking > ABI client-side and only breaking it server-side. See also: > > http://list

Re: [PATCH weston] Convert resources to use wl_resource_init

2013-02-28 Thread Kristian Høgsberg
On Thu, Feb 28, 2013 at 01:07:35PM +0200, Pekka Paalanen wrote: > On Wed, 27 Feb 2013 21:31:31 -0600 > Jason Ekstrand wrote: > > > This patch goes along with the 4 wayland patches and updates weston to build > > against them. The main question I have here is what we should do with the > > shell

Re: [PATCH 1/2] compositor: Make sure surface->transform.matrix,inverse are always valid

2013-02-28 Thread Kristian Høgsberg
On Thu, Feb 28, 2013 at 12:27:41PM +0200, Pekka Paalanen wrote: > On Wed, 20 Feb 2013 12:46:42 -0500 > Kristian Høgsberg wrote: > > > On Wed, Feb 20, 2013 at 01:05:42PM +0200, Pekka Paalanen wrote: > > > On Tue, 19 Feb 2013 17:04:50 -0500 > > > Kristian Høgsberg wrote: > > > > > > > First step

Re: [PATCH web] docs: update qt5 documentation

2013-02-28 Thread darxus
Great, thanks, committed. HTML didn't validate, so I corrected that after: http://cgit.freedesktop.org/wayland/wayland-web/commit/?id=bb8b63cf1f5b6e210572d04678adcddebbccfb2b You had a inside a , unquoted url, and an un-escaped "&" (should be "&"). I like http://www.htmlhelp.com/tools/valida

[PATCH web] docs: update qt5 documentation

2013-02-28 Thread Giulio Camuffo
--- qt5.html | 55 +-- 1 file changed, 25 insertions(+), 30 deletions(-) diff --git a/qt5.html b/qt5.html index 657f96b..fc09ce3 100644 --- a/qt5.html +++ b/qt5.html @@ -11,16 +11,10 @@ Qt 5 on Wayland -Wayland support in the Qt 5 toolkit i

Re: [PATCH 4/5] compositor: Implement pointer_lock

2013-02-28 Thread Kristian Høgsberg
On Tue, Feb 26, 2013 at 02:58:37PM +0200, Pekka Paalanen wrote: > On Mon, 25 Feb 2013 21:33:24 -0500 > Kristian Høgsberg wrote: > > > The pointer lock extension lets a client lock pointer motion and receive > > relative pointer motion events. This patch implements the weston side > > of the exte

[PATCH] tests: attach the surface's buffer when moving it

2013-02-28 Thread Giulio Camuffo
move_client() needs to attach the buffer, even if it was attached already, because since 184df50 configure() will be called only on newly attached surfaces, but the one that sets the test surface position is the configure function. --- tests/weston-test-client-helper.c | 5 - 1 file changed, 4

Re: [PATCH 5/5] resizor: Lock pointer on click

2013-02-28 Thread Kristian Høgsberg
On Tue, Feb 26, 2013 at 02:16:37PM +0200, Pekka Paalanen wrote: > On Mon, 25 Feb 2013 21:33:25 -0500 > Kristian Høgsberg wrote: > > > We add a resize mode to resizor where you can click to resize it using > > pointer lock and relative motion. > > --- > > clients/resizor.c | 74 > > +

Re: [PATCH 4/5] compositor: Implement pointer_lock

2013-02-28 Thread Kristian Høgsberg
On Tue, Feb 26, 2013 at 01:55:11PM +0200, Pekka Paalanen wrote: > On Mon, 25 Feb 2013 21:33:24 -0500 > Kristian Høgsberg wrote: > > > The pointer lock extension lets a client lock pointer motion and receive > > relative pointer motion events. This patch implements the weston side > > of the exte

[PATCH v2] compositor: send a pointer motion if the focus surface moves under the pointer

2013-02-28 Thread Giulio Camuffo
This makes weston_device_repick() return an int, 1 if the pointer moved on the same surface, 0 otherwise. notify_motion sends the motion event on its own, while weston_compositor_repick() checks if it returns 1 and it sends the motion event using the time given to it by weston_output_repaint() as t

Re: [PATCH weston] Convert resources to use wl_resource_init

2013-02-28 Thread Pekka Paalanen
On Wed, 27 Feb 2013 21:31:31 -0600 Jason Ekstrand wrote: > This patch goes along with the 4 wayland patches and updates weston to build > against them. The main question I have here is what we should do with the > shell surface and the input panel surface. For some reason, the creation of > the

Re: [PATCH 1/2] compositor: Make sure surface->transform.matrix,inverse are always valid

2013-02-28 Thread Pekka Paalanen
On Wed, 20 Feb 2013 12:46:42 -0500 Kristian Høgsberg wrote: > On Wed, Feb 20, 2013 at 01:05:42PM +0200, Pekka Paalanen wrote: > > On Tue, 19 Feb 2013 17:04:50 -0500 > > Kristian Høgsberg wrote: > > > > > First step towards getting rid of the transform.enabled optimization. > > > --- > > > src/

Re: [PATCH 1/2] animation: zoom: don't start animation with alpha set to 0

2013-02-28 Thread Pekka Paalanen
On Wed, 27 Feb 2013 16:08:30 -0500 Kristian Høgsberg wrote: > On Sun, Feb 24, 2013 at 06:43:33AM +0100, Rune K. Svendsen wrote: > > From: "Rune K. Svendsen" > > > > I was doing some research on why I thought enabling the zoom/fade > > animations added latency (a gap between releasing the launch

Re: [PATCH 2/2] compositor: Remove surface->transform.enabled

2013-02-28 Thread Pekka Paalanen
On Wed, 27 Feb 2013 17:17:55 -0500 Kristian Høgsberg wrote: > On Thu, Feb 21, 2013 at 11:05:41AM +0200, Pekka Paalanen wrote: > > On Wed, 20 Feb 2013 11:52:18 -0500 > > Kristian Høgsberg wrote: > > > > > We have matrix.type now and can rely on that for optimizing > > > transformations > > > an

Re: [PATCH v3] compositor: send a pointer motion if the focus surface moves under the pointer

2013-02-28 Thread Pekka Paalanen
On Wed, 27 Feb 2013 16:01:18 -0500 Kristian Høgsberg wrote: > On Thu, Feb 21, 2013 at 04:36:20PM +0100, Giulio Camuffo wrote: > > --- > > src/compositor.c | 19 +++ > > 1 file changed, 15 insertions(+), 4 deletions(-) > > > > diff --git a/src/compositor.c b/src/compositor.c > >