[PATCH weston 5/5] tests: list available tests if an invalid test name is given

2013-09-10 Thread Peter Hutterer
--- tests/weston-test-runner.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/weston-test-runner.c b/tests/weston-test-runner.c index fefb93b..ed5baf0 100644 --- a/tests/weston-test-runner.c +++ b/tests/weston-test-runner.c @@ -82,6 +82,7 @@ int main(int argc, char *argv[])

[PATCH weston 4/5] tests: support -h/--help for the tests

2013-09-10 Thread Peter Hutterer
Including listing the tests available in that binary --- tests/weston-test-runner.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/tests/weston-test-runner.c b/tests/weston-test-runner.c index 7cc1cbe..fefb93b 100644 --- a/tests/weston-test-runner.c +++ b/tests/weston-test

[PATCH weston 3/5] tests: include config.h in weston-test-runner

2013-09-10 Thread Peter Hutterer
--- tests/weston-test-runner.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/weston-test-runner.c b/tests/weston-test-runner.c index 27ea9e4..7cc1cbe 100644 --- a/tests/weston-test-runner.c +++ b/tests/weston-test-runner.c @@ -20,6 +20,7 @@ * OF THIS SOFTWARE. */ +#include "confi

[PATCH weston 2/5] tests: use variable for test name in weston-tests-env

2013-09-10 Thread Peter Hutterer
Slightly more readable and makes it easier to switch to use $2 for something in the future (if that's ever needed). --- tests/weston-tests-env | 17 - 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/tests/weston-tests-env b/tests/weston-tests-env index 2e5fa95..b7322

[PATCH weston 1/5] tests: always build tests

2013-09-10 Thread Peter Hutterer
check_PROGRAMS and friends are only built during make check. Which is a great way of introducing compiler errors in tests. Always build them, TESTS defines what's being run during make check. --- tests/Makefile.am | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/tes

[PATCH 1/3] Add support for table-driven testing.

2013-09-10 Thread Sam Spilsbury
The new TEST_P macro takes a function name and a "data" argument to point to an arbitrary array of known size of test data. This allows multiple tests to be run with different datasets. The array is stored as a void * but advanced by a known size on each iteration. The data for each invocation of

RE: [PATCH 1/3] Add support for table-driven testing.

2013-09-10 Thread Eoff, Ullysses A
> -Original Message- > From: Sam Spilsbury [mailto:smspil...@gmail.com] > Sent: Tuesday, September 10, 2013 5:33 PM > To: Eoff, Ullysses A > Cc: wayland-devel@lists.freedesktop.org > Subject: Re: [PATCH 1/3] Add support for table-driven testing. > > Hi, > > > On Wed, Sep 11, 2013 at 1:37

[PATCH 2/3] Split vertex clipping code out into vertex-clipping.c in shared/

2013-09-10 Thread Sam Spilsbury
--- shared/Makefile.am | 4 +- shared/vertex-clipping.c | 317 +++ shared/vertex-clipping.h | 65 ++ src/gl-renderer.c| 292 +-- 4 files changed, 389 insertions(+), 289 deletions(-) create

[PATCH 3/3] Added tests for the vertex clipping code.

2013-09-10 Thread Sam Spilsbury
This tests (via the table-driven testing method) that the correct number of vertices and also the correct vertices themselves are generated for an clip box and polygon of up to eight vertices. Also add a libshared_test.a so that we don't have to build weston-test-runner all the time --- tests/Mak

[PATCH 1/3] Add support for table-driven testing.

2013-09-10 Thread Sam Spilsbury
The new TEST_P macro takes a function name and a "data" argument to point to an arbitrary array of known size of test data. This allows multiple tests to be run with different datasets. The array is stored as a void * but advanced by a known size on each iteration. The data for each invocation of

[PATCH 0/3] Tests for vertex clipping (v2)

2013-09-10 Thread Sam Spilsbury
This patch series adds some tests for the vertex clipping code. It splits the main parts of calculate_edges out into clip_transformed exported in shared/vertex-clipping.c and adds support for table-driven testing in tests/weston-test-runner.h . Finally, it makes weston-test-runner.c compile into

Re: [PATCH 3/3] Added tests for the vertex clipping code.

2013-09-10 Thread Sam Spilsbury
Hi, On Wed, Sep 11, 2013 at 1:33 AM, Eoff, Ullysses A wrote: >> -Original Message- >> From: wayland-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org >> [mailto:wayland-devel- >> bounces+ullysses.a.eoff=intel@lists.freedesktop.org] On Behalf Of Sam >> Spilsbury >> Sent:

Re: [PATCH 1/3] Add support for table-driven testing.

2013-09-10 Thread Sam Spilsbury
Hi, On Wed, Sep 11, 2013 at 1:37 AM, Eoff, Ullysses A wrote: > > > -Original Message- > > From: wayland-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org > > [mailto:wayland-devel- > > bounces+ullysses.a.eoff=intel@lists.freedesktop.org] On Behalf Of Sam > > Spilsbury >

Re: [PATCH] wayland-util: fix wrap-back of wl_map->free_list when client id >= 0x80000000

2013-09-10 Thread Jason Ekstrand
Chang, We cannot change WL_SERVER_ID_START. I'm not 100% sure why we chose that value but it's probably because there are significantly more client-side allocated objects than server-side. However, at this point, it is fairly well baked into the protocol and changing it would break backwards comp

Re: [RFC wayland] System compositor protocol

2013-09-10 Thread Tristan Schmelcher
I'm not qualified to comment on the protocol, but I would definitely like to see a standard for Wayland system compositors. I am going to be porting a pixel orbiter to the Wayland stack (original code at https://github.com/TristanSchmelcher/pixelorbiter) and the only secure way to design it is as a

[RFC 0/2] Notify clients when the native mode is switched

2013-09-10 Thread Hardening
This suite of patches implements notification of clients when the mode is switched. For now, most clients aren't ready for being notified and take appropriate actions. The shell behaves strangely during a mode_switch(), anyway I think the underlying logic is ok. Comments are welcome and appreci

[PATCH 1/2] Rename current, origin, scale, origin_scale

2013-09-10 Thread Hardening
This patch renames that fields to have consistent names. --- src/compositor-drm.c | 58 +++ src/compositor-fbdev.c| 4 ++-- src/compositor-headless.c | 4 ++-- src/compositor-rdp.c | 12 +- src/compositor-rpi.c | 4 ++-- src

[PATCH 2/2] Notify clients on mode_switch()

2013-09-10 Thread Hardening
This patch implements the notification of clients during mode_switch. As discussed on IRC, clients are notified of mode_switch only when the "native" mode is changed and activated. That means that if the native mode is changed and the compositor had activated a temporary mode for a fullscreen surfa

RE: [PATCH 3/3] Added tests for the vertex clipping code.

2013-09-10 Thread Eoff, Ullysses A
> -Original Message- > From: wayland-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org > [mailto:wayland-devel- > bounces+ullysses.a.eoff=intel@lists.freedesktop.org] On Behalf Of Sam > Spilsbury > Sent: Tuesday, September 10, 2013 9:15 AM > To: wayland-devel@lists.freedes

RE: [PATCH weston 0/6] ivi-shell proposal

2013-09-10 Thread nobuhiko_tanibata
2013-09-10 20:13 に michael.schu...@bmw.de さんは書きました: Hi Jason, Tanibata-san Hi Jason, Thank you very much for feedback. Michale & Nobuhiko, First of all, thank you for the clarification and thank you for sending this to the list and being willing to work with the FOSS community to try and ma

Re: [PATCH] client: Add a way to get a pointer to the display's default queue

2013-09-10 Thread Neil Roberts
(I hope you don't remind me replying to the whole list.) Marc Chalain writes: > I'm not really agree about Mesa. In my mind it isn't a simple client, > it's a library with parts of wayland. I'm not sur that simple clients > need to know the main queue. But I agree with you about the headers. I

RE: [PATCH 1/3] Add support for table-driven testing.

2013-09-10 Thread Eoff, Ullysses A
> -Original Message- > From: wayland-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org > [mailto:wayland-devel- > bounces+ullysses.a.eoff=intel@lists.freedesktop.org] On Behalf Of Sam > Spilsbury > Sent: Tuesday, September 10, 2013 9:15 AM > To: wayland-devel@lists.freedes

Re: [PATCH] client: Add a way to get a pointer to the display's default queue

2013-09-10 Thread Neil Roberts
Hi, Marc Chalain writes: > I wonder if there isn't a problem when some client application uses > this function from a child thread. If I remember, only the main thread > must use the main queue. If you give an API to use it, developers will > want to use it, and may not understand the usage. I

[PATCH v2] nested: Add a renderer using subsurfaces

2013-09-10 Thread Neil Roberts
I wrote: > • For some reason if you move the cursor to somewhere near the > middle of the surface then it will change to the resize grabber and > it will let you resize the window. It looks like this is happening because the toy toolkit gets confused about the pointer position when it gets motion

[PATCH 3/3] Added tests for the vertex clipping code.

2013-09-10 Thread Sam Spilsbury
This tests (via the table-driven testing method) that the correct number of vertices and also the correct vertices themselves are generated for an clip box and polygon of up to eight vertices. --- tests/Makefile.am| 11 ++- tests/vertex-clip-test.c | 219 ++

[PATCH 2/3] Split vertex clipping code out into vertex-clipping.c in shared/

2013-09-10 Thread Sam Spilsbury
--- shared/vertex-clipping.c | 317 +++ shared/vertex-clipping.h | 65 ++ src/Makefile.am | 2 + src/gl-renderer.c| 292 +-- 4 files changed, 388 insertions(+), 288 deletions(-) create

[PATCH 1/3] Add support for table-driven testing.

2013-09-10 Thread Sam Spilsbury
The new TEST_P macro takes a function name and a "data" argument to point to an arbitrary array of known size of test data. This allows multiple tests to be run with different datasets. The array is stored as a void * but advanced by a known size on each iteration. The data for each invocation of

[PATCH 0/3] Tests for vertex clipping

2013-09-10 Thread Sam Spilsbury
This patch series adds some tests for the vertex clipping code. It splits the main parts of calculate_edges out into clip_transformed exported in shared/vertex-clipping.c and adds support for table-driven testing in tests/weston-test-runner.h . Sam Spilsbury (3): Add support for table-driven t

Re: [PATCH] client: Add a way to get a pointer to the display's default queue

2013-09-10 Thread Marc Chalain
Hello, I wonder if there isn't a problem when some client application uses this function from a child thread. If I remember, only the main thread must use the main queue. If you give an API to use it, developers will want to use it, and may not understand the usage. I read your patch about EGL_WL_c

[PATCH] client: Add a way to get a pointer to the display's default queue

2013-09-10 Thread Neil Roberts
Adds a function called wl_display_get_main_queue which just returns a pointer to the default event queue. This will be useful in order to implement the EGL_WL_create_wayland_buffer_from_image extension. The buffers created within Mesa's Wayland platform are created using the the wl_drm object as a

[PATCH v2] wayland: Add an extension to create wl_buffers from EGLImages

2013-09-10 Thread Neil Roberts
This adds an extension called EGL_WL_create_wayland_buffer_from_image which adds the following single function: struct wl_buffer * eglCreateWaylandBufferFromImageWL(EGLDisplay dpy, EGLImageKHR image); The function creates a wl_buffer which shares its contents with the given EGLImage. The expected

Re: [PATCH 0/10] Add a mechanism for nested compositors to use subsurfaces without blitting

2013-09-10 Thread Neil Roberts
I wrote: > • It seems to cause the nested-client to cycle between three buffers > while rendering whereas previously it would only use two. I may have > messed something up with the ordering between the frame callbacks and > the buffer release events but I'm not sure yet. I think I got to the bott

[PATCH] Add missing return

2013-09-10 Thread Mariusz Ceier
Signed-off-by: Mariusz Ceier --- src/data-device.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/data-device.c b/src/data-device.c index ec3df33..8de5063 100644 --- a/src/data-device.c +++ b/src/data-device.c @@ -393,6 +393,7 @@ weston_seat_start_drag(struct weston_seat *seat, w

RE: [PATCH weston 0/6] ivi-shell proposal

2013-09-10 Thread Michael.Schuldt
Hi Jason, Tanibata-san > Hi Jason, > > Thank you very much for feedback. > >> Michale & Nobuhiko, >> >> First of all, thank you for the clarification and thank you for >> sending this to the list and being willing to work with the FOSS >> community to try and make a standard. I'm sorry that thi

Re: [PATCH weston 0/6] ivi-shell proposal

2013-09-10 Thread nobuhiko_tanibata
Hi Jason, Thank you very much for feedback. Michale & Nobuhiko, First of all, thank you for the clarification and thank you for sending this to the list and being willing to work with the FOSS community to try and make a standard. I'm sorry that this reply is not inline. I think that would get

Re: How to check if OpenGL is supported?

2013-09-10 Thread Pekka Paalanen
On Tue, 10 Sep 2013 19:57:32 +1000 Campbell Barton wrote: > On Tue, Sep 10, 2013 at 4:58 PM, Pekka Paalanen > wrote: > > On Tue, 10 Sep 2013 16:49:27 +1000 > > Campbell Barton wrote: > > > >> Software OpenGL is fine if I can use it to get Blender window up > >> and running in weston. > >> I've

Re: How to check if OpenGL is supported?

2013-09-10 Thread Campbell Barton
On Tue, Sep 10, 2013 at 4:58 PM, Pekka Paalanen wrote: > On Tue, 10 Sep 2013 16:49:27 +1000 > Campbell Barton wrote: > >> On Tue, Sep 10, 2013 at 4:45 PM, Pekka Paalanen >> wrote: >> > On Tue, 10 Sep 2013 16:13:32 +1000 >> > Campbell Barton wrote: >> > >> >> On Tue, Sep 10, 2013 at 3:49 PM, Pek

Re: How to check if OpenGL is supported?

2013-09-10 Thread Pekka Paalanen
On Tue, 10 Sep 2013 16:49:27 +1000 Campbell Barton wrote: > On Tue, Sep 10, 2013 at 4:45 PM, Pekka Paalanen > wrote: > > On Tue, 10 Sep 2013 16:13:32 +1000 > > Campbell Barton wrote: > > > >> On Tue, Sep 10, 2013 at 3:49 PM, Pekka Paalanen > >> wrote: > >> > On Tue, 10 Sep 2013 14:06:55 +1000