<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 20 November 2014 at 23:59, Bryce Harrington <span dir="ltr"><<a href="mailto:bryce@osg.samsung.com" target="_blank">bryce@osg.samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Nov 19, 2014 at 12:00:10PM +0100, Marek Chalupa wrote:<br>
> This env variable is used for turning off the leak checks in tests.<br>
> Rename it to WAYLAND_TESTS_NO_LEAK_CHECK, so that it is consistent<br>
> with WAYLAND_TESTS_NO_TIMEOUTS.<br>
<br>
</span>Should these both be 'WAYLAND_TEST_'?  In weston there is already a<br>
WESTON_TEST_CLIENT_PATH.<br>
<br>
If WAYLAND_TESTS_* is better, then maybe WESTON_TEST_CLIENT_PATH should<br>
be changed for consistency.<br>
<br></blockquote><div><br></div><div>Yep, let's see what others think :) I'll either modify this patch or follow-up with Weston patch then.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Apart from that, LGTM.<br>
<br>
Reviewed-by: Bryce Harrington <<a href="mailto:b.harrington@samsung.com">b.harrington@samsung.com</a>><br>
<div><div class="h5"><br>
> Signed-off-by: Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>><br>
> ---<br>
>  tests/test-runner.c | 5 ++++-<br>
>  1 file changed, 4 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/tests/test-runner.c b/tests/test-runner.c<br>
> index 09c50e2..a6c75a9 100644<br>
> --- a/tests/test-runner.c<br>
> +++ b/tests/test-runner.c<br>
> @@ -42,6 +42,9 @@ static void (*sys_free)(void*);<br>
>  static void* (*sys_realloc)(void*, size_t);<br>
>  static void* (*sys_calloc)(size_t, size_t);<br>
><br>
> +/* when set to 1, check if tests are not leaking memory and opened files.<br>
> + * It is turned on by default. It can be turned off by<br>
> + * WAYLAND_TESTS_NO_LEAK_CHECK environment variable. */<br>
>  int leak_check_enabled;<br>
><br>
>  /* when this var is set to 0, every call to test_set_timeout() is<br>
> @@ -234,7 +237,7 @@ int main(int argc, char *argv[])<br>
>       sys_malloc = dlsym(RTLD_NEXT, "malloc");<br>
>       sys_free = dlsym(RTLD_NEXT, "free");<br>
><br>
> -     leak_check_enabled = !getenv("NO_ASSERT_LEAK_CHECK");<br>
> +     leak_check_enabled = !getenv("WAYLAND_TESTS_NO_LEAK_CHECK");<br>
>       timeouts_enabled = !getenv("WAYLAND_TESTS_NO_TIMEOUTS");<br>
><br>
>       if (argc == 2 && strcmp(argv[1], "--help") == 0)<br>
> --<br>
> 2.1.0<br>
><br>
</div></div>> _______________________________________________<br>
> wayland-devel mailing list<br>
> <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br></blockquote><div><br></div><div>Thanks,<br>Marek <br></div></div><br></div></div>