[PATCH wayland v3 2/2] tests: Add test for wl_list_length

Pekka Paalanen ppaalanen at gmail.com
Mon Sep 5 12:14:12 UTC 2016


On Sun,  4 Sep 2016 13:23:05 -0700
Yong Bakos <junk at humanoriented.com> wrote:

> From: Yong Bakos <ybakos at humanoriented.com>
> 
> list-test.c did not cover wl_list_length, so add one test that specifically
> tests this method.
> 
> Signed-off-by: Yong Bakos <ybakos at humanoriented.com>
> ---
> v3: no changes
> v2: no changes
> 
>  tests/list-test.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/tests/list-test.c b/tests/list-test.c
> index 21ca4ec..0752618 100644
> --- a/tests/list-test.c
> +++ b/tests/list-test.c
> @@ -57,6 +57,19 @@ TEST(list_insert)
>  	assert(e.link.prev == &list);
>  }
> 
> +TEST(list_length)
> +{
> +	struct wl_list list;
> +	struct element e;
> +
> +	wl_list_init(&list);
> +	assert(wl_list_length(&list) == 0);
> +	wl_list_insert(&list, &e.link);
> +	assert(wl_list_length(&list) == 1);
> +	wl_list_remove(&e.link);
> +	assert(wl_list_length(&list) == 0);
> +}
> +
>  TEST(list_iterator)
>  {
>  	struct wl_list list;
> --
> 2.7.2

Hi Yong,

when re-sending patches, you should add any Reviewed-by etc. tags you
have already received, unless you think you have changed the patch
enough for them to not apply anymore. I see v2 had them, so I used
those.

I think this patch is small enough to land right now even though we are
in fixes-only phase, so pushed:
   d9ef882..2bdbb74  master -> master


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20160905/d6a6883b/attachment.sig>


More information about the wayland-devel mailing list