On Mon, 13 Jun 2011 21:31:09 -0500 Michael Roth <mdr...@linux.vnet.ibm.com> wrote:
> Base definitions/includes for Visiter interface used by generated > visiter/marshalling code. > > Includes a GenericList type. Our lists require an embedded element. > Since these types are generated, if you want to use them in a different > type of data structure, there's no easy way to add another embedded > element. The solution is to have non-embedded lists and that what this is. > > Signed-off-by: Michael Roth <mdr...@linux.vnet.ibm.com> > --- > Makefile.objs | 6 +++ > qapi/qapi-types-core.h | 21 ++++++++++ > qapi/qapi-visit-core.c | 101 > ++++++++++++++++++++++++++++++++++++++++++++++++ > qapi/qapi-visit-core.h | 68 ++++++++++++++++++++++++++++++++ > 4 files changed, 196 insertions(+), 0 deletions(-) > create mode 100644 qapi/qapi-types-core.h > create mode 100644 qapi/qapi-visit-core.c > create mode 100644 qapi/qapi-visit-core.h > > diff --git a/Makefile.objs b/Makefile.objs > index a7807e8..68d7b5a 100644 > --- a/Makefile.objs > +++ b/Makefile.objs > @@ -364,6 +364,12 @@ endif > > libcacard-y = cac.o event.o vcard.o vreader.o vcard_emul_nss.o > vcard_emul_type.o card_7816.o > > +###################################################################### > +# qapi > + > +qapi-nested-y = qapi-visit-core.o > +qapi-obj-y = $(addprefix qapi/, $(qapi-nested-y)) To build this I have to add qapi-obj-y to common-obj-y and add the qapi directory to the DIR variable in configure (so that it builds in a different directory). I saw that you send the configure change in a different patch, but it has to be done here. > + > vl.o: QEMU_CFLAGS+=$(GPROF_CFLAGS) > > vl.o: QEMU_CFLAGS+=$(SDL_CFLAGS) > diff --git a/qapi/qapi-types-core.h b/qapi/qapi-types-core.h > new file mode 100644 > index 0000000..de733ab > --- /dev/null > +++ b/qapi/qapi-types-core.h > @@ -0,0 +1,21 @@ > +/* > + * Core Definitions for QAPI-generated Types > + * > + * Copyright IBM, Corp. 2011 > + * > + * Authors: > + * Anthony Liguori <aligu...@us.ibm.com> > + * > + * This work is licensed under the terms of the GNU LGPL, version 2.1 or > later. > + * See the COPYING.LIB file in the top-level directory. > + * > + */ > + > +#ifndef QAPI_TYPES_CORE_H > +#define QAPI_TYPES_CORE_H > + > +#include <stdbool.h> > +#include <stdint.h> > +#include "error.h" > + > +#endif > diff --git a/qapi/qapi-visit-core.c b/qapi/qapi-visit-core.c > new file mode 100644 > index 0000000..948818e > --- /dev/null > +++ b/qapi/qapi-visit-core.c > @@ -0,0 +1,101 @@ > +#include "qapi/qapi-visit-core.h" No license info. > + > +void visit_start_handle(Visitor *v, void **obj, const char *kind, const char > *name, Error **errp) > +{ > + if (!error_is_set(errp) && v->start_handle) { > + v->start_handle(v, obj, kind, name, errp); > + } > +} > + > +void visit_end_handle(Visitor *v, Error **errp) > +{ > + if (!error_is_set(errp) && v->end_handle) { > + v->end_handle(v, errp); > + } > +} > + > +void visit_start_struct(Visitor *v, void **obj, const char *kind, const char > *name, size_t size, Error **errp) > +{ > + if (!error_is_set(errp)) { > + v->start_struct(v, obj, kind, name, size, errp); > + } > +} > + > +void visit_end_struct(Visitor *v, Error **errp) > +{ > + if (!error_is_set(errp)) { > + v->end_struct(v, errp); > + } > +} > + > +void visit_start_list(Visitor *v, const char *name, Error **errp) > +{ > + if (!error_is_set(errp)) { > + v->start_list(v, name, errp); > + } > +} > + > +GenericList *visit_next_list(Visitor *v, GenericList **list, Error **errp) > +{ > + if (!error_is_set(errp)) { > + return v->next_list(v, list, errp); > + } > + > + return 0; > +} > + > +void visit_end_list(Visitor *v, Error **errp) > +{ > + if (!error_is_set(errp)) { > + v->end_list(v, errp); > + } > +} > + > +void visit_start_optional(Visitor *v, bool *present, const char *name, Error > **errp) > +{ > + if (!error_is_set(errp) && v->start_optional) { > + v->start_optional(v, present, name, errp); > + } > +} > + > +void visit_end_optional(Visitor *v, Error **errp) > +{ > + if (!error_is_set(errp) && v->end_optional) { > + v->end_optional(v, errp); > + } > +} > + > +void visit_type_enum(Visitor *v, int *obj, const char *kind, const char > *name, Error **errp) > +{ > + if (!error_is_set(errp)) { > + v->type_enum(v, obj, kind, name, errp); > + } > +} > + > +void visit_type_int(Visitor *v, int64_t *obj, const char *name, Error **errp) > +{ > + if (!error_is_set(errp)) { > + v->type_int(v, obj, name, errp); > + } > +} > + > +void visit_type_bool(Visitor *v, bool *obj, const char *name, Error **errp) > +{ > + if (!error_is_set(errp)) { > + v->type_bool(v, obj, name, errp); > + } > +} > + > +void visit_type_str(Visitor *v, char **obj, const char *name, Error **errp) > +{ > + if (!error_is_set(errp)) { > + v->type_str(v, obj, name, errp); > + } > +} > + > +void visit_type_number(Visitor *v, double *obj, const char *name, Error > **errp) > +{ > + if (!error_is_set(errp)) { > + v->type_number(v, obj, name, errp); > + } > +} > diff --git a/qapi/qapi-visit-core.h b/qapi/qapi-visit-core.h > new file mode 100644 > index 0000000..8350a6d > --- /dev/null > +++ b/qapi/qapi-visit-core.h > @@ -0,0 +1,68 @@ > +/* > + * Core Definitions for QAPI Visitor Classes > + * > + * Copyright IBM, Corp. 2011 > + * > + * Authors: > + * Anthony Liguori <aligu...@us.ibm.com> > + * > + * This work is licensed under the terms of the GNU LGPL, version 2.1 or > later. > + * See the COPYING.LIB file in the top-level directory. > + * > + */ > +#ifndef QAPI_VISITOR_CORE_H > +#define QAPI_VISITOR_CORE_H > + > +#include "qapi/qapi-types-core.h" > +#include "error.h" > +#include <stdlib.h> > + > +typedef struct GenericList > +{ > + void *value; > + struct GenericList *next; > +} GenericList; > + > +typedef struct Visitor Visitor; > + > +struct Visitor > +{ > + /* Must be set */ > + void (*start_struct)(Visitor *v, void **obj, const char *kind, const > char *name, size_t size, Error **errp); > + void (*end_struct)(Visitor *v, Error **errp); > + > + void (*start_list)(Visitor *v, const char *name, Error **errp); > + GenericList *(*next_list)(Visitor *v, GenericList **list, Error **errp); > + void (*end_list)(Visitor *v, Error **errp); > + > + void (*type_enum)(Visitor *v, int *obj, const char *kind, const char > *name, Error **errp); > + > + void (*type_int)(Visitor *v, int64_t *obj, const char *name, Error > **errp); > + void (*type_bool)(Visitor *v, bool *obj, const char *name, Error **errp); > + void (*type_str)(Visitor *v, char **obj, const char *name, Error **errp); > + void (*type_number)(Visitor *v, double *obj, const char *name, Error > **errp); > + > + /* May be NULL */ > + void (*start_optional)(Visitor *v, bool *present, const char *name, > Error **errp); > + void (*end_optional)(Visitor *v, Error **errp); > + > + void (*start_handle)(Visitor *v, void **obj, const char *kind, const > char *name, Error **errp); > + void (*end_handle)(Visitor *v, Error **errp); > +}; > + > +void visit_start_handle(Visitor *v, void **obj, const char *kind, const char > *name, Error **errp); > +void visit_end_handle(Visitor *v, Error **errp); > +void visit_start_struct(Visitor *v, void **obj, const char *kind, const char > *name, size_t size, Error **errp); > +void visit_end_struct(Visitor *v, Error **errp); > +void visit_start_list(Visitor *v, const char *name, Error **errp); > +GenericList *visit_next_list(Visitor *v, GenericList **list, Error **errp); > +void visit_end_list(Visitor *v, Error **errp); > +void visit_start_optional(Visitor *v, bool *present, const char *name, Error > **errp); > +void visit_end_optional(Visitor *v, Error **errp); > +void visit_type_enum(Visitor *v, int *obj, const char *kind, const char > *name, Error **errp); > +void visit_type_int(Visitor *v, int64_t *obj, const char *name, Error > **errp); > +void visit_type_bool(Visitor *v, bool *obj, const char *name, Error **errp); > +void visit_type_str(Visitor *v, char **obj, const char *name, Error **errp); > +void visit_type_number(Visitor *v, double *obj, const char *name, Error > **errp); > + > +#endif