[PATCH wayland 3/4] tests: Test wl_array_release
Pekka Paalanen
ppaalanen at gmail.com
Tue Sep 20 08:34:34 UTC 2016
On Fri, 16 Sep 2016 15:37:38 -0700
Yong Bakos <junk at humanoriented.com> wrote:
> From: Yong Bakos <ybakos at 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 <ybakos at 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 {
-------------- 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/20160920/6c2840c5/attachment.sig>
More information about the wayland-devel
mailing list