On Thu, Apr 25, 2019 at 15:28:31 +0200, Ján Tomko wrote:
> On Mon, Apr 15, 2019 at 06:02:01PM +0200, Peter Krempa wrote:
> > Simplify virQEMUQAPISchemaTraverse by separating out the necessary
> > operations for given 'meta-type' into separate functions.
> >
> > Signed-off-by: Peter Krempa <pkre...@redhat.com>
> > ---
> > src/qemu/qemu_qapi.c | 117 ++++++++++++++++++++++++++++---------------
> > 1 file changed, 78 insertions(+), 39 deletions(-)
> >
> > diff --git a/src/qemu/qemu_qapi.c b/src/qemu/qemu_qapi.c
> > index 2652061f4e..5995e5006e 100644
> > --- a/src/qemu/qemu_qapi.c
> > +++ b/src/qemu/qemu_qapi.c
> > @@ -105,68 +105,107 @@ static int
> > virQEMUQAPISchemaTraverse(const char *baseName,
> > char **query,
> > virHashTablePtr schema,
> > - virJSONValuePtr *type)
> > + virJSONValuePtr *type);
> > +
> > +
> > +static int
> > +virQEMUQAPISchemaTraverseObject(virJSONValuePtr cur,
> > + char **query,
> > + virHashTablePtr schema,
> > + virJSONValuePtr *type)
> > {
> > - virJSONValuePtr curtype;
> > virJSONValuePtr obj;
> > - const char *metatype;
> > const char *querytype = NULL;
> > - const char *querystr;
> > - char modifier;
> > + const char *querystr = *query;
> > + char modifier = *querystr;
> > +
> > + if (!c_isalpha(modifier))
> > + querystr++;
> > +
> > + if (modifier == '+') {
> > + querytype = virQEMUQAPISchemaObjectGetType("variants",
> > + querystr,
> > + "case", cur);
> > + } else {
> > + obj = virQEMUQAPISchemaObjectGet("members", querystr, "name", cur);
> > +
> > + if (modifier == '*' &&
> > + !virJSONValueObjectHasKey(obj, "default"))
> > + return 0;
> > +
> > + querytype = virQEMUQAPISchemaTypeFromObject(obj);
> > + }
> >
> > - if (!(curtype = virHashLookup(schema, baseName)))
>
> The object-specific code lost the
> if (!querytype)
> return 0;
> conditionIt is not necessary as virHashLookup returns NULL if @name is NULL and it's called right as the first thing in virQEMUQAPISchemaTraverse which is called from virQEMUQAPISchemaTraverseObject. Dropping that part makes it work the same in all of the individual workers.
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
