[PATCH wayland v3] abort if a listener function is NULL

2014-10-01 Thread Ryo Munakata
Signed-off-by: Ryo Munakata --- src/connection.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/connection.c b/src/connection.c index f292853..f516c85 100644 --- a/src/connection.c +++ b/src/connection.c @@ -931,6 +931,11 @@ wl_closure_invoke(struct wl_closure *closure, uint32_t fl

Re: [PATCH wayland v2] assert if a listener function is NULL

2014-10-01 Thread Giulio Camuffo
2014-10-01 14:36 GMT+03:00 Ryo Munakata : > Signed-off-by: Ryo Munakata > --- > src/connection.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/src/connection.c b/src/connection.c > index f292853..424edd4 100644 > --- a/src/connection.c > +++ b/src/connection.c > @@ -931,6 +931,11

[PATCH wayland 0/4] Implement role conflict errors

2014-10-01 Thread Pekka Paalanen
From: Pekka Paalanen Hi, this is my attempt on the wl_surface role conflict (trying to change a surface role) error patches. Jasper's original version is here: http://lists.freedesktop.org/archives/wayland-devel/2014-August/016810.html http://lists.freedesktop.org/archives/wayland-devel/2014-Aug

[PATCH weston 2/4] protocol: add role error code to xdg_shell

2014-10-01 Thread Pekka Paalanen
From: Pekka Paalanen Needed for properly reporting role violations from xdg_shell.get_xdg_surface and .get_xdg_popup. Signed-off-by: Pekka Paalanen --- protocol/xdg-shell.xml | 9 + 1 file changed, 9 insertions(+) diff --git a/protocol/xdg-shell.xml b/protocol/xdg-shell.xml index 2758

Re: [PATCH wayland v4 1/5] protocol: define the concept of wl_surface role

2014-10-01 Thread Pekka Paalanen
On Fri, 22 Aug 2014 11:35:40 +0300 Pekka Paalanen wrote: > On Thu, 21 Aug 2014 09:08:56 -0700 > Jason Ekstrand wrote: > > > That applies to the whole series, BTW > > > > > > On Thu, Aug 21, 2014 at 9:03 AM, Jason Ekstrand > > wrote: > > > > > LGTM > > > Reviewed-by: Jason Ekstrand > > Tha

Wayland, Weston and libinput to patchwork?

2014-10-01 Thread Pekka Paalanen
Hi, at least with Wayland and Weston, we have bit of hard time tracking the patches that need attention. I think I am currently the only one who actually keeps a backlog, my backlog is not public, and I cannot be a maintainer 100% all the time, so this won't scale or work for too long. How about

Re: [RFC weston 13/16] compositor: Add a function to test if images transformed by a matrix should be bilinearly filtered

2014-10-01 Thread Jason Ekstrand
Allow me to chip in here. Sorry that I haven't had a chance to really look over things carefully. I have been reading this thread, just haven't had a chance to respond. On Wed, Oct 1, 2014 at 12:41 AM, Pekka Paalanen wrote: > On Tue, 30 Sep 2014 14:35:24 -0500 > Derek Foreman wrote: > > > Tha

Re: [PATCH wayland 0/4] Implement role conflict errors

2014-10-01 Thread Jasper St. Pierre
Thanks for taking this on. I'm disappointed about the duplication, but if somebody else does the work, I can't really complain. The whole series is: Reviewed-by: Jasper St. Pierre On Wed, Oct 1, 2014 at 7:50 AM, Pekka Paalanen wrote: > From: Pekka Paalanen > > Hi, > > this is my attempt on t

[PATCH weston 3/4] compositor: send error for surface role resets

2014-10-01 Thread Pekka Paalanen
From: Pekka Paalanen With the more accurate definition of wl_surface roles in Wayland, enforce the restriction: a role is always set permanently, and attempting to change it is a protocol error. This patch is based on Jasper's patch: http://lists.freedesktop.org/archives/wayland-devel/2014-Augus

[PATCH weston 4/4] tests: add role conflict sub-surface vs. wl_shell_surface

2014-10-01 Thread Pekka Paalanen
From: Pekka Paalanen Add tests for triggering the role conflict when a wl_surface is already a wl_shell_surface and then attempted to be made into a sub-surface, and vice versa. Signed-off-by: Pekka Paalanen --- Makefile.am| 5 ++ tests/roles-test.c | 139

[PATCH wayland 1/1] protocol: define error codes for role reassignment

2014-10-01 Thread Pekka Paalanen
From: Pekka Paalanen On the interfaces where it was missing: - wl_data_device - wl_shell - wl_pointer add an error code for requests that set a wl_surface role when the wl_surface already has a different role. This is needed for compositors to appropriately report wl_surface role violations. Si

Re: [PATCH wayland v2] assert if a listener function is NULL

2014-10-01 Thread Ryo Munakata
On Wed, 1 Oct 2014 14:49:33 +0300 Giulio Camuffo wrote: > 2014-10-01 14:36 GMT+03:00 Ryo Munakata : > > Signed-off-by: Ryo Munakata > > --- > > src/connection.c | 5 + > > 1 file changed, 5 insertions(+) > > > > diff --git a/src/connection.c b/src/connection.c > > index f292853..424edd4 100

[PATCH wayland v2] assert if a listener function is NULL

2014-10-01 Thread Ryo Munakata
Signed-off-by: Ryo Munakata --- src/connection.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/connection.c b/src/connection.c index f292853..424edd4 100644 --- a/src/connection.c +++ b/src/connection.c @@ -931,6 +931,11 @@ wl_closure_invoke(struct wl_closure *closure, uint32_t fl

[PATCH wayland] assert if a listener function is NULL

2014-10-01 Thread Ryo Munakata
Signed-off-by: Ryo Munakata --- src/connection.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/connection.c b/src/connection.c index f292853..c61275d 100644 --- a/src/connection.c +++ b/src/connection.c @@ -931,6 +931,11 @@ wl_closure_invoke(struct wl_closure *closure, uint32_t fl

Re: [RFC weston 13/16] compositor: Add a function to test if images transformed by a matrix should be bilinearly filtered

2014-10-01 Thread Pekka Paalanen
On Tue, 30 Sep 2014 14:35:24 -0500 Derek Foreman wrote: > Thanks for taking a look! > > On 26/09/14 05:48 PM, Bill Spitzak wrote: > > 90 degree rotation about x or y will require filtering. > > Yup, you're right. > > > You test y scale twice, must be a typo. I think you intended to test z, > >

Re: There is a bad documentation in the website

2014-10-01 Thread Pekka Paalanen
On Tue, 23 Sep 2014 02:39:52 +0800 (CST) Googol wrote: > Hi, when I start download the pdf in your website, I found a pdf file can not > read any more. > I think it is important to the others, though we can still use the html. > My system is Windows 7 x64, with Adobe Reader. > Here is the addres