diff mbox series

thunk: improve readability of allocation loop

Message ID a09acf9e093c09a70896bb9886370cf7f049f941.camel@gmail.com (mailing list archive)
State New, archived
Headers show
Series thunk: improve readability of allocation loop | expand

Commit Message

Aarushi Mehta March 8, 2019, 7:43 p.m. UTC 
Signed-off-by: Aarushi Mehta <mehta.aaru20@gmail.com>
---
 thunk.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stefan Hajnoczi March 9, 2019, 2:28 p.m. UTC | #1 
On Fri, Mar 8, 2019 at 7:43 PM Aarushi Mehta <mehta.aaru20@gmail.com> wrote:
>
> Signed-off-by: Aarushi Mehta <mehta.aaru20@gmail.com>
> ---
>  thunk.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Laurent Vivier March 9, 2019, 2:52 p.m. UTC | #2 
On 08/03/2019 20:43, Aarushi Mehta wrote:
> Signed-off-by: Aarushi Mehta <mehta.aaru20@gmail.com>
> ---
>  thunk.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/thunk.c b/thunk.c
> index 17f3d320bb..7f31cffe09 100644
> --- a/thunk.c
> +++ b/thunk.c
> @@ -86,7 +86,7 @@ void thunk_register_struct(int id, const char *name, const argtype *types)
>  #endif
>      /* now we can alloc the data */
>  
> -    for(i = 0;i < 2; i++) {
> +    for (i = 0; i < ARRAY_SIZE(se->field_offsets); i++) {
>          offset = 0;
>          max_align = 1;
>          se->field_offsets[i] = g_new(int, nb_fields);
> 

Applied to my trivial-patches branch.

Thanks,
Laurent
diff mbox series

Patch

diff --git a/thunk.c b/thunk.c
index 17f3d320bb..7f31cffe09 100644
--- a/thunk.c
+++ b/thunk.c
@@ -86,7 +86,7 @@  void thunk_register_struct(int id, const char *name, const argtype *types)
 #endif
     /* now we can alloc the data */
 
-    for(i = 0;i < 2; i++) {
+    for (i = 0; i < ARRAY_SIZE(se->field_offsets); i++) {
         offset = 0;
         max_align = 1;
         se->field_offsets[i] = g_new(int, nb_fields);