[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qapi: fix qapi_dealloc_type_size parameter type
From: |
mdroth |
Subject: |
Re: [Qemu-devel] [PATCH] qapi: fix qapi_dealloc_type_size parameter type |
Date: |
Tue, 27 Nov 2012 14:54:30 -0600 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, Nov 27, 2012 at 01:11:25PM -0700, Bruce Rogers wrote:
> The second parameter to qapi_dealloc_type_size should be a uint64_t *,
> not a size_t *. This was causing our 32 bit x86 build to fail, since
> warnings are treated as errors.
>
> Signed-off-by: Bruce Rogers <address@hidden>
Doh, I should've caught this in review. Yes, qapi's type generator uses
uint64_t as the underlying type for 'size', and the visitor interface
calls for uint64_t*, so that's what the implementation should use.
Reviewed-by: Michael Roth <address@hidden>
> ---
> qapi/qapi-dealloc-visitor.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/qapi/qapi-dealloc-visitor.c b/qapi/qapi-dealloc-visitor.c
> index a07b171..75214e7 100644
> --- a/qapi/qapi-dealloc-visitor.c
> +++ b/qapi/qapi-dealloc-visitor.c
> @@ -132,7 +132,7 @@ static void qapi_dealloc_type_number(Visitor *v, double
> *obj, const char *name,
> {
> }
>
> -static void qapi_dealloc_type_size(Visitor *v, size_t *obj, const char *name,
> +static void qapi_dealloc_type_size(Visitor *v, uint64_t *obj, const char
> *name,
> Error **errp)
> {
> }
> --
> 1.7.7
>