[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH 27/29] tests/hbitmap: use ARRAY_S
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-trivial] [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
- Re: [Qemu-trivial] [Qemu-devel] [PATCH 25/29] libqos: use ARRAY_SIZE macro, (continued)
[Qemu-trivial] [PATCH 27/29] tests/hbitmap: use ARRAY_SIZE macro, Philippe Mathieu-Daudé, 2017/07/18
[Qemu-trivial] [PATCH 28/29] tests/qapi: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
[Qemu-trivial] [PATCH 29/29] tests/qapi: use ARRAY_SIZE macro, Philippe Mathieu-Daudé, 2017/07/18
Message not available