On Sun, 4 Sep 2016 13:23:05 -0700 Yong Bakos <[email protected]> wrote:
> From: Yong Bakos <[email protected]> > > list-test.c did not cover wl_list_length, so add one test that specifically > tests this method. > > Signed-off-by: Yong Bakos <[email protected]> > --- > 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
pgpS1oIKVcEOf.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
