[PATCH wayland-protocols] input-method: Cleanup some grammar

2016-09-16 Thread Bryce Harrington
Fix which vs. that, and rephrase a few descriptions to be clearer. Signed-off-by: Bryce Harrington --- unstable/input-method/input-method-unstable-v1.xml | 25 +++--- 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/unstable/input-method/input-method-unstable-v1.xm

[PATCH wayland-protocols] idle-inhibit: Lead with a verb in request description

2016-09-16 Thread Bryce Harrington
Signed-off-by: Bryce Harrington --- unstable/idle-inhibit/idle-inhibit-unstable-v1.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/unstable/idle-inhibit/idle-inhibit-unstable-v1.xml b/unstable/idle-inhibit/idle-inhibit-unstable-v1.xml index 70372fc..9c06cdc 100644 ---

[PATCH wayland 2/4] wl_array: Set data to null after free

2016-09-16 Thread Yong Bakos
From: Yong Bakos Explicitly set the data member to NULL during wl_array_release, preventing the dangling pointer but, more importantly, making it testable. Signed-off-by: Yong Bakos --- src/wayland-util.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/wayland-util.c b/src/wayland-util

[PATCH wayland 3/4] tests: Test wl_array_release

2016-09-16 Thread Yong Bakos
From: Yong Bakos array-test.c did not cover wl_array_release, so add one test that specifically tests this method. Signed-off-by: Yong Bakos --- tests/array-test.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/tests/array-test.c b/tests/array-test.c index fe53240..a5856fd 100

[PATCH wayland 0/4] wl_array: documentation and test

2016-09-16 Thread Yong Bakos
From: Yong Bakos This series adds documentation to wl_array and its methods. You can ignore the warning during documentation building; it is caused by a reference to `wl_list_for_each`, which is anticipated in a separate patch (up for review). There is one implementation change: setting the `dat

[PATCH wayland 4/4] array-test: Include wayland-util.h and simplify init test

2016-09-16 Thread Yong Bakos
From: Yong Bakos Include wayland-util.h instead of wayland-private.h, because that's where wl_array is and nothing in wayland-private.h is required. Remove the useless repeated testing of wl_array_init, because if it fails once out of thousands of iterations we're all doomed anyway. Signed-off-

[PATCH wayland 1/4] util: Document wl_array

2016-09-16 Thread Yong Bakos
From: Yong Bakos Add doxygen comments for wl_array and its methods. Signed-off-by: Yong Bakos --- src/wayland-util.h | 66 +- 1 file changed, 61 insertions(+), 5 deletions(-) diff --git a/src/wayland-util.h b/src/wayland-util.h index cacc122

Re: [PATCH wayland-web] Drop the ubuntu 12.04 build directions.

2016-09-16 Thread Yong Bakos
On Sep 16, 2016, at 2:29 PM, Bryce Harrington wrote: > > It is unlikely anyone still needs directions on how to install on this > old distro -- Ubuntu 12.04 is scheduled to hit end-of-life this April. > > Further, no developers (to my knowledge) still test on 12.04, so the > directions have like

[PATCH wayland-web] Drop the ubuntu 12.04 build directions.

2016-09-16 Thread Bryce Harrington
It is unlikely anyone still needs directions on how to install on this old distro -- Ubuntu 12.04 is scheduled to hit end-of-life this April. Further, no developers (to my knowledge) still test on 12.04, so the directions have likely bitrotted anyway. (Bill had a machine running 12.04 and was kee

Re: [PATCH wayland 0/4] Untangle the symbol export duplication

2016-09-16 Thread Pekka Paalanen
On Fri, 16 Sep 2016 14:47:39 +0100 Emil Velikov wrote: > Hi Pekka, > > Thanks again for keeping up. > > On 16 September 2016 at 10:46, Pekka Paalanen wrote: > > Hi, > > > > there seems to be two different issues discussed here. > > > You cought me, yes there are. > > > 1) libwayland-client

Re: [PATCH libxkbcommon 2/2] README: Add basic build directions

2016-09-16 Thread Bryce Harrington
On Fri, Sep 16, 2016 at 09:35:09AM +0300, Ran Benita wrote: > On Thu, Sep 15, 2016 at 02:31:55PM -0700, Bryce Harrington wrote: > > In particular, highlight the use of configure flags to control locating > > X11 keyboard stuff when building for Wayland. > > > > Of particular note, if the locale ro

Re: [PATCH wayland v1 1/1] wayland-scanner.pc.in: prepend pc_sysrootdir

2016-09-16 Thread Joe Konno
On Fri, 16 Sep 2016 17:30:18 +0200 Arnaud Vrac wrote: > On Fri, Sep 16, 2016 at 5:19 PM, Joe Konno > wrote: > > > On Fri, 16 Sep 2016 10:53:32 +0300 > > Pekka Paalanen wrote: > > > > > On Thu, 15 Sep 2016 15:22:12 -0700 > > > Joe Konno wrote: > > > > > > > From: Joe Konno > > > > > > > >

Re: [PATCH wayland v1 1/1] wayland-scanner.pc.in: prepend pc_sysrootdir

2016-09-16 Thread Arnaud Vrac
On Fri, Sep 16, 2016 at 5:19 PM, Joe Konno wrote: > On Fri, 16 Sep 2016 10:53:32 +0300 > Pekka Paalanen wrote: > > > On Thu, 15 Sep 2016 15:22:12 -0700 > > Joe Konno wrote: > > > > > From: Joe Konno > > > > > > In a cross-compilation environment with packages depending on > > > wayland-scanner

Re: [PATCH wayland v1 1/1] wayland-scanner.pc.in: prepend pc_sysrootdir

2016-09-16 Thread Joe Konno
On Fri, 16 Sep 2016 10:53:32 +0300 Pekka Paalanen wrote: > On Thu, 15 Sep 2016 15:22:12 -0700 > Joe Konno wrote: > > > From: Joe Konno > > > > In a cross-compilation environment with packages depending on > > wayland-scanner, ensure the path to wayland-scanner is correct. Without > > this pat

Re: [PATCH weston 07/14 v3] weston: Port RDP backend to new output handling API

2016-09-16 Thread Pekka Paalanen
On Thu, 18 Aug 2016 18:42:35 +0200 Armin Krezović wrote: > This is a complete port of the RDP backend that uses > recently added output handling API for output > configuration. > > Output can be configured at runtime by passing the > necessary configuration parameters, which can be > filled in m

Re: [PATCH wayland 0/4] Untangle the symbol export duplication

2016-09-16 Thread Emil Velikov
Hi Pekka, Thanks again for keeping up. On 16 September 2016 at 10:46, Pekka Paalanen wrote: > Hi, > > there seems to be two different issues discussed here. > You cought me, yes there are. > 1) libwayland-client and libwayland-server export the same interface > symbols each, and cannot stop exp

Re: [PATCH wayland 0/4] Untangle the symbol export duplication

2016-09-16 Thread Pekka Paalanen
On Fri, 16 Sep 2016 01:03:40 +0100 Emil Velikov wrote: > On 15 September 2016 at 11:47, Pekka Paalanen wrote: > > On Wed, 14 Sep 2016 16:57:17 +0100 > > Emil Velikov wrote: > > > >> Hi Pekka, > >> > >> Huge thanks for the input. > >> > >> On 14 September 2016 at 14:35, Pekka Paalanen wrote:

Re: [PATCH wayland v1 1/1] wayland-scanner.pc.in: prepend pc_sysrootdir

2016-09-16 Thread Pekka Paalanen
On Thu, 15 Sep 2016 15:22:12 -0700 Joe Konno wrote: > From: Joe Konno > > In a cross-compilation environment with packages depending on > wayland-scanner, ensure the path to wayland-scanner is correct. Without > this patch, the path will _not_ point to the target environment but to > the host's