[PATCH 4/4] queue-test: Add another assertion

[Pekka Paalanen]

On Thu, 11 Sep 2014 21:43:49 +0200
Karsten Otto <karsten.otto at posteo.de> wrote:

> From: Philip Withnall <philip at tecnocode.co.uk>
> Date: Fri, 15 Feb 2013 12:57:23 +0000
> 
> Ensure that the round trip succeeds.
> 
> [kao: adjusted patch to current test framework style]
> 
> Signed-off-by: Philip Withnall <philip at tecnocode.co.uk>
> Signed-off-by: Karsten Otto <ottoka at posteo.de>
> ---
> tests/queue-test.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/queue-test.c b/tests/queue-test.c
> index 96f2100..6e2e932 100644
> --- a/tests/queue-test.c
> +++ b/tests/queue-test.c
> @@ -66,7 +66,7 @@ client_test_proxy_destroy(void)
> 	assert(registry != NULL);
> 	wl_registry_add_listener(registry, &registry_listener,
> 				 &counter);
> -	wl_display_roundtrip(display);
> +	assert(wl_display_roundtrip(display) != -1);
> 
> 	assert(counter == 1);
> 

Hi,

sorry, all four patches are corrupted: there is no space in front of the
context lines. Could you try again with git-send-email?

The same goes for your "Do not abort() the compositor when dup fails
during fd marshalling".

The link http://www.freebsd.org/cgi/query-pr.cgi?pr=docs/99356#reply2
does not work anymore. Can you dig up a new one?

I commented on patch 3, the rest look good so far.


Thanks,
pq