On Fri, 16 Sep 2016 15:37:38 -0700 Yong Bakos <j...@humanoriented.com> wrote:
> From: Yong Bakos <yba...@humanoriented.com>
>
> array-test.c did not cover wl_array_release, so add one test that specifically
> tests this method.
>
> Signed-off-by: Yong Bakos <yba...@humanoriented.com>
> ---
> tests/array-test.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/tests/array-test.c b/tests/array-test.c
> index fe53240..a5856fd 100644
> --- a/tests/array-test.c
> +++ b/tests/array-test.c
> @@ -45,6 +45,16 @@ TEST(array_init)
> }
> }
>
> +TEST(array_release)
> +{
> + struct wl_array array;
> + wl_array_init(&array);
Hi
> + array.data = calloc(1, sizeof(int));
This is something one should never do, it is asking for problems. Why
don't you use wl_array_add()?
Thanks,
pq
> + assert(array.data != NULL);
> + wl_array_release(&array);
> + assert(array.data == NULL);
> +}
> +
> TEST(array_add)
> {
> struct mydata {
pgpN4L16WPTfg.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel
