Re: Window stacking / raising design

2012-01-27 Thread Bill Spitzak
Pekka Paalanem thought this text that I sent was a better description of design goals for window stacking: * Client must make the final decision about raising. This is a requirement so that drag & drop can be implemented, as dragging an object must not raise the window it is in, and only the c

Re: [PATCH 1/2] Add separate page with Qt 5 instructions for Wayland

2012-01-27 Thread Kristian Høgsberg
On Fri, Jan 27, 2012 at 5:03 PM, Tiago Vignatti wrote: > From: Tiago Vignatti Thanks, this and 2/2 pushed. Kristian > Signed-off-by: Tiago Vignatti > --- >  qt5.html      |  134 > + >  toolkits.html |   61 +- >  

Re: [PATCH 2/2] compositor-drm: add sprite support

2012-01-27 Thread Kristian Høgsberg
On Fri, Jan 27, 2012 at 4:41 PM, Jesse Barnes wrote: > Add support for assigning surfaces to overlay sprites using the new > assign_planes hook. Yeah, looks good overall, comments below. > NOTE: this patch fails to handle fbs correctly; a live fb/surface pair > may be removed while still active!

[PATCH 2/2] toolkits: update a bit the instructions

2012-01-27 Thread Tiago Vignatti
From: Tiago Vignatti Removed unnecessary comment for EGL settings. Also, remove the ToC given this page from now on tends to be short. Signed-off-by: Tiago Vignatti --- toolkits.html |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/toolkits.html b/toolkits.html inde

[PATCH 1/2] Add separate page with Qt 5 instructions for Wayland

2012-01-27 Thread Tiago Vignatti
From: Tiago Vignatti Signed-off-by: Tiago Vignatti --- qt5.html | 134 + toolkits.html | 61 +- 2 files changed, 136 insertions(+), 59 deletions(-) create mode 100644 qt5.html diff --git a/qt5.html b/qt5.h

[PATCH 2/2] compositor-drm: add sprite support

2012-01-27 Thread Jesse Barnes
Add support for assigning surfaces to overlay sprites using the new assign_planes hook. NOTE: this patch fails to handle fbs correctly; a live fb/surface pair may be removed while still active! --- configure.ac |2 +- src/compositor-drm.c | 413 +++

[PATCH 1/2] compositor: add an assign_planes hook to the output

2012-01-27 Thread Jesse Barnes
This allows each output back end to optimize drawing using overlay planes and cursors (yet to be integrated). If a surface is assigned to a plane, the back end should clear its damage field so that the later repaint code won't look at it. --- src/compositor-drm.c |1 + src/compositor-open

Re: [PULL] Surface transformations v1

2012-01-27 Thread Bill Spitzak
I have no idea if you need this, but here is free code to do the matrix inversion. This is code I wrote and is public domain (although I certainly referred to a book to generate it, but I can't see that being a copyright problem). The matrix struct uses aRC as the variable name for the entry i

Re: [PATCH] Implement CONFIG_KEY_UNSIGNED_INTEGER since strtol() does not work when trying to assign 32 bits of data into a regular signed int on 32 bit systems. Use corresponding strtoul() instead.

2012-01-27 Thread Kristian Høgsberg
On Fri, Jan 27, 2012 at 3:25 PM, Scott Moreau wrote: Thanks. I fixed up the header and rolled the comment fixup patch into this one as well. Kristian >  clients/desktop-shell.c |    4 ++-- >  shared/config-parser.c  |   10 ++ >  shared/config-parser.h  |    1 + >  3 files changed, 13 i

[PATCH 2/2] Cleanup comments.

2012-01-27 Thread Scott Moreau
--- shared/config-parser.h |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/shared/config-parser.h b/shared/config-parser.h index a83aa4c..2731138 100644 --- a/shared/config-parser.h +++ b/shared/config-parser.h @@ -24,10 +24,10 @@ #define CONFIGPARSER_H enum confi

[PATCH] Implement CONFIG_KEY_UNSIGNED_INTEGER since strtol() does not work when trying to assign 32 bits of data into a regular signed int on 32 bit systems. Use corresponding strtoul() instead.

2012-01-27 Thread Scott Moreau
--- clients/desktop-shell.c |4 ++-- shared/config-parser.c | 10 ++ shared/config-parser.h |1 + 3 files changed, 13 insertions(+), 2 deletions(-) diff --git a/clients/desktop-shell.c b/clients/desktop-shell.c index 4848449..3d5119a 100644 --- a/clients/desktop-shell.c +++ b/

Re: [PATCH] desktop-shell: print what failed on exec

2012-01-27 Thread Kristian Høgsberg
On Fri, Jan 27, 2012 at 2:50 AM, Pekka Paalanen wrote: > Signed-off-by: Pekka Paalanen That's a good improvement, thanks. Kristian > --- > > This is for both master and 0.85 branches. > > clients/desktop-shell.c |    2 +- >  1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/clie

Re: [PATCH] compositor: Damage output when restoring X window.

2012-01-27 Thread Kristian Høgsberg
On Fri, Jan 27, 2012 at 5:39 AM, Scott Moreau wrote: When do you see the damage? Are you not using a compositing window manager? We probably need to do x11_compositor_find_output and then weston_output_damage in the XCB_EXPOSE: case instead. Kristian >  src/compositor-x11.c |    2 ++ >  1 fi

Re: [PATCH] strtol() fails on 32 bit systems when trying to parse a 32 bit 'string' fromt the config file, because it returns long which is signed. Use strtoll to solve the problem.

2012-01-27 Thread Kristian Høgsberg
On Fri, Jan 27, 2012 at 10:53 AM, Scott Moreau wrote: > --- > I'm not sure this is the correct fix but if nothing else, this can serve as a > bug report until the tracker is up. The problem is that config-parser sets > variables to incorrect values on 32 bit systems which causes problems. > Spe

Re: [PATCH] window: fix resizing of windows backed by shm buffers

2012-01-27 Thread Kristian Høgsberg
On Fri, Jan 27, 2012 at 10:17 AM, Ander Conselvan de Oliveira wrote: > When window_attach_surface() calls window_get_resize_dx_dy(), > window->resize_edges is cleared. However if there is already a pending > surface to be attached, the resize won't be done until the following > call to window_atta

[PATCH] strtol() fails on 32 bit systems when trying to parse a 32 bit 'string' fromt the config file, because it returns long which is signed. Use strtoll to solve the problem.

2012-01-27 Thread Scott Moreau
--- I'm not sure this is the correct fix but if nothing else, this can serve as a bug report until the tracker is up. The problem is that config-parser sets variables to incorrect values on 32 bit systems which causes problems. Specifically, this fixes a bug where the background color value is w

[PATCH] window: fix resizing of windows backed by shm buffers

2012-01-27 Thread Ander Conselvan de Oliveira
When window_attach_surface() calls window_get_resize_dx_dy(), window->resize_edges is cleared. However if there is already a pending surface to be attached, the resize won't be done until the following call to window_attach_surface(). In this next call, since resize_edges is now zero, the top-left

[PULL] Surface transformations v1

2012-01-27 Thread Pekka Paalanen
Hi Kristian, and everyone, this is the bulk of the surface transformations work. For non-transformed surfaces, it should not regress. A quick Valgrind run showed no new issues. Surfaces can now have any number of arbitrary projective-linear (4x4 matrix) transformations. The transformations are us

Re: [PATCH 2/2] Check if background image exists instead of blindly using it.

2012-01-27 Thread Scott Moreau
It looks like 8129bc0fb9700e014455702446f21eb004ce448a already fixes this problem already. I was slightly out-of-date. ___ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel

[PATCH 2/2] Check if background image exists instead of blindly using it.

2012-01-27 Thread Scott Moreau
--- clients/desktop-shell.c | 25 ++--- 1 files changed, 14 insertions(+), 11 deletions(-) diff --git a/clients/desktop-shell.c b/clients/desktop-shell.c index f50f425..17f049f 100644 --- a/clients/desktop-shell.c +++ b/clients/desktop-shell.c @@ -374,17 +374,20 @@ backgroun

[PATCH] compositor: Damage output when restoring X window.

2012-01-27 Thread Scott Moreau
--- src/compositor-x11.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/compositor-x11.c b/src/compositor-x11.c index 09213f7..3afd6fa 100644 --- a/src/compositor-x11.c +++ b/src/compositor-x11.c @@ -599,6 +599,8 @@ x11_compositor_handle_event(int fd, uint32_t mask,