<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 27, 2015 at 1:24 PM, Daniel Stone <span dir="ltr"><<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<span class=""><br>
On 19 December 2014 at 13:53, Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>> wrote:<br>
> diff --git a/tests/test-compositor.h b/tests/test-compositor.h<br>
> index c41b17b..f77a5db 100644<br>
> --- a/tests/test-compositor.h<br>
> +++ b/tests/test-compositor.h<br>
> @@ -95,3 +95,9 @@ struct client_info *client_create_with_name(struct display *d,<br>
>                                             void (*client_main)(void),<br>
>                                             const char *name);<br>
>  #define client_create(d, c) client_create_with_name((d), (c), (#c))<br>
> +<br>
> +#define TEST_CLIENT_NOT_USING_WL_DISPLAY_CONNECT               \<br>
> +       do {                                                    \<br>
> +               extern int not_using_wl_display_connect;        \<br>
> +               not_using_wl_display_connect = 1;               \<br>
> +       } while(0);<br>
<br>
</span>I really like this series, but this bit seems a bit fragile. How about<br>
something like the two attached patches, which seems to work for me?<br>
The first one can be squashed into this patch, and the second into the<br>
follow-up.<br></blockquote><div><br></div><div>Yeah, your solution is much better than mine, I like it. The Bill's addition looks quite good too. . Feel free to squeeze it/follow-up. Or do you want me to re-post version 2?<br><div><br></div><div>Thanks,<br>Marek <br></div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Cheers,<br>
Daniel<br></blockquote><br></div></div></div>