[PATCH wayland 2/8] test-compositor: extend leak checks into clients

Daniel Stone daniel at fooishbar.org
Tue Jan 27 04:24:47 PST 2015


Hi,

On 19 December 2014 at 13:53, Marek Chalupa <mchqwerty at gmail.com> wrote:
> diff --git a/tests/test-compositor.h b/tests/test-compositor.h
> index c41b17b..f77a5db 100644
> --- a/tests/test-compositor.h
> +++ b/tests/test-compositor.h
> @@ -95,3 +95,9 @@ struct client_info *client_create_with_name(struct display *d,
>                                             void (*client_main)(void),
>                                             const char *name);
>  #define client_create(d, c) client_create_with_name((d), (c), (#c))
> +
> +#define TEST_CLIENT_NOT_USING_WL_DISPLAY_CONNECT               \
> +       do {                                                    \
> +               extern int not_using_wl_display_connect;        \
> +               not_using_wl_display_connect = 1;               \
> +       } while(0);

I really like this series, but this bit seems a bit fragile. How about
something like the two attached patches, which seems to work for me?
The first one can be squashed into this patch, and the second into the
follow-up.

Cheers,
Daniel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Tests-Simpler-check-for-WAYLAND_SOCKET-fd-consumptio.patch
Type: text/x-patch
Size: 2764 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20150127/e1e3c73a/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-fixup-for-3-8.patch
Type: text/x-patch
Size: 2916 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20150127/e1e3c73a/attachment-0001.bin>


More information about the wayland-devel mailing list