[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] migration: Fix compilation with older compilers
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PATCH] migration: Fix compilation with older compilers |
Date: |
Wed, 14 Jun 2017 10:50:24 +0200 |
On Wed, 14 Jun 2017 10:34:34 +0200
Juan Quintela <address@hidden> wrote:
> That typedefs are needed on both files. New compilers (F25 where I
Only LoadStateHandler is actually needed by both files but I guess
it is ok to keep these typedefs together.
Reviewed-by: Greg Kurz <address@hidden>
> work) don't complain about repeating a typedef. But older ones
> complain.
>
> Signed-off-by: Juan Quintela <address@hidden>
> ---
> include/migration/register.h | 3 ---
> include/migration/vmstate.h | 1 -
> include/qemu/typedefs.h | 2 ++
> 3 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/include/migration/register.h b/include/migration/register.h
> index 717c617..d9498d9 100644
> --- a/include/migration/register.h
> +++ b/include/migration/register.h
> @@ -14,9 +14,6 @@
> #ifndef MIGRATION_REGISTER_H
> #define MIGRATION_REGISTER_H
>
> -typedef void SaveStateHandler(QEMUFile *f, void *opaque);
> -typedef int LoadStateHandler(QEMUFile *f, void *opaque, int version_id);
> -
> typedef struct SaveVMHandlers {
> /* This runs inside the iothread lock. */
> SaveStateHandler *save_state;
> diff --git a/include/migration/vmstate.h b/include/migration/vmstate.h
> index f3f3c2a..e85fbd8 100644
> --- a/include/migration/vmstate.h
> +++ b/include/migration/vmstate.h
> @@ -29,7 +29,6 @@
>
> #include "migration/qjson.h"
>
> -typedef int LoadStateHandler(QEMUFile *f, void *opaque, int version_id);
> typedef struct VMStateInfo VMStateInfo;
> typedef struct VMStateDescription VMStateDescription;
> typedef struct VMStateField VMStateField;
> diff --git a/include/qemu/typedefs.h b/include/qemu/typedefs.h
> index 51958bf..f745d5f 100644
> --- a/include/qemu/typedefs.h
> +++ b/include/qemu/typedefs.h
> @@ -96,5 +96,7 @@ typedef struct uWireSlave uWireSlave;
> typedef struct VirtIODevice VirtIODevice;
> typedef struct Visitor Visitor;
> typedef struct node_info NodeInfo;
> +typedef void SaveStateHandler(QEMUFile *f, void *opaque);
> +typedef int LoadStateHandler(QEMUFile *f, void *opaque, int version_id);
>
> #endif /* QEMU_TYPEDEFS_H */
pgp54Gzkm_sQj.pgp
Description: OpenPGP digital signature