qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [Qemu-devel] [PATCH 27/29] tests/hbitmap: use ARRAY_SIZ


From: Marc-André Lureau
Subject: Re: [Qemu-block] [Qemu-devel] [PATCH 27/29] tests/hbitmap: use ARRAY_SIZE macro
Date: Tue, 18 Jul 2017 03:52:26 -0700

On Mon, Jul 17, 2017 at 11:10 PM, Philippe Mathieu-Daudé
<address@hidden> wrote:
> Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>

Reviewed-by: Marc-André Lureau <address@hidden>


> ---
>  tests/test-hbitmap.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/tests/test-hbitmap.c b/tests/test-hbitmap.c
> index 1acb353889..628e72122b 100644
> --- a/tests/test-hbitmap.c
> +++ b/tests/test-hbitmap.c
> @@ -813,7 +813,7 @@ static void test_hbitmap_serialize_basic(TestHBitmapData 
> *data,
>      size_t buf_size;
>      uint8_t *buf;
>      uint64_t positions[] = { 0, 1, L1 - 1, L1, L2 - 1, L2, L2 + 1, L3 - 1 };
> -    int num_positions = sizeof(positions) / sizeof(positions[0]);
> +    int num_positions = ARRAY_SIZE(positions);
>
>      hbitmap_test_init(data, L3, 0);
>      g_assert(hbitmap_is_serializable(data->hb));
> @@ -838,7 +838,7 @@ static void test_hbitmap_serialize_part(TestHBitmapData 
> *data,
>      size_t buf_size;
>      uint8_t *buf;
>      uint64_t positions[] = { 0, 1, L1 - 1, L1, L2 - 1, L2, L2 + 1, L3 - 1 };
> -    int num_positions = sizeof(positions) / sizeof(positions[0]);
> +    int num_positions = ARRAY_SIZE(positions);
>
>      hbitmap_test_init(data, L3, 0);
>      buf_size = L2;
> @@ -880,7 +880,7 @@ static void test_hbitmap_serialize_zeroes(TestHBitmapData 
> *data,
>      int64_t next;
>      uint64_t min_l1 = MAX(L1, 64);
>      uint64_t positions[] = { 0, min_l1, L2, L3 - min_l1};
> -    int num_positions = sizeof(positions) / sizeof(positions[0]);
> +    int num_positions = ARRAY_SIZE(positions);
>
>      hbitmap_test_init(data, L3, 0);
>
> --
> 2.13.2
>
>



-- 
Marc-André Lureau



reply via email to

[Prev in Thread] Current Thread [Next in Thread]