From 8af5d6bd7c73b51a7b0cec6993cccd2466dea2f1 Mon Sep 17 00:00:00 2001
From: Peter Krempa <pkrempa@redhat.com>
Date: Wed, 10 Apr 2019 14:14:10 +0200
Subject: [PATCH] qemu: qapi: Separate virQEMUQAPISchemaTraverse into functions
 by object type
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Simplify virQEMUQAPISchemaTraverse by separating out the necessary
operations for given 'meta-type' into separate functions.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@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)))
+    return virQEMUQAPISchemaTraverse(querytype, query + 1, schema, type);
+}
+
+
+static int
+virQEMUQAPISchemaTraverseArray(virJSONValuePtr cur,
+                               char **query,
+                               virHashTablePtr schema,
+                               virJSONValuePtr *type)
+{
+    const char *querytype;
+
+    /* arrays are just flattened by default */
+    if (!(querytype = virJSONValueObjectGetString(cur, "element-type")))
+        return 0;
+
+    return virQEMUQAPISchemaTraverse(querytype, query, schema, type);
+}
+
+
+static int
+virQEMUQAPISchemaTraverseCommand(virJSONValuePtr cur,
+                                 char **query,
+                                 virHashTablePtr schema,
+                                 virJSONValuePtr *type)
+{
+    const char *querytype;
+
+    if (!(querytype = virJSONValueObjectGetString(cur, *query)))
+        return 0;
+
+    return virQEMUQAPISchemaTraverse(querytype, query + 1, schema, type);
+}
+
+
+static int
+virQEMUQAPISchemaTraverse(const char *baseName,
+                          char **query,
+                          virHashTablePtr schema,
+                          virJSONValuePtr *type)
+{
+    virJSONValuePtr cur;
+    const char *metatype;
+
+    if (!(cur = virHashLookup(schema, baseName)))
         return 0;
 
     if (!*query) {
         if (type)
-            *type = curtype;
+            *type = cur;
 
         return 1;
     }
 
-    if (!(metatype = virJSONValueObjectGetString(curtype, "meta-type")))
+    if (!(metatype = virJSONValueObjectGetString(cur, "meta-type")))
         return 0;
 
-    /* flatten arrays by default */
     if (STREQ(metatype, "array")) {
-        if (!(querytype = virJSONValueObjectGetString(curtype, "element-type")))
-            return 0;
+        return virQEMUQAPISchemaTraverseArray(cur, query, schema, type);
     } else if (STREQ(metatype, "object")) {
-        querystr = *query;
-        modifier = **query;
-
-        if (!c_isalpha(modifier))
-            querystr++;
-
-        if (modifier == '+') {
-            querytype = virQEMUQAPISchemaObjectGetType("variants",
-                                                       querystr,
-                                                       "case", curtype);
-        } else {
-            obj = virQEMUQAPISchemaObjectGet("members", querystr,
-                                             "name", curtype);
-
-            if (modifier == '*' &&
-                !virJSONValueObjectHasKey(obj, "default"))
-                return 0;
-
-            querytype = virQEMUQAPISchemaTypeFromObject(obj);
-        }
-        query++;
+        return virQEMUQAPISchemaTraverseObject(cur, query, schema, type);
     } else if (STREQ(metatype, "command") ||
                STREQ(metatype, "event")) {
-        if (!(querytype = virJSONValueObjectGetString(curtype, *query)))
-            return 0;
-        query++;
+        return virQEMUQAPISchemaTraverseCommand(cur, query, schema, type);
     } else {
         /* alternates, basic types and enums can't be entered */
         return 0;
     }
 
-    if (!querytype)
-        return 0;
-
-    return virQEMUQAPISchemaTraverse(querytype, query, schema, type);
+    return 0;
 }
 
 
-- 
GitLab