diff options
author | Greg Kurz <groug@kaod.org> | 2018-06-08 10:39:09 +0200 |
---|---|---|
committer | Dr. David Alan Gilbert <dgilbert@redhat.com> | 2018-06-15 14:40:56 +0100 |
commit | ea134caa08ca86689a56665f30bebeb38c5cd1ab (patch) | |
tree | 11eee7c50514930846903917df6bc1378745c0e2 /migration | |
parent | 2702c2d3eb74e3908c0c5dbf3a71c8987595a86e (diff) | |
download | qemu-ea134caa08ca86689a56665f30bebeb38c5cd1ab.zip qemu-ea134caa08ca86689a56665f30bebeb38c5cd1ab.tar.gz qemu-ea134caa08ca86689a56665f30bebeb38c5cd1ab.tar.bz2 |
typedefs: add QJSON
Since commit 83ee768d6247b, we now have two places that define the
QJSON type:
$ git grep 'typedef struct QJSON QJSON'
include/migration/vmstate.h:typedef struct QJSON QJSON;
migration/qjson.h:typedef struct QJSON QJSON;
This breaks docker-test-build@centos6:
In file included from /tmp/qemu-test/src/migration/savevm.c:59:
/tmp/qemu-test/src/migration/qjson.h:16: error: redefinition of typedef
'QJSON'
/tmp/qemu-test/src/include/migration/vmstate.h:30: note: previous
declaration of 'QJSON' was here
make: *** [migration/savevm.o] Error 1
This happens because CentOS 6 has an old GCC 4.4.7. Even if redefining
a typedef with the same type is permitted since GCC 4.6, unless -pedantic
is passed, we don't really need to do that on purpose. Let's have a
single definition in <qemu/typedefs.h> instead.
Signed-off-by: Greg Kurz <groug@kaod.org>
Message-Id: <152844714981.11789.3657734445739553287.stgit@bahia.lan>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Diffstat (limited to 'migration')
-rw-r--r-- | migration/qjson.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/migration/qjson.h b/migration/qjson.h index 2978b5f..41664f2 100644 --- a/migration/qjson.h +++ b/migration/qjson.h @@ -13,8 +13,6 @@ #ifndef QEMU_QJSON_H #define QEMU_QJSON_H -typedef struct QJSON QJSON; - QJSON *qjson_new(void); void qjson_destroy(QJSON *json); void json_prop_str(QJSON *json, const char *name, const char *str); |