Re: [PATCH wayland] tests/resources-test: Fix wl_resource_post_event call

2014-01-06 Thread Jason Ekstrand
On Mon, Jan 6, 2014 at 8:34 AM, Mariusz Ceier wrote: > I think both patches (and original code too obviously) are wrong - > WL_DISPLAY_ERROR (opcode 0) event has signature "ous", so Quentin > patch calls event with too many arguments and ignores types, and Marek > patch calls event with too few ar

Re: [PATCH wayland] tests/resources-test: Fix wl_resource_post_event call

2014-01-06 Thread Mariusz Ceier
I think both patches (and original code too obviously) are wrong - WL_DISPLAY_ERROR (opcode 0) event has signature "ous", so Quentin patch calls event with too many arguments and ignores types, and Marek patch calls event with too few arguments and incorrect type for the second argument. Imo, corre

Re: [PATCH wayland] tests/resources-test: Fix wl_resource_post_event call

2014-01-06 Thread Marek Ch
The call is wrong, I posted it here: http://lists.freedesktop.org/archives/wayland-devel/2013-November/012141.html But without any reaction. Good that somobody else renewed it :) On 05/01/2014, Quentin Glidic wrote: > From: Quentin Glidic > > Signed-off-by: Quentin Glidic > --- > > I do not kno

[PATCH wayland] tests/resources-test: Fix wl_resource_post_event call

2014-01-05 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- I do not know if this call is wrong of this the test just reveals a bug but without this, the wl_resource_post_event_array is called with random values tests/resources-test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g