Second and final release candidate for 1.5 (1.4.93)

2014-05-12 Thread Kristian Høgsberg
Hi, Here's the second release candidate for Wayland and Weston 1.5: ee6d6367ef6c3f6b46e39ce6b4568f316e2d1667 wayland-1.4.93.tar.xz 8511544e6be4385a87d2111c87f7f397c63c43ff wayland 1.4.93 tag 9d81584d203cc10ad7d9c49b18ffe2642ee5d7d4 weston-1.4.93.tar.xz 7098948463836104c28f6adc0b41cd08

Re: [PATCH weston] window: Ignore input events from subsurfaces

2014-05-12 Thread Kristian Høgsberg
On Tue, May 06, 2014 at 03:25:40PM +0300, Ander Conselvan de Oliveira wrote: > From: Ander Conselvan de Oliveira > > Toytoolkit was not designed to handle input from subsurfaces and > instead it expects subsurfaces to have an empty input region. That way > input events for subsurfaces are generat

Re: [PATCH V2] event: assert wl_event_source pointer is NULL.

2014-05-12 Thread Bill Spitzak
The reason functions should not validate pointers that are supposed to be non-NULL is because it is misleading. An "if (x)..." in the code is a very very strong indicator to a programmer that x can legitimately have a NULL value. No amount of documentation is going to change that programmer's m

Re: [PATCH] Do not distribute generated headers

2014-05-12 Thread Kristian Høgsberg
On Wed, May 07, 2014 at 02:09:35PM +0200, Thierry Reding wrote: > From: Thierry Reding > > The wayland-server-protocol.h and wayland-client-protocol.h headers are > currently being shipped in tarballs created using make dist. This causes > out-of-tree builds to fail since make will detect that th

Re: [PATCH] doc: Remove obsolete doxygen tags

2014-05-12 Thread Kristian Høgsberg
On Mon, May 12, 2014 at 08:40:03PM +0800, Boyan Ding wrote: > --- > doc/doxygen/wayland.doxygen.in | 28 > 1 file changed, 28 deletions(-) > > diff --git a/doc/doxygen/wayland.doxygen.in b/doc/doxygen/wayland.doxygen.in > index dd98594..e64512f 100644 > --- a/doc/doxy

Re: [PATCH V2] event: assert wl_event_source pointer is NULL.

2014-05-12 Thread Kristian Høgsberg
On Mon, May 12, 2014 at 08:31:14AM +, Bryce W. Harrington wrote: > On Mon, May 12, 2014 at 11:26:01AM +0530, Srivardhan Hebbar wrote: > > Signed-off-by: Srivardhan Hebbar > > --- > > src/event-loop.c |6 +- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/src/e

Re: [PATCH weston 2/2] build: let 'make clean' remove logs/ again

2014-05-12 Thread Kristian Høgsberg
On Mon, May 12, 2014 at 10:08:57AM +0300, Pekka Paalanen wrote: > From: Pekka Paalanen > > Before in the recursive automake setting, we had tests/logs/ for > explicitly created test log files. There is a Makefile rule to > remove the logs directory on 'make clean'. The rule broke on moving to > n

Re: [PATCH weston] desktop-shell: Fix order of desktop_shell messages

2014-05-12 Thread Kristian Høgsberg
On Sat, May 10, 2014 at 10:43:34AM +0200, Jonas Ådahl wrote: > There was a bug in wayland-scanner that failed to detect when an > message with implicitly set version (i.e. version 1) came after a > message with a newer version. This patch fixes the weston desktop shell > protocol to pass again. >

[PATCH] xdg-shell: Rename set_transient_for to set_parent

2014-05-12 Thread Jasper St. Pierre
It's a confusing name that comes from the ICCCM. The ICCCM is best forgotten about. With the addition of the potential new "transient" role meaning a parent-relative toplevel like a long-lived popup, used for e.g. tooltips, the set_transient_for name will become even more confusing. --- clients/s

[PATCH 2/2] fullscreen: Remove dirty sizing trick

2014-05-12 Thread Jasper St. Pierre
xdg-shell mandates that the FULLSCREEN state means that we must match the size that we were configured to, at least by default. Other states or protocol extensions might relax this requirement, but at least for now implement the behavior specified in the protocol documentation. --- clients/fullscr

[PATCH 1/2] fullscreen: Remove our own focused state tracking

2014-05-12 Thread Jasper St. Pierre
This is decided by xdg-shell now. --- clients/fullscreen.c | 21 +++-- 1 file changed, 3 insertions(+), 18 deletions(-) diff --git a/clients/fullscreen.c b/clients/fullscreen.c index fa8028a..789d243 100644 --- a/clients/fullscreen.c +++ b/clients/fullscreen.c @@ -50,7 +50,7 @@ st

[PATCH] doc: Remove obsolete doxygen tags

2014-05-12 Thread Boyan Ding
--- doc/doxygen/wayland.doxygen.in | 28 1 file changed, 28 deletions(-) diff --git a/doc/doxygen/wayland.doxygen.in b/doc/doxygen/wayland.doxygen.in index dd98594..e64512f 100644 --- a/doc/doxygen/wayland.doxygen.in +++ b/doc/doxygen/wayland.doxygen.in @@ -310,22 +31

RE: [PATCH V2] event: assert wl_event_source pointer is NULL.

2014-05-12 Thread Srivardhan
> -Original Message- > From: wayland-devel [mailto:wayland-devel- > boun...@lists.freedesktop.org] On Behalf Of Pekka Paalanen > Sent: Monday, May 12, 2014 2:48 PM > To: Bryce W. Harrington > Cc: Srivardhan Hebbar; wayland-devel@lists.freedesktop.org > Subject: Re: [PATCH V2] event: asser

Re: [PATCH V2] event: assert wl_event_source pointer is NULL.

2014-05-12 Thread Pekka Paalanen
On Mon, 12 May 2014 11:49:46 +0300 Pekka Paalanen wrote: > On Mon, 12 May 2014 08:31:14 + > "Bryce W. Harrington" wrote: > > > On Mon, May 12, 2014 at 11:26:01AM +0530, Srivardhan Hebbar wrote: > > > Signed-off-by: Srivardhan Hebbar > > > --- > > > src/event-loop.c |6 +- > > > 1

Re: [PATCH V2] event: assert wl_event_source pointer is NULL.

2014-05-12 Thread Pekka Paalanen
On Mon, 12 May 2014 08:31:14 + "Bryce W. Harrington" wrote: > On Mon, May 12, 2014 at 11:26:01AM +0530, Srivardhan Hebbar wrote: > > Signed-off-by: Srivardhan Hebbar > > --- > > src/event-loop.c |6 +- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/src/even

Re: [PATCH V2] event: assert wl_event_source pointer is NULL.

2014-05-12 Thread Bryce W. Harrington
On Mon, May 12, 2014 at 11:26:01AM +0530, Srivardhan Hebbar wrote: > Signed-off-by: Srivardhan Hebbar > --- > src/event-loop.c |6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/src/event-loop.c b/src/event-loop.c > index 9790cde..57e3fed 100644 > --- a/src/event-lo

Re: [PATCH weston 2/2] build: let 'make clean' remove logs/ again

2014-05-12 Thread Bryce W. Harrington
Yep, I ran into this myself but wasn't sure if it was intentional or not. Reviewed-by: Bryce Harrington On Mon, May 12, 2014 at 10:08:57AM +0300, Pekka Paalanen wrote: > From: Pekka Paalanen > > Before in the recursive automake setting, we had tests/logs/ for > explicitly created test log file

Re: [PATCH V2] event: assert wl_event_source pointer is NULL.

2014-05-12 Thread Hardening
Le 12/05/2014 07:56, Srivardhan Hebbar a écrit : Signed-off-by: Srivardhan Hebbar --- src/event-loop.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/event-loop.c b/src/event-loop.c index 9790cde..57e3fed 100644 --- a/src/event-loop.c +++ b/src/event-loop.c @@

Re: [RFCv3 weston 00/15] Wayland Presentation extension

2014-05-12 Thread Pekka Paalanen
On Fri, 7 Mar 2014 14:03:48 +0200 Pekka Paalanen wrote: > From: Pekka Paalanen > > Hi all, > > here is the third RFC of the Wayland Presentation protocol, now > with a complete implementation! Hi all, I have rebased the RFCv3 version on top of the current Weston master, and added a few fix

Re: [PATCH] gitignore log files, now in root directory

2014-05-12 Thread Pekka Paalanen
On Wed, 7 May 2014 13:30:52 +0300 Pekka Paalanen wrote: > On Tue, 11 Mar 2014 18:11:43 + > "Bryce W. Harrington" wrote: > > > > > Signed-off-by: Bryce Harrington > > --- > > .gitignore |2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/.gitignore b/.gitignore > > index

[PATCH weston 2/2] build: let 'make clean' remove logs/ again

2014-05-12 Thread Pekka Paalanen
From: Pekka Paalanen Before in the recursive automake setting, we had tests/logs/ for explicitly created test log files. There is a Makefile rule to remove the logs directory on 'make clean'. The rule broke on moving to non-recursive make, since now we have logs/, not tests/logs/. Fix the rule t

[PATCH weston 1/2] gitignore log files, now in root directory

2014-05-12 Thread Pekka Paalanen
From: "Bryce W. Harrington" [paalanen: added also *.trs to ignore] Signed-off-by: Bryce Harrington Signed-off-by: Pekka Paalanen --- .gitignore | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.gitignore b/.gitignore index e0a73c0..f9a6a7b 100644 --- a/.gitignore +++ b/.gitignore @@ -2,